Skip to:
Content

BuddyPress.org

Opened 23 months ago

Closed 22 months ago

Last modified 22 months ago

#8892 closed defect (bug) (fixed)

Need to removed unused variable.

Reported by: upadalavipul's profile upadalavipul Owned by: imath's profile imath
Milestone: 12.0.0 Priority: normal
Severity: normal Version:
Component: Core Keywords: has-patch
Cc:

Description

I have reviewed the BuddyPress code and found that some of the files in unused variables define and it's not used anywhere. Please check the below files.

Files:

  • src/bp-blogs/bp-blogs-activity.php
  • src/bp-blogs/bp-blogs-filters.php
  • src/bp-members/classes/class-bp-members-invitation-manager.php
  • src/bp-members/classes/class-bp-members-list-table.php
  • src/bp-members/classes/class-bp-members-ms-list-table.php

Attachments (9)

8892.patch​ (2.9 KB) - added by upadalavipul 23 months ago.
8892.2.patch​ (3.8 KB) - added by upadalavipul 23 months ago.
few more files in found unused variable.
8892.3.patch​ (2.8 KB) - added by upadalavipul 22 months ago.
few more files in found unused variable.
8892.4.patch​ (3.2 KB) - added by upadalavipul 22 months ago.
few more files in found unused variable.
8892.5.patch​ (3.2 KB) - added by upadalavipul 22 months ago.
few more files in found unused variable.
#8892.patch​ (1.7 KB) - added by kajalgohel 22 months ago.
Removed unused variable from the few files
8892.6.patch​ (917 bytes) - added by viralsampat 22 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
8892.7.patch​ (6.0 KB) - added by upadalavipul 22 months ago.
few more files in found unused variable.
8892.8.patch​ (1.1 KB) - added by upadalavipul 22 months ago.
few more files in found unused variable.

Download all attachments as: .zip

Change History (19)

@upadalavipul
23 months ago

#1 @imath
23 months ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 12.0.0

Thanks for your review & patch. I will check it asap 👌

#2 @imath
23 months ago

  • Owner set to imath
  • Resolution set to fixed
  • Status changed from new to closed

In 13476:

Remove never used inited variables in Blogs & Members components

Props upadalavipul

Fixes #8892

@upadalavipul
23 months ago

few more files in found unused variable.

@upadalavipul
22 months ago

few more files in found unused variable.

#3 @imath
22 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Thanks for these new patches @upadalavipul I'll check & commit them asap.

This ticket was mentioned in ​PR #113 on ​buddypress/buddypress by ​@imath.
22 months ago #4

Remove never used inited variables in various functions

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/8892

#5 @imath
22 months ago

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

In 13497:

Core, Activity & Notifications: remove never used inited variables

Props upadalavipul

Fixes #8892
Closes ​https://github.com/buddypress/buddypress/pull/113

@upadalavipul
22 months ago

few more files in found unused variable.

@upadalavipul
22 months ago

few more files in found unused variable.

#6 @imath
22 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Thanks for these new patches @upadalavipul 👍

This ticket was mentioned in ​PR #115 on ​buddypress/buddypress by ​@imath.
22 months ago #7

Remove more never used inited variables in various functions

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/8892

@kajalgohel
22 months ago

Removed unused variable from the few files

@viralsampat
22 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

#8 @imath
22 months ago

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

In 13500:

Core, Activity: remove more never used inited variables in functions

Props upadalavipul, kajalgohel, viralsampat

Fixes #8892
Closes ​https://github.com/buddypress/buddypress/pull/115

@upadalavipul
22 months ago

few more files in found unused variable.

@upadalavipul
22 months ago

few more files in found unused variable.

This ticket was mentioned in ​PR #118 on ​buddypress/buddypress by ​@imath.
22 months ago #9

Remove even more never used inited variables in various functions!

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/8892

#10 @imath
22 months ago

In 13504:

Core, Messages & Templates: remove never used inited vars in functions

Props upadalavipul

Fixes #8892
Closes ​https://github.com/buddypress/buddypress/pull/118

Note: See TracTickets for help on using tickets.