Skip to:
Content

bbPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#1760 closed enhancement (fixed)

Remove slugs from bbPress class and rely on object cache

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.1 Priority: normal
Severity: normal Version: 2.0
Component: Tools - Code Improvements Keywords:
Cc:

Description

Originally bbPress loaded its slugs at run-time into class variables for safe keeping. Now that all slugs are reliable options in the database, we can remove these pointers and rely on the options and object caches to handle this.

This reduces code duplication, removes class variables that are no longer needed, and is a net performance and memory usage win.

Change History (4)

#1 @johnjamesjacoby
13 years ago

  • Component changed from Actions/Filters to Code Improvement

#2 @johnjamesjacoby
13 years ago

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

(In [3760]) Remove slug variables from bbPress class and rely on options and object cache going forward.

  • Net performance and memory usage win
  • Removes class variables that are already in object cache
  • Reduces code duplication
  • Fixes #1760

#3 @johnjamesjacoby
13 years ago

(In [3761]) Use bbp_get_view_slug() in bbp_get_view_url(). See #1760.

#4 @johnjamesjacoby
13 years ago

(In [3762]) Introduce functions for getting the unique rewrite ID's for user/view/edit.

  • Use new functions where needed
  • Clean up variable usages in bbPress class
  • See #1760
Note: See TracTickets for help on using tickets.