[
https://issues.apache.org/jira/browse/OFBIZ-11274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16971213#comment-16971213 ]
Daniel Watford commented on OFBIZ-11274:
----------------------------------------
[~mthl] I've not been able to find a good candidate lookup that would benefit from using set_multvalues. This might suggest it isn't really needed, or might just be down to my lack of experience with the various ofbiz components.
Thinking on this further, perhaps having the lookup form specify the inputs to be populated (as is the case with set_multivalues) would couple the lookup form and calling page too tightly, and might be something to be discouraged.
Having lookup forms limited to calling set_value with with an ID value (and possibly a description) might be enough. I can retrieve any other information needed for display by hooking into the 'lookup:changed' jquery event fired when a lookup field is updated.
I'm happy to close this ticket unless others have reason to explore set_multivalues() further.
> Cannot set property 'value' of null in lookup.js set_multivalues() function
> ---------------------------------------------------------------------------
>
> Key: OFBIZ-11274
> URL:
https://issues.apache.org/jira/browse/OFBIZ-11274> Project: OFBiz
> Issue Type: Bug
> Components: themes
> Affects Versions: Trunk
> Reporter: Daniel Watford
> Priority: Major
> Labels: pull-request-available
> Attachments: OFBIZ-11274-lookup-set-multi.patch
>
>
> Attempts to call lookup.js function set_multivalues() from a lookup screen fail with the console logging error:
> {noformat}
> Uncaught TypeError: Cannot set property 'value' of null
> at set_multivalues (fieldlookup.js:769)
> at <anonymous>:1:1{noformat}
> No framework or plugins appear to currently use this function.
> The set_multivalues() function should be useful for populating multiple fields (e.g. a Postal Address) on a form from a lookup.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
Locked
