Make WordPress Core

Opened 2 years ago

Closed 15 months ago

Last modified 15 months ago

#55132 closed enhancement (fixed)

Unused variable in in WP_Plugins_List_Table::single_row()

Reported by: azouamauriac's profile azouamauriac Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.3 Priority: normal
Severity: normal Version:
Component: Plugins Keywords:
Focuses: coding-standards Cc:

Description

in WP_Plugins_List_Table::single_row there some unused variables ($available_updates, $sortable, $primary).

some histories: [47835] [32644]

Attachments (2)

55132.diff (714 bytes) - added by azouamauriac 2 years ago.
55132.2.diff (792 bytes) - added by azouamauriac 2 years ago.
Remove space for phpcs. this last is good.

Download all attachments as: .zip

Change History (5)

@azouamauriac
2 years ago

@azouamauriac
2 years ago

Remove space for phpcs. this last is good.

#1 @SergeyBiryukov
15 months ago

  • Milestone changed from Awaiting Review to 6.3
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

Hi there, thanks for the patch!

It looks like this exact line also exists in a few other list tables:

list( $columns, $hidden, $sortable, $primary ) = $this->get_column_info();

So I think I'd like to leave it as is for consistency, even though $sortable and $primary are not used here.

The other change looks good to me.

#2 @SergeyBiryukov
15 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 55571:

Coding Standards: Remove unused variable in WP_Plugins_List_Table::single_row().

Follow-up to [47835], [48669].

Props azouamauriac.
Fixes #55132.

#3 @SergeyBiryukov
15 months ago

  • Summary changed from Unused variable to Unused variable in in WP_Plugins_List_Table::single_row()
Note: See TracTickets for help on using tickets.