Opened 7 months ago

Closed 7 months ago

Last modified 7 months ago

#22280 closed enhancement (wontfix)

Correcting class-http.php to use WordPress naming conventions

Reported by: danhgilmore Owned by:
Priority: normal Milestone:
Component: Formatting Version: 3.5
Severity: minor Keywords: has-patch
Cc:

Description

WordPress convention is to use underscores versus camelCase. I have changed the names of all the functions to adhere to WordPress standards.

Attachments (1)

class-http.diff (6.7 KB) - added by danhgilmore 7 months ago.

Download all attachments as: .zip

Change History (3)

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

While this is our convention, it is too late for these to change, as these are all publicly accessible methods that plugins may be referencing directly. Note that renaming the methods themselves wouldn't be enough either way, as we'd need to update where they are called as well.

We *could* solve this with __call() but some of these are static methods, which means we'd need callStatic() in PHP 5.3. Ultimately, using magic methods to rename a few methods for standards reasons is a bit overkill anyway.

@Nacin - Totally understand, but just FYI, I changed where the functions were being called in my second upload (FWIW)

Note: See TracTickets for help on using tickets.