WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#30200 closed defect (bug) (fixed)

Documentation error for activate_plugin hook

Reported by: wedi Owned by: SergeyBiryukov
Milestone: 4.1 Priority: normal
Severity: normal Version: 4.1
Component: Plugins Keywords:
Focuses: docs Cc:

Description

The documentation for the activate_plugin hook is erroneous. In the beginning it says that the hook gets fired on plugin deactivation. When you read further it suddenly says activation. This is probably a simple copy&paste error.

https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/plugin.php#L560

My original report for reference: https://meta.trac.wordpress.org/ticket/606


While investigating, @georgestephanis noticed a bad reference here:

https://core.trac.wordpress.org/browser/trunk/src/wp-admin/plugins.php#L154

It should refer to wp-admin/includes/plugin.php -- not plugins.php -- no s.


The attached patch fixes both issues.

Attachments (1)

documentation.diff (1.3 KB) - added by wedi 5 years ago.

Download all attachments as: .zip

Change History (3)

@wedi
5 years ago

#1 @SergeyBiryukov
5 years ago

  • Component changed from Administration to Plugins
  • Milestone changed from Awaiting Review to 4.1

The first part was introduced in [27769] and fixed in [29977].

The second part was introduced in [26345].

#2 @SergeyBiryukov
5 years ago

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

In 30117:

Use correct file name in a hook comment.

props wedi.
fixes #30200.

Note: See TracTickets for help on using tickets.