If you don't, you get a big ol' fatal error about theme_comment being declared twice.

I'm thinking an extra submit handler on the admin/build/modules screen that disables the module, along with a drupal_set_message() saying the two are incompatible.

CommentFileSizeAuthor
#3 disable-comment.patch1.22 KBwebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Closed (fixed)

Nevermind! This has already been taken care of in the newest CVS version.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Closed (fixed) » Active

Ah, but the Fatal error: Cannot redeclare views_query_handler_field_last_comment_name() wasn't. ;)

So setting back to active. Will take a look after supper. http://drupal.org/node/54463 might have some clues.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Needs review
StatusFileSize
new disable-comment.patch1.22 KB

Here we go.

This patch does two things:
#1: Disables comment module whenever node comment is enabled.
#2: Makes the comment module checkbox disabled so that it won't get re-enabled accidentally.

robertdouglass’s picture

robertdouglass commented
Version: » 5.x-1.0-beta
Status: Needs review » Fixed

After a quick review I applied. Further testing needed, but that's why it's BETA.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)