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,16 +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 ul { 37 - list-style-type: none; 38 - padding: 0; 39 - } 40 40 `; 41 41 constructor() { 42 42 super(); ... ... @@ -48,18 +48,15 @@ 48 48 this.rolesFilter = []; 49 49 this.filterUrl = new URL(`${document.location.origin}/rest/v1/collabs?search=&offset=0&orderField=title&order=asc&limit=10000&favorite=false&roles=`); 50 50 this.results = []; 51 - this.resultsCache = []; 52 - this.collabFiltersResults = []; 53 53 } 54 - firstUpdated() { 55 - this.renderRoot.getElementById('searchInput').focus(); 56 - } 57 57 toggleRoleFilter(roleName) { 48 + console.log("toggleRoleFilter", roleName, this.rolesFilter, this.rolesFilter.indexOf(roleName)); 58 58 if(this.rolesFilter.indexOf(roleName) === -1) { 59 59 this.rolesFilter.push(roleName); 60 60 } else { 61 - this.rolesFilter = this.rolesFilter.filter(role => role !== roleName);52 + this.rolesFilter.filter(role => role !== roleName); 62 62 } 54 + console.log(this.rolesFilter); 63 63 this[roleName] = !this[roleName]; 64 64 this.filterUrl.searchParams.set('roles', this.rolesFilter.join("+")); 65 65 this.handleSearch(); ... ... @@ -67,26 +67,24 @@ 67 67 setSearchText(e) { 68 68 this.searchText = e.target.value; 69 69 } 70 - handleSearch(textSearch) { 71 - if(this.searchText !== "" || textSearch) { 62 + handleSearch() { 63 + console.log(this.rolesFilter) 64 + if(this.searchText !== "") { 72 72 if(this.searchText !== this.prevSearchText) { 73 - this. clearResults();66 + console.log(this.searchText, this.prevSearchText); 74 74 this.prevSearchText = this.searchText; 75 - this.fetchWikiSearch().then(results => {76 - this. resultsCache=results;77 - if(this.hasActiveFilters()) {78 - this.fetchFilterSearch().then(this.filterResults);79 - }else{80 - this.results=results;81 - }82 - })83 - }elseif(this.searchText===this.prevSearchText && this.hasActiveFilters()) {68 + Promise.all([ 69 + this.fetchWikiSearch(), 70 + this.fetchFilterSearch() 71 + ]).then(([wikiSearchResults, collabSearchResults]) => { 72 + this.results = wikiSearchResults; 73 + console.log(wikiSearchResults, collabSearchResults); 74 + }); 75 + } else if (this.searchText === this.prevSearchText) { 76 + console.log("Only apply filter on results"); 84 84 this.fetchFilterSearch().then(this.filterResults) 85 - } else { 86 - this.results = this.resultsCache; 87 87 } 88 88 } else { 89 - this.clearResults(); 90 90 console.log("no search text, should return filter search results if any"); 91 91 this.fetchFilterSearch().then(collabs => { 92 92 console.log("Format and display collabs", collabs); ... ... @@ -95,6 +95,7 @@ 95 95 } 96 96 hasActiveFilters() { 97 97 // we only look at role filters for POC. 88 + console.log('rolesfilter', this.rolesFilter); 98 98 return this.rolesFilter.length > 0; 99 99 } 100 100 fetchWikiSearch() { ... ... @@ -104,37 +104,27 @@ 104 104 fetchFilterSearch() { 105 105 return new Promise((resolve) => { 106 106 if(this.hasActiveFilters()) { 107 - fetch(this.filterUrl).then(res => res.json()).then((res) => { 108 - this.collabFiltersResults = res; 109 - resolve(res); 110 - }); 98 + fetch(this.filterUrl).then(res => res.json()).then(resolve); 111 111 } else { 112 - resolve( []);100 + resolve(null); 113 113 } 114 114 }); 115 115 } 116 116 filterResults = (filteredCollabs) => { 105 + console.log("filteredCollabs:", filteredCollabs, "search results:", this.results); 117 117 const collabSpaces = filteredCollabs.map(collab => `Collabs.${collab.name}`); 118 - this.results = this.resultsCache.filter(result => collabSpaces.some(space => result.space.startsWith(space))); 107 + const filteredResults = this.results.filter(result => collabSpaces.some(space => result.space.startsWith(space))); 108 + console.log(filteredResults); 119 119 } 120 - clearResults() { 121 - this.results = []; 122 - this.resultsCache = []; 123 - } 124 124 getLink(xwikiDocSpace) { 125 125 return `/bin/view/${xwikiDocSpace.replaceAll('.', '/')}`; 126 126 } 127 - handleKeyPress(e) { 128 - if (e.keyCode === 13) { 129 - this.handleSearch(true); 130 - } 131 - } 132 132 render() { 133 133 return html` 134 134 <div> 135 135 <div> 136 - <input id="searchInput"@change="${this.setSearchText}" type="text" placeholder="Search..."@keypress="${this.handleKeyPress}"/>137 - <button @click="${ () =>this.handleSearch(true)}">Search</button>117 + <input @change="${this.setSearchText}" type="text" placeholder="Search..." /> 118 + <button @click="${this.handleSearch}">Search</button> 138 138 </div> 139 139 <div> 140 140 <label><input type="checkbox" @change="${() => this.toggleRoleFilter('administrator')}" .checked="${this.administratorFilter}"/> Administrator</label> ... ... @@ -150,16 +150,6 @@ 150 150 </div> 151 151 `)} 152 152 </div> 153 - <div class="rawResults"> 154 - <div> 155 - <strong>XWiki Search Results</strong> 156 - <ul>${this.resultsCache.map((res) => html`<li>${res.space}</li>`)}</ul> 157 - </div> 158 - <div> 159 - <strong>Collab Search Results (Filters)</strong> 160 - <ul>${this.collabFiltersResults.map((res) => html`<li>${res.name}</li>`)}</ul> 161 - </div> 162 - </div> 163 163 </div> 164 164 ` 165 165 } ... ... @@ -187,5 +187,7 @@ 187 187 } 188 188 }); 189 189 </script> 161 + 190 190 <clb-unified-search></clb-unified-search> 163 + 191 191 {{/html}}