WordPress.org

Make WordPress Core

Opened 6 months ago

Last modified 6 weeks ago

#48267 new defect (bug)

IXR: Modernize IXR library

Reported by: kraftbj Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: XML-RPC Keywords:
Focuses: Cc:

Description

Motivated by #47678, I started updating func_get_args calls in a plugin that extended the IXR classes provided by Core and realized that I was getting ahead of myself and Core.

With this coming from a seemingly abandoned upstream library where we haven't modernize the code in some time, before doing more (PHPCS, inline docs, etc), what is the status of this part of the code? Open to being modernized?

Attaching a simple patch for the spread operator to start the discussion.

Subtasks:

  • Remove PHP 4 and pre-PHP 5.6 conventions.
  • PHPCS.
  • Inline documentation.

Related: #46484

Attachments (1)

48267.diff (1.4 KB) - added by kraftbj 6 months ago.
Add spread operator

Download all attachments as: .zip

Change History (4)

@kraftbj
6 months ago

Add spread operator

#1 @SergeyBiryukov
6 months ago

  • Milestone changed from Awaiting Review to 5.4

#2 in reply to: ↑ description @SergeyBiryukov
5 months ago

Replying to kraftbj:

With this coming from a seemingly abandoned upstream library where we haven't modernize the code in some time, before doing more (PHPCS, inline docs, etc), what is the status of this part of the code? Open to being modernized?

Looking at the recent commits, of which [38389] is probably the most substantial:

it appears to be treated as "adopted" rather than external, so I think we should bring it in line with the rest of core.

#3 @audrasjb
6 weeks ago

  • Milestone changed from 5.4 to Future Release

Hi,

With 5.4 Beta 3 approaching and the Beta period reserved for bugs introduced during the cycle, this is being moved to Future Release. If any maintainer or committer feels this should be included or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.

Note: See TracTickets for help on using tickets.