[
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.
