Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#9295 closed enhancement (fixed)

rename to class-simplepie.php?

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


Currently SimplePie is included as wp-includes/, 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)

#2 @rmccue
15 years ago

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

rmccue: gsnedders: Why is SP called 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

#3 @ryan
15 years ago

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

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

Note: See TracTickets for help on using tickets.