Make WordPress Core

Opened 14 years ago

Closed 13 years ago

#15236 closed defect (bug) (duplicate)

Delete plugin confirmation should strip_tags from plugin author

Reported by: ampt's profile ampt Owned by:
Milestone: Priority: normal
Severity: trivial Version: 3.1
Component: Plugins Keywords: has-patch
Focuses: Cc:

Description

On the plugins page if I click delete Hello Dolly the confirmation page displays:

You are about to remove the following plugin:

  • Hello Dolly by <a href="http://ma.tt/" title="Visit author homepage">Matt Mullenweg</a>

The attached patch strips tags from the plugin author before outputting to the browser

Attachments (2)

15236.patch (1.2 KB) - added by ampt 14 years ago.
15236.2.diff (645 bytes) - added by ampt 14 years ago.

Download all attachments as: .zip

Change History (5)

@ampt
14 years ago

#1 @nacin
14 years ago

  • Milestone changed from Awaiting Review to 3.1

Hmm. We shouldn't need to strip HTML there. We should have Author_Name without the link for get_plugin_data(), the same thing we did for themes in [14658].

The problem here is a change in functionality in either 2.9 or 3.0, for Author to include the link by default. We just didn't update it across core.

@ampt
14 years ago

#2 @ampt
14 years ago

Thanks nacin, attached is another patch with that fixes this with get_plugin_data

#3 @nacin
13 years ago

  • Milestone 3.1 deleted
  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.