Fix: Sort fields by sequence (adempiere#650)

Author: elsiosanchez

src/components/ADempiere/DataTable/Components/BatchEntry.vue

@@ -35,31 +35,30 @@
             size="small"
             class="field-component-bacht-entry"
           >
-            <template
+            <el-col
               v-for="(field, key) in fieldsListBatchEntry"
+              :key="key"
+              :span="6"
             >
-              <el-col :key="key" :span="6">
-                <el-form-item
-                  :label="field.name"
-                  :required="field.isMandatory"
-                  style="padding: 0px !important;"
-                  class="label-field-title"
-                >
-                  <fields-display-definitions
-                    ref="fieldsDisplay"
-                    :field="field"
-                    :update-field="updateField"
-                    :persistence-data="persistenceBachtEntry"
-                    :is-new-record="true"
-                    :field-list="fieldsListBatchEntry"
-                    :is-panel-general="true"
-                    :is-read-only="!isCreateRecord"
-                    :container-manager="containerManager"
-                    :is-value-bacht-entry="attributesBachtEntry[field.column_name]"
-                  />
-                </el-form-item>
-              </el-col>
-            </template>
+              <el-form-item
+                :label="field.name"
+                style="padding: 0px !important;"
+                class="label-field-title"
+              >
+                <fields-display-definitions
+                  ref="fieldsDisplay"
+                  :field="field"
+                  :update-field="updateField"
+                  :persistence-data="persistenceBachtEntry"
+                  :is-new-record="true"
+                  :field-list="fieldsListBatchEntry"
+                  :is-panel-general="true"
+                  :is-read-only="!isCreateRecord"
+                  :container-manager="containerManager"
+                  :is-value-bacht-entry="attributesBachtEntry[field.column_name]"
+                />
+              </el-form-item>
+            </el-col>
           </el-form>
         </el-col>
         <el-col :span="24">
@@ -220,7 +219,7 @@ export default defineComponent({
       })
     })
 
-    fieldsListBatchEntry.value = fieldsList.value.filter(fieldAttributes => fieldAttributes.is_quick_entry)
+    fieldsListBatchEntry.value = fieldsList.value.filter(fieldAttributes => fieldAttributes.is_quick_entry).sort((a, b) => a.sequence - b.sequence)
 
     const defaultValues = computed(() => {
       const isSalesTransactionContext = isSalesTransaction({

src/components/ADempiere/FieldsDisplayDefinitionsBachtEntry/FieldNumber.vue

@@ -24,7 +24,6 @@
     :placeholder="fieldMetadata.description"
     size="mini"
     :precision="precision"
-    class="field-number"
     style="width: 100%;"
     :disabled="isReadOnly"
     @change="saveFieldValue(fieldMetadata.value, fieldMetadata)"

src/components/ADempiere/FieldsDisplayDefinitionsBachtEntry/FieldSelect.vue

@@ -28,7 +28,7 @@
     :placeholder="fieldMetadata.description"
     :remote-method="remoteMethod"
     :loading="isLoadingSearch"
-    style="padding-right: 10px;width: 100%;"
+    style="width: 100%;"
     :disabled="isReadOnly"
     @visible-change="showList"
     @change="saveFieldValue"

src/components/ADempiere/FieldsDisplayDefinitionsBachtEntry/FieldText.vue

@@ -25,7 +25,7 @@
     :rows="rowsField"
     :type="typeTextBox"
     :disabled="isReadOnly"
-    style="padding-right: 10px;width: 100%;"
+    style="width: 100%;"
     @input="saveFieldValue(fieldMetadata.value, fieldMetadata)"
   />
 </template>