flag_flag_definitions() needs a module_exists().

CommentFileSizeAuthor
#15 1736486.15.flag_conditional_comment6x2x.patch1.01 KBsocketwench
#14 1736486.14.flag_conditional_comment7x2x.patch887 bytessocketwench
#11 1736486.11.flag_conditional_comment_7x2x.patch896 bytessocketwench
#6 1736486.6.flag_.conditional-comment-type.patch903 bytesjoachim
#2 1736486.flag_.conditional-comment-module.patch1 KBjoachim

Comments

joachim’s picture

joachim commented
Issue tags: +Needs backport to D6, +Novice

Tagging.

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 1736486.flag_.conditional-comment-module.patch1 KB

Let's see if this patch I wrote on 6 works on 7!!!!

Status: Needs review » Needs work

The last submitted patch, 1736486.flag_.conditional-comment-module.patch, failed testing.

joachim’s picture

joachim commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Status: Needs work » Postponed

I'm an idiot -- this needs fixing on 7--3 too.

Postponing until the hook changes are decided on.

joachim’s picture

joachim commented
Status: Postponed » Active

Hook changes made; this can now be looked at.

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 1736486.6.flag_.conditional-comment-type.patch903 bytes

Status: Needs review » Needs work

The last submitted patch, 1736486.6.flag_.conditional-comment-type.patch, failed testing.

socketwench’s picture

socketwench commented
Status: Needs work » Reviewed & tested by the community

Looks good to me. Tests clean.

joachim’s picture

joachim commented
Status: Reviewed & tested by the community » Fixed

Thanks for the review!

Issue #1736486 by joachim: Fixed comment flags are defined when comment module is absent.

joachim’s picture

joachim commented
Version: 7.x-3.x-dev » 7.x-2.x-dev
Status: Fixed » Patch (to be ported)

Needs porting to 7-2.

socketwench’s picture

socketwench commented
StatusFileSize
new 1736486.11.flag_conditional_comment_7x2x.patch896 bytes

Backport to 7.x-2.x.

socketwench’s picture

socketwench commented
Status: Patch (to be ported) » Needs review

Status: Needs review » Needs work

The last submitted patch, 1736486.11.flag_conditional_comment.patch, failed testing.

socketwench’s picture

socketwench commented
Status: Needs work » Needs review
StatusFileSize
new 1736486.14.flag_conditional_comment7x2x.patch887 bytes
socketwench’s picture

socketwench commented
Version: 7.x-2.x-dev » 6.x-2.x-dev
StatusFileSize
new 1736486.15.flag_conditional_comment6x2x.patch1.01 KB

D6 backport.

joachim’s picture

joachim commented
Status: Needs review » Fixed

Committed. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.