Merge pull request #4838 from atmire/w2p-136225_fix-upstream-private-items-issue-11569-8_x

tdonohue · web-flow · commit 19924302bca2 · 2025-11-19T15:52:31.000-06:00
Add noindex robots meta tag for non-discoverable items (8_x)
diff --git a/src/app/core/metadata/head-tag.service.spec.ts b/src/app/core/metadata/head-tag.service.spec.ts
@@ -24,6 +24,7 @@ import {
   MockBitstream1,
   MockBitstream2,
   MockBitstream3,
+  NonDiscoverableItemMock,
 } from '../../shared/mocks/item.mock';
 import { getMockTranslateService } from '../../shared/mocks/translate.service.mock';
 import {
@@ -128,6 +129,37 @@ describe('HeadTagService', () => {
     );
   });
 
+  describe(`robots tag`, () => {
+    it(`should be set to noindex for non-discoverable items`, fakeAsync(() => {
+      (headTagService as any).processRouteChange({
+        data: {
+          value: {
+            dso: createSuccessfulRemoteDataObject(NonDiscoverableItemMock),
+          },
+        },
+      });
+      tick();
+      expect(meta.addTag).toHaveBeenCalledWith({
+        name: 'robots',
+        content: 'noindex',
+      });
+    }));
+    it(`should not be set for discoverable items`, fakeAsync(() => {
+      (headTagService as any).processRouteChange({
+        data: {
+          value: {
+            dso: createSuccessfulRemoteDataObject(ItemMock),
+          },
+        },
+      });
+      tick();
+      expect(meta.addTag).not.toHaveBeenCalledWith({
+        name: 'robots',
+        content: 'noindex',
+      });
+    }));
+  });
+
   it('items page should set meta tags', fakeAsync(() => {
     (headTagService as any).processRouteChange({
       data: {
diff --git a/src/app/core/metadata/head-tag.service.ts b/src/app/core/metadata/head-tag.service.ts
@@ -173,6 +173,8 @@ export class HeadTagService {
 
   protected setDSOMetaTags(): void {
 
+    this.setNoIndexTag();
+
     this.setTitleTag();
     this.setDescriptionTag();
 
@@ -210,6 +212,15 @@ export class HeadTagService {
 
   }
 
+  /**
+   * Add <meta name="robots" content="noindex">  to the <head> if non-discoverable item
+   */
+  protected setNoIndexTag(): void {
+    if (this.currentObject.value instanceof Item && this.currentObject.value.isDiscoverable === false) {
+      this.addMetaTag('robots', 'noindex');
+    }
+  }
+
   /**
    * Add <meta name="title" ... >  to the <head>
    */
diff --git a/src/app/shared/mocks/item.mock.ts b/src/app/shared/mocks/item.mock.ts
@@ -293,4 +293,55 @@ export const ItemMock: Item = Object.assign(new Item(), {
   },
   ),
 });
+
+export const NonDiscoverableItemMock: Item = Object.assign(new Item(), {
+  handle: '10673/7',
+  lastModified: '2017-04-24T19:44:08.178+0000',
+  isArchived: true,
+  isDiscoverable: false,
+  isWithdrawn: false,
+  bundles: createSuccessfulRemoteDataObject$(createPaginatedList([
+    MockOriginalBundle,
+  ])),
+  _links:{
+    self: {
+      href: 'https://dspace7.4science.it/dspace-spring-rest/api/core/items/0ec7ff22-f211-40ab-a69e-c819b0b1f358',
+    },
+  },
+  id: '0ec7ff22-f211-40ab-a69e-c819b0b1f358',
+  uuid: '0ec7ff22-f211-40ab-a69e-c819b0b1f358',
+  type: 'item',
+  metadata: {
+    'dc.date.accessioned': [
+      {
+        language: null,
+        value: '1650-06-26T19:58:25Z',
+      },
+    ],
+    'dc.date.available': [
+      {
+        language: null,
+        value: '1650-06-26T19:58:25Z',
+      },
+    ],
+    'dc.date.issued': [
+      {
+        language: null,
+        value: '1650-06-26',
+      },
+    ],
+    'dc.identifier.uri': [
+      {
+        language: null,
+        value: 'http://dspace7.4science.it/xmlui/handle/10673/7',
+      },
+    ],
+    'dc.title': [
+      {
+        language: 'en_US',
+        value: 'Test Non-Discoverable',
+      },
+    ],
+  },
+});
 /* eslint-enable @typescript-eslint/no-shadow */
