[jira] Commented: (OFBIZ-1133) Unescaped quotes break lookup screens functionality

["Jacopo Cappellato (JIRA)" <ji...@apache.org>]

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

Jacopo Cappellato commented on OFBIZ-1133:
------------------------------------------

I agree with Christian: we can remove the second parameter (internalName) from all the product's lookup screens. That was an old experiment that I did :-( to provide a mean to display the internal name after a product id was selected... pre-Ajax age



> Unescaped quotes break lookup screens functionality
> ---------------------------------------------------
>
>                 Key: OFBIZ-1133
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1133
>             Project: OFBiz
>          Issue Type: Bug
>    Affects Versions: SVN trunk, Release Branch 4.0
>            Reporter: Oleg Andreyev
>            Assignee: Jacques Le Roux
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: patch_ofbiz_LookupProductAndPrice-branch.diff, patch_ofbiz_LookupProductAndPrice-trunk.diff
>
>
> Example, it's impossible to select product in LookupProductAndPrice (used in EditQuoteItem) if product internal name contains quotas or apostrophe. The reason is obvious. The script "javascript:set_values('10112', 'Test product - Monitor 17", gray')" has error, unnecessary quote. I think the problem is common.
> Simple solution is to remove second parameter. There is no field internalName in opener form. But it's feckless way.
> Have anybody suggestion how rightly correct this problem? Modify HtmlFormRenderer?
> Please explain me why we don't store escaped string in database. This is generally accepted solution.

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