Changes for page Context Search
Last modified by bougault on 2023/02/20 16:29
Summary
-
Page properties (1 modified, 0 added, 0 removed)
Details
- Page properties
-
- Content
-
... ... @@ -1,7 +1,6 @@ 1 1 {{html clean="false"}} 2 2 <!-- 3 - !!! POC. Code must be reviewed / optimized and packaged for use in production !!! 4 - !!! Many optimisations can be done !!! 3 + !!! POC. Code must be reviewed / optimized and packaged for use in production !!! 5 5 --> 6 6 <script type="module"> 7 7 import {LitElement, html, css} from 'https://cdn.jsdelivr.net/gh/lit/dist@2/core/lit-core.min.js'; ... ... @@ -8,15 +8,21 @@ 8 8 window.addEventListener('DOMContentLoaded', function() { 9 9 10 10 class UnifiedSearch extends LitElement { 10 + /* 11 11 static properties = { 12 - results: { type: Array }, 13 - resultsCache: { type: Array }, 14 - collabFiltersResults: { type: Array} 12 + searchText: { type: String }, 13 + favoritesFilter: { type: Boolean }, 14 + administratorFilter: { type: Boolean }, 15 + editorFilter: { type: Boolean }, 16 + viewerFilter: { type: Boolean }, 17 + publicFilter: { type: Boolean }, 18 + privateFilter: { type: Boolean }, 15 15 } 20 + */ 21 + static properties = { 22 + results: { type: Array } 23 + } 16 16 static styles = css` 17 - :host > div { 18 - padding-bottom: 4em; 19 - } 20 20 .result { 21 21 margin-top: 1em; 22 22 } ... ... @@ -27,19 +27,6 @@ 27 27 a { 28 28 color: var(--color-brand-primary, blue); 29 29 } 30 - .rawResults { 31 - width: 100%; 32 - display: flex; 33 - font-size: 12px; 34 - margin-top: 2em; 35 - } 36 - .rawResults > div { 37 - width: 100%; 38 - } 39 - .rawResults ul { 40 - list-style-type: none; 41 - padding: 0; 42 - } 43 43 `; 44 44 constructor() { 45 45 super(); ... ... @@ -51,12 +51,7 @@ 51 51 this.rolesFilter = []; 52 52 this.filterUrl = new URL(`${document.location.origin}/rest/v1/collabs?search=&offset=0&orderField=title&order=asc&limit=10000&favorite=false&roles=`); 53 53 this.results = []; 54 - this.resultsCache = []; 55 - this.collabFiltersResults = []; 56 56 } 57 - firstUpdated() { 58 - this.renderRoot.getElementById('searchInput').focus(); 59 - } 60 60 toggleRoleFilter(roleName) { 61 61 if(this.rolesFilter.indexOf(roleName) === -1) { 62 62 this.rolesFilter.push(roleName); ... ... @@ -63,6 +63,7 @@ 63 63 } else { 64 64 this.rolesFilter = this.rolesFilter.filter(role => role !== roleName); 65 65 } 53 + console.log(this.rolesFilter); 66 66 this[roleName] = !this[roleName]; 67 67 this.filterUrl.searchParams.set('roles', this.rolesFilter.join("+")); 68 68 this.handleSearch(); ... ... @@ -70,26 +70,24 @@ 70 70 setSearchText(e) { 71 71 this.searchText = e.target.value; 72 72 } 73 - handleSearch(textSearch) { 74 - if(this.searchText !== "" || textSearch) { 61 + handleSearch() { 62 + console.log(this.rolesFilter) 63 + if(this.searchText !== "") { 75 75 if(this.searchText !== this.prevSearchText) { 76 - this. clearResults();65 + console.log(this.searchText, this.prevSearchText); 77 77 this.prevSearchText = this.searchText; 78 - this.fetchWikiSearch().then(results => {79 - this. resultsCache=results;80 - if(this.hasActiveFilters()) {81 - this.fetchFilterSearch().then(this.filterResults);82 - }else{83 - this.results=results;84 - }85 - })86 - }elseif(this.searchText===this.prevSearchText && this.hasActiveFilters()) {67 + Promise.all([ 68 + this.fetchWikiSearch(), 69 + this.fetchFilterSearch() 70 + ]).then(([wikiSearchResults, collabSearchResults]) => { 71 + this.results = wikiSearchResults; 72 + console.log(wikiSearchResults, collabSearchResults); 73 + }); 74 + } else if (this.searchText === this.prevSearchText) { 75 + console.log("Only apply filter on results"); 87 87 this.fetchFilterSearch().then(this.filterResults) 88 - } else { 89 - this.results = this.resultsCache; 90 90 } 91 91 } else { 92 - this.clearResults(); 93 93 console.log("no search text, should return filter search results if any"); 94 94 this.fetchFilterSearch().then(collabs => { 95 95 console.log("Format and display collabs", collabs); ... ... @@ -98,6 +98,7 @@ 98 98 } 99 99 hasActiveFilters() { 100 100 // we only look at role filters for POC. 87 + console.log('rolesfilter', this.rolesFilter); 101 101 return this.rolesFilter.length > 0; 102 102 } 103 103 fetchWikiSearch() { ... ... @@ -107,39 +107,27 @@ 107 107 fetchFilterSearch() { 108 108 return new Promise((resolve) => { 109 109 if(this.hasActiveFilters()) { 110 - fetch(this.filterUrl).then(res => res.json()).then((res) => { 111 - this.collabFiltersResults = res; 112 - resolve(res); 113 - }); 97 + fetch(this.filterUrl).then(res => res.json()).then(resolve); 114 114 } else { 115 - this.collabFiltersResults = res; 116 - resolve([]); 99 + resolve(null); 117 117 } 118 118 }); 119 119 } 120 120 filterResults = (filteredCollabs) => { 104 + console.log("filteredCollabs:", filteredCollabs, "search results:", this.results); 121 121 const collabSpaces = filteredCollabs.map(collab => `Collabs.${collab.name}`); 122 - this.results = this.resultsCache.filter(result => collabSpaces.some(space => result.space.startsWith(space))); 106 + const filteredResults = this.results.filter(result => collabSpaces.some(space => result.space.startsWith(space))); 107 + console.log(filteredResults); 123 123 } 124 - clearResults() { 125 - this.results = []; 126 - this.resultsCache = []; 127 - this.collabFiltersResults = []; 128 - } 129 129 getLink(xwikiDocSpace) { 130 130 return `/bin/view/${xwikiDocSpace.replaceAll('.', '/')}`; 131 131 } 132 - handleKeyPress(e) { 133 - if (e.keyCode === 13) { 134 - this.handleSearch(true); 135 - } 136 - } 137 137 render() { 138 138 return html` 139 139 <div> 140 140 <div> 141 - <input id="searchInput"@change="${this.setSearchText}" type="text" placeholder="Search..."@keypress="${this.handleKeyPress}"/>142 - <button @click="${ () =>this.handleSearch(true)}">Search</button>116 + <input @change="${this.setSearchText}" type="text" placeholder="Search..." /> 117 + <button @click="${this.handleSearch}">Search</button> 143 143 </div> 144 144 <div> 145 145 <label><input type="checkbox" @change="${() => this.toggleRoleFilter('administrator')}" .checked="${this.administratorFilter}"/> Administrator</label> ... ... @@ -155,16 +155,6 @@ 155 155 </div> 156 156 `)} 157 157 </div> 158 - <div class="rawResults"> 159 - <div> 160 - <strong>XWiki Search Results</strong> 161 - <ul>${this.resultsCache.map((res) => html`<li>${res.space}</li>`)}</ul> 162 - </div> 163 - <div> 164 - <strong>Collab Search Results (Filters)</strong> 165 - <ul>${this.collabFiltersResults.map((res) => html`<li>${res.name}</li>`)}</ul> 166 - </div> 167 - </div> 168 168 </div> 169 169 ` 170 170 } ... ... @@ -192,5 +192,7 @@ 192 192 } 193 193 }); 194 194 </script> 160 + 195 195 <clb-unified-search></clb-unified-search> 162 + 196 196 {{/html}}