Introduced by #3053165: [Addressbook part 2] Complete the UI by allowing choice between multiple addressbook profiles, doesn't trigger a notice cause it's wrapped in an empty() check.

CommentFileSizeAuthor
#4 3068649-4-undefined-variable.patch952 bytesbojanz

Comments

bojanz created an issue. See original summary.

bojanz credited NicolasGraph.

bojanz’s picture

bojanz commented

Crediting NicolasGraph, who caught it.

bojanz’s picture

bojanz commented
Status: Active » Needs review
StatusFileSize
new 3068649-4-undefined-variable.patch952 bytes

Let's see if this breaks any tests.

Status: Needs review » Needs work

The last submitted patch, 4: 3068649-4-undefined-variable.patch, failed testing. View results

  • bojanz committed 60d9919 on 8.x-2.x 
    Issue #3068649 by bojanz, NicolasGraph: Undefined variable $...
bojanz’s picture

bojanz commented
Status: Needs work » Fixed

Okay, the test failures are legit. We don't actually want or need the triggering element check. Removed it.

Status: Fixed » Closed (fixed)

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