svn commit: r1809377 - /ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1809377 - /ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml

jleroux@apache.org
Author: jleroux
Date: Sat Sep 23 08:34:52 2017
New Revision: 1809377

URL: http://svn.apache.org/viewvc?rev=1809377&view=rev
Log:
Implemented: Required field indicator (*) is missing from the required field on
add content role.
(OFBIZ-9749)

1. Go to Content.
2. Find Content, select a content from the list.
3. Click on roles from the submenu

Content Id, Role Type Id and party Id are required fields, required field
indicators are missing from the UI.
createContentRole service throws an error if the fields are missing.

Here is the link to the form,
https://demo-trunk.ofbiz.apache.org/content/control/addContentRole

Thanks: Rohit Rai

Modified:
    ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml

Modified: ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml?rev=1809377&r1=1809376&r2=1809377&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml (original)
+++ ofbiz/ofbiz-framework/trunk/applications/content/widget/content/ContentForms.xml Sat Sep 23 08:34:52 2017
@@ -359,11 +359,11 @@ under the License.
     <form name="AddContentRole" target="addContentRole" title="" type="single"
         header-row-style="header-row" default-table-style="basic-table">
         <auto-fields-service service-name="createContentRole"/>
-        <field name="contentId" widget-style="buttontext" >
+        <field name="contentId" widget-style="buttontext" required-field="true">
             <lookup target-form-name="LookupContent"/>
         </field>
-        <field name="partyId" widget-style="buttontext"><lookup target-form-name="LookupPerson"/></field>
-        <field name="roleTypeId">
+        <field name="partyId" widget-style="buttontext" required-field="true"><lookup target-form-name="LookupPerson"/></field>
+        <field name="roleTypeId" required-field="true">
             <drop-down allow-empty="true">
                 <entity-options entity-name="RoleType" key-field-name="roleTypeId">
                     <entity-order-by field-name="description"/>