Make WordPress Core

Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#42837 closed defect (bug) (fixed)

wp_normalize_path does not take into account stream wrappers

Reported by: calin Owned by: dd32
Priority: normal Milestone: 4.9.7
Component: Filesystem API Version: 3.9
Severity: normal Keywords:
Cc: Focuses:

Description

wp_normalize_path does not take into account stream paths using php stream wrappers. I've attached a patch to fix this.

Attachments (2)

wp_normalize_path.diff​ (632 bytes ) - added by calin 9 years ago.
wp_normalize_path_tests.diff​ (580 bytes ) - added by calin 9 years ago.

Download all attachments as: .zip

Change History (11)

#1 @swissspidy
9 years ago

  • Component General → Filesystem API
  • Keywords has-patch added
  • Version trunk → 3.9

#2 @dd32
9 years ago

  • Keywords needs-unit-tests added

This would be a good candidate for some unit tests covering the current and new behaviour here.

#3 @calin
9 years ago

@dd32 I've attached a patch for tests.

#4 @dd32
9 years ago

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

In 42387:

Filesystem: Allow wp_normalise_path() to handle PHP stream wrappers such as php:// correctly.

Props calin, dd32.
Fixes #42837.

#5 @dd32
9 years ago

  • Milestone Awaiting Review → 5.0

#6 @johnbillion
8 years ago

In 43406:

Media: Update the @since tag for wp_normalize_path() as this has been backported to th 3.9 (and all other) branches.

See #42837

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


8 years ago

#8 @johnbillion
8 years ago

  • Keywords has-patch needs-unit-tests removed
  • Resolution fixed
  • Status closed → reopened

[42387] and [43406] should be merged into 5.0

#9 @pento
8 years ago

  • Milestone 5.0 → 4.9.7
  • Resolution → fixed
  • Status reopened → closed

They were both merged into the 4.9 branch in [43393], so already exist in the 5.0 branch.

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