-> > Here's the corrected patch, introducing warn() statements appropriately -> > and removing internal dependence on the dict-like interface. -> -> Thanks for spending the time to make a patch. However, I think -> SessionManager should provide some kind of API for accessing the -> managed sessions. Forcing the application poke around in .sessions -> breaks encapsulation and is a step backward, IMHO. Upon reflection, -> while the dict interface is not ideal maybe it's not so bad either. -> Maybe it's not worth changing.
Hi, Neil, in my rewrite/adaptation of the sessions code, I couldn't find a use for anything other than the start_request, end_request, abort_request, get_session, and expire_session functions. I don't think anyone but the session manager class itself needs to poke around in .sessions directly. cheers, --titus _______________________________________________ Quixote-users mailing list [email protected] http://mail.mems-exchange.org/mailman/listinfo/quixote-users
