Renderer expander converts all context-resolved values from the model to string type

Description

(07:55:06) Isle of Yura: og I noticed one funny thing
(07:55:33) Isle of Yura: when using repeat expander for example
(07:55:41) Isle of Yura: and trying to resove pathAs value
(07:55:57) Isle of Yura: boolean is converted into string

(07:57:37) Isle of Yura: i noticed it depends on noCopy option
(07:57:39) Isle of Yura: i tihnk
(07:57:57) AntranigBasman: Interesting
(07:58:14) Isle of Yura: and expandLight, and the fact that that option cant propagate
(07:58:23) AntranigBasman: Can't propagate?
(07:58:37) Isle of Yura: yes as part of renderer expander resolution
(07:58:46) Isle of Yura: repeat expander that is
(07:58:47) AntranigBasman: What do you mean by propagate?
(07:59:10) Isle of Yura: i mean it wont get through even if i were to specify it somewhere
(08:00:23) AntranigBasman: Can you give an example?
(08:00:45) Isle of Yura: one sec
(08:03:29) Isle of Yura: so kasper has a branch in his github, where he 's trying to use condition expander
(08:03:36) Isle of Yura: and as an arg he passes a boolean
(08:03:50) Isle of Yura: but when we get to calling the function that will resolve condition
(08:04:06) Isle of Yura: that boolean turns to a string

https://github.com/collectionspace/ui/blob/master/src/main/webapp/js/Header.js

(08:10:41) Isle of Yura: so line 83
(08:10:51) Isle of Yura: ${{itemName}.hide} is actually a boolean
(08:11:07) Isle of Yura: but when cspace.header.assertMenuItemDisplay is called, it's already a string arg

Environment

None

Activity

Show:

Antranig Basman March 2, 2011 at 4:15 AM

Fixed at revision c6298c9db2fca7f385d1dfd83bbdd23fa328b03c

Antranig Basman March 2, 2011 at 3:26 AM

The code in question reads:

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created March 2, 2011 at 3:25 AM
Updated March 3, 2014 at 6:37 PM
Resolved March 2, 2011 at 4:15 AM