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

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

> From Order manager, Order entry. Then from 'Browse Categories left menu'
> ... After adding this product to cart, in shopping cart items there is broken
> link "Surveys:"

I see it. That link is missing the surveyId. So that's why you're deriving the 
surveyId in ViewSurveyResponses.bsh .

> Can you tell me please how you got this error "You have already responded to
> this survey; multiple responses are not allowed" ? Some surveys (like Mouse
> Hand Poll and Mini-Poll Poll from demo data ) has allowMultiple='N',

Yes, already answered survey. Problem is, I can't order the item again! That's 
the bug.

> ordermgr/controller.xml:477
>    <request-map uri="additemsurvey">
>        <security https="false" auth="false"/>
>
> it should be https="true" auth="true" Is there a reason for not doing so?

I think some surveys can be taken without having to be logged. isAnonymous "Y". 
Test those?


> 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