Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#10119 closed defect (bug) (fixed)

Wrong data type in edit-page-form.php

Reported by:	pairg	Owned by:	pairg
Priority:	normal	Milestone:	2.8.1
Component:	Warnings/Notices	Version:	2.8
Severity:	minor	Keywords:	has-patch tested
Cc:		Focuses:

Description

PHP Warning in "/wp-admin/page.php?action=edit&post=<number>"

Warning: in_array() [function.in-array]: Wrong datatype for second argument in /wp-admin/edit-page-form.php on line 355

Attachments (3)

edit-page-form_rev11553_patch.patch (617 bytes ) - added by pairg 17 years ago.: /wp-admin/edit-page-form.php patch
10119.diff (416 bytes ) - added by Denis-de-Bernardy 17 years ago.
10119.2.diff (416 bytes ) - added by Denis-de-Bernardy 17 years ago.

Download all attachments as: .zip

Change History (17)

@pairg
17 years ago

Attachment edit-page-form_rev11553_patch.patch added

/wp-admin/edit-page-form.php patch

#1 @pairg
17 years ago

Owner set to pairg
Status new → assigned

#2 @pairg
17 years ago

Resolution → fixed
Status assigned → closed

#3 @pairg
17 years ago

How I can commit my changes to SVN?
Please somebody commit my patch!

#4 @ShaneF
17 years ago

Resolution fixed
Status closed → reopened

It must be tested first before it is committed. I am re-opening this so we can test it. Please be patient!

#5 @Denis-de-Bernardy
17 years ago

Keywords dup added

testing report 17

#6 @Denis-de-Bernardy
17 years ago

Keywords dup removed

sweet :-)

@Denis-de-Bernardy
17 years ago

Attachment 10119.diff added

#7 @Denis-de-Bernardy
17 years ago

Issue with the original patch is that (array) false === array(0=>false).

attached a slightly different patch, but it may have side effects

#8 @Denis-de-Bernardy
17 years ago

Priority high → normal

#9 follow-up: ↓ 11 @Denis-de-Bernardy
17 years ago

@pairg -- how did you end up with this error in the first place? never ran into it myself.

#10 @Denis-de-Bernardy
17 years ago

Keywords reporter-feedback added

#11 in reply to: ↑ 9 @ShaneF
17 years ago

Replying to Denis-de-Bernardy:

@pairg -- how did you end up with this error in the first place? never ran into it myself.

Ditto.

@Denis-de-Bernardy
17 years ago

Attachment 10119.2.diff added

#12 @Denis-de-Bernardy
17 years ago

Keywords tested added; reporter-feedback removed

#13 @ryan
17 years ago

Resolution → fixed
Status reopened → closed

(In [11571]) Return empty array instead of false from get_editable_user_ids. Props Denis-de-Bernardy, pairg. fixes #10119 for trunk

#14 @ryan
17 years ago

(In [11572]) Return empty array instead of false from get_editable_user_ids. Props Denis-de-Bernardy, pairg. fixes #10119 for 2.8.1

Note: See TracTickets for help on using tickets.

Download in other formats: