From 0ded4eb250519abdc01bdf0dc0493f83cb0620a6 Mon Sep 17 00:00:00 2001 From: Chris McDonough Date: Wed, 24 Aug 2011 13:00:40 -0400 Subject: explain better --- pyramid/config/__init__.py | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/pyramid/config/__init__.py b/pyramid/config/__init__.py index baae824ac..80fd51d69 100644 --- a/pyramid/config/__init__.py +++ b/pyramid/config/__init__.py @@ -303,17 +303,22 @@ class Configurator( self.add_view(exceptionresponse_view, context=IExceptionResponse) self.add_view(exceptionresponse_view,context=WebobWSGIHTTPException) - # commit before adding default_view_mapper, as the - # exceptionresponse_view above requires the superdefault view - # mapper + # commit below because: + # + # - the default exceptionresponse_view requires the superdefault view + # mapper, so we need to configure it before adding default_view_mapper + # + # - provided default renderers should be overrideable without requiring + # the user to commit before calling config.add_renderer self.commit() - # The following registrations should be treated as if the methods had - # been called after configurator construction (commit should not be - # called after this). Rationale: user-supplied implementations - # should be preferred rather than add-on author implementations (as - # per automatic conflict resolution). + # self.commit() should not be called after this point because the + # following registrations should be treated as analogues of methods + # called by the user after configurator construction. Rationale: + # user-supplied implementations should be preferred rather than + # add-on author implementations with the help of automatic conflict + # resolution. if authentication_policy and not authorization_policy: authorization_policy = ACLAuthorizationPolicy() # default -- cgit v1.2.3