WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#9068 closed defect (bug) (fixed)

HTTP API Cookies trigger E_NOTICE

Reported by: beaulebens's profile beaulebens Owned by:
Milestone: 2.8 Priority: low
Severity: minor Version: 2.8
Component: HTTP API Keywords:
Focuses: Cc:

Description

When creating a WP_Http_Cookie using a source array, if you don't set all elements then it generates E_NOTICE errors.

Patch attached checks values before trying to use them.

Attachments (3)

cookie-notices.diff​ (989 bytes) - added by beaulebens 17 years ago.
9068.sivel.diff​ (527 bytes) - added by sivel 17 years ago.
cookies_check.9068.diff​ (640 bytes) - added by filosofo 17 years ago.

Download all attachments as: .zip

Change History (12)

@beaulebens
17 years ago

#1 @azaozz
17 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [10524]) Fix PHP notices in HTTP API Cookies, props beaulebens, fixes #9068

#2 @westi
17 years ago

  • Version set to 2.8

#3 @sivel
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#4 @sivel
17 years ago

Update for one additional place in the HTTP API that causes notice in new cookies functionality.

@sivel
17 years ago

#5 @ryan
17 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [10633]) Fix notice. Props sivel. fixes #9068

#6 follow-up: @aaroncampbell
17 years ago

Is there a reason that we couldn't use empty() in [10633] rather than isset() and count()?

#7 @filosofo
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Also should probably cast cookies to array. Patch attached.

@filosofo
17 years ago

#8 in reply to: ↑ 6 @sivel
17 years ago

Replying to aaroncampbell:

Is there a reason that we couldn't use empty() in [10633] rather than isset() and count()?

That does sound better. I guess I should have looked closer at what the code was doing rather than just a way to stop the notice.

#9 @westi
17 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [10639]) Better Notice fixing and place a protective cast in the cookie handler. Fixes #9068 props filosofo.

Note: See TracTickets for help on using tickets.