Skip to content

Eliminate non-existent index notices #1

Merged
merged 1 commit into from Dec 29, 2013

5 participants

@TheFrozenFire

Adds an isset($this->cchunk["appendlater"]) check to lines 425 and 433 to eliminate non-existent index notices.

@TheFrozenFire TheFrozenFire Eliminate non-existent index notices
Adds an isset($this->cchunk["appendlater"]) check to lines 425 and 433 to eliminate non-existent index notices.
fd1b439
@whatthejeff

I just ran into this issue as well. Might be simpler to do:

!empty($this->cchunk["appendlater"])

instead of:

isset($this->cchunk["appendlater"]) && $this->cchunk["appendlater"]
@mumumu
mumumu commented Jun 27, 2013

whatthejeff

+1. your fix is simpler.
I ran into this issue just now. This is a minor issue, but want to include fix in next minor release.

@php-pulls php-pulls merged commit fd1b439 into php:master Dec 29, 2013
@bjori
php.net member
bjori commented Dec 29, 2013

I'm so sorry, I never saw this PR!
Merged now, thanks!

@TheFrozenFire TheFrozenFire deleted the TheFrozenFire:patch-1 branch Dec 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.