Switch back caching policy to must-revalidate (update upgrader constraint)

See merge request !22

bt5/officejs_test/TestTemplateItem/portal_components/test.erp5.testOfficeJSScenario.py

@@ -136,7 +136,7 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
     self.assertEquals(response.status, 200)
     self.assertEquals(
       response.getheader('Cache-Control'),
-      'max-age=600, stale-while-revalidate=360000, stale-if-error=31536000, public'
+      'max-age=0, public, must-revalidate'
     )
     self.assertTrue(
       'text/html;' in response.getheader('Content-Type'),
@@ -152,12 +152,11 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
     )
     self.assertEquals(
       response.getheader('Vary').replace(' ', ''),
-      'Cookie,Authorization,Accept-Encoding'
+      'Accept-Language,Cookie,Authorization,Accept-Encoding'
     )
     # First access to Base_getWebDocumentDrivenModificationDate writes in ZODB
     self.tic()
 
-
     ###################################
     ### Anonymous: access the redirect application appcache
     connection.request(
@@ -168,7 +167,8 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
 
     page_content = response.read()
     self.assertEquals("""CACHE MANIFEST
-# development / %s""" % web_section.Base_getWebDocumentDrivenModificationDate().rfc822(),
+CACHE:
+NETWORK:""",
       page_content
     )
     self.assertEquals(response.status, 200)
@@ -258,7 +258,7 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
     self.assertEquals(response.status, 200)
     self.assertEquals(
       response.getheader('Cache-Control'),
-      'max-age=600, stale-while-revalidate=360000, stale-if-error=31536000, public'
+      'max-age=0, public, must-revalidate'
     )
     self.assertTrue(
       'text/html;' in response.getheader('Content-Type'),
@@ -274,7 +274,7 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
     )
     self.assertEquals(
       response.getheader('Vary').replace(' ', ''),
-      'Cookie,Authorization,Accept-Encoding'
+      'Accept-Language,Cookie,Authorization,Accept-Encoding'
     )
 
     ###################################
@@ -306,7 +306,7 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
     self.assertEquals(response.status, 200)
     self.assertEquals(
       response.getheader('Cache-Control'),
-      'max-age=600, stale-while-revalidate=360000, stale-if-error=31536000, public'
+      'max-age=0, public, must-revalidate'
     )
     self.assertTrue(
       'text/html;' in response.getheader('Content-Type'),
@@ -322,7 +322,7 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
     )
     self.assertEquals(
       response.getheader('Vary').replace(' ', ''),
-      'Cookie,Authorization,Accept-Encoding'
+      'Accept-Language,Cookie,Authorization,Accept-Encoding'
     )
 
     ###################################
@@ -335,7 +335,8 @@ class TestOfficeJSScenario(testOfficeJSAppstoreMixin):
 
     page_content = response.read()
     self.assertEquals("""CACHE MANIFEST
-# %s / %s""" % (web_section.getId(), web_section.Base_getWebDocumentDrivenModificationDate().rfc822()),
+CACHE:
+NETWORK:""",
       page_content
     )
     self.assertEquals(response.status, 200)

bt5/officejs_test/TestTemplateItem/portal_components/test.erp5.testOfficeJSScenario.xml

@@ -45,7 +45,11 @@
         <item>
             <key> <string>text_content_warning_message</string> </key>
             <value>
-              <tuple/>
+              <tuple>
+                <string>W:113, 28: Unused variable \'api_path\' (unused-variable)</string>
+                <string>W:113, 38: Unused variable \'api_query\' (unused-variable)</string>
+                <string>W:114,  6: Unused variable \'api_fragment\' (unused-variable)</string>
+              </tuple>
             </value>
         </item>
         <item>
@@ -100,24 +104,28 @@
   </record>
   <record id="4" aka="AAAAAAAAAAQ=">
     <pickle>
-      <global name="WorkflowHistoryList" module="Products.ERP5Type.patches.WorkflowTool"/>
+      <global name="WorkflowHistoryList" module="Products.ERP5Type.Workflow"/>
     </pickle>
     <pickle>
-      <tuple>
-        <none/>
-        <list>
-          <dictionary>
-            <item>
-                <key> <string>action</string> </key>
-                <value> <string>validate</string> </value>
-            </item>
-            <item>
-                <key> <string>validation_state</string> </key>
-                <value> <string>validated</string> </value>
-            </item>
-          </dictionary>
-        </list>
-      </tuple>
+      <dictionary>
+        <item>
+            <key> <string>_log</string> </key>
+            <value>
+              <list>
+                <dictionary>
+                  <item>
+                      <key> <string>action</string> </key>
+                      <value> <string>validate</string> </value>
+                  </item>
+                  <item>
+                      <key> <string>validation_state</string> </key>
+                      <value> <string>validated</string> </value>
+                  </item>
+                </dictionary>
+              </list>
+            </value>
+        </item>
+      </dictionary>
     </pickle>
   </record>
 </ZopeData>

bt5/officejs_test/TestTemplateItem/portal_components/test.erp5.testOfficeJSScenarioRjsUI.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="Test Component" module="erp5.portal_type"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>_recorded_property_dict</string> </key>
-            <value>
-              <persistent> <string encoding="base64">AAAAAAAAAAI=</string> </persistent>
-            </value>
-        </item>
-        <item>
-            <key> <string>default_reference</string> </key>
-            <value> <string>testOfficeJSScenarioRjsUI</string> </value>
-        </item>
-        <item>
-            <key> <string>description</string> </key>
-            <value>
-              <none/>
-            </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>test.erp5.testOfficeJSScenarioRjsUI</string> </value>
-        </item>
-        <item>
-            <key> <string>portal_type</string> </key>
-            <value> <string>Test Component</string> </value>
-        </item>
-        <item>
-            <key> <string>sid</string> </key>
-            <value>
-              <none/>
-            </value>
-        </item>
-        <item>
-            <key> <string>text_content_error_message</string> </key>
-            <value>
-              <tuple/>
-            </value>
-        </item>
-        <item>
-            <key> <string>text_content_warning_message</string> </key>
-            <value>
-              <tuple/>
-            </value>
-        </item>
-        <item>
-            <key> <string>version</string> </key>
-            <value> <string>erp5</string> </value>
-        </item>
-        <item>
-            <key> <string>workflow_history</string> </key>
-            <value>
-              <persistent> <string encoding="base64">AAAAAAAAAAM=</string> </persistent>
-            </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-  <record id="2" aka="AAAAAAAAAAI=">
-    <pickle>
-      <global name="PersistentMapping" module="Persistence.mapping"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>data</string> </key>
-            <value>
-              <dictionary/>
-            </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-  <record id="3" aka="AAAAAAAAAAM=">
-    <pickle>
-      <global name="PersistentMapping" module="Persistence.mapping"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>data</string> </key>
-            <value>
-              <dictionary>
-                <item>
-                    <key> <string>component_validation_workflow</string> </key>
-                    <value>
-                      <persistent> <string encoding="base64">AAAAAAAAAAQ=</string> </persistent>
-                    </value>
-                </item>
-              </dictionary>
-            </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-  <record id="4" aka="AAAAAAAAAAQ=">
-    <pickle>
-      <global name="WorkflowHistoryList" module="Products.ERP5Type.patches.WorkflowTool"/>
-    </pickle>
-    <pickle>
-      <tuple>
-        <none/>
-        <list>
-          <dictionary>
-            <item>
-                <key> <string>action</string> </key>
-                <value> <string>validate</string> </value>
-            </item>
-            <item>
-                <key> <string>validation_state</string> </key>
-                <value> <string>validated</string> </value>
-            </item>
-          </dictionary>
-        </list>
-      </tuple>
-    </pickle>
-  </record>
-</ZopeData>

bt5/officejs_test/bt/template_test_id_list

 test.erp5.testOfficeJSSecurity
 test.erp5.testOfficeJSScenario
-test.erp5.testOfficeJSScenarioRjsUI
 test.erp5.testFunctionalOfficeJSAppstorePublisherUI
 test.erp5.testFunctionalOfficeJSAppstoreSiteUI
 test.erp5.testOfficeJSScenarioAppstore

bt5/officejs_upgrader/SkinTemplateItem/portal_skins/officejs_upgrader/WebSection_checkAppstoreApplicationCachingPolicyConsistency.py

@@ -4,10 +4,10 @@ error_list = []
 person = web_section.getSourceValue(portal_type='Person')
 if (web_section.getParentId() == 'application-list') \
    and (web_section.getAggregate() == 'web_page_module/gadget_ojs_appstore_redirect_page_html') \
-   and (web_section.getCachingPolicy() == 'must-revalidate'):
-  error_list.append('"Must-revalidate" caching policy is deprecated')
+   and (web_section.getCachingPolicy() == 'one-hour-max-modification-date'):
+  error_list.append('"One-hour-max-modification-date" caching policy is deprecated')
 
   if fixit:
-    web_section.edit(caching_policy='one-hour-max-modification-date')
+    web_section.edit(caching_policy='must-revalidate')
 
 return error_list