﻿id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
10798	RegexUrlResolver's _get_reverse_dict appends objects of unknown type to the reverse dict	fas	fas	"In _get_reverse_dict:
{{{
if isinstance(pattern, RegexURLResolver):
    # ...
else:
    bits = normalize(p_pattern)
    self._reverse_dict.appendlist(pattern.callback, (bits, p_pattern))
    self._reverse_dict.appendlist(pattern.name, (bits, p_pattern))
}}}

In the else branch it is implicitly assumed that pattern is of type RegexURLPattern (it uses its properties). This does not have to be the case. For example, one could create a different kind of RegexURLResolver which does not inherit RegexURLResolver. This resolver would be handled as a RegexURLPattern instance and added to the reverse dict. 
Patch is a short one: replace ""else"" with ""elif isinstance(pattern, RegexURLPattern):""."		closed	Core (Other)	1.0		wontfix	url, urlresolver, regexurlpattern, regexurlresolver		Unreviewed	1	0	0	0	0	0
