Problem/Motivation

See title, this seems to happen for errors that do not have a method as source but are e.g. parsed from the php error log file. Only happens if you use --xml I think.

Proposed resolution

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#3 prevent-notices-if-function-is-not-a-method-2417727-3-interdiff.txt605 bytesBerdir
#3 prevent-notices-if-function-is-not-a-method-2417727-3.patch802 bytesBerdir
#1 prevent-notices-if-function-is-not-a-method-2417727-1.patch801 bytesBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Active » Needs review
FileSize
prevent-notices-if-function-is-not-a-method-2417727-1.patch801 bytes

Hard to test, but I'll try adding this as a patch to my d8modulestatus builds to see if it helps to get rid of the annoying notices in http://d8ms.worldempire.ch/build.log.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

Btw, we currently also create filenames like "Drupal\somemodule\Tests\SomeTest.xml", which a lot of operating systems and tools can't handle properly, but that's a separate issue.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
FileSize
prevent-notices-if-function-is-not-a-method-2417727-3.patch802 bytes
prevent-notices-if-function-is-not-a-method-2417727-3-interdiff.txt605 bytes

Berdir--.

Berdir queued 3: prevent-notices-if-function-is-not-a-method-2417727-3.patch for re-testing.

bzrudi71’s picture

bzrudi71 CreditAttribution: bzrudi71 commented
Status: Needs review » Reviewed & tested by the community

Nice, no more notices with patch applied. Found nothing obvious during review so RTBC.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

@bzrudi71: Right, I saw those notices in your postgresql/sqlite logs as well, wanted to ping you about it :) Thanks.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed c42d69a and pushed to 8.0.x. Thanks!

  • alexpott committed c42d69a on 8.0.x 
    Issue #2417727 by Berdir: Notice: Undefined offset: 1 in core/scripts/...

Status: Fixed » Closed (fixed)

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