You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2012/05/20 12:20:59 UTC

svn commit: r1340661 - /ofbiz/trunk/applications/content/script/org/ofbiz/content/survey/SurveyServices.xml

Author: jleroux
Date: Sun May 20 10:20:59 2012
New Revision: 1340661

URL: http://svn.apache.org/viewvc?rev=1340661&view=rev
Log:
A patch from Harsha Chadhar for "Can not add "Question Options" within the Survey section of the Content application." https://issues.apache.org/jira/browse/OFBIZ-4879

For a Survey Question of Type= Selected option, while trying to edit a survey question option the following error is encountered :
In set-pk-fields a value was not found with the specified valueAcsr: lookupKeyValue, not setting fields
URL : https://demo-trunk.ofbiz.apache.org:8443/content/control/EditSurveyQuestions?surveyId=1501
Screenlet:
Survey Options .

jleroux: this screen is not much user friendly (for instance when you edit the survey question option you are not redirected to the screenlet anchor)

Modified:
    ofbiz/trunk/applications/content/script/org/ofbiz/content/survey/SurveyServices.xml

Modified: ofbiz/trunk/applications/content/script/org/ofbiz/content/survey/SurveyServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/content/script/org/ofbiz/content/survey/SurveyServices.xml?rev=1340661&r1=1340660&r2=1340661&view=diff
==============================================================================
--- ofbiz/trunk/applications/content/script/org/ofbiz/content/survey/SurveyServices.xml (original)
+++ ofbiz/trunk/applications/content/script/org/ofbiz/content/survey/SurveyServices.xml Sun May 20 10:20:59 2012
@@ -282,7 +282,6 @@
         </check-permission>
         <check-errors/>
         <entity-one entity-name="SurveyQuestionOption" value-field="lookedUpValue"/>
-        <set-pk-fields map="parameters" value-field="lookupKeyValue"/>
         <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
         <store-value value-field="lookedUpValue"/>
     </simple-method>