1 files changed, 61 insertions, 53 deletions
diff --git a/archaeological_files/ishtar_menu.py b/archaeological_files/ishtar_menu.py
index 4458c5b8f..b120d18d1 100644
--- a/archaeological_files/ishtar_menu.py
+++ b/archaeological_files/ishtar_menu.py
@@ -19,66 +19,74 @@
 
 from django.utils.translation import ugettext_lazy as _
 
-from archaeological_operations.models import Operation
 from ishtar_common.menu_base import SectionItem, MenuItem
 
 from archaeological_operations.models import AdministrativeAct
 
 import models
 
+# be carreful: each access_controls must be relevant with check_rights in urls
+
 MENU_SECTIONS = [
-    (20, SectionItem('file_management', _(u"Archaeological file"),
-        childs=[
-                MenuItem('file_search', _(u"Search"),
-                    model=models.File,
-                    access_controls=['view_file', 'view_own_file']),
-                MenuItem('file_creation', _(u"Creation"),
-                    model=models.File,
-                    access_controls=['add_file', 'add_own_file']),
-                MenuItem('file_modification', _(u"Modification"),
-                    model=models.File,
-                    access_controls=['change_file', 'change_own_file']),
-                MenuItem('file_closing', _(u"Closing"),
-                  model=models.File,
-                  access_controls=['close_file']),
-                MenuItem('file_deletion', _(u"Deletion"),
-                    model=models.File,
-                    access_controls=['delete_file', 'delete_own_file']),
-                SectionItem('admin_act_files', _(u"Administrative act"),
-            childs=[
-                MenuItem('file_administrativeactfil_search',
-                         _(u"Search"),
-                  model=AdministrativeAct,
-                  access_controls=['change_file', 'change_own_file']),
-                MenuItem('file_administrativeactfil',
-                         _(u"Add"),
-                  model=AdministrativeAct,
-                  access_controls=['change_file', 'change_own_file']),
-                MenuItem('file_administrativeactfil_modification',
-                         _(u"Modification"),
-                  model=AdministrativeAct,
-                  access_controls=['change_file', 'change_own_file']),
-                MenuItem('file_administrativeactfil_deletion',
-                         _(u"Deletion"),
-                  model=AdministrativeAct,
-                  access_controls=['delete_file', 'delete_own_file']),
-                MenuItem('file_administrativeact_document',
-                         _(u"Documents"),
-                  model=AdministrativeAct,
-                  access_controls=['change_file', 'change_own_file']),
-                ],),
-        ]),
-    ),
-    (100, SectionItem('dashboard', _(u"Dashboard"),
-        childs=[
-                MenuItem('dashboard_main', _(u"General informations"),
-                    model=models.File,
-                    access_controls=['change_file', 'change_own_file']),
-                MenuItem('dashboard_file', _(u"Archaeological files"),
-                    model=models.File,
-                    access_controls=['change_file', 'change_own_file']),
-        ]),
-    ),
+    (20,
+     SectionItem(
+         'file_management', _(u"Archaeological file"),
+         childs=[
+             MenuItem(
+                 'file_search', _(u"Search"),
+                 model=models.File,
+                 access_controls=['view_file', 'view_own_file']),
+             MenuItem(
+                 'file_creation', _(u"Creation"),
+                 model=models.File,
+                 access_controls=['add_file', 'add_own_file']),
+             MenuItem(
+                 'file_modification', _(u"Modification"),
+                 model=models.File,
+                 access_controls=['change_file', 'change_own_file']),
+             MenuItem(
+                 'file_closing', _(u"Closing"),
+                 model=models.File,
+                 access_controls=['close_file']),
+             MenuItem(
+                 'file_deletion', _(u"Deletion"),
+                 model=models.File,
+                 access_controls=['delete_file', 'delete_own_file']),
+             SectionItem(
+                 'admin_act_files', _(u"Administrative act"),
+                 childs=[
+                     MenuItem('file_administrativeactfil_search',
+                              _(u"Search"),
+                              model=AdministrativeAct,
+                              access_controls=['change_administrativeact']),
+                     MenuItem('file_administrativeactfil',
+                              _(u"Add"),
+                              model=AdministrativeAct,
+                              access_controls=['change_administrativeact']),
+                     MenuItem('file_administrativeactfil_modification',
+                              _(u"Modification"),
+                              model=AdministrativeAct,
+                              access_controls=['change_administrativeact']),
+                     MenuItem('file_administrativeactfil_deletion',
+                              _(u"Deletion"),
+                              model=AdministrativeAct,
+                              access_controls=['change_administrativeact']),
+                     MenuItem('file_administrativeact_document',
+                              _(u"Documents"),
+                              model=AdministrativeAct,
+                              access_controls=['change_administrativeact']),
+                 ],)]),),
+    (100,
+     SectionItem(
+         'dashboard', _(u"Dashboard"),
+         childs=[MenuItem('dashboard_main', _(u"General informations"),
+                          model=models.File,
+                          access_controls=['change_file', 'change_own_file']),
+                 MenuItem('dashboard_file', _(u"Archaeological files"),
+                          model=models.File,
+                          access_controls=['change_file', 'change_own_file']),
+                 ]),
+     ),
 ]
 """
         SectionItem('dashboard', _(u"Dashboard"),