This was pretty before I touched it, just a couple idiosyncrasies. Still worried that our ctools_form_comment_form_alter() has a chance of not getting included in the module_implements cache, but having trouble reproducing said bug. If we do end up finding this is a problem I think we can utilize hook_module_implements_alter() but my first patch will not include that as we really need to test this and see if we can figure out the conditions in which it breaks.
The most promising (logically) candidate for this is to clear cache and visit a non-ctools generated comment_form page (comment/reply/#node) or something, but even under these conditions it appears to be getting picked up, so I'm punting and asking for help at this point.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 954762_1.patch | 972 bytes | eclipsegc |
| #1 | 954762.patch | 874 bytes | eclipsegc |
Comments
Comment #1
eclipsegc commentedComment #2
eclipsegc commentedComment #3
eclipsegc commentedfixed the comment_post_forbidden theme call.
Comment #4
merlinofchaos commentedCommitted.