Closed (works as designed)
Project:
Real Name
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Apr 2009 at 04:53 UTC
Updated:
1 Oct 2009 at 15:49 UTC
Jump to comment: Most recent file
Comments
Comment #1
eromba commentedComment #2
nancydruThank you for the patch.
I'm not crazy about adding module-specific checks into this module because there are potentially 2,000 such requests waiting. I also cannot possibly download and test every module that is requested.
"Module_exists" can be too expensive to put into hook_nodeapi. Is there a function that can be checked with "function_exists" that would be a faster and reliable way? For example "authorship_menu" probably exists.
Comment #3
eromba commentedGood idea, Nancy. Here's an updated patch that checks for the "_authorship_name_rewrite" function instead of the Authorship module as a whole.
Comment #4
nancydruI can't be adding support for 4,000 individual modules; it would be too big a performance impact. That module needs to properly use theme_username.