Context Navigation

← Previous Ticket
Next Ticket →

#61420 closed defect (bug) (fixed)

Plugins: Remove unassigned sprintf from wp_get_plugin_action_button()

Reported by:	hellofromTonya	Owned by:	hellofromTonya
Priority:	normal	Milestone:	6.5.5
Component:	Plugins	Version:	6.5
Severity:	minor	Keywords:	has-patch commit dev-reviewed
Cc:		Focuses:

Description

[57545] introduced wp_get_plugin_action_button(). After speaking with @costdev, it appears an unassigned sprintf was accidentally included (the code line starts here). A copy/paste whoopsie.

        sprintf(
                '<a class="install-now button" data-slug="%s" href="%s" aria-label="%s" data-name="%s" role="button">%s</a>',
                esc_attr( $data->slug ),
                esc_url( $status['url'] ),
                /* translators: %s: Plugin name and version. */
                esc_attr( sprintf( _x( 'Install %s now', 'plugin' ), $name ) ),
                esc_attr( $name ),
                _x( 'Install Now', 'plugin' )
        );

As this code does not assign to a variable, it is not used and thus dead code. This ticket will remove it.

Change History (10)

This ticket was mentioned in PR #6792 on WordPress/wordpress-develop by @hellofromTonya.

2 years ago #1

Keywords has-patch added

Removes the unused / unassigned sprintf code from wp_get_plugin_action_button().

Trac ticket: https://core.trac.wordpress.org/ticket/61420

@rajinsharwar commented on PR #6792:

2 years ago #2

Wondering how this was here, unnoticed...

#3 @hellofromTonya
2 years ago

Keywords commit added

Patch: https://github.com/WordPress/wordpress-develop/pull/6792

2 approvals. CI all green. Committing.

#4 @hellofromTonya
2 years ago

Resolution → fixed
Status assigned → closed

In 58402:

Plugins: Remove unassigned sprintf in wp_get_plugin_action_button().

Removes an unassigned sprintf that was accidentally included as part of [57545] in the wp_get_plugin_action_button(). A copy/paste whoopsie.

The actual used code is assigned a wee bit lower in the function within the install case.

Follow-up to [57545].

Props hellofromTonya, costdev, rajinsharwar.
Fixes #61420.

@hellofromTonya commented on PR #6792:

2 years ago #5

Thank you both. Committed via https://core.trac.wordpress.org/changeset/58402.

#6 @hellofromTonya
2 years ago

@jorbin @costdev this is another good candidate to backport to 6.5 branch if a 6.5.5 is planned. Also noting, it might be a good one to backport anyway as it doesn't affect production code, but rather removes dead code to avoid confusion.

#7 @jorbin
2 years ago

Keywords dev-reviewed added
Milestone 6.6 → 6.5.5

@hellofromTonya I agree. I think we should backport it so that it goes out if a 6.5.5 happens even if it's not a driving force for a 6.5.5

I've also reviewed [58402] and am approving it for backport.

#8 @jorbin
2 years ago

Resolution fixed
Status closed → reopened

#9 @hellofromTonya
2 years ago

Awesome. I'll backport it now. Thanks @jorbin.

#10 @hellofromTonya
2 years ago

Resolution → fixed
Status reopened → closed

In 58404:

Plugins: Remove unassigned sprintf in wp_get_plugin_action_button().

Removes an unassigned sprintf that was accidentally included as part of [57545] in the wp_get_plugin_action_button(). A copy/paste whoopsie.

The actual used code is assigned a wee bit lower in the function within the install case.

Follow-up to [57545].

Reviewed by jorbin.
Merges [58402] to the 6.5 branch.

Props hellofromTonya, costdev, rajinsharwar.
Fixes #61420.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#61420 closed defect (bug) (fixed)

Plugins: Remove unassigned sprintf from wp_get_plugin_action_button()

Description

Change History (10)

This ticket was mentioned in ​PR #6792 on ​WordPress/wordpress-develop by ​@hellofromTonya. 2 years ago #1

​@rajinsharwar commented on ​PR #6792: 2 years ago #2

#3 @hellofromTonya 2 years ago

#4 @hellofromTonya 2 years ago

​@hellofromTonya commented on ​PR #6792: 2 years ago #5

#6 @hellofromTonya 2 years ago

#7 @jorbin 2 years ago

#8 @jorbin 2 years ago

#9 @hellofromTonya 2 years ago

#10 @hellofromTonya 2 years ago