[wp-trac] [WordPress Trac] #54111: Delete attachment with sizes not working on Windows file system

WordPress Trac noreply at wordpress.org
Sun Jan 7 17:15:32 UTC 2024


#54111: Delete attachment with sizes not working on Windows file system
-------------------------------------------------+-------------------------
 Reporter:  kybernetikservices                   |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  6.5
Component:  Media                                |     Version:  5.8.1
 Severity:  normal                               |  Resolution:
 Keywords:  reporter-feedback has-patch dev-     |     Focuses:
  feedback needs-testing                         |
-------------------------------------------------+-------------------------
Changes (by bgoewert):

 * keywords:
     reporter-feedback has-patch dev-feedback needs-testing needs-testing-
     info
     => reporter-feedback has-patch dev-feedback needs-testing


Comment:

 == Reproduction Report
 After hours of setting up IIS and testing, I was unable to reproduce the
 issue.

 @kybernetikservices, if you could provide a report similar to this and
 test a bare wordpress installation with the plugin provided here, that
 would be most helpful. This will verify whether it is an issue with
 WordPress or your server setup.

 But from what I can tell, at least with 6.4, it seems to be either an
 issue with a plugin or a configuration issue with the web server.

 === Environment
 - OS: Pop_OS! 22.04
 - Server: IIS
 - PHP: 8.2
 - WP: 6.4
 - Browser: Firefox
 - Theme: TT4
 - Active Plugins:
 [https://core.trac.wordpress.org/attachment/ticket/54111/54111_v1.php
 54111_v1.php]

 === Actual Results
 - ❌ All attachements and size variations were deleted upon post deletion
 (not reproduced)


 === Additional Notes
 - IIS is a pain
 - The `wp_delete_file_from_directory()` function, called in
 `wp_delete_attachment_files()`, utlizes `wp_normalize_path()` ''and''
 `realpath()` to validate the path before deleting, similar to what
 `path_is_absolute()` does.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/54111#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list