From f6ad2f32756047fb8af4597cea841800577e402c Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Tue, 4 Sep 2018 16:25:48 -0300
Subject: [PATCH] Prevents breaking form hooks when theme is not updated. Bulk
 edition second select now automatically selects when only one option is
 available.

---
 .../components/bulk-edition/bulk-edition-modal.vue   | 12 +++++++++++-
 src/admin/js/mixins.js                               |  4 ++--
 src/admin/js/theme-main.js                           |  5 ++---
 src/admin/theme-items-list.vue                       |  1 -
 4 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/admin/components/bulk-edition/bulk-edition-modal.vue b/src/admin/components/bulk-edition/bulk-edition-modal.vue
index 88aea6397..b3364819a 100644
--- a/src/admin/components/bulk-edition/bulk-edition-modal.vue
+++ b/src/admin/components/bulk-edition/bulk-edition-modal.vue
@@ -25,11 +25,13 @@
                         <template 
                                 v-for="(metadatum, index) in metadata">
                             <option
+                                    :key="index"
                                     v-if="metadatum.id"
                                     :value="metadatum.id">
                                 {{ metadatum.name }}
                             </option>
                             <option
+                                    :key="index"
                                     v-if="index === Object.keys(metadata).length-1"
                                     value="status">
                                 {{ $i18n.get('label_status') }}
@@ -42,6 +44,7 @@
                             v-if="bulkEditionProcedures[criterion] &&
                             bulkEditionProcedures[criterion].metadatumID"
                             :disabled="!!bulkEditionProcedures[criterion].action"
+                            :value="bulkEditionProcedures[criterion].action ? bulkEditionProcedures[criterion].action : undefined"
                             class="tainacan-bulk-edition-field tainacan-bulk-edition-field-not-last"
                             :placeholder="$i18n.get('instruction_select_a_action')"
                             @input="addToBulkEditionProcedures($event, 'action', criterion)">
@@ -57,7 +60,6 @@
                             <option
                                     v-for="(edtAct, key) in editionActionsForNotMultiple"
                                     :value="edtAct"
-                                    :selected="Object.keys(editionActionsForNotMultiple).length === 1"
                                     :key="key">
                                 {{ edtAct }}
                             </option>
@@ -533,11 +535,19 @@
                 return found ? found : {};
             },
             addToBulkEditionProcedures(value, key, criterion){
+
                 if(Array.isArray(value)){
                     value = value[0];
                 }
 
                 this.$set(this.bulkEditionProcedures[criterion], `${key}`, value);
+                
+                if (key == 'metadatumID') {
+                    if (this.getMetadataByID(this.bulkEditionProcedures[criterion].metadatumID).multiple != 'yes') {
+                        let value = Object.values(this.editionActionsForNotMultiple)[0];
+                        this.addToBulkEditionProcedures(value, 'action', criterion);
+                    }
+                }
             }
         },
     }
diff --git a/src/admin/js/mixins.js b/src/admin/js/mixins.js
index 1565d3aaa..23bd09cb4 100644
--- a/src/admin/js/mixins.js
+++ b/src/admin/js/mixins.js
@@ -65,7 +65,7 @@ export const formHooks = {
             ];
             // Gets data from existing extra form hooks
             for (let position of positions) {
-                if (this.formHooks[entity][position] && this.formHooks[entity][position] != undefined) {
+                if (this.formHooks[entity] && this.formHooks[entity][position] && this.formHooks[entity][position] != undefined) {
                     let formElement = document.getElementById('form-' + entity + '-' + position);
                     if (formElement) {  
                         for (let element of formElement.elements) {
@@ -95,7 +95,7 @@ export const formHooks = {
             ];
             // Gets data from existing extra form hooks
             for (let position of positions) {
-                if (this.formHooks[entity][position] && this.formHooks[entity][position] != undefined) {
+                if (this.formHooks[entity] && this.formHooks[entity][position] && this.formHooks[entity][position] != undefined) {
                     let formElement = document.getElementById('form-' + entity + '-' + position);
                     
                     if (formElement) {   
diff --git a/src/admin/js/theme-main.js b/src/admin/js/theme-main.js
index 9bdfaa462..35182d981 100644
--- a/src/admin/js/theme-main.js
+++ b/src/admin/js/theme-main.js
@@ -110,9 +110,8 @@ export const ThemeItemsListing =  new Vue({
     },
     render: h => h(ThemeItemsList),
     beforeMount () {
-        console.log("oie")
-        //if (this.$el.attributes['collection-id'] != undefined)
-        this.collectionId = this.$el.attributes['collection-id'].value;
+        if (this.$el.attributes['collection-id'] != undefined)
+            this.collectionId = this.$el.attributes['collection-id'].value;
         if (this.$el.attributes['default-view-mode'] != undefined)
             this.defaultViewMode = this.$el.attributes['default-view-mode'].value;
         if (this.$el.attributes['enabled-view-modes'] != undefined)
diff --git a/src/admin/theme-items-list.vue b/src/admin/theme-items-list.vue
index d443bd966..b99225cf1 100644
--- a/src/admin/theme-items-list.vue
+++ b/src/admin/theme-items-list.vue
@@ -20,7 +20,6 @@
 export default {
     name: "ThemeItemsList",
     created() {
-        console.log("oi")
         this.$userPrefs.init(); 
     }
 }