I'm not comfortable with this code.  Themes are already registered, so I don't see why they all need to be listed here, in this spot which is a performance-critical piece of code.  Can you a) make this only called once on the initialization of the formatter, and b) use Theme.find?