WordPress.org
Get WordPress

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#58101 closed defect (bug) (wontfix)

Remove the Unused variables in the wp-includes folder.

Reported by: upadalavipul's profile upadalavipul Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: External Libraries Keywords: close
Focuses: coding-standards Cc:

Description

I have reviewed the code and found Unused variables in some of the below files. Please below files:

Files:

  1. wp-includes/ID3/module.audio-video.quicktime.php
  2. wp-includes/ID3/module.tag.lyrics3.php

Attachments (1)

58101.patch​ (1.8 KB) - added by upadalavipul 2 years ago.

Download all attachments as: .zip

Change History (3)

@upadalavipul
2 years ago

#1 @audrasjb
2 years ago

  • Component changed from General to External Libraries
  • Keywords close added
  • Version trunk deleted

Hello and thanks for the ticket/patch,
However these files are part of the ID3 external library so the issue should be reported upstream:
​https://github.com/JamesHeinrich/getID3

Last edited 2 years ago by audrasjb (previous) (diff)

#2 @SergeyBiryukov
2 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Hi there, thanks for the patch!

As noted above, GetID3 is an external library and should not be patched here, any suggested changes to its files should be submitted upstream.

Note: See TracTickets for help on using tickets.