Merge pull request #76 from jonnitto/feature/noindexmixin

FEATURE: Add NoindexMixin
neos · Nov 13, 2018 · 7a939cb · 7a939cb
2 parents 380f7f5 + 11374cf
commit 7a939cb
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 1 deletion.
diff --git a/Classes/Fusion/XmlSitemapImplementation.php b/Classes/Fusion/XmlSitemapImplementation.php
@@ -201,7 +201,7 @@ protected function resolveImages(NodeInterface $node, array &$item)
      */
     protected function isDocumentNodeToBeIndexed(NodeInterface $node): bool
     {
-        return !$node->getNodeType()->isOfType('Neos.Neos:Shortcut') && $node->isVisible()
+        return !$node->getNodeType()->isOfType('Neos.Seo:NoindexMixin') && $node->isVisible()
             && ($this->getRenderHiddenInIndex() || !$node->isHiddenInIndex()) && $node->isAccessible()
             && $node->getProperty('metaRobotsNoindex') !== true;
     }

diff --git a/Configuration/NodeTypes.Mixins.yaml b/Configuration/NodeTypes.Mixins.yaml
@@ -74,6 +74,20 @@
           group: 'seometa'
           position: 40
 
+#
+# SEO Noindex tag mixin
+#
+'Neos.Seo:NoindexMixin':
+  abstract: true
+  properties:
+    metaRobotsNoindex:
+      defaultValue: true
+      ui:
+        inspector:
+          editorOptions:
+            disabled: true
+
+
 #
 # Twitter Card mixin
 #

diff --git a/Configuration/NodeTypes.yaml b/Configuration/NodeTypes.yaml
@@ -21,6 +21,7 @@
   superTypes:
     'Neos.Seo:TitleTagMixin': false
     'Neos.Seo:SeoMetaTagsMixin': false
+    'Neos.Seo:NoindexMixin': true
     'Neos.Seo:TwitterCardMixin': false
     'Neos.Seo:CanonicalLinkMixin': false
     'Neos.Seo:OpenGraphMixin': false

diff --git a/Documentation/index.rst b/Documentation/index.rst
@@ -108,6 +108,10 @@ To include contained images of pages in the xml sitemap use the following fusion
         body.includeImageUrls = true
     }
 
+By default all shortcuts are ignored in the sitemap. They inherit from the prototype `Neos.Seo:NoindexMixin`.
+If you have other document types that should not appear in the sitemap you can also let them inherit from
+that prototype.
+
 Alternate Language Tag
 ------------------------
 
@@ -194,6 +198,7 @@ enabled in the `Configuration/NodeTypes.yaml` file, along with an inspector tab:
     superTypes:
       'Neos.Seo:TitleTagMixin': false
       'Neos.Seo:SeoMetaTagsMixin': false
+      'Neos.Seo:NoindexMixin': true
       'Neos.Seo:TwitterCardMixin': false
       'Neos.Seo:CanonicalLinkMixin': false
       'Neos.Seo:OpenGraphMixin': false