Bullet-proof some extension and URL uninitialization code.
Review Request #5849 — Created May 21, 2014 and submitted — Latest diff uploaded
We hit some nasty issues where uninitializing an extension failed hard,
preventing all future attempts at uninitializing. These had to do with
the dynamically-set/removedextension.admin_site
attribute, which was
assumed to be available ifhas_admin_site
wasTrue
.We now check for the existence of
admin_site
instead, and also catch
attempts at removing non-existing URLs inDynamicURLResolver
.
Unit tests passed.