Faced problem on multilingual site where pcp module is used. Possible solution is to replace key 'und' with constant LANGUAGE_NONE

CommentFileSizeAuthor
#5 replace_und_with_language_none-2484811-5.patch732 bytesBarisW
#1 replace_und_with_language_none-2484811-1.patch731 bytesuladzimir.karol
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

uladzimir.karol’s picture

uladzimir.karol CreditAttribution: uladzimir.karol commented
FileSize
replace_und_with_language_none-2484811-1.patch731 bytes

Here is my quick fix 'und'->LANGUAGE_NONE for pcp

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Active » Needs review

Thanks, good catch. Committed it right away, but forgot to run the tests first. Please set issues with patched to "Needs review" to trigger the test bot.

Status: Needs review » Needs work

The last submitted patch, 1: replace_und_with_language_none-2484811-1.patch, failed testing.

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Needs work » Needs review
FileSize
replace_und_with_language_none-2484811-5.patch732 bytes

Here's the same patch, rerolled.

Status: Needs review » Needs work

The last submitted patch, 5: replace_und_with_language_none-2484811-5.patch, failed testing.

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Needs work » Fixed

Don't know why the patch fails (maybe because I already committed it). Let's close this.

Status: Fixed » Closed (fixed)

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