WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#45420 closed defect (bug) (fixed)

requiring wp-admin/includes/image.php twice in WP_REST_Attachments_Controller::create_item

Reported by: david.binda Owned by: pento
Priority: normal Milestone: 5.0.2
Component: REST API Version: 5.0
Severity: minor Keywords: has-patch fixed-5.0
Cc: Focuses:

Description

The r43773 introduced require_once ABSPATH . 'wp-admin/includes/image.php' twice in WP_REST_Attachments_Controller::create_item method.

While the require_once mitigates the possible side-effect of redeclaring the same functions twice on the media endpoint, it should be really only required once, since it's enough :)

Attachments (1)

45420.diff​ (652 bytes ) - added by david.binda 8 years ago.

Download all attachments as: .zip

Change History (9)

@david.binda
8 years ago

#1 @desrosj
8 years ago

  • Keywords has-patch added
  • Milestone Awaiting Review → 5.0.1
  • Severity normal → minor

Thanks, @david.binda. Your patch looks good to me.

Marking this as a 5.0.1 item, but it's relatively minor and can be punted if need be (mainly depending on how quickly 5.0.1 is released).

#2 @desrosj
8 years ago

  • Keywords commit added

#3 @pento
8 years ago

  • Milestone 5.0.1 → 5.0.2

#4 @pento
8 years ago

  • Owner set to pento
  • Status new → assigned

#5 @pento
8 years ago

In 44206:

REST API: Remove a duplicate require_once().

In WP_REST_Attachments_Controller::create_item(), wp-admin/includes/image.php was being require_once()-ed... twice. The superflous require_once() has been quietly removed.

Props david.binda.
See #45420.

#6 @pento
8 years ago

  • Keywords fixed-5.0 added

#7 @desrosj
8 years ago

  • Keywords commit removed

#8 @SergeyBiryukov
8 years ago

  • Resolution → fixed
  • Status assigned → closed

In 44309:

REST API: Remove a duplicate require_once().

In WP_REST_Attachments_Controller::create_item(), wp-admin/includes/image.php was being require_once()-ed... twice. The superflous require_once() has been quietly removed.

Props david.binda.
Merges [44206] to trunk.
Fixes #45420.

Note: See TracTickets for help on using tickets.