WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#57907 closed defect (bug) (fixed)

Fix WP_Error in copy_dir()

Reported by: afragen Owned by: SergeyBiryukov
Priority: normal Milestone: 6.3
Component: Filesystem API Version: 5.7
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

The WP_Error returned for ::dirlist() has an incorrect data argument. It should be based on the $from.

Change History (8)

This ticket was mentioned in ​PR #4222 on ​WordPress/wordpress-develop by ​@afragen.
4 years ago #1

The WP_Errror returned in copy_dir() for ::dirlist() seems to be incorrect. It should be $from not $to.

Trac ticket: https://core.trac.wordpress.org/ticket/57907

#2 @costdev
4 years ago

  • Milestone Awaiting Review → 6.3

#3 @costdev
4 years ago

  • Keywords commit added

​PR 4222 looks good to me. ::dirlist() is run on $from, so basename( $from ) is the correct thing to pass to the WP_Error object.

Marking for commit consideration.

#4 @afragen
4 years ago

  • Version 5.1.1 → 5.7

#5 @pravinparmar2404
4 years ago

Tested the patch and it looks good to me.

#6 @SergeyBiryukov
4 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 55538:

Filesystem API: Return correct error data from copy_dir() if the directory listing failed.

The ::dirlist() method is run on the $from directory, so basename( $from ) is the correct thing to pass to the WP_Error object.

Follow-up to [50149].

Props afragen, costdev, pravinparmar2404.
Fixes #57907.

​@SergeyBiryukov commented on ​PR #4222:
4 years ago #7

Thanks for the PR! Merged in r55538.

#8 @hellofromTonya
4 years ago

@SergeyBiryukov should this be reopened to backport to 6.2 branch? Never mind, re-read > I thought this was addressing the move dir introduced in 6.2. Sorry for the confusion.

Last edited 4 years ago by hellofromTonya (previous) (diff)
Note: See TracTickets for help on using tickets.