Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 11 years ago

#9561 closed enhancement (fixed)

plugins_dir() and plugins_basename() should also work for plugins in the MU plugins dir

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: Plugins Keywords: has-patch tested
Focuses: Cc:

Description

r10737 introduces a sweet new feature, but if a plugin in there relies on plugin_dir_url() to auto-magically get its url in order to include styles and scripts, it won't work as expected.

Attachments (2)

9561.diff (2.0 KB) - added by Denis-de-Bernardy 16 years ago.
missing_slash.patch (772 bytes) - added by alexrabe 16 years ago.

Download all attachments as: .zip

Change History (7)

#1 @DD32
16 years ago

+1 This looks like a good idea to me.

#2 @azaozz
16 years ago

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

(In [11069]) Make plugin_dir() and plugin_basename() work for mu-plugins directory, props Denis-de-Bernardy, fixes #9561

#3 @alexrabe
16 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Since [11069] returns plugin_basename( dirname(FILE)) a slash at the beginning. If I'm not wrong then it's missing, see attached patch

#4 @azaozz
16 years ago

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

(In [11072]) Remove slash from plugin_basename(), props alexrabe, fixes #9561

This ticket was mentioned in IRC in #wordpress-dev by ericandrewlewis. View the logs.


11 years ago

Note: See TracTickets for help on using tickets.