Review Board 1.7.22


CrawlerBeansPropHandler doesn't set list properties for Spring PropertyOverrideConfigurer correctly

Review Request #4454 - Created March 22, 2012 and submitted

brian Foster
OODT-428
Reviewers
oodt
lmzxq.tom, mattmann, pramirez, rickdn
oodt
It currently was setting lists keys to: value1,value2,value3,...

This patch will instead change it to:
key[0] = value1
key[1] = value2
Wrote unit-test
Review request changed
Updated (March 26, 2012, 8:55 p.m.)
  • still need a unit-test for this... will write when i get the time

    Wrote unit-test
- For case optionInstance.getValues().isEmpty() now throws RuntimeException
- Added Unit-tests
Ship it!
Posted (March 26, 2012, 10:10 p.m.)
Sounds good LGTM.
Ship it!
Posted (March 27, 2012, 12:02 a.m.)
LGTM.