The current code in project module allows project/X alias that could break project_issues.module or project.module.
Attached patch tries to be more thorough in preventing this.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | proj_name_blacklist_2.patch | 1.57 KB | dww |
| proj_name_blacklist_1.patch | 1.57 KB | pwolanin |
Comments
Comment #1
dwwGood catch. However, this taxonomy checking is over-agressive. You're excluding all possible project-related terms, even the module categories. This is bad, since A) some module categories are valid project names and B) project/[module_category] links are never used.
New patch that fixes this (tiny change). Otherwise, looks good [sic] given the insane handling of menu paths in project*. :(
Comment #2
dwwAfter more review and a little testing, committed to HEAD. Thanks.
Comment #3
(not verified) commented