WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#10119 closed defect (bug) (fixed)

Wrong data type in edit-page-form.php

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

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 5 years ago.
/wp-admin/edit-page-form.php patch
10119.diff (416 bytes) - added by Denis-de-Bernardy 5 years ago.
10119.2.diff (416 bytes) - added by Denis-de-Bernardy 5 years ago.

Download all attachments as: .zip

Change History (17)

pairg5 years ago

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

comment:1 pairg5 years ago

  • Owner set to pairg
  • Status changed from new to assigned

comment:2 pairg5 years ago

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

comment:3 pairg5 years ago

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

comment:4 ShaneF5 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

comment:5 Denis-de-Bernardy5 years ago

  • Keywords dup added

testing report 17

comment:6 Denis-de-Bernardy5 years ago

  • Keywords dup removed

sweet :-)

Denis-de-Bernardy5 years ago

comment:7 Denis-de-Bernardy5 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

comment:8 Denis-de-Bernardy5 years ago

  • Priority changed from high to normal

comment:9 follow-up: Denis-de-Bernardy5 years ago

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

comment:10 Denis-de-Bernardy5 years ago

  • Keywords reporter-feedback added

comment:11 in reply to: ↑ 9 ShaneF5 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-Bernardy5 years ago

comment:12 Denis-de-Bernardy5 years ago

  • Keywords tested added; reporter-feedback removed

comment:13 ryan5 years ago

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

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

comment:14 ryan5 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.