Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#49590 closed defect (bug) (duplicate)

Hello Dolly default plugin and get_plugins()

Reported by: xfirefartx's profile xFireFartx Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.4
Component: Plugins Keywords:
Focuses: Cc:


I just noticed something weird: The hello-dolly plugin which is included by default on wordpress installs is just added as hello.php in the plugins folder.
This causes the get_plugins() call to return only hello.php as a plugin slug for this as it is not in a folder with the slug name like in the example over here:

Can we maybe move the plugin into the folder hello-dolly by default so it behaves like a real plugin? I think there is no other way to get the slug without parsing the PluginURI field which I would like to avoid.

Current output from get_plugins():

    [hello.php] => Array
            [Name] => Hello Dolly
            [PluginURI] =>
            [Version] => 1.7.2
            [Description] => This is not just a plugin, it symbolizes the hope and enthusiasm of an entire generation summed up in two words sung most famously by Louis Armstrong: Hello, Dolly. When activated you will randomly see a lyric from <cite>Hello, Dolly</cite> in the upper right of your admin screen on every page.
            [Author] => Matt Mullenweg
            [AuthorURI] =>
            [TextDomain] =>
            [DomainPath] =>
            [Network] =>
            [RequiresWP] =>
            [RequiresPHP] =>
            [Title] => Hello Dolly
            [AuthorName] => Matt Mullenweg

This behaviour is present with 5.4 beta 1

Change History (4)

#1 @xFireFartx
4 years ago

Another option would be to parse the slug inside wordpress core and return it as a seperate field on the plugin data

#2 @SergeyBiryukov
4 years ago

  • Component changed from General to Plugins
  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Hi there, welcome back to WordPress Trac!

Thanks for the ticket, we're already tracking this issue in #29540, #37351, and #49338.

#3 @xFireFartx
4 years ago

  • Resolution duplicate deleted
  • Status changed from closed to reopened

All of the tickets are closed as invalid without further comment?

#4 @ocean90
4 years ago

  • Resolution set to duplicate
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.