slapos_panel: allow project manager to see customer name in the support request

master/bt5/slapos_erp5/TestTemplateItem/portal_components/test.erp5.testSlapOSCodingStyle.py

@@ -234,6 +234,7 @@ def makeTestSlapOSCodingStyleTestCase(tested_business_template):
         'slapos_panel/Document_getNewsDict',
         'slapos_panel/Event_getAttachmentList',
         'slapos_panel/Event_getSafeSourceTitle',
+        'slapos_panel/Event_getUnsafeSourceTitle',
         'slapos_panel/InstanceNode_addSlapOSAllocationSupply',
         'slapos_panel/InstanceTreeModule_selectRequestProject',
         'slapos_panel/InstanceTree_redirectToManualDepositPayment',

master/bt5/slapos_panel/SkinTemplateItem/portal_skins/slapos_panel/Event_getSafeSourceTitle.py

-return context.getSourceTitle(checked_permission='View') or context.getSourceProjectTitle(checked_permission='View') or context.getFollowUpTitle(checked_permission='View') or None
+return context.Event_getUnsafeSourceTitle() or context.getSourceTitle(checked_permission='View') or context.getSourceProjectTitle(checked_permission='View') or context.getFollowUpTitle(checked_permission='View') or None

master/bt5/slapos_panel/SkinTemplateItem/portal_skins/slapos_panel/Event_getUnsafeSourceTitle.py

+from zExceptions import Unauthorized
+if REQUEST is not None:
+  raise Unauthorized
+
+# Allow project manager to see who is sending message for this ticket
+# this helps providing better support if their are multiple project's manager handling tickets
+# Customer will not see which manager replied.
+if context.Base_hasSlapOSProjectUserGroup(project_relation='source_project', manager=True, agent=True) or context.Base_hasSlapOSProjectUserGroup(project_relation='destination_project', manager=True, agent=True):
+  return context.getSourceTitle(checked_permission='View')
+return None

master/bt5/slapos_panel/SkinTemplateItem/portal_skins/slapos_panel/Event_getUnsafeSourceTitle.xml

+<?xml version="1.0"?>
+<ZopeData>
+  <record id="1" aka="AAAAAAAAAAE=">
+    <pickle>
+      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
+    </pickle>
+    <pickle>
+      <dictionary>
+        <item>
+            <key> <string>_bind_names</string> </key>
+            <value>
+              <object>
+                <klass>
+                  <global name="_reconstructor" module="copy_reg"/>
+                </klass>
+                <tuple>
+                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
+                  <global name="object" module="__builtin__"/>
+                  <none/>
+                </tuple>
+                <state>
+                  <dictionary>
+                    <item>
+                        <key> <string>_asgns</string> </key>
+                        <value>
+                          <dictionary>
+                            <item>
+                                <key> <string>name_container</string> </key>
+                                <value> <string>container</string> </value>
+                            </item>
+                            <item>
+                                <key> <string>name_context</string> </key>
+                                <value> <string>context</string> </value>
+                            </item>
+                            <item>
+                                <key> <string>name_m_self</string> </key>
+                                <value> <string>script</string> </value>
+                            </item>
+                            <item>
+                                <key> <string>name_subpath</string> </key>
+                                <value> <string>traverse_subpath</string> </value>
+                            </item>
+                          </dictionary>
+                        </value>
+                    </item>
+                  </dictionary>
+                </state>
+              </object>
+            </value>
+        </item>
+        <item>
+            <key> <string>_params</string> </key>
+            <value> <string>REQUEST=None</string> </value>
+        </item>
+        <item>
+            <key> <string>_proxy_roles</string> </key>
+            <value>
+              <tuple>
+                <string>Manager</string>
+              </tuple>
+            </value>
+        </item>
+        <item>
+            <key> <string>id</string> </key>
+            <value> <string>Event_getUnsafeSourceTitle</string> </value>
+        </item>
+      </dictionary>
+    </pickle>
+  </record>
+</ZopeData>

master/bt5/slapos_panel_ui_test/PathTemplateItem/portal_tests/slapos_panel_zuite/testCustomerInstanceScenario.zpt

@@ -347,6 +347,11 @@
         <td>//div[contains(@data-gadget-url, 'gadget_thread_reader.html')]//div[contains(@data-gadget-url, 'gadget_html_viewer.html')]//p[text()='Why I can not use my instance yet?']</td>
         <td></td>
       </tr>
+      <tr>
+        <td>assertElementPresent</td>
+        <td>//div[contains(@data-gadget-url, 'gadget_thread_reader.html')]//div[contains(@class, "post_content")]//strong[text()='Manual test Project Customer']</td>
+        <td></td>
+      </tr>
 
 
       <tr>