Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.This is part of the effort to break up core page handlers that no longer have to be in the main modules. So far I split out just comment_admin_settings(). I tried cutting out comment_admin() and related functions, but I keep getting a white screen and I don't know how to fix it. Maybe someone with more skills can step in.
For more info see the drupal groups page at http://groups.drupal.org/node/4179/
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | comment_49.patch | 31.43 KB | Crell |
| #1 | comment_48.patch | 37.93 KB | Crell |
| comment_pages.patch | 9.41 KB | bennybobw | |
Comments
Comment #1
Crell CreditAttribution: Crell commentedHere's a more complete version.
Comment #2
catchComment module has been split already.
Comment #3
Crell CreditAttribution: Crell commentedNo it has not. It had theme functions factored out, not page handlers. That still needs to be done, but of course the module has been updated since this patch was written so it no longer applies.
Comment #4
catchWhoops, sorry!
Comment #5
Crell CreditAttribution: Crell commentedTake 3...
Comment #6
catchPatch applied cleanly.
I added, deleted, edited and previewed comments. Changed settings, enabled comment controls and played with those. No errors, notices, warnings or anything else unexpected.
RTBC.
Comment #7
Dries CreditAttribution: Dries commentedGood job. I've committed this to CVS HEAD. Thanks.
Comment #8
(not verified) CreditAttribution: commentedAutomatically closed -- issue fixed for two weeks with no activity.