Modified the Config library so that an array of items can be set using set_item() #1189

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@JonoB

I was not sure if we should be testing for empty values in the array, but the original version doesn't, so I left it as-is.

@cryode

I personally would rather see this as a new set_items() method. What do others think?

@JonoB

I dont think that set_items() is consistent with the rest of core.

@narfbg

What if you want to actually set an item to be an array? This change would break it.

@narfbg narfbg closed this Nov 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment