Changeset 6157

Timestamp:

12/12/2016 01:51:49 PM (7 years ago)

Author:

xknown

Message:

Add PHP 5.x style constructors.

Keep also the PHP 4.x style constructors to avoid breaking code that depends on these methods.

See #3033

Location:

branches/0.9

Files:

• bb-admin/class-install.php (modified) (2 diffs)
• bb-includes/cache.php (modified) (1 diff)
• bb-includes/class-phpass.php (modified) (2 diffs)
• bb-includes/classes.php (modified) (5 diffs)
• bb-includes/locale.php (modified) (1 diff)
• bb-includes/script-loader.php (modified) (3 diffs)
• bb-includes/streams.php (modified) (5 diffs)
• bb-includes/wp-classes.php (modified) (3 diffs)
• bb-plugins/bozo.php (modified) (2 diffs)

branches/0.9/bb-admin/class-install.php

@@ -95 +95 @@
      * @return boolean
      **/
-    function BB_Install($caller)
+    function __construct($caller)
     {
         $this->caller = $caller;
@@ -103 +103 @@
         
         return true;
+    }
+
+    function BB_Install($caller)
+    {
+        $this->__construct($caller);
     }
     

branches/0.9/bb-includes/cache.php

@@ -6 +6 @@
     var $flush_time = 172800; // 2 days
 
-    function BB_Cache() {
+    function __construct() {
         if ( false === bb_get_option( 'use_cache' ) || !is_writable(BB_PATH . 'bb-cache/') )
             $this->use_cache = false;
         else
             $this->flush_old();
+    }
+
+    function BB_Cache() {
+        $this->__construct();
     }
 

branches/0.9/bb-includes/class-phpass.php

@@ -31 +31 @@
     var $random_state;
 
-    function PasswordHash($iteration_count_log2, $portable_hashes)
+    function __construct($iteration_count_log2, $portable_hashes)
     {
         $this->itoa64 = './0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz';
@@ -42 +42 @@
 
         $this->random_state = microtime() . getmypid();
+    }
+
+    function PasswordHash($iteration_count_log2, $portable_hashes)
+    {
+        $this->__construct($iteration_count_log2, $portable_hashes);
     }
 

branches/0.9/bb-includes/classes.php

@@ -17 +17 @@
 
     // Can optionally pass unique id string to help out filters
-    function BB_Query( $type = 'topic', $query = '', $id = '' ) {
+    function __construct( $type = 'topic', $query = '', $id = '' ) {
         if ( !empty($query) )
             $this->query($type, $query, $id);
+    }
+
+    function BB_Query( $type = 'topic', $query = '', $id = '' ) {
+        $this->__construct( $type, $query, $id );
     }
 
@@ -782 +786 @@
 
     // Can optionally pass unique id string to help out filters
-    function BB_Query_Form( $type = 'topic', $defaults = '', $allowed = '', $id = '' ) {
+    function __construct( $type = 'topic', $defaults = '', $allowed = '', $id = '' ) {
         $this->defaults = wp_parse_args( $defaults );
         $this->allowed  = wp_parse_args( $allowed );
         if ( !empty($defaults) || !empty($allowed) )
             $this->query_from_env($type, $defaults, $allowed, $id);
+    }
+
+    function BB_Query_Form( $type = 'topic', $defaults = '', $allowed = '', $id = '' ) {
+        $this->__construct( $type, $defaults, $allowed, $id );
     }
 
@@ -933 +941 @@
     var $_current_file;
 
-    function BB_DIR_MAP( $root, $args = '' ) {
+    function __construct( $root, $args = '' ) {
         if ( !is_dir( $root ) ) {
             $this->error = new WP_Error( 'bb_dir_map', __('Not a valid directory') );
@@ -946 +954 @@
         $this->_current_root = $this->root = rtrim($root, '/\\');
         $this->map();
+    }
+
+    function BB_Dir_Map( $root, $args = '' ) {
+        $this->__construct( $root, $args );
     }
 
@@ -1184 +1196 @@
     }
 
-    function BB_Loop( &$elements ) {
+    function __construct( &$elements ) {
         $this->elements = $elements;
         if ( !is_array($this->elements) || empty($this->elements) )
             return $this->elements = false;
+    }
+
+    function BB_Loop( &$elements ) {
+        $this->__construct( $elements );
     }
 

branches/0.9/bb-includes/locale.php

@@ -165 +165 @@
     }
 
-    function BB_Locale() {
+    function __construct() {
         $this->init();
         $this->register_globals();
+    }
+
+    function BB_Locale() {
+        $this->__construct();
     }
 }

branches/0.9/bb-includes/script-loader.php

@@ -7 +7 @@
     var $to_print = array();
 
+    function __construct() {
+        $this->default_scripts();
+    }
+
     function BB_Scripts() {
-        $this->default_scripts();
+        $this->__construct();
     }
 
@@ -209 +213 @@
     var $l10n = array();
 
-    function _BB_Script() {
+    function __construct() {
         @list($this->handle, $this->src, $this->deps, $this->ver ) = func_get_args();
         if ( !is_array($this->deps) )
@@ -215 +219 @@
         if ( !$this->ver )
             $this->ver = false;
+    }
+
+    function _BB_Script() {
+        $this->__construct();
     }
 

branches/0.9/bb-includes/streams.php

@@ -56 +56 @@
   var $_str;
 
-  function StringReader($str='') {
+  function __construct($str='') {
     $this->_str = $str;
     $this->_pos = 0;
+  }
+
+  function StringReader($str='') {
+      $this->__construct($str);
   }
 
@@ -93 +97 @@
   var $_length;
 
-  function FileReader($filename) {
+  function __construct($filename) {
     if (file_exists($filename)) {
 
@@ -107 +111 @@
       return false;
     }
+  }
+
+  function FileReader($filename) {
+    $this->__construct($filename);
   }
 
@@ -149 +157 @@
 // over it (it assumes knowledge of StringReader internals)
 class CachedFileReader extends StringReader {
-  function CachedFileReader($filename) {
+  function __construct($filename) {
     if (file_exists($filename)) {
 
@@ -167 +175 @@
     }
   }
+
+  function CachedFileReader($filename) {
+    $this->__construct($filename);
+  }
 }
 

branches/0.9/bb-includes/wp-classes.php

@@ -6 +6 @@
     var $error_data = array();
 
-    function WP_Error($code = '', $message = '', $data = '') {
+    function __construct($code = '', $message = '', $data = '') {
         if ( empty($code) )
             return;
@@ -14 +14 @@
         if ( ! empty($data) )
             $this->error_data[$code] = $data;
+    }
+
+    function WP_Error($code = '', $message = '', $data = '') {
+        $this->__construct($code, $message, $data);
     }
 
@@ -93 +97 @@
     var $responses = array();
 
-    function WP_Ajax_Response( $args = '' ) {
+    function __construct( $args = '' ) {
         if ( !empty($args) )
             $this->add($args);
+    }
+
+    function WP_Ajax_Response( $args = '' ) {
+        $this->__construct( $args );
     }
 

branches/0.9/bb-plugins/bozo.php

@@ -278 +278 @@
         var $title = '';
 
-        function BB_Bozo_Users( $page = '' ) { // constructor
+        function __construct( $page = '' ) { // constructor
             $this->raw_page = ( '' == $page ) ? false : (int) $page;
             $this->page = (int) ( '' == $page ) ? 1 : $page;
@@ -286 +286 @@
             $this->query();
             $this->do_paging();
+        }
+
+        function BB_Bozo_Users( $page = '' ) {
+            $this->__construct( $page );
         }