[erp5_web_renderjs_ui] Selecting nothing means all are selected

Same behaviour than xhtml style

bt5/erp5_web_renderjs_ui/PathTemplateItem/web_page_module/rjs_gadget_erp5_pt_form_list_js.js

@@ -205,25 +205,30 @@
 
     .allowPublicAcquisition("triggerListboxSelectAction", function triggerListboxSelectAction(argument_list) {
       var action = argument_list[0],
-        uid_list = argument_list[1],
+        checked_uid_list = argument_list[1],
+        unchecked_uid_list = argument_list[2],
         gadget = this,
         i,
         search_query,
         query_list = [];
       if ((action === 'include') || (action === 'exclude')) {
-        if (uid_list.length === 0) {
+        if (checked_uid_list.length === 0) {
+          // If nothing is checked, use all unchecked values (same as xhtml style)
+          checked_uid_list = unchecked_uid_list;
+        }
+        if (checked_uid_list.length === 0) {
           // XXX Queries do not correctly handle empty uid list
           return gadget.redirect({
             command: 'reload'
           });
         }
 
-        for (i = 0; i < uid_list.length; i += 1) {
+        for (i = 0; i < checked_uid_list.length; i += 1) {
           query_list.push(new SimpleQuery({
             key: "catalog.uid",
             type: "simple",
             operator: (action === 'include') ? "=" : "!=",
-            value: uid_list[i]
+            value: checked_uid_list[i]
           }));
         }
         if (gadget.state.extended_search) {

bt5/erp5_web_renderjs_ui/PathTemplateItem/web_page_module/rjs_gadget_erp5_pt_form_list_js.xml

@@ -230,7 +230,7 @@
             </item>
             <item>
                 <key> <string>serial</string> </key>
-                <value> <string>972.27121.4308.45824</string> </value>
+                <value> <string>972.27145.17258.43178</string> </value>
             </item>
             <item>
                 <key> <string>state</string> </key>
@@ -248,7 +248,7 @@
                     </tuple>
                     <state>
                       <tuple>
-                        <float>1545232470.9</float>
+                        <float>1545233919.1</float>
                         <string>UTC</string>
                       </tuple>
                     </state>

bt5/erp5_web_renderjs_ui_test/PathTemplateItem/portal_tests/renderjs_ui_page_templates_zuite/testFormListExcludeItemWithoutSelecting.zpt

@@ -100,7 +100,7 @@
 <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/wait_for_listbox_loaded" />
 <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/assert_listbox_not_selectable" />
 
-<tal:block tal:define="pagination_configuration python: {'header': '(1 - 3 / 10)', 'footer': 'Records 1 - 3 / 10'}">
+<tal:block tal:define="pagination_configuration python: {'header': '(1 - 3 / 7)', 'footer': 'Records 1 - 3 / 7'}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_listbox_pagination_text" />
 </tal:block>
 
@@ -122,18 +122,18 @@
 </tal:block>
 
 <!-- Check the listbox cell -->
-<tal:block tal:define="cell_configuration python: {'text': 'Title 0', 'line': 0, 'index': 1}">
+<tal:block tal:define="cell_configuration python: {'text': 'Title 3', 'line': 0, 'index': 1}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_listbox_cell_text" />
 </tal:block>
-<tal:block tal:define="cell_configuration python: {'text': 'Title 1', 'line': 1, 'index': 1}">
+<tal:block tal:define="cell_configuration python: {'text': 'Title 4', 'line': 1, 'index': 1}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_listbox_cell_text" />
 </tal:block>
-<tal:block tal:define="cell_configuration python: {'text': 'Title 2', 'line': 2, 'index': 1}">
+<tal:block tal:define="cell_configuration python: {'text': 'Title 5', 'line': 2, 'index': 1}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_listbox_cell_text" />
 </tal:block>
 
 <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/triggle_filter_and" />
-<tal:block tal:define="filter_section_configuration python: {'key': 'TEXT', 'value': 'title', 'index': 0}">
+<tal:block tal:define="filter_section_configuration python: {'key': 'TEXT', 'value': 'title', 'index': 3}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_filter_section" />
 </tal:block>
 

bt5/erp5_web_renderjs_ui_test/PathTemplateItem/portal_tests/renderjs_ui_page_templates_zuite/testFormListIncludeItemWithoutSelecting.zpt

@@ -100,7 +100,7 @@
 <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/wait_for_listbox_loaded" />
 <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/assert_listbox_not_selectable" />
 
-<tal:block tal:define="pagination_configuration python: {'header': '(1 - 3 / 10)', 'footer': 'Records 1 - 3 / 10'}">
+<tal:block tal:define="pagination_configuration python: {'header': '(3)', 'footer': '3 Records'}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_listbox_pagination_text" />
 </tal:block>
 
@@ -133,7 +133,7 @@
 </tal:block>
 
 <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/triggle_filter_and" />
-<tal:block tal:define="filter_section_configuration python: {'key': 'TEXT', 'value': 'title', 'index': 0}">
+<tal:block tal:define="filter_section_configuration python: {'key': 'TEXT', 'value': 'title', 'index': 1}">
   <tal:block metal:use-macro="here/Zuite_CommonTemplateForRenderjsUi/macros/check_filter_section" />
 </tal:block>