Added project shortname to filter. Resolves #1653.

DataBiosphere · Apr 20, 2021 · b9b21d5 · b9b21d5
1 parent 3e01802
commit b9b21d5
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/spa/src/app/files/_ngrx/files.effects.ts b/spa/src/app/files/_ngrx/files.effects.ts
@@ -248,6 +248,6 @@ export class FilesEffects {
      */
     private isAnyProjectSelected(searchTermsByFacetName: Map<string, Set<SearchTerm>>): boolean {
 
-        return searchTermsByFacetName.has(FileFacetName.PROJECT) || searchTermsByFacetName.has(FileFacetName.PROJECT_ID);
+        return searchTermsByFacetName.has(FileFacetName.PROJECT_ID);
     }
 }
diff --git a/spa/src/app/files/_ngrx/search/search.selectors.ts b/spa/src/app/files/_ngrx/search/search.selectors.ts
@@ -30,8 +30,7 @@ export const selectSelectedSearchTermsBySearchKey = createSelector(selectSearch,
 // Selected project search terms - returns the current set of selected projects
 export const selectSelectedProjectSearchTerms = createSelector(selectSearch, (state) => {
     return state.selectedSearchTerms.filter((searchTerm: SearchTerm) => {
-        return searchTerm.getSearchKey() === FileFacetName.PROJECT ||
-            searchTerm.getSearchKey() === FileFacetName.PROJECT_ID;
+            return searchTerm.getSearchKey() === FileFacetName.PROJECT_ID;
     });
 });
 

diff --git a/spa/src/app/files/entity/entity-request.service.ts b/spa/src/app/files/entity/entity-request.service.ts
@@ -109,7 +109,7 @@ export class EntityRequestService {
     }
 
     /**
-     * Remove project facet and/or project IDs from list of search terms as we do not want to restrict the table result
+     * Remove project ID facet from list of search terms as we do not want to restrict the table result
      * set to just the selected project.
      *
      * @param {Map<string, Set<SearchTerm>>} searchTermsByFacetName
@@ -120,7 +120,6 @@ export class EntityRequestService {
         searchTermsByFacetName: Map<string, Set<SearchTerm>>, selectedEntity: string): Map<string, Set<SearchTerm>> {
 
         const filteredSearchTerms = new Map(searchTermsByFacetName);
-        filteredSearchTerms.delete(FileFacetName.PROJECT);
         filteredSearchTerms.delete(FileFacetName.PROJECT_ID);
         return filteredSearchTerms;
     }

diff --git a/spa/src/app/files/facet/file-facet/file-facet-name-display.model.ts b/spa/src/app/files/facet/file-facet/file-facet-name-display.model.ts
@@ -9,6 +9,7 @@
 export enum FileFacetNameDisplay {
     "disease" = "Disease Status",
     "libraryConstructionApproach" = "Library Construction Method",
+    "project" = "Project Label",
     "projectId" = "Project",
     "sampleEntityType" = "Sample Type",
     "specimenOrgan" = "Organ",

diff --git a/spa/src/app/files/search/http/search-term-http.service.spec.ts b/spa/src/app/files/search/http/search-term-http.service.spec.ts
@@ -42,12 +42,12 @@ describe("SearchTermHttpService:", () => {
         /**
          * Confirm project facet is not added as a search term.
          */
-        it("excludes project facet from search terms", () => {
+        it("binds project facet in search terms", () => {
 
             const searchTerms = searchTermHttpService.bindSearchTerms(PROJECTS_ENTITY_API_RESPONSE.termFacets);
             expect(searchTerms).toBeTruthy();
             const projectSearchTerm = searchTerms.find((searchTerm) => searchTerm.facetName === FileFacetName.PROJECT);
-            expect(projectSearchTerm).toBeFalsy();
+            expect(projectSearchTerm).toBeTruthy();
         });
 
 

diff --git a/spa/src/app/files/search/http/search-term-http.service.ts b/spa/src/app/files/search/http/search-term-http.service.ts
@@ -40,6 +40,7 @@ export class SearchTermHttpService {
         "organismAgeValue",
         "pairedEnd",
         "preservationMethod",
+        "project",
         "projectTitle",
         "publicationTitle",
         "sampleDisease",