Party Services bug - Please review

[Chris Juettner]

I believe I have found and corrected a bug in these simple methods named 
"createPartyContent" and "updatePartyContent" located in PartyServices.xml.  
 
What is happening is the method is setting the dataResourceId in the map prior 
to the DataResource row being created.  
This is a problem because the dataResourceId doesn't exist yet.  
The result is that the uploaded file name is persisted with just the path and 
extension.  
 
For example:     
C:\eclipse\workspace\ofbiz\applications\content\uploads\1200348109149\.doc
 
So I added a call to the updateDataResource service following the 
createDataResource because the dataResourceId is now available. 
 
Please review for accuracy and make the change if appropriate.
 
Thanks
 
Chris Juettner
Integral Business Solutions
 
 
File: PartyServices.xml

createPartyContent
Original code:
 
           <!-- create the data resource object -->
           <set from-field="parameters._uploadedFile_fileName" 
field="dataResource.dataResourceName"/>
           <set from-field="parameters._uploadedFile_contentType" 
field="dataResource.mimeTypeId"/>
           <set value="${uploadPath}/${dataResource.dataResourceId}" 
field="dataResource.objectInfo"/>
           <if-not-empty field-name="extension">
              <set 
value="${uploadPath}/${dataResource.dataResourceId}.${extension.fileExtensionId}"
 field="dataResource.objectInfo"/>
           </if-not-empty>
           <set value="LOCAL_FILE" field="dataResource.dataResourceTypeId"/>
           <call-service service-name="createDataResource" 
in-map-name="dataResource" break-on-error="false">
               <result-to-field result-name="dataResourceId" 
field-name="dataResourceId"/>
               <result-to-field result-name="dataResource" 
field-name="dataResourceMap"/>
           </call-service>           
 
 
Fixed code:

           <!-- create the data resource object -->
           <set from-field="parameters._uploadedFile_fileName" 
field="dataResource.dataResourceName"/>
           <set from-field="parameters._uploadedFile_contentType" 
field="dataResource.mimeTypeId"/>
           <set value="LOCAL_FILE" field="dataResource.dataResourceTypeId"/>
           <call-service service-name="createDataResource" 
in-map-name="dataResource" break-on-error="false">
               <result-to-field result-name="dataResourceId" 
field-name="dataResourceId"/>
           </call-service>
                
           <set-service-fields service-name="updateDataResource" 
map-name="dataResource" to-map-name="dataResource"/>           
           <set from-field="dataResourceId" 
field="dataResource.dataResourceId"/>
           <call-service service-name="updateDataResource" 
in-map-name="dataResource" break-on-error="false" />
           <set from-field="dataResource.dataResourceId" 
field="lookup.dataResourceId"/>
           <find-by-primary-key use-cache="true" entity-name="DataResource" 
value-name="dataResourceMap" map-name="lookup"/>
 
 
updatePartyContent
Original code:
 
               <!-- create the data resource object -->
               <set from-field="parameters._uploadedFile_fileName" 
field="dataResource.dataResourceName"/>
               <set from-field="parameters._uploadedFile_contentType" 
field="dataResource.mimeTypeId"/>
               <set value="${uploadPath}/${dataResource.dataResourceId}" 
field="dataResource.objectInfo"/>
               <if-not-empty field-name="extension">
                   <set 
value="${uploadPath}/${dataResource.dataResourceId}.${extension.fileExtensionId}"
 field="dataResource.objectInfo"/>
               </if-not-empty>
               <set value="LOCAL_FILE" field="dataResource.dataResourceTypeId"/>
               <call-service service-name="createDataResource" 
in-map-name="dataResource" break-on-error="false">
                   <result-to-field result-name="dataResourceId" 
field-name="dataResourceId"/>
                   <result-to-field result-name="dataResource" 
field-name="dataResourceMap"/>
               </call-service>    
 
Fixed code:

               <!-- create the data resource object -->
               <set from-field="parameters._uploadedFile_fileName" 
field="dataResource.dataResourceName"/>
               <set from-field="parameters._uploadedFile_contentType" 
field="dataResource.mimeTypeId"/>
               <set value="LOCAL_FILE" field="dataResource.dataResourceTypeId"/>
               <call-service service-name="createDataResource" 
in-map-name="dataResource" break-on-error="false">
                   <result-to-field result-name="dataResourceId" 
field-name="dataResourceId"/>
               </call-service>
               <set value="${uploadPath}/${dataResourceId}" 
field="dataResource.objectInfo"/>
               <if-not-empty field-name="extension">
                     <set 
value="${uploadPath}/${dataResourceId}.${extension.fileExtensionId}" 
field="dataResource.objectInfo"/>
               </if-not-empty>
               <set-service-fields service-name="updateDataResource" 
map-name="dataResource" to-map-name="dataResource"/>           
               <set from-field="dataResourceId" 
field="dataResource.dataResourceId"/>
               <call-service service-name="updateDataResource" 
in-map-name="dataResource" break-on-error="false" />
               <set from-field="dataResource.dataResourceId" 
field="lookup.dataResourceId"/>
               <find-by-primary-key use-cache="true" entity-name="DataResource" 
value-name="dataResourceMap" map-name="lookup"/>