Changes between Version 1 and Version 2 of Ticket #22678, comment 2
- Timestamp:
- May 22, 2014, 4:42:12 PM (10 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #22678, comment 2
v1 v2 1 I intended to convey that above: the issue is the integer . Strings work, integers don't.1 I intended to convey that above: the issue is the integer key. Strings work, integers don't. 2 2 The form as built 'looked' like reasonable code, but had a murky and hard to track problem. 3 3 Most code paths blow up on integers, but not this one. 4 4 5 5 6 The proposed fix is to assert on string for the key. 6 The proposed fix is to assert on string for the key.... or almost as good perform the string 7 concatenation (e.g. 'id_'+key) in all cases, so it blows up nice and fast with any form of key that 8 does not actually fully work.