Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim’s picture

Issue tags: +Needs backport to D6, +Novice

Tagging.

joachim’s picture

Status: Active » Needs review
FileSize
1 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

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

Status: Postponed » Active

Hook changes made; this can now be looked at.

joachim’s picture

Status: Active » Needs review
FileSize
903 bytes

Status: Needs review » Needs work

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

socketwench’s picture

Status: Needs work » Reviewed & tested by the community

Looks good to me. Tests clean.

joachim’s picture

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

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

Backport to 7.x-2.x.

socketwench’s picture

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

Status: Needs work » Needs review
FileSize
887 bytes
socketwench’s picture

Version: 7.x-2.x-dev » 6.x-2.x-dev
FileSize
1.01 KB

D6 backport.

joachim’s picture

Status: Needs review » Fixed

Committed. Thanks!

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