59415: Browse-Entries empty response handling and doc fix

resources/i18n/en.json

@@ -293,7 +293,8 @@
         "author": "By Author",
         "subject": "By Subject"
       }
-    }
+    },
+    "empty": "No items to show."
   },
   "admin": {
     "registries": {

src/app/+browse-by/+browse-by-metadata-page/browse-by-metadata-page.component.ts

@@ -71,8 +71,8 @@ export class BrowseByMetadataPageComponent implements OnInit {
 
   /**
    * The value we're browing items for
-   * - When the value is not empty, we're browing items
-   * - When the value is empty, we're browing browse-entries (values for the given metadata definition)
+   * - When the value is not empty, we're browsing items
+   * - When the value is empty, we're browsing browse-entries (values for the given metadata definition)
    */
   value = '';
 

src/app/core/data/browse-entries-response-parsing.service.ts

@@ -30,10 +30,12 @@ export class BrowseEntriesResponseParsingService extends BaseResponseParsingServ
   }
 
   parse(request: RestRequest, data: DSpaceRESTV2Response): RestResponse {
-    if (isNotEmpty(data.payload) && isNotEmpty(data.payload._embedded)
-      && Array.isArray(data.payload._embedded[Object.keys(data.payload._embedded)[0]])) {
-      const serializer = new DSpaceRESTv2Serializer(BrowseEntry);
-      const browseEntries = serializer.deserializeArray(data.payload._embedded[Object.keys(data.payload._embedded)[0]]);
+    if (isNotEmpty(data.payload)) {
+      let browseEntries = [];
+      if (isNotEmpty(data.payload._embedded) && Array.isArray(data.payload._embedded[Object.keys(data.payload._embedded)[0]])) {
+        const serializer = new DSpaceRESTv2Serializer(BrowseEntry);
+        browseEntries = serializer.deserializeArray(data.payload._embedded[Object.keys(data.payload._embedded)[0]]);
+      }
       return new GenericSuccessResponse(browseEntries, data.statusCode, this.processPageInfo(data.payload));
     } else {
       return new ErrorResponse(

src/app/shared/browse-by/browse-by.component.html

@@ -7,6 +7,9 @@
       [objects]="objects">
     </ds-viewable-collection>
   </div>
-  <ds-loading *ngIf="!objects || objects?.payload?.page.length <= 0" message="{{'loading.browse-by' | translate}}"></ds-loading>
+  <ds-loading *ngIf="!objects || objects?.isLoading" message="{{'loading.browse-by' | translate}}"></ds-loading>
   <ds-error *ngIf="objects?.hasFailed" message="{{'error.browse-by' | translate}}"></ds-error>
+  <div *ngIf="!objects?.isLoading && objects?.payload?.page.length === 0" class="alert alert-info w-100" role="alert">
+    {{'browse.empty' | translate}}
+  </div>
 </ng-container>