Problem/Motivation

There are many php notices in the newsletter editions tab. see attached picture.

Proposed resolution

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 many_php_notices_in-2455525-1.patch673 bytesAnushka-mp
Screen Shot 2015-03-19 at 14.09.28.png195.38 KBAnushka-mp
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anushka-mp’s picture

Anushka-mp CreditAttribution: Anushka-mp at MD Systems GmbH commented
Issue summary: View changes
FileSize
many_php_notices_in-2455525-1.patch673 bytes
Anushka-mp’s picture

Anushka-mp CreditAttribution: Anushka-mp at MD Systems GmbH commented
Status: Active » Needs review
miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented

Oh, head is currently broken? https://qa.drupal.org/pifr/test/1002418
I guess we need to fix this, too...

Status: Needs review » Needs work

The last submitted patch, 1: many_php_notices_in-2455525-1.patch, failed testing.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs work » Reviewed & tested by the community

Yes, exactly. Patch is green and the notices are visible on the re-tested branch test: https://qa.drupal.org/pifr/test/977208

I think I still can't commit, so just setting to RTBC for now.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Fixed.

Status: Fixed » Closed (fixed)

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