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

Timestamp:

04/15/2015 02:33:11 AM (12 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']   );