Private (non-discoverable) Items missing 'noindex' meta tag in DSpace 7 and onwards

[ZahraaChreim-Atmire]

Description:
In DSpace 6, Private (non-discoverable) items used to include a
<meta name="robots" content="noindex"> tag (Issue #5346 / PR #2593) to prevent them from being indexed by search engines.

With the move to DSpace 7 and onwards, this behavior was lost. Private/non-discoverable items no longer output a noindex meta tag.

Expected:
Non-discoverable items should include <meta name="robots" content="noindex"> in the HTML.

Actual:
No such tag is added in the Angular UI.

[alanorth]

Well spotted @ZahraaChreim-Atmire. I think the fix is simple:

diff --git a/src/app/core/metadata/head-tag.service.ts b/src/app/core/metadata/head-tag.service.ts
index b83052083..0960ea4c8 100644
--- a/src/app/core/metadata/head-tag.service.ts
+++ b/src/app/core/metadata/head-tag.service.ts
@@ -194,6 +194,12 @@ export class HeadTagService {
       this.setCitationDissertationNameTag();
     }
 
+    if (this.currentObject.value instanceof Item) {
+      if (!this.currentObject.value.isDiscoverable) {
+        this.addMetaTag('robots', 'noindex');
+      }
+    }
+
     // this.setCitationJournalTitleTag();
     // this.setCitationVolumeTag();
     // this.setCitationIssueTag();

I don't know if the implementation should be any more complicated than that? Let me know what you think and I can submit a PR.

[ZahraaChreim-Atmire]

Well spotted @ZahraaChreim-Atmire. I think the fix is simple:

diff --git a/src/app/core/metadata/head-tag.service.ts b/src/app/core/metadata/head-tag.service.ts
index b83052083..0960ea4c8 100644
--- a/src/app/core/metadata/head-tag.service.ts
+++ b/src/app/core/metadata/head-tag.service.ts
@@ -194,6 +194,12 @@ export class HeadTagService {
this.setCitationDissertationNameTag();
}

• if (this.currentObject.value instanceof Item) {

•  if (!this.currentObject.value.isDiscoverable) {
  

•    this.addMetaTag('robots', 'noindex');
  

•  }
  

• }
• // this.setCitationJournalTitleTag();
  // this.setCitationVolumeTag();
  // this.setCitationIssueTag();
  I don't know if the implementation should be any more complicated than that? Let me know what you think and I can submit a PR.

@alanorth Thank you for your comment. That's 100% correct! I have already submitted the PR.