Brain fart or bug?
Lucee 5, I'd expect the NULLs ...
# luceeb
bhartsfield
07/27/2022, 4:42 PMBrain fart or bug?
Lucee 5, I'd expect the NULLs to be -1 in the output but they are still null
dump( [nullValue(), 6, nullValue(), 10].map( ( value ) => value ?: -1 ) );bhartsfield
07/27/2022, 4:42 PMIt works when doing it this way...
Copy code
d2 = [];
for( value in [nullValue(), 6, nullValue(), 10] ) d2.append( value ?: -1 )
dump(d2);bhartsfield
07/27/2022, 4:43 PMw
websolete
07/27/2022, 4:46 PMbugfart?
b
bhartsfield
07/27/2022, 4:47 PMfartbug?
bhartsfield
07/27/2022, 4:47 PMhttps://www.pestworld.org/media/561856/brown-marmorated-stink-bug-_l0o9225.jpg▾
w
websolete
07/27/2022, 4:47 PMalso know as 'clackers'
websolete
07/27/2022, 4:48 PMfwiw, seems like an implementation inconsistency with map
b
bhartsfield
07/27/2022, 4:51 PMto support it being map and not the Hound Dog, same result with this (NULLs should be 0 but are still NULL)
[nullValue(),6,nullValue(),10].map( ( value ) => val(value) )bhartsfield
07/27/2022, 4:52 PMThis converts the GOOD values to 'wth' but leaves the nulls...
[nullValue(),6,nullValue(),10].map( ( value ) => 'wth' );b
bdw429s
07/27/2022, 4:53 PM@bhartsfield This is a Lucee bug if I'm understanding what you're doing
b
bhartsfield
07/27/2022, 4:54 PMDefinitely seems like map bug
b
bdw429s
07/27/2022, 4:54 PMLucee's map/reduce/each functinos all skip null values
bdw429s
07/27/2022, 4:54 PMI just ran across this
bdw429s
07/27/2022, 4:54 PMThere are tickets in JIRA for it
b
bhartsfield
07/27/2022, 4:54 PMahh
b
bdw429s
07/27/2022, 4:54 PMI had to change my code to a for loop
bdw429s
07/27/2022, 4:54 PMbdw429s
07/27/2022, 4:55 PMbdw429s
07/27/2022, 4:55 PMb
bhartsfield
07/27/2022, 4:55 PMThanks for the sanity check. Voted for the two open issues
b
bdw429s
07/27/2022, 4:55 PMbdw429s
07/27/2022, 4:56 PMYou'd think something that obviously broken would be fixed by now 😕
b
bhartsfield
07/27/2022, 4:56 PMoof... that's an old one
z
zackster
07/28/2022, 7:46 AMas it's somewhat a potential breaking change, i've added this to the 6.0 sprint, coz we are breaking stuff that needs to be broken with 6