svn commit: r1820949 - /ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml

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

svn commit: r1820949 - /ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml

paulfoxworthy
Author: paulfoxworthy
Date: Fri Jan 12 06:50:17 2018
New Revision: 1820949

URL: http://svn.apache.org/viewvc?rev=1820949&view=rev
Log:
Fixed: Find Employee Position does not allow "don't care" for boolean fields (OFBIZ-10158)

In the demo app, go to https://demo-stable.ofbiz.apache.org/humanres/control/FindEmplPositions, click on New Employee Position, and toggle one of the four flags, e.g. set Temporary Flag to N.

Now on the Find screen, there is no way to find all employee positions: it insists on a Yes or No for the flags. This patch allows a blank "don't care" option.

Thanks: Larissa Zanoni at Blockfreight for discovering this issue.

Modified:
    ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml

Modified: ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml?rev=1820949&r1=1820948&r2=1820949&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml (original)
+++ ofbiz/ofbiz-framework/trunk/applications/humanres/widget/forms/EmplPositionForms.xml Fri Jan 12 06:50:17 2018
@@ -276,25 +276,25 @@ under the License.
         <auto-fields-entity entity-name="EmplPosition" default-field-type="find"/>
         <field name="noConditionFind"><hidden value="Y"/><!-- if this isn't there then with all fields empty no query will be done --></field>
         <field name="salaryFlag">
-            <drop-down allow-empty="false" current="selected">
+            <drop-down allow-empty="true" current="selected">
                 <option key="Y" description="${uiLabelMap.CommonYes}"/>
                 <option key="N" description="${uiLabelMap.CommonNo}"/>
             </drop-down>
         </field>
         <field name="exemptFlag">
-            <drop-down allow-empty="false" current="selected">
+            <drop-down allow-empty="true" current="selected">
                 <option key="Y" description="${uiLabelMap.CommonYes}"/>
                 <option key="N" description="${uiLabelMap.CommonNo}"/>
             </drop-down>
         </field>
         <field name="fulltimeFlag">
-            <drop-down allow-empty="false" current="selected">
+            <drop-down allow-empty="true" current="selected">
                 <option key="Y" description="${uiLabelMap.CommonYes}"/>
                 <option key="N" description="${uiLabelMap.CommonNo}"/>
             </drop-down>
         </field>
         <field name="temporaryFlag">
-            <drop-down allow-empty="false" current="selected">
+            <drop-down allow-empty="true" current="selected">
                 <option key="Y" description="${uiLabelMap.CommonYes}"/>
                 <option key="N" description="${uiLabelMap.CommonNo}"/>
             </drop-down>