Merge pull request #3512 from plone/petschki-iconresolver

Fix @@iconresolver resolving subpaths (eg. "contenttype/document")
plone · Apr 27, 2022 · 19f845d · 19f845d
2 parents 16ca65e + ccbf261
commit 19f845d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/Products/CMFPlone/browser/icons.py b/Products/CMFPlone/browser/icons.py
@@ -64,9 +64,13 @@ class IconsView(BrowserView):
 
     prefix = "plone.icon."
     defaulticon = "++plone++icons/plone.svg"
+    name = ""
 
     def publishTraverse(self, request, name):
-        self.name = name
+        if self.name:
+            # fix traversing to eg. "contenttype/document"
+            self.name += "/"
+        self.name += name
         return self
 
     def __call__(self):

diff --git a/news/3500.bugfix b/news/3500.bugfix
@@ -0,0 +1,2 @@
+fix `@@iconresolver` to resolve names with "/" correctly (eg. "contenttype/document")
+[petschki]
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		fix `@@iconresolver` to resolve names with "/" correctly (eg. "contenttype/document")
		[petschki]