app: default to full fieldlist on queries to enable querying

js/erp5_loader.js

@@ -4555,19 +4555,29 @@
    * Generate an array of fields from a dict
    * @method makeSelectList
    * @param {object} scheme of gadget to retrieve fields from
+   * @param {object} field_dict
    * @return {array} select_list
    **/
-  storage.makeSelectList = function (scheme) {
-    var i, j, field, select_list = [];
-
-    for (i = 0; i < scheme.length; i += 1) {
-      for (j = 0; j < scheme[i].field_list.length; j += 1) {
-        field = scheme[i].field_list[j];
-        if (field.field && select_list.indexOf(field.field) === -1) {
-          select_list.push(field.field);
+  storage.makeSelectList = function (scheme, field_dict) {
+    var i, j, field, prop, select_list = [];
+
+    if (scheme) {
+      for (i = 0; i < scheme.length; i += 1) {
+        for (j = 0; j < scheme[i].field_list.length; j += 1) {
+          field = scheme[i].field_list[j];
+          if (field.field && select_list.indexOf(field.field) === -1) {
+            select_list.push(field.field);
+          }
+        }
+      }
+    } else {
+      for (prop in field_dict) {
+        if (field_dict.hasOwnProperty(prop)) {
+          select_list.push(prop);
         }
       }
     }
+
     // if nothing is to be fetched, make sure we return at least the _id
     if (select_list.length === 0 && select_list.indexOf("_id") === -1) {
       select_list.push("_id");
@@ -6326,16 +6336,26 @@
 
       // only fetch columns based on gadget configuration
       } else {
-        if (pass.data_dict.field_dict &&
-              storage &&
-                app.storage_dict.property_dict.force_field_definitions &&
-                  pass.config_dict.scheme
-            ) {
+
+        // force definitions = we only get what we need
+        if (app.storage_dict.property_dict.force_field_definitions &&
+            pass.config_dict.scheme
+          ) {
           pass.data_dict.field_list = storage.makeSelectList(
             pass.config_dict.scheme
           );
+
+        // else get all columns, otherwise we cannot search later?
+        } else {
+          pass.data_dict.field_list = storage.makeSelectList(
+            undefined,
+            pass.data_dict.field_dict
+          );
         }
 
+
+        // we shouldn't have to regenerate the query here only because
+        // we now add the field_list
         pass.state.query = storage.parseQuery(
           pass.config_dict.initial_query,
           pass.config_dict.portal_type_source,