Improve modularity

1 files changed, 45 insertions, 52 deletions

diff --git a/archaeological_operations/ishtar_menu.py b/archaeological_operations/ishtar_menu.py
index 5a22efd05..a0852147a 100644
--- a/archaeological_operations/ishtar_menu.py
+++ b/archaeological_operations/ishtar_menu.py
@@ -20,7 +20,6 @@
 from django.utils.translation import ugettext_lazy as _, pgettext_lazy
 
 from ishtar_common.menu_base import SectionItem, MenuItem
-from ishtar_common.models import get_current_profile
 
 import models
 
@@ -55,6 +54,38 @@ MENU_SECTIONS = [
                 access_controls=['change_operation',
                                  'change_own_operation']),
             SectionItem(
+                'admin_act_operations',
+                _(u"Administrative act"),
+                profile_restriction='files',
+                childs=[
+                    MenuItem(
+                        'operation_administrativeactop_search',
+                        _(u"Search"),
+                        model=models.AdministrativeAct,
+                        access_controls=[
+                            'change_administrativeact']),
+                    MenuItem(
+                        'operation_administrativeactop',
+                        _(u"Add"),
+                        model=models.AdministrativeAct,
+                        access_controls=['change_administrativeact']),
+                    MenuItem(
+                        'operation_administrativeactop_modification',
+                        _(u"Modification"),
+                        model=models.AdministrativeAct,
+                        access_controls=['change_administrativeact']),
+                    MenuItem(
+                        'operation_administrativeactop_deletion',
+                        _(u"Deletion"),
+                        model=models.AdministrativeAct,
+                        access_controls=['change_administrativeact']),
+                    MenuItem(
+                        'operation_administrativeact_document',
+                        _(u"Documents"),
+                        model=models.AdministrativeAct,
+                        access_controls=['change_administrativeact']),
+                ],),
+            SectionItem(
                 'operation_source', _(u"Documentation"),
                 childs=[
                     MenuItem('operation_source_search',
@@ -81,6 +112,19 @@ MENU_SECTIONS = [
         ]),
      ),
     (
+        35, SectionItem(
+            'administrativact_management', _(u"Administrative Act"),
+            profile_restriction='files',
+            childs=[
+                MenuItem(
+                    'administrativact_register',
+                    pgettext_lazy('admin act register', u"Register"),
+                    model=models.AdministrativeAct,
+                    access_controls=['view_administrativeact',
+                                     'view_own_administrativeact']),
+            ])
+    ),
+    (
         102, SectionItem(
             'dashboard', _(u"Dashboard"),
             childs=[
@@ -95,54 +139,3 @@ MENU_SECTIONS = [
             ]),
     ),
 ]
-
-if get_current_profile().files:
-    MENU_SECTIONS.insert(
-        1,
-        (
-            35, SectionItem(
-                'administrativact_management', _(u"Administrative Act"),
-                childs=[
-                    MenuItem(
-                        'administrativact_register',
-                        pgettext_lazy('admin act register', u"Register"),
-                        model=models.AdministrativeAct,
-                        access_controls=['view_administrativeact',
-                                         'view_own_administrativeact']),
-                ])
-        )
-    )
-    MENU_SECTIONS[0][1].childs.insert(
-        5,
-        SectionItem(
-            'admin_act_operations',
-            _(u"Administrative act"),
-            childs=[
-                MenuItem(
-                    'operation_administrativeactop_search',
-                    _(u"Search"),
-                    model=models.AdministrativeAct,
-                    access_controls=[
-                        'change_administrativeact']),
-                MenuItem(
-                    'operation_administrativeactop',
-                    _(u"Add"),
-                    model=models.AdministrativeAct,
-                    access_controls=['change_administrativeact']),
-                MenuItem(
-                    'operation_administrativeactop_modification',
-                    _(u"Modification"),
-                    model=models.AdministrativeAct,
-                    access_controls=['change_administrativeact']),
-                MenuItem(
-                    'operation_administrativeactop_deletion',
-                    _(u"Deletion"),
-                    model=models.AdministrativeAct,
-                    access_controls=['change_administrativeact']),
-                MenuItem(
-                    'operation_administrativeact_document',
-                    _(u"Documents"),
-                    model=models.AdministrativeAct,
-                    access_controls=['change_administrativeact']),
-            ],)
-    )