Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#56333 closed enhancement (fixed)

Remove _wp_image_belongs_to_attachment function

Reported by: mukesh27's profile mukesh27 Owned by: adamsilverstein's profile adamsilverstein
Milestone: 6.1 Priority: normal
Severity: normal Version: 6.1
Component: Media Keywords: has-patch
Focuses: Cc:

Description

In #55443, we introduced _wp_image_belongs_to_attachment() to skip already existing image files, but we changed the approach to ensure unique filenames and changed the generate_filename function.

Check out the comment on the PR here: https://github.com/WordPress/wordpress-develop/pull/2393#issuecomment-1185012318

Change History (6)

This ticket was mentioned in PR #3061 on WordPress/wordpress-develop by mukeshpanchal27.


3 years ago
#1

  • Keywords has-patch added; needs-patch removed

#2 @adamsilverstein
3 years ago

Good catch @mukesh27 this was part of the original code, but we changed the approach and this is no longer used or needed.

This ticket was mentioned in PR #3070 on WordPress/wordpress-develop by adamsilverstein.


3 years ago
#3

Trac ticket:

#4 @adamsilverstein
3 years ago

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

In 53845:

Media: remove unused _wp_image_belongs_to_attachment function.

Remove the _wp_image_belongs_to_attachment function which was left over from a an unused branch and added inadvertently as part of r53751.

Props mukesh27.
Fixes #56333.

This ticket was mentioned in Slack in #core-media by antpb. View the logs.


3 years ago

Note: See TracTickets for help on using tickets.