id summary reporter owner description type status priority component severity resolution keywords cc release
11469 Exceptions in AccountModule._reset_password_enabled are not trapped in Trac 1.0.2dev Ryan J Ollos Steffen Hoffmann "While investigating #10980 using Trac 1.0.2dev, with the reporter's configuration I immediately saw a warning:
{{{#!html
close
Warning:
Error with navigation contributor ""AccountModule""
}}}
It looks like some changes are needed after [trac 12068].
{{{#!diff
diff --git a/accountmanagerplugin/trunk/acct_mgr/web_ui.py b/accountmanagerplugi
index 0611206..3713dfe 100644
--- a/accountmanagerplugin/trunk/acct_mgr/web_ui.py
+++ b/accountmanagerplugin/trunk/acct_mgr/web_ui.py
@@ -19,7 +19,8 @@ from genshi.builder import tag
from trac import perm, util
from trac.core import Component, implements
-from trac.config import Configuration, BoolOption, IntOption, Option
+from trac.config import Configuration, ConfigurationError, BoolOption, \
+ IntOption, Option
from trac.env import open_environment
from trac.prefs import IPreferencePanelProvider
from trac.util import hex_entropy
@@ -106,7 +107,7 @@ class AccountModule(CommonTemplateProvider):
def _reset_password_enabled(self, log=False):
try:
self.store.hash_method
- except AttributeError:
+ except (AttributeError, ConfigurationError):
return False
return is_enabled(self.env, self.__class__) and \
self.reset_password and (self._write_check(log) != []) and \
}}}
" defect closed normal AccountManagerPlugin normal fixed