You must log in to post
Please turn it on in your browser preferences.
for specific sites, we recommend the
Review Request #4508 -
Created March 27, 2012 and updated
April 24, 2012, 2:54 a.m.
Functionally fine, just a couple of niggles
This should probably be a constant
I'm pretty sure that now we have a dedicated class for the configuration we don't need a subclass to hold the constants?
For consistency I think this should go into the configuration and throw a ConfigurationException