From bdcf3704e66aa991faf36f253faa1446125310df Mon Sep 17 00:00:00 2001
From: eduardohumberto <eduardo.humberto1992@gmail.com>
Date: Wed, 12 Sep 2018 22:51:19 -0300
Subject: [PATCH 1/8] endpoint facets - priority for selected values

---
 .../class-tainacan-rest-facets-controller.php | 118 +++++++++++++++---
 .../filter-types/checkbox/Checkbox.vue        |   4 +-
 .../filter-types/filter-types-mixin.js        |   8 +-
 .../filter-types/taxonomy/Checkbox.vue        |   2 +-
 4 files changed, 109 insertions(+), 23 deletions(-)

diff --git a/src/api/endpoints/class-tainacan-rest-facets-controller.php b/src/api/endpoints/class-tainacan-rest-facets-controller.php
index 2a9a01fbc..8f6d6e182 100644
--- a/src/api/endpoints/class-tainacan-rest-facets-controller.php
+++ b/src/api/endpoints/class-tainacan-rest-facets-controller.php
@@ -90,6 +90,8 @@ class REST_Facets_Controller extends REST_Controller {
 			$options = $metadatum->get_metadata_type_options();
 			$args = $this->prepare_filters($request);
 
+			// handle filter with relationship metadata
+
 			if( $metadatum_type === 'Tainacan\Metadata_Types\Relationship' ){
 
 				$selected =  $this->getRelationshipSelectedValues($request, $metadatum->get_id());
@@ -99,14 +101,13 @@ class REST_Facets_Controller extends REST_Controller {
 					$args['posts_per_page'] = $request['number'];
 				}
 
-				if( $selected ){
-					//$args['postin'] = $selected;
-				}
-
 				$items = $this->items_repository->fetch($args, $options['collection_id'], 'WP_Query');
+				$ids = [];
+
 				if ($items->have_posts()) {
 					while ( $items->have_posts() ) {
 						$items->the_post();
+						$ids[] = (string) $items->post->ID;
 		
 						$item = new Entities\Item($items->post);
 						$prepared_item = $restItemsClass->prepare_item_for_response($item, $request);
@@ -117,10 +118,34 @@ class REST_Facets_Controller extends REST_Controller {
 					wp_reset_postdata();
 				}
 
+				// retrieve selected items
+
+				if( $selected && $request['getSelected'] && $request['getSelected'] === '1' ){
+					foreach( $selected as $index => $item_id ){
+
+						if( in_array($item_id,$ids) ){
+							continue;
+						}
+
+						$item = new Entities\Item($item_id);
+						$prepared_item = $restItemsClass->prepare_item_for_response($item, $request);
+						$response[$index] = $prepared_item;
+
+						if( isset($request['number']) && ($index+1) >= $request['number']){
+							break;
+						}
+					}
+				}
+
 				$this->total_items  = $items->found_posts;
 				$this->total_pages = ceil($this->total_items / (int) $items->query_vars['posts_per_page']);
 
-			} else if ( $metadatum_type === 'Tainacan\Metadata_Types\Taxonomy' ){
+			} 
+
+			// handle filter with Taxonomy metadata
+			
+			else if ( $metadatum_type === 'Tainacan\Metadata_Types\Taxonomy' ){
+
 				$this->taxonomy = $this->taxonomy_repository->fetch($options['taxonomy_id']);
 				$selected = $this->getTaxonomySelectedValues($request, $options['taxonomy_id']);
 
@@ -131,22 +156,42 @@ class REST_Facets_Controller extends REST_Controller {
 
 				} else {
 
-					if( $selected ){
-						//$args['include'] = $selected;
+					$terms = $this->terms_repository->fetch($args, $this->taxonomy);
+
+					// retrieve selected items
+
+					if( $selected && $request['getSelected'] && $request['getSelected'] === '1' ){
+						$ids = $this->get_terms_ids( $terms );
+
+						foreach( $selected as $index => $term_id ){
+
+							if( in_array($term_id,$ids) ){
+								continue;
+							}
+
+							$term_selected = $this->terms_repository->fetch($term_id, $this->taxonomy);
+							array_unshift($terms, $term_selected);
+
+							if( isset($request['number']) && ($index+1) >= $request['number']){
+								break;
+							}
+						}
 					}
 
-					$terms = $this->terms_repository->fetch($args, $this->taxonomy);
 					$restTermClass = new REST_Terms_Controller();
 				}
 
-				
 				foreach ($terms as $term) {
 					array_push($response, $restTermClass->prepare_item_for_response( $term, $request ));
 				}
 
 				$this->set_pagination_properties_term_type( $args, $response );
 
-			} else {
+			} 
+			
+			// handle filter with Text metadata
+
+			else {
 
 				$metadatum_id = $metadatum->get_id();
 				$offset = '';
@@ -174,10 +219,25 @@ class REST_Facets_Controller extends REST_Controller {
 					}
 				}
 
-				if($selected){
-					//foreach( $selected as $value ){
-						//$response[] = ['mvalue' => $value];
-					//}
+				// retrieve selected items
+
+				if( $selected && $request['getSelected'] && $request['getSelected'] === '1'){
+					$rawValues = $this->get_values( $response );
+					
+					foreach( $selected as $index => $value ){
+
+						if( in_array($value,$rawValues) ){
+							continue;
+						}
+
+						$row = ['mvalue' => $value, 'metadatum_id' => $metadatum_id ];
+						$response[$index] = $row;
+
+						if( isset($request['number']) && ($index+1) >= $request['number']){
+							break;
+						}
+					}
+					
 				}
 
 				$this->set_pagination_properties_text_type( $collection_id, $metadatum_id, ($request['search']) ? $request['search'] : '' , $offset, $number );
@@ -343,7 +403,7 @@ class REST_Facets_Controller extends REST_Controller {
 		if( isset($request['current_query']['metaquery']) ){
 
 			foreach( $request['current_query']['metaquery'] as $metaquery ){
-				if( $metaquery['key'] === $metadatum_id ){
+				if( $metaquery['key'] == $metadatum_id ){
 
 					return $metaquery['value'];
 						
@@ -368,7 +428,7 @@ class REST_Facets_Controller extends REST_Controller {
 		if( isset($request['current_query']['metaquery']) ){
 
 			foreach( $request['current_query']['metaquery'] as $metaquery ){
-				if( $metaquery['key'] === $metadatum_id ){
+				if( $metaquery['key'] == $metadatum_id ){
 
 					return $metaquery['value'];
 						
@@ -379,6 +439,32 @@ class REST_Facets_Controller extends REST_Controller {
 
 		return [];
 	}
+
+	/**
+	 * 
+	 */
+	private function get_terms_ids( $terms ){
+		$ids = [];
+
+		foreach( $terms as $term ){
+            $ids[] = (string) $term->WP_Term->term_id;
+		}
+
+		return $ids;
+	}
+
+	/**
+	 * 
+	 */
+	private function get_values( $rows ){
+		$values = [];
+
+		foreach( $rows as $row ){
+            $values[] = $row['mvalue'];
+		}
+
+		return $values;
+	}
 }
 
 ?>
\ No newline at end of file
diff --git a/src/classes/filter-types/checkbox/Checkbox.vue b/src/classes/filter-types/checkbox/Checkbox.vue
index 01c2f2936..79b93549f 100644
--- a/src/classes/filter-types/checkbox/Checkbox.vue
+++ b/src/classes/filter-types/checkbox/Checkbox.vue
@@ -112,7 +112,7 @@
                     let collectionTarget = ( this.metadatum_object && this.metadatum_object.metadata_type_options.collection_id ) ?
                         this.metadatum_object.metadata_type_options.collection_id : this.collection_id;
 
-                    promise = this.getValuesRelationship( collectionTarget, null, [], 0, this.filter.max_options);
+                    promise = this.getValuesRelationship( collectionTarget, null, [], 0, this.filter.max_options, false, '1');
                     promise
                         .then(() => {
                             if(this.options.length > this.filter.max_options){
@@ -122,7 +122,7 @@
                             this.$console.error(error);
                     })
                 } else {
-                    promise = this.getValuesPlainText( this.metadatum, null, this.isRepositoryLevel, [], 0, this.filter.max_options );
+                    promise = this.getValuesPlainText( this.metadatum, null, this.isRepositoryLevel, [], 0, this.filter.max_options, false, '1' );
                     promise
                         .then(() => {
                             if(this.options.length > this.filter.max_options){
diff --git a/src/classes/filter-types/filter-types-mixin.js b/src/classes/filter-types/filter-types-mixin.js
index 828bd1181..b815fba35 100644
--- a/src/classes/filter-types/filter-types-mixin.js
+++ b/src/classes/filter-types/filter-types-mixin.js
@@ -18,10 +18,10 @@ export const filter_type_mixin = {
         query: {}
     },
     methods: {
-        getValuesPlainText(metadatumId, search, isRepositoryLevel, valuesToIgnore, offset, number, isInCheckboxModal) {
+        getValuesPlainText(metadatumId, search, isRepositoryLevel, valuesToIgnore, offset, number, isInCheckboxModal, getSelected = '0') {
             let query_items = { 'current_query': this.query };
 
-            let url = `/collection/${this.collection}/facets/${metadatumId}?`;
+            let url = `/collection/${this.collection}/facets/${metadatumId}?getSelected=${getSelected}&`;
 
             if(offset != undefined && number != undefined){
                 url += `offset=${offset}&number=${number}&`;
@@ -102,9 +102,9 @@ export const filter_type_mixin = {
                     this.$console.error(error);
                 });
         },
-        getValuesRelationship(collectionTarget, search, valuesToIgnore, offset, number, isInCheckboxModal) {
+        getValuesRelationship(collectionTarget, search, valuesToIgnore, offset, number, isInCheckboxModal, getSelected = '0') {
             let query_items = { 'current_query': this.query };
-            let url = '/collection/' + this.filter.collection_id + '/facets/' + this.filter.metadatum.metadatum_id + '?';
+            let url = '/collection/' + this.filter.collection_id + '/facets/' + this.filter.metadatum.metadatum_id + `?getSelected=${getSelected}&`;
 
             if(offset != undefined && number != undefined){
                 url += `offset=${offset}&number=${number}`;
diff --git a/src/classes/filter-types/taxonomy/Checkbox.vue b/src/classes/filter-types/taxonomy/Checkbox.vue
index f3b736746..468aa0b76 100644
--- a/src/classes/filter-types/taxonomy/Checkbox.vue
+++ b/src/classes/filter-types/taxonomy/Checkbox.vue
@@ -99,7 +99,7 @@
                 let query_items = { 'current_query': this.query };
 
                 axios.get('/collection/'+ this.collection +'/facets/' + this.metadatum 
-                + `?hideempty=0&order=asc&parent=0&number=${this.filter.max_options}&` + qs.stringify(query_items))
+                + `?getSelected=1&hideempty=0&order=asc&parent=0&number=${this.filter.max_options}&` + qs.stringify(query_items))
                     .then( res => {
 
                         for (let item of res.data) {

From 22c5737ccd478a0a9b9abe8f09b0439e88940200 Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Thu, 13 Sep 2018 09:48:08 -0300
Subject: [PATCH 2/8] Uses facets endpoint only to retrieve options on
 checkboxes list.

---
 .../other/checkbox-filter-modal.vue           | 50 +------------------
 .../filter-types/checkbox/Checkbox.vue        | 19 ++-----
 .../filter-types/taxonomy/Checkbox.vue        | 15 +-----
 3 files changed, 6 insertions(+), 78 deletions(-)

diff --git a/src/admin/components/other/checkbox-filter-modal.vue b/src/admin/components/other/checkbox-filter-modal.vue
index b91d1de24..342a87d2e 100644
--- a/src/admin/components/other/checkbox-filter-modal.vue
+++ b/src/admin/components/other/checkbox-filter-modal.vue
@@ -438,8 +438,6 @@
             applyFilter() {
                 this.$parent.close();
 
-                let selectedOptions = [];
-
                 if(this.isTaxonomy){
                     this.$eventBusSearch.$emit('input', {
                         filter: 'checkbox',
@@ -448,24 +446,7 @@
                         metadatum_id: this.metadatum_id,
                         collection_id: this.collection_id,
                         terms: this.selected
-                    });
-                    
-                    for (let selected of this.selected) {
-                        for(let i in this.finderColumns){
-                            let valueIndex = this.finderColumns[i].findIndex(option => option.value == selected);
-                            
-                            if (valueIndex >= 0) {
-                                selectedOptions.push(this.finderColumns[i][valueIndex]);
-                            }
-                        }
-                    }
-
-                    this.$eventBusSearch.$emit('sendValuesToTags', {
-                        filterId: this.filter.id,
-                        value: selectedOptions,
-                    });
-                    
-                    this.$emit('appliedCheckBoxModal', selectedOptions);
+                    });         
                 } else {
                     this.$eventBusSearch.$emit('input', {
                         filter: 'checkbox',
@@ -474,36 +455,9 @@
                         collection_id: this.collection_id ? this.collection_id : this.filter.collection_id,
                         value: this.selected,
                     });
-                     
-                    // if(!isNaN(this.selected[0])){
-                    //     for (let option of this.options) {
-                    //         let valueIndex = this.selected.findIndex(item => item == option.value);
-
-                    //         if (valueIndex >= 0) {
-                    //             selectedOptions.push(this.options[valueIndex].label);
-                    //         }
-                    //     }
-                    // }
-                    if(Array.isArray(this.selected)){
-                        for (let aSelected of this.selected) {
-                            let valueIndex = this.options.findIndex(option => option.value == aSelected);
-                            
-                            if (valueIndex >= 0) {
-                                selectedOptions.push(this.options[valueIndex]);
-                            }
-                        }
-                    } else {
-                        let valueIndex = this.options.findIndex(option => option.value == this.selected);
-                        
-                        if (valueIndex >= 0) {
-                            selectedOptions.push(this.options[valueIndex]);
-                        }
-                    }
-
-                    this.$emit('appliedCheckBoxModal', selectedOptions);
                 }
 
-                
+                this.$emit('appliedCheckBoxModal');
             }
         }
     }
diff --git a/src/classes/filter-types/checkbox/Checkbox.vue b/src/classes/filter-types/checkbox/Checkbox.vue
index 79b93549f..735ff1b01 100644
--- a/src/classes/filter-types/checkbox/Checkbox.vue
+++ b/src/classes/filter-types/checkbox/Checkbox.vue
@@ -1,7 +1,7 @@
 <template>
     <div class="block">
         <div
-                v-for="(option, index) in options"
+                v-for="(option, index) in options.slice(0, filter.max_options)"
                 :key="index"
                 class="metadatum">
             <b-checkbox
@@ -12,7 +12,7 @@
             </b-checkbox>
             <div
                     class="view-all-button-container"
-                    v-if="option.seeMoreLink && index == options.length - 1"
+                    v-if="option.seeMoreLink && index == options.slice(0, filter.max_options).length - 1"
                     @click="openCheckboxModal()"
                     v-html="option.seeMoreLink"/>
         </div>
@@ -200,22 +200,9 @@
                         query: this.query
                     },
                     events: {
-                        appliedCheckBoxModal: (options) => { this. appliedCheckBoxModal(options) }
+                        appliedCheckBoxModal: () => this.loadOptions()
                     }
                 });
-            },
-            appliedCheckBoxModal(options) {
-                this.options = this.options.concat(options)
-                for(let i = 0; i < this.options.length; ++i) {
-                    for(let j = i + 1; j < this.options.length; ++j) {
-                        if(this.options[i].value == this.options[j].value)
-                            this.options.splice(j--, 1);
-                    }
-                    if (i == this.options.length - 1)
-                        this.options[i].seeMoreLink = `<a style="font-size: 0.75rem;"> ${ this.$i18n.get('label_view_all') } </a>`;
-                }
-                
-                this.selectedValues();
             }
         }
     }
diff --git a/src/classes/filter-types/taxonomy/Checkbox.vue b/src/classes/filter-types/taxonomy/Checkbox.vue
index 468aa0b76..2532c547b 100644
--- a/src/classes/filter-types/taxonomy/Checkbox.vue
+++ b/src/classes/filter-types/taxonomy/Checkbox.vue
@@ -213,23 +213,10 @@
                         query: this.query
                     },                    
                     events: {
-                        appliedCheckBoxModal: (options) => { this. appliedCheckBoxModal(options) }
+                        appliedCheckBoxModal: () => this.loadOptions()
                     },
                     width: 'calc(100% - 8.333333333%)',
                 });
-            },
-            appliedCheckBoxModal(options) {
-                this.options = this.options.concat(options)
-                for(let i = 0; i < this.options.length; ++i) {
-                    for(let j = i + 1; j < this.options.length; ++j) {
-                        if(this.options[i].value == this.options[j].value)
-                            this.options.splice(j--, 1);
-                    }
-                    if (i == this.options.length - 1)
-                        this.options[i].seeMoreLink = `<a style="font-size: 0.75rem;"> ${ this.$i18n.get('label_view_all') } </a>`;
-                }
-                
-                this.selectedValues();
             }
         }
     }

From dfd2e5b650c22a13e2b8dc83bf43ed5a3c1fc10a Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Thu, 13 Sep 2018 10:04:56 -0300
Subject: [PATCH 3/8] Adds view mode dropdown back to repository level items on
 them.

---
 src/admin/js/theme-main.js                | 3 ++-
 src/admin/pages/lists/items-page.vue      | 2 +-
 src/admin/pages/lists/term-items-page.vue | 4 ++--
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/admin/js/theme-main.js b/src/admin/js/theme-main.js
index f10919575..b638a33ca 100644
--- a/src/admin/js/theme-main.js
+++ b/src/admin/js/theme-main.js
@@ -120,7 +120,8 @@ export const ThemeItemsListing =  new Vue({
 
         if (this.$el.attributes['enabled-view-modes'] != undefined)
             this.enabledViewModes = this.$el.attributes['enabled-view-modes'].value.split(',');
-
+        console.log(this.enabledViewModes)
+        console.log(this.defaultViewMode)
         if (this.$el.attributes['term-id'] != undefined)
             this.termId = this.$el.attributes['term-id'].value;
         if (this.$el.attributes['taxonomy'] != undefined)
diff --git a/src/admin/pages/lists/items-page.vue b/src/admin/pages/lists/items-page.vue
index 0ef94e3d1..79ed7a410 100644
--- a/src/admin/pages/lists/items-page.vue
+++ b/src/admin/pages/lists/items-page.vue
@@ -271,7 +271,7 @@
 
                 <!-- View Modes Dropdown -->
                 <div 
-                        v-if="isOnTheme && !isRepositoryLevel"
+                        v-if="isOnTheme"
                         class="search-control-item">
                     <b-field>
                         <b-dropdown
diff --git a/src/admin/pages/lists/term-items-page.vue b/src/admin/pages/lists/term-items-page.vue
index 03ca55668..f1961dbad 100644
--- a/src/admin/pages/lists/term-items-page.vue
+++ b/src/admin/pages/lists/term-items-page.vue
@@ -271,7 +271,7 @@
                 </div>
 
                 <!-- View Modes Dropdown -->
-                <!-- <div 
+                <div 
                         v-if="isOnTheme"
                         class="search-control-item">
                     <b-field>
@@ -372,7 +372,7 @@
                             </b-dropdown-item>
                         </b-dropdown>
                     </b-field>
-                </div> -->
+                </div>
 
                 <!-- Text simple search (used on mobile, instead of the one from filter list)-->
                 <div class="is-hidden-tablet search-control-item">

From 5bbb1bc10c4f63c177810bbb7d03ab774e4a1b0a Mon Sep 17 00:00:00 2001
From: Mateus Machado Luna <mateus.m.luna@gmail.com>
Date: Thu, 13 Sep 2018 10:20:03 -0300
Subject: [PATCH 4/8] Removes console log

---
 src/admin/js/theme-main.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/admin/js/theme-main.js b/src/admin/js/theme-main.js
index b638a33ca..f10919575 100644
--- a/src/admin/js/theme-main.js
+++ b/src/admin/js/theme-main.js
@@ -120,8 +120,7 @@ export const ThemeItemsListing =  new Vue({
 
         if (this.$el.attributes['enabled-view-modes'] != undefined)
             this.enabledViewModes = this.$el.attributes['enabled-view-modes'].value.split(',');
-        console.log(this.enabledViewModes)
-        console.log(this.defaultViewMode)
+
         if (this.$el.attributes['term-id'] != undefined)
             this.termId = this.$el.attributes['term-id'].value;
         if (this.$el.attributes['taxonomy'] != undefined)

From e6b19852d517814287ffb960d79812c8e74c90ad Mon Sep 17 00:00:00 2001
From: weryques <weryques@outlook.com>
Date: Thu, 13 Sep 2018 10:35:54 -0300
Subject: [PATCH 5/8] Now is not possible edit or open items on trash

---
 src/admin/components/lists/items-list.vue | 38 ++++++++++++++++-------
 src/admin/tainacan-admin-i18n.php         |  1 +
 2 files changed, 27 insertions(+), 12 deletions(-)

diff --git a/src/admin/components/lists/items-list.vue b/src/admin/components/lists/items-list.vue
index b84ad1ef9..c2caac3a0 100644
--- a/src/admin/components/lists/items-list.vue
+++ b/src/admin/components/lists/items-list.vue
@@ -40,7 +40,7 @@
                     </button>
 
                     <b-dropdown-item
-                            v-if="$route.params.collectionId && $userCaps.hasCapability('edit_others_posts')"
+                            v-if="$route.params.collectionId && $userCaps.hasCapability('edit_others_posts') && !isOnTrash"
                             @click="openBulkEditionModal()">
                         {{ $i18n.get('label_edit_selected_items') }}
                     </b-dropdown-item>
@@ -119,7 +119,8 @@
                             v-if="item.current_user_can_edit"
                             class="actions-area"
                             :label="$i18n.get('label_actions')">
-                        <a 
+                        <a
+                                v-if="!isOnTrash"
                                 id="button-edit"   
                                 :aria-label="$i18n.getFrom('items','edit_item')" 
                                 @click.prevent.stop="goToItemEditPage(item)">
@@ -203,7 +204,8 @@
                             v-if="item.current_user_can_edit"
                             class="actions-area"
                             :label="$i18n.get('label_actions')">
-                        <a 
+                        <a
+                                v-if="!isOnTrash"
                                 id="button-edit"   
                                 :aria-label="$i18n.getFrom('items','edit_item')" 
                                 @click.prevent.stop="goToItemEditPage(item)">
@@ -273,7 +275,8 @@
                             v-if="item.current_user_can_edit"
                             class="actions-area"
                             :label="$i18n.get('label_actions')">
-                        <a 
+                        <a
+                                v-if="!isOnTrash"
                                 id="button-edit"   
                                 :aria-label="$i18n.getFrom('items','edit_item')" 
                                 @click.prevent.stop="goToItemEditPage(item)">
@@ -388,8 +391,8 @@
                                     autoHide: false,
                                     placement: 'auto-start'
                                 }"
-                                v-for="(column, index) in tableMetadata"
-                                :key="index"
+                                v-for="(column, columnIndex) in tableMetadata"
+                                :key="columnIndex"
                                 v-if="collectionId != undefined && column.display && column.metadata_type_object != undefined && (column.metadata_type_object.related_mapped_prop == 'title')"
                                 @click="onClickItem($event, item, index)"
                                 v-html="item.metadata != undefined ? renderMetadata(item.metadata, column) : ''" />  
@@ -400,8 +403,8 @@
                                     autoHide: false,
                                     placement: 'auto-start'
                                 }"
-                                v-for="(column, index) in tableMetadata"
-                                :key="index"
+                                v-for="(column, columnIndex) in tableMetadata"
+                                :key="columnIndex"
                                 v-if="collectionId == undefined && column.display && column.metadata_type_object != undefined && (column.metadata_type_object.related_mapped_prop == 'title')"
                                 @click="onClickItem($event, item, index)"
                                 v-html="item.title != undefined ? item.title : ''" />                             
@@ -411,7 +414,8 @@
                             v-if="item.current_user_can_edit"
                             class="actions-area"
                             :label="$i18n.get('label_actions')">
-                        <a 
+                        <a
+                                v-if="!isOnTrash"
                                 id="button-edit"   
                                 :aria-label="$i18n.getFrom('items','edit_item')" 
                                 @click.prevent.stop="goToItemEditPage(item)">
@@ -607,7 +611,8 @@
                                 class="actions-cell"
                                 :label="$i18n.get('label_actions')">
                             <div class="actions-container">
-                                <a 
+                                <a
+                                        v-if="!isOnTrash"
                                         id="button-edit"   
                                         :aria-label="$i18n.getFrom('items','edit_item')" 
                                         @click.prevent.stop="goToItemEditPage(item)">
@@ -858,10 +863,19 @@ export default {
             });
         },
         onClickItem($event, item, index) {
-            if ($event.ctrlKey) {
+            if ($event.ctrlKey || $event.shiftKey) {
                 this.$set(this.selectedItems, index, !this.selectedItems[index]);
             } else {
-                this.$router.push(this.$routerHelper.getItemPath(item.collection_id, item.id));
+                if(this.isOnTrash){
+                    this.$toast.open({
+                        duration: 3000,
+                        message: this.$i18n.get('info_warning_remove_from_trash_first'),
+                        position: 'is-bottom',
+                        type: 'is-warning'
+                    });
+                } else {
+                    this.$router.push(this.$routerHelper.getItemPath(item.collection_id, item.id));
+                }
             }
         },
         goToItemEditPage(item) {
diff --git a/src/admin/tainacan-admin-i18n.php b/src/admin/tainacan-admin-i18n.php
index b889e6ad4..5627e3b6a 100644
--- a/src/admin/tainacan-admin-i18n.php
+++ b/src/admin/tainacan-admin-i18n.php
@@ -385,6 +385,7 @@ return apply_filters( 'tainacan-admin-i18n', [
 	'info_showing_taxonomies'                        => __( 'Showing taxonomies ', 'tainacan' ),
 	'info_showing_events'                            => __( 'Showing events ', 'tainacan' ),
 	'info_showing_processes'                         => __( 'Showing processes ', 'tainacan' ),
+	'info_warning_remove_from_trash_first'           => __( 'Remove this item from trash first'),
 	'info_to'                                        => __( ' to ', 'tainacan' ),
 	'info_of'                                        => __( ' of ', 'tainacan' ),
 	'info_created_by'                                => __( 'Created by: ', 'tainacan' ),

From 4974066fc182052ffc58155677baad4a37d07eec Mon Sep 17 00:00:00 2001
From: Leo Germani <leogermani@hacklab.com.br>
Date: Thu, 13 Sep 2018 14:05:28 -0300
Subject: [PATCH 6/8] fix hook to add extra metadata to items #96

---
 .../class-tainacan-rest-items-controller.php  | 31 +++++++++++--------
 1 file changed, 18 insertions(+), 13 deletions(-)

diff --git a/src/api/endpoints/class-tainacan-rest-items-controller.php b/src/api/endpoints/class-tainacan-rest-items-controller.php
index de30eabdd..66892f292 100644
--- a/src/api/endpoints/class-tainacan-rest-items-controller.php
+++ b/src/api/endpoints/class-tainacan-rest-items-controller.php
@@ -139,8 +139,24 @@ class REST_Items_Controller extends REST_Controller {
 	public function prepare_item_for_response( $item, $request ) {
 		if(!empty($item)){
 
+			/**
+			 * Use this filter to add additional post_meta to the api response
+			 * Use the $request object to get the context of the request and other variables
+			 * For example, id context is edit, you may want to add your meta or not.
+			 * 
+			 * Also take care to do any permissions verification before exposing the data
+			 */
+			$extra_metadata = apply_filters('tainacan-api-response-item-meta', [], $request);
+			$extra_metadata_values = [];
+
+			foreach ($extra_metadata as $extra_meta) {
+				$extra_metadata_values[$extra_meta] = get_post_meta($item->get_id(), $extra_meta, true);
+			}
+			
 			if(!isset($request['fetch_only'])) {
 				$item_arr = $item->_toArray();
+				
+				$item_arr = array_merge($extra_metadata_values, $item_arr);
 
 				if ( $request['context'] === 'edit' ) {
 					$item_arr['current_user_can_edit'] = $item->can_edit();
@@ -168,6 +184,8 @@ class REST_Items_Controller extends REST_Controller {
 			}
 
 			$item_arr = $this->filter_object_by_attributes($item, $attributes_to_filter);
+			
+			$item_arr = array_merge($extra_metadata_values, $item_arr);
 
 			if(is_array($attributes_to_filter) && array_key_exists('meta', $attributes_to_filter)){
 
@@ -183,19 +201,6 @@ class REST_Items_Controller extends REST_Controller {
 			$item_arr['url'] = get_permalink( $item_arr['id'] );
 			$item_arr['exposer_urls'] = \Tainacan\Exposers\Exposers::get_exposer_urls(get_rest_url(null, "{$this->namespace}/{$this->rest_base}/{$item->get_id()}/"));
 			
-			/**
-			 * Use this filter to add additional post_meta to the api response
-			 * Use the $request object to get the context of the request and other variables
-			 * For example, id context is edit, you may want to add your meta or not.
-			 * 
-			 * Also take care to do any permissions verification before exposing the data
-			 */
-			$extra_metadata = apply_filters('tainacan-api-response-item-meta', [], $request);
-
-			foreach ($extra_metadata as $extra_meta) {
-				$item_arr[$extra_meta] = get_post_meta($item_arr['id'], $extra_meta, true);
-			}
-			
 			return $item_arr;
 		}
 

From 108cce551cd4aa303e55a059175d4d7ebb82ce4a Mon Sep 17 00:00:00 2001
From: vnmedeiros <vnicius.nm.ba@gmail.com>
Date: Thu, 13 Sep 2018 17:07:32 -0300
Subject: [PATCH 7/8] add auto HTML link tag in text values #128 and HTML break
 line tag in textarea #116

---
 .../text/class-tainacan-text.php              | 25 +++++++-
 .../textarea/class-tainacan-textarea.php      | 25 +++++++-
 .../traits/class-tainacan-formatter-text.php  | 23 +++++++
 tests/test-item-metadata.php                  | 61 ++++++++++++++++++-
 4 files changed, 131 insertions(+), 3 deletions(-)
 create mode 100644 src/classes/traits/class-tainacan-formatter-text.php

diff --git a/src/classes/metadata-types/text/class-tainacan-text.php b/src/classes/metadata-types/text/class-tainacan-text.php
index 78fc0513b..da4540584 100644
--- a/src/classes/metadata-types/text/class-tainacan-text.php
+++ b/src/classes/metadata-types/text/class-tainacan-text.php
@@ -8,6 +8,7 @@ defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
  * Class TainacanMetadatumType
  */
 class Text extends Metadata_Type {
+	use \Tainacan\Traits\Formatter_Text;
 
     function __construct(){
         // call metadatum type constructor
@@ -26,5 +27,27 @@ class Text extends Metadata_Type {
                                item_id="'.$itemMetadata->get_item()->get_id().'"    
                                value=\''.json_encode( $itemMetadata->get_value() ).'\'  
                                name="'.$itemMetadata->get_metadatum()->get_name().'"></tainacan-text>';
-    }
+	}
+	
+	/**
+	 * Get the value as a HTML string with links
+	 * @return string
+	 */
+	public function get_value_as_html(\Tainacan\Entities\Item_Metadata_Entity $item_metadata) {
+		$value = $item_metadata->get_value();
+		$return = '';
+		if ( $item_metadata->is_multiple() ) {
+			$total = sizeof($value);
+			$count = 0;
+			foreach ( $value as $el ) {
+				$return .= $this->make_clickable_links($el);
+				$count ++;
+				if ($count <= $total)
+					$return .= ', ';
+			}
+		} else {
+			$return = $this->make_clickable_links($value);
+		}
+		return $return;
+	}
 }
\ No newline at end of file
diff --git a/src/classes/metadata-types/textarea/class-tainacan-textarea.php b/src/classes/metadata-types/textarea/class-tainacan-textarea.php
index 18997da3c..f1dfc4780 100644
--- a/src/classes/metadata-types/textarea/class-tainacan-textarea.php
+++ b/src/classes/metadata-types/textarea/class-tainacan-textarea.php
@@ -8,6 +8,7 @@ defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
  * Class TainacanMetadatumType
  */
 class Textarea extends Metadata_Type {
+	use \Tainacan\Traits\Formatter_Text;
 
     function __construct(){
         // call metadatum type constructor
@@ -21,11 +22,33 @@ class Textarea extends Metadata_Type {
      * @return string
      */
 
-    public function render( $itemMetadata ){
+    public function render( $itemMetadata ) {
         return '<tainacan-textarea 
                         metadatum_id ="'.$itemMetadata->get_metadatum()->get_id().'" 
                         item_id="'.$itemMetadata->get_item()->get_id().'"    
                         value=\''.json_encode( $itemMetadata->get_value() ).'\'  
                         name="'.$itemMetadata->get_metadatum()->get_name().'"></tainacan-textarea>';
     }
+
+	/**
+	 * Get the value as a HTML string with links and breakline tag.
+	 * @return string
+	 */
+	public function get_value_as_html(\Tainacan\Entities\Item_Metadata_Entity $item_metadata) {
+		$value = $item_metadata->get_value();
+		$return = '';
+		if ( $item_metadata->is_multiple() ) {
+			$total = sizeof($value);
+			$count = 0;
+			foreach ( $value as $el ) {
+				$return .= nl2br($this->make_clickable_links($el));
+				$count ++;
+				if ($count <= $total)
+					$return .= ', ';
+			}
+		} else {
+			$return = nl2br($this->make_clickable_links($value));
+		}
+		return $return;
+	}
 }
\ No newline at end of file
diff --git a/src/classes/traits/class-tainacan-formatter-text.php b/src/classes/traits/class-tainacan-formatter-text.php
new file mode 100644
index 000000000..5dcdfd53f
--- /dev/null
+++ b/src/classes/traits/class-tainacan-formatter-text.php
@@ -0,0 +1,23 @@
+<?php
+
+namespace Tainacan\Traits;
+
+defined( 'ABSPATH' ) or die( 'No script kiddies please!' );
+
+/**
+ * Class for formatter texts
+ * @author Vinicius Nunus - L3P/Medialab
+ *
+ */
+trait Formatter_Text {
+
+	/**
+	 * 
+	 * @return string Texts with url's transformed in html tag <a>
+	 */
+	public function make_clickable_links($text) {
+		$url = '~(?:(http|https|ftp|ftps)://([^\s<]+)|(www\.[^\s<]+?\.[^\s<]+))(?<![\.,:])~i';
+		$text = preg_replace($url, '<a href="$0" target="_blank" title="$0">$0</a>', $text);
+		return $text;
+	}
+}
\ No newline at end of file
diff --git a/tests/test-item-metadata.php b/tests/test-item-metadata.php
index b999ca1c9..289346ed3 100644
--- a/tests/test-item-metadata.php
+++ b/tests/test-item-metadata.php
@@ -243,5 +243,64 @@ class Item_Metadata extends TAINACAN_UnitTestCase {
         //first 2 metadata are repository metadata
         $this->assertTrue( in_array('metadado', $names) );
         
-    }
+	}
+	
+	function test_metadata_text_textarea() {
+		$Tainacan_Metadata = \Tainacan\Repositories\Metadata::get_instance();
+
+        $collection = $this->tainacan_entity_factory->create_entity(
+	        'collection',
+	        array(
+		        'name' => 'teste'
+	        ),
+	        true
+		);
+		
+		$i = $this->tainacan_entity_factory->create_entity(
+		    'item',
+		    array(
+			    'title'       => 'item teste',
+			    'description' => 'description',
+			    'collection'  => $collection,
+			    'status'      => 'publish'
+		    ),
+		    true
+	    );
+
+	    $metadatum_text = $this->tainacan_entity_factory->create_entity(
+	        'metadatum',
+	        array(
+		        'name'              => 'metadadoText',
+		        'description'       => 'descricao',
+		        'collection_id'     => $collection->get_id(),
+		        'metadata_type'  => 'Tainacan\Metadata_Types\Text',
+	        ),
+	        true
+		);
+		
+		$metadatum_textarea = $this->tainacan_entity_factory->create_entity(
+	        'metadatum',
+	        array(
+		        'name'              => 'metadadoTextarea',
+		        'description'       => 'descricao',
+		        'collection_id'     => $collection->get_id(),
+		        'metadata_type'  => 'Tainacan\Metadata_Types\Textarea',
+	        ),
+	        true
+		);
+		
+		$value_text = 'GOOGLE: www.google.com';
+		$item_metadata_text = new \Tainacan\Entities\Item_Metadata_Entity($i, $metadatum_text);
+		$item_metadata_text->set_value($value_text);
+		
+		$value_textarea = 'GOOGLE: www.google.com \n GOOGLE: https://www.google.com';
+		$item_metadata_textarea = new \Tainacan\Entities\Item_Metadata_Entity($i, $metadatum_textarea);
+		$item_metadata_textarea->set_value($value_textarea);
+
+		$response_text = 'GOOGLE: <a href="www.google.com" target="_blank" title="www.google.com">www.google.com</a>';
+		$response_textarea = 'GOOGLE: <a href="www.google.com" target="_blank" title="www.google.com">www.google.com</a> \n GOOGLE: <a href="https://www.google.com" target="_blank" title="https://www.google.com">https://www.google.com</a>';
+
+		$this->assertEquals($item_metadata_text->get_value_as_html(), $response_text);
+		$this->assertEquals($item_metadata_textarea->get_value_as_html(), $response_textarea);
+	}
 }
\ No newline at end of file

From 8036b7e8c143b5bfcb3db559af3c1486c991ef8c Mon Sep 17 00:00:00 2001
From: eduardohumberto <eduardo.humberto1992@gmail.com>
Date: Thu, 13 Sep 2018 22:37:07 -0300
Subject: [PATCH 8/8] test importer - verify connection

---
 tests/test-importer.php | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/tests/test-importer.php b/tests/test-importer.php
index 1a9a59392..93cde402b 100644
--- a/tests/test-importer.php
+++ b/tests/test-importer.php
@@ -500,12 +500,17 @@ class ImporterTests extends TAINACAN_UnitTestCase {
         $this->assertFalse( is_numeric($document_id) );
 
         $attachments = $items[0]->get_attachments();
-        $this->assertEquals( 1, count( $attachments ) );
 
-        $this->assertTrue( count($items[2]->get_attachments()) > 0 );
+        if(@file_get_contents ( 'https://www.w3schools.com/w3css/img_lights.jpg' ))
+            $this->assertEquals( 1, count( $attachments ) );
+
+        if(@file_get_contents ( 'https://www.youtube.com/watch?v=V8dpmD4HG5s&start_radio=1&list=RDEMZS6OrHEAut8dOA38mVtVpg' ))
+            $this->assertTrue( count($items[2]->get_attachments()) > 0 );
 
         $document_id = $items[2]->get_document();
-        $this->assertTrue( is_numeric($document_id) );
+
+        if(@file_get_contents ( 'https://www.codeproject.com/KB/GDI-plus/ImageProcessing2/img.jpg' ))
+            $this->assertTrue( is_numeric($document_id) );
 
         $document = $items[3]->get_document();
         $this->assertTrue( wp_http_validate_url($document) !== false );