WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#9295 closed enhancement (fixed)

rename simplepie.inc to class-simplepie.php?

Reported by: DD32 Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: Feeds Keywords: 2nd-opinion
Focuses: Cc:

Description

Currently SimplePie is included as wp-includes/simplepie.inc, I somewhat think it'll look cleaner if its included the same way as the other external libraries, mainly as class-<class name or package>.php

Change History (3)

comment:2 rmccue5 years ago

+1 for class-simplepie.php. On the subject of why it actually is .inc:

rmccue: gsnedders: Why is SP called simplepie.inc and not simplepie.php?
gsnedders: rmccue: Because nobody thought to change it since 0.80?
[...]
gsnedders: rmccue: Or, more seriously, it makes the code open

comment:3 ryan5 years ago

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

(In [10747]) rename simplepie.inc to class-simplepie.php. fixes #9295

Note: See TracTickets for help on using tickets.