WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

#49163 closed defect (bug) (fixed)

Some typing mistakes in \wp-admin\includes\class-pclzip.php

Reported by: passoniate Owned by: SergeyBiryukov
Priority: normal Milestone: 5.4
Component: Filesystem API Version: 5.4
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

Some typing mistakes in \wp-admin\includes\class-pclzip.php

Attachments (1)

class-pclzip.php.patch​ (4.4 KB ) - added by passoniate 7 years ago.
patch

Download all attachments as: .zip

Change History (3)

@passoniate
7 years ago

patch

#1 @SergeyBiryukov
7 years ago

  • Component General → Filesystem API
  • Milestone Awaiting Review → 5.4
  • Owner set to SergeyBiryukov
  • Status new → reviewing

Thanks for the patch!

Per comment:3:ticket:38464, class-pclzip.php used to be treated as an external library and was excluded from any documentation fixes. We only modify third-party libraries when it's needed to fix a bug, like in [46105/trunk/src/wp-admin/includes/class-pclzip.php] or [38101/trunk/src/wp-admin/includes/class-pclzip.php].

That said, looking at the commit history, there were some whitespace changes in [12042/trunk/wp-admin/includes/class-pclzip.php] and documentation additions in [8645/trunk/wp-admin/includes/class-pclzip.php].

Given that the ​upstream library hasn't been updated in 10 years since the last time it was merged, I think it's safe to treat it as "adopted" rather than external, same as jquery.imgareaselect.js or class-IXR.php, and make these fixes.

#2 @SergeyBiryukov
6 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 47123:

Docs: Fix typos in wp-admin/includes/class-pclzip.php.

With the upstream library not updated in 10 years since the last time it was merged, it should be safe to treat it as "adopted" rather than external, and make these fixes.

Props passoniate, ottok.
Fixes #49163. See #38464.

Note: See TracTickets for help on using tickets.