Changeset 5676 for trunk/src/includes/common/classes.php

Timestamp:

04/15/2015 02:33:11 AM (11 years ago)

Author:

johnjamesjacoby

Message:

All: Update default values for $args parameters from empty strings to empty arrays.

This is a micro-optimization to avoid unnecessary calls to wp_parse_str() inside bbp_parse_args().

File:

• trunk/src/includes/common/classes.php (modified) (3 diffs)

trunk/src/includes/common/classes.php

@@ -61 +61 @@
      * @since bbPress (r2700)
      *
-     * @param mixed $args Required. Supports these args:
+     * @param array $args Required. Supports these args:
      *  - name: Unique name (for internal identification)
      *  - id: Unique ID (normally for custom post type)
@@ -71 +71 @@
      * @uses BBP_Component::setup_actions() Setup the hooks and actions
      */
-    public function __construct( $args = '' ) {
+    public function __construct( $args = array() ) {
         if ( empty( $args ) ) {
             return;
@@ -90 +90 @@
      * @uses apply_filters() Calls 'bbp_{@link BBP_Component::name}_slug'
      */
-    private function setup_globals( $args = '' ) {
+    private function setup_globals( $args = array() ) {
         $this->name = $args['name'];
         $this->id   = apply_filters( 'bbp_' . $this->name . '_id',   $args['id']   );