[ 
https://issues.apache.org/jira/browse/OFBIZ-1227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12528437
 ] 

Jonathon Wong commented on OFBIZ-1227:
--------------------------------------

>> Yes, already answered survey. Problem is, I can't order the item again! 
>> That's the bug.
> Are you sure about it ? Which product ? There are only two surveys from demo
> data with allowMultiple='N' and they are not from type "CART_ADD" , which
> means they are not supposed to appears while adding to cart .

I was testing "Mouse Hand Poll" with GZ-2644 . Shouldn't the allowMultiple "N" 
mean that the survey is only asked once, and that the product can be bought 
again? Or should allowMultiple "N" not be used with "Add-To Cart" survey 
trigger? I tried triggers "Check-Out" and "Sales Order Placement", but those 
don't seem to work.

>> I think some surveys can be taken without having to be logged. isAnonymous 
>> "Y". Test those?
> This is possible in ecommerce application. I will change it in order manager,
> where in general you are always logged in. There is meaning to let it
> auth="false"

Agreed. I'm not sure why it's auth="false" in OrderMgr. Maybe it was 
cut-and-pasted from Ecommerce controller.xml .

> survey screen fix
> -----------------
>
>                 Key: OFBIZ-1227
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1227
>             Project: OFBiz
>          Issue Type: Bug
>          Components: content
>    Affects Versions: SVN trunk
>            Reporter: Bilgin Ibryam
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: survey.patch, survey.patch
>
>
> Here i will explain the bugs fixed:
> if you add a product with survey to shopping cart and try to see the survey, 
> the given link is broken. I add some code the get the surveyId from 
> surveyResponseId.
> If you search for survey from find survey screen (for example with id = 1) 
> you will get all the surveys stating with id=1, but at this stage the survey 
> menu will appears, thinking that we choose a survey with id 1. And this 
> survey menu is broken(surveyId=1, it is only a search criteria, not an 
> existing survey id). Removing some unused fields from FindSurveyscreen and 
> CommonSurveyDecorator fixed these bugs.
> I also have a question/proposal: Isn't it better to change the links ( from 
> shopping cart and order review screens )  from  ViewSurveyResponses to 
> EditSurveyResponse. If we change it, clicking on the survey link from an 
> order/shoppingcart item, will show us the exact survey response and ability 
> to edit it, instead of all the responses to the survey, as it is now ?
> Bilgin Ibryam

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to