Author: jleroux
Date: Sun Oct 18 12:26:15 2015
New Revision: 1709271

URL: http://svn.apache.org/viewvc?rev=1709271&view=rev
Log:
Completes r1709192 for Changes for "SalaryStep entity missing From and Thru 
dates" https://issues.apache.org/jira/browse/OFBIZ-5758

The dateModified field was not modified (not even before auto-engine was used). 
I found that while changing updateSalaryStep from auto-engine to simple for 
lastModifiedByUserLogin and setting the createdByUserLogin field in 
createalaryStep services. It's now fixed
     




Modified:
    
ofbiz/trunk/applications/humanres/script/org/ofbiz/humanres/HumanResServices.xml
    ofbiz/trunk/applications/humanres/servicedef/services.xml

Modified: 
ofbiz/trunk/applications/humanres/script/org/ofbiz/humanres/HumanResServices.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/humanres/script/org/ofbiz/humanres/HumanResServices.xml?rev=1709271&r1=1709270&r2=1709271&view=diff
==============================================================================
--- 
ofbiz/trunk/applications/humanres/script/org/ofbiz/humanres/HumanResServices.xml
 (original)
+++ 
ofbiz/trunk/applications/humanres/script/org/ofbiz/humanres/HumanResServices.xml
 Sun Oct 18 12:26:15 2015
@@ -226,9 +226,19 @@
         <make-value entity-name="SalaryStep" value-field="newEntity"/>
         <set-pk-fields map="parameters" value-field="newEntity"/>
         <set-nonpk-fields map="parameters" value-field="newEntity"/>
+        <set field="newEntity.createdByUserLogin" 
from-field="parameters.userLogin.userLoginId"/>
         <sequenced-id sequence-name="SalaryStep" 
field="newEntity.salaryStepSeqId"/>
         <field-to-result field="salaryStepSeqId" 
result-name="salaryStepSeqId"/>
         <create-value value-field="newEntity"/>
-    </simple-method>    
+    </simple-method>
+    
+    <simple-method method-name="updateSalaryStep" short-description="Update 
Salary Step">
+        <entity-one entity-name="SalaryStep" value-field="lookedUpValue"/>
+        <set-nonpk-fields value-field="lookedUpValue" map="parameters"/>
+        <set field="lookedUpValue.lastModifiedByUserLogin" 
from-field="parameters.userLogin.userLoginId"/>
+        <now field="fromDate"/>
+        <set field="lookedUpValue.dateModified" from-field="fromDate"/>
+        <store-value value-field="lookedUpValue"/>
+    </simple-method>
 
 </simple-methods>

Modified: ofbiz/trunk/applications/humanres/servicedef/services.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/humanres/servicedef/services.xml?rev=1709271&r1=1709270&r2=1709271&view=diff
==============================================================================
--- ofbiz/trunk/applications/humanres/servicedef/services.xml (original)
+++ ofbiz/trunk/applications/humanres/servicedef/services.xml Sun Oct 18 
12:26:15 2015
@@ -292,7 +292,8 @@ under the License.
         <override type="String" mode="IN" name="salaryStepSeqId" 
optional="true"/>
     </service>
 
-    <service name="updateSalaryStep" engine="entity-auto" 
default-entity-name="SalaryStep" invoke="update" auth="true">
+    <service name="updateSalaryStep" engine="simple" 
default-entity-name="SalaryStep"
+             
location="component://humanres/script/org/ofbiz/humanres/HumanResServices.xml" 
invoke="updateSalaryStep" auth="true">
         <description>Update Salary Step</description>
         <permission-service service-name="humanResManagerPermission" 
main-action="UPDATE"/>
         <auto-attributes mode="IN" include="pk" optional="false"/>


Reply via email to