fix #30

djangocms_reversion2/cms_toolbars.py

@@ -7,15 +7,32 @@
 from cms.toolbar_pool import toolbar_pool
 from cms.toolbar_base import CMSToolbar
 
+from djangocms_reversion2.utils import is_version_page
+
 
 @toolbar_pool.register
 class Reversion2sModifier(CMSToolbar):
 
     def populate(self):
         page = self.request.current_page
+
+        # if the current page is an archived page we want to disable the toolbar
+        is_archieved = is_version_page(page)
+        if is_archieved:
+            # disable all menus
+            for name, menu in self.request.toolbar.menus.items():
+                menu.disabled = True
+            # remove the publish button
+
+
         if page and page.publisher_is_draft:
             reversion_menu = self.toolbar.get_or_create_menu('djangocms_reversion2', _('Reversion'))
 
+            if is_archieved:
+                reversion_menu.add_item(LinkItem(_('This is an archieved version'), url=''))
+                return
+
+
             reversion_menu.add_modal_item(
                 _('Create a snapshot of current page'),
                 url=self.get_url('admin:djangocms_reversion2_pageversion_add', query_args={'draft': page.id,
@@ -55,3 +72,6 @@ def get_url(self, viewname, arguments=None, query_args=None):
             url=reverse(viewname=viewname, kwargs=arguments),
             query=query_dict.urlencode()
         )
+
+
+

djangocms_reversion2/templates/admin/view_revision.html

@@ -10,9 +10,11 @@
 
     <script type="text/javascript">
         function inject_code() {
-            var iframe_document = document.getElementById('myframe').contentWindow.document;
+            iframe_document = document.getElementById('myframe').contentWindow.document;
             iframe_document.getElementsByClassName('cms-toolbar')[0].remove();
             iframe_document.getElementsByClassName('cms-toolbar-trigger')[0].remove();
+            // monkey patch the missing CMS context
+            window.CMS = document.getElementById('myframe').contentWindow.CMS;
         }
     </script>
 

setup.py

@@ -21,9 +21,8 @@
     'Topic :: Software Development :: Libraries :: Application Frameworks',
 ]
 
-# ToDo: django>=1.8.17,<1.9 is <1.9 really a hard requirement?
 REQUIREMENTS = [
-    'django>=1.8.17',
+    'django>=1.8.17,<1.9',
     'django-cms>=3.4.3',
     'diff-match-patch>=20110725.1',
     'django-sekizai>=0.9.0'