WordPress.org
Get WordPress

Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#9295 closed enhancement (fixed)

rename simplepie.inc to class-simplepie.php?

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

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)

#2 @rmccue
18 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

#3 @ryan
18 years ago

  • Resolution → fixed
  • Status new → closed

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

Note: See TracTickets for help on using tickets.