Rattail

I've pondered this for years, and wanted to get some thoughts down. At this time I remain unconvinced that a separate package is needed, but surely some sort of refactor is in order eventually.

Below are various thoughts so far.

The RattailConfig class has served pretty well but it needs some love, esp. since it is the "most fundamental" part of Rattail. I say that because e.g. in a new Python shell, the first thing to do is usually call make_config().

Now the AppHandler class has blurred the lines a bit, since it's also (effectively) a singleton, so 1-to-1 with the global config object (got with config.get_app()). Some of the methods on RattailConfig probably belong on AppHandler instead.

Or maybe there should be some make_app() function, and the AppHandler returned should have methods to read/write config settings, instead of passing around the config object..? IDK, it seems like calling config.get() is better than app.get_setting()? (Or am I just being nostalgic?) It would also mean calls like somefunc(config) would be replaced by somefunc(app) in which case that may be "difficult" to refactor anytime soon.

Splitting out the main Config class to a separate package seems like a good idea, to make it available for non-Rattail projects. But the question is, does it actually do something useful enough for anyone else? To my mind it offers:

• support for typical INI file format
• support for file "inheritance" (recursive includes)
• support for settings in a DB table
• (new!) support for cache layer for settings in DB

If I were to bother splitting it out, it should be more "modular" somehow and let config values come from other sources etc.

But today I did yet another search on PyPI, and found the python-configuration package which actually seems to do already, what I had in mind. It appears at first glance that it should be possible to add a custom Configuration for the DB settings table logic, and presumably the Beaker cache as well.

Which then begs the question, if python-configuration is so swell then why am I not just using it? (Well first of all, I've looked to see what's out there before, but today was the first time I found it; apparently came out Jan 2019.) But maybe it should get used..TBD. Probably depends on how much of the config-related things AppHandler needs to take on, which e.g. then might delegate to either a RattailConfig or Configuration to read/write settings?