From 4966d8388e7d10e768ee7d5037cf19dcb06bfb93 Mon Sep 17 00:00:00 2001
From: Philip Bauer <bauer@starzel.de>
Date: Thu, 27 Jun 2019 19:43:27 +0200
Subject: [PATCH] add plone.resource dependency and change to plone:static

---
 .../howto_override_comments_viewlet.txt       | 34 +------------------
 plone/app/discussion/browser/configure.zcml   |  5 +--
 plone/app/discussion/browser/controlpanel.pt  |  2 +-
 plone/app/discussion/browser/moderation.pt    |  2 +-
 .../discussion/profiles/default/metadata.xml  |  1 +
 .../discussion/profiles/default/registry.xml  |  2 +-
 6 files changed, 8 insertions(+), 38 deletions(-)

diff --git a/docs/source/howtos/howto_override_comments_viewlet.txt b/docs/source/howtos/howto_override_comments_viewlet.txt
index 11c0248..b45aba4 100644
--- a/docs/source/howtos/howto_override_comments_viewlet.txt
+++ b/docs/source/howtos/howto_override_comments_viewlet.txt
@@ -135,36 +135,4 @@ comments.py file with our custom version of the comments viewlet::
                 return "%s/memberhome/%s" % (self.context.portal_url(), username)
 
 To override the comments viewlet template, we create a comment.pt file in the
-same directory and copy the contents from::
-
-    ...
-
-
-Override the comments viewlet Javascript
-----------------------------------------
-
-Overriding the comments viewlet javascript works just like overriding the
-comments viewlet. We register the javascript file for our custom browser
-layer and remove the existing javascript file in
-profiles/default/jsregistry.xml::
-
-    <?xml version="1.0"?>
-    <object name="portal_javascripts">
-     <!-- Remove plone.app.discussion comments javascript -->
-     <javascript
-         id="++resource++plone.app.discussion.javascripts/comments.js"
-         remove="True"
-         />
-     <!-- Register a custom version of the plone.app.discussion javascript -->
-     <javascript
-         id="++resource++example.myaddonproduct.javascripts/comments.js" />
-    </object>
-
-browser/configure.zcml::
-
-    <!-- Resource directory for javascripts -->
-    <browser:resourceDirectory
-      name="example.myaddonproduct.javascripts"
-      directory="javascripts"
-      layer="example.myaddonprocuts.interfaces.IBrowserLayer"
-      />
+same directory and copy the contents from the original.
diff --git a/plone/app/discussion/browser/configure.zcml b/plone/app/discussion/browser/configure.zcml
index 6a0efa2..7bb5aac 100644
--- a/plone/app/discussion/browser/configure.zcml
+++ b/plone/app/discussion/browser/configure.zcml
@@ -1,6 +1,7 @@
 <configure
     xmlns="http://namespaces.zope.org/zope"
     xmlns:browser="http://namespaces.zope.org/browser"
+    xmlns:plone="http://namespaces.plone.org/plone"
     xmlns:zcml="http://namespaces.zope.org/zcml"
     i18n_domain="plone">
 
@@ -129,10 +130,10 @@
         />
 
     <!-- Resource directory for javascripts -->
-    <browser:resourceDirectory
+    <plone:static
         name="plone.app.discussion.javascripts"
+        type="plone"
         directory="javascripts"
-        layer="..interfaces.IDiscussionLayer"
         />
 
     <!-- Control panel -->
diff --git a/plone/app/discussion/browser/controlpanel.pt b/plone/app/discussion/browser/controlpanel.pt
index b62a447..d525fcb 100644
--- a/plone/app/discussion/browser/controlpanel.pt
+++ b/plone/app/discussion/browser/controlpanel.pt
@@ -70,7 +70,7 @@
     </div>
 
   <script type="text/javascript"
-          tal:attributes="src string:${portal_url}/++resource++plone.app.discussion.javascripts/controlpanel.js">
+          tal:attributes="src string:${portal_url}/++plone++plone.app.discussion.javascripts/controlpanel.js">
   </script>
 
 </article>
diff --git a/plone/app/discussion/browser/moderation.pt b/plone/app/discussion/browser/moderation.pt
index 797c18a..073d4d2 100644
--- a/plone/app/discussion/browser/moderation.pt
+++ b/plone/app/discussion/browser/moderation.pt
@@ -18,7 +18,7 @@
                     b_start request/b_start | b_start;
                     moderation_enabled view/moderation_enabled;">
         <script type="text/javascript"
-            tal:attributes="src string:${context/portal_url}/++resource++plone.app.discussion.javascripts/moderation.js">
+            tal:attributes="src string:${context/portal_url}/++plone++plone.app.discussion.javascripts/moderation.js">
         </script>
 
         <h1 class="documentFirstHeading" i18n:translate="heading_moderate_comments">
diff --git a/plone/app/discussion/profiles/default/metadata.xml b/plone/app/discussion/profiles/default/metadata.xml
index f14a707..6557251 100644
--- a/plone/app/discussion/profiles/default/metadata.xml
+++ b/plone/app/discussion/profiles/default/metadata.xml
@@ -1,6 +1,7 @@
 <metadata>
  <version>1000</version>
  <dependencies>
+  <dependency>profile-plone.resource:default</dependency>
   <dependency>profile-plone.app.registry:default</dependency>
  </dependencies>
 </metadata>
diff --git a/plone/app/discussion/profiles/default/registry.xml b/plone/app/discussion/profiles/default/registry.xml
index 5c046f4..ba7fe98 100644
--- a/plone/app/discussion/profiles/default/registry.xml
+++ b/plone/app/discussion/profiles/default/registry.xml
@@ -7,7 +7,7 @@
 
   <records prefix="plone.resources/plone-app-discussion"
             interface='Products.CMFPlone.interfaces.IResourceRegistry'>
-      <value key="js">++resource++plone.app.discussion.javascripts/comments.js</value>
+      <value key="js">++plone++plone.app.discussion.javascripts/comments.js</value>
   </records>
   <records prefix="plone.bundles/plone-legacy"
             interface='Products.CMFPlone.interfaces.IBundleRegistry'>