From ccbc9539ca8f7a56089affb0bc1a6feb1f65272e Mon Sep 17 00:00:00 2001 From: Chris McDonough Date: Tue, 9 Nov 2010 15:14:31 -0500 Subject: sqla idiomatic suggestions from rob --- TODO.txt | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/TODO.txt b/TODO.txt index ec3feac9a..f0bd20631 100644 --- a/TODO.txt +++ b/TODO.txt @@ -95,3 +95,22 @@ - Option for route_url to omit the host and port (perhaps a different function named ``route_path``). + +- SQLAlchemy idiomatics: + + mcdonc: those paster templates all look pretty good... the + only thing i'd consider is adjusting your config variable names to match + exactly what sqlalchemy uses as parameter names, see here: + http://www.sqlalchemy.org/docs/core/engines.html + + mcdonc: especially in the pylons_sqla ini file, where the db + initialization is mixed in w/ the app config... + + ... i'd use "sqlalchemy.PARAMETER" for all of the sqla + settings, so it could easily be handed to engine_from_config w/o any need + to parse by hand + + mcdonc: in the other ini files, where sqlalchemy is given its + own part, the "sqlalchemy." prefix probably isn't necessary, but matching + the parameter names (e.g. 'url' instead of 'db_string') is still probably + a good idea -- cgit v1.2.3