Posted by NancyDru on July 16, 2009 at 4:06pm
2 followers
Jump to:
| Project: | Drupal core |
| Version: | 6.x-dev |
| Component: | comment.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
| Issue tags: | comment delete, Performance |
Issue Summary
In order for contribs to call comment_multiple_delete_confirm(), they must precede this call with a module_load_include('inc', 'comment', 'comment.admin'). If the delete function is in comment.module, the extra load is unnecessary. If I can get CVS working again for D7, I will create a patch, but it is very simple: just cut the function from comment.admin.inc and paste it into comment.module.
This problem was found in D6, so it would be real nice if the fix could be back-ported.
Comments
#1
Irrelevant for D7, just drupal_function_exists() that function.