Added parent::__construct(); to each component class

Samuel Akopyan · Samuel Akopyan · commit 0e9762a95823 · 2020-11-21T23:58:32.000+02:00
diff --git a/framework/components/CClientScript.php b/framework/components/CClientScript.php
@@ -76,6 +76,7 @@ class CClientScript extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
     }
 
     /**
diff --git a/framework/components/CComponent.php b/framework/components/CComponent.php
@@ -60,6 +60,7 @@ public static function __callStatic($method, $args)
      *     return parent::init(__CLASS__);
      * }
      * </pre>
+     * @return mixed
      */
     private static function _parentInit($className = __CLASS__)
     {
diff --git a/framework/components/CDbHttpSession.php b/framework/components/CDbHttpSession.php
@@ -66,6 +66,8 @@ class CDbHttpSession extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         $this->_db = CDatabase::init();
 
         @session_set_save_handler(
diff --git a/framework/components/CHttpCookie.php b/framework/components/CHttpCookie.php
@@ -43,6 +43,8 @@ class CHttpCookie extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         if (CConfig::get('cookies.domain') != '') {
             $this->setDomain(CConfig::get('cookies.domain'));
         }
diff --git a/framework/components/CHttpRequest.php b/framework/components/CHttpRequest.php
@@ -101,6 +101,8 @@ class CHttpRequest extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         $this->_csrfValidation  = (CConfig::get('validation.csrf.enable') === true) ? true : false;
         $this->_csrfExclude     = CConfig::exists('validation.csrf.exclude') ? CConfig::get('validation.csrf.exclude')
             : [];
diff --git a/framework/components/CHttpSession.php b/framework/components/CHttpSession.php
@@ -64,6 +64,8 @@ class CHttpSession extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         if ($this->_cookieMode !== 'only') {
             $this->_setCookieMode($this->_cookieMode);
         }
diff --git a/framework/components/CLocalTime.php b/framework/components/CLocalTime.php
@@ -28,6 +28,7 @@ class CLocalTime extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
     }
 
     /**
diff --git a/framework/components/CLogger.php b/framework/components/CLogger.php
@@ -44,6 +44,8 @@ class CLogger extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         $this->_enabled       = CConfig::get('log.enable') !== '' ? CConfig::get('log.enable') : false;
         $this->_logPath       = APPHP_PATH.DS.(CConfig::get('log.path') !== '' ? CConfig::get('log.path') : 'protected/tmp/logs/');
         $this->_fileExtension = CConfig::exists('log.fileExtension') && CConfig::get('log.fileExtension') !== '' ? ltrim(CConfig::get('log.fileExtension'), '.') : 'php';
diff --git a/framework/components/CMessageSource.php b/framework/components/CMessageSource.php
@@ -29,6 +29,8 @@ class CMessageSource extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         $this->_basePath = dirname(__FILE__);
     }
 
diff --git a/framework/components/CMobileDetect.php b/framework/components/CMobileDetect.php
@@ -32,6 +32,7 @@ class CMobileDetect extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
     }
 
     /**
diff --git a/framework/components/CShoppingCart.php b/framework/components/CShoppingCart.php
@@ -67,6 +67,8 @@ class CShoppingCart extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         // Grab the shopping cart array from the session and initialize it
         $this->_cartContent = A::app()->getSession()->get('shopping_cart_content', null);
         if ($this->_cartContent === null) {
diff --git a/framework/components/CUri.php b/framework/components/CUri.php
@@ -46,6 +46,8 @@ class CUri extends CComponent
      */
     function __construct()
     {
+        parent::__construct();
+
         $this->_uriString = $this->_detectUri();
         $this->_explodeSegments();
     }

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ class CClientScript extends CComponent`
`76`	`76`	`*/`
`77`	`77`	`function __construct()`
`78`	`78`	`{`
	`79`	`+ parent::__construct();`
`79`	`80`	`}`
`80`	`81`
`81`	`82`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,7 @@ public static function __callStatic($method, $args)`
`60`	`60`	`* return parent::init(__CLASS__);`
`61`	`61`	`* }`
`62`	`62`	`* </pre>`
	`63`	`+ * @return mixed`
`63`	`64`	`*/`
`64`	`65`	`private static function _parentInit($className = __CLASS__)`
`65`	`66`	`{`
Original file line number	Diff line number	Diff line change
`@@ -66,6 +66,8 @@ class CDbHttpSession extends CComponent`
`66`	`66`	`*/`
`67`	`67`	`function __construct()`
`68`	`68`	`{`
	`69`	`+ parent::__construct();`
	`70`	`+`
`69`	`71`	`$this->_db = CDatabase::init();`
`70`	`72`
`71`	`73`	`@session_set_save_handler(`
Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,8 @@ class CHttpCookie extends CComponent`
`43`	`43`	`*/`
`44`	`44`	`function __construct()`
`45`	`45`	`{`
	`46`	`+ parent::__construct();`
	`47`	`+`
`46`	`48`	`if (CConfig::get('cookies.domain') != '') {`
`47`	`49`	`$this->setDomain(CConfig::get('cookies.domain'));`
`48`	`50`	`}`
Original file line number	Diff line number	Diff line change
`@@ -101,6 +101,8 @@ class CHttpRequest extends CComponent`
`101`	`101`	`*/`
`102`	`102`	`function __construct()`
`103`	`103`	`{`
	`104`	`+ parent::__construct();`
	`105`	`+`
`104`	`106`	`$this->_csrfValidation = (CConfig::get('validation.csrf.enable') === true) ? true : false;`
`105`	`107`	`$this->_csrfExclude = CConfig::exists('validation.csrf.exclude') ? CConfig::get('validation.csrf.exclude')`
`106`	`108`	`: [];`
Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,8 @@ class CHttpSession extends CComponent`
`64`	`64`	`*/`
`65`	`65`	`function __construct()`
`66`	`66`	`{`
	`67`	`+ parent::__construct();`
	`68`	`+`
`67`	`69`	`if ($this->_cookieMode !== 'only') {`
`68`	`70`	`$this->_setCookieMode($this->_cookieMode);`
`69`	`71`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ class CLocalTime extends CComponent`
`28`	`28`	`*/`
`29`	`29`	`function __construct()`
`30`	`30`	`{`
	`31`	`+ parent::__construct();`
`31`	`32`	`}`
`32`	`33`
`33`	`34`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -44,6 +44,8 @@ class CLogger extends CComponent`
`44`	`44`	`*/`
`45`	`45`	`function __construct()`
`46`	`46`	`{`
	`47`	`+ parent::__construct();`
	`48`	`+`
`47`	`49`	`$this->_enabled = CConfig::get('log.enable') !== '' ? CConfig::get('log.enable') : false;`
`48`	`50`	`$this->_logPath = APPHP_PATH.DS.(CConfig::get('log.path') !== '' ? CConfig::get('log.path') : 'protected/tmp/logs/');`
`49`	`51`	`$this->_fileExtension = CConfig::exists('log.fileExtension') && CConfig::get('log.fileExtension') !== '' ? ltrim(CConfig::get('log.fileExtension'), '.') : 'php';`
Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,8 @@ class CMessageSource extends CComponent`
`29`	`29`	`*/`
`30`	`30`	`function __construct()`
`31`	`31`	`{`
	`32`	`+ parent::__construct();`
	`33`	`+`
`32`	`34`	`$this->_basePath = dirname(__FILE__);`
`33`	`35`	`}`
`34`	`36`
Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,7 @@ class CMobileDetect extends CComponent`
`32`	`32`	`*/`
`33`	`33`	`function __construct()`
`34`	`34`	`{`
	`35`	`+ parent::__construct();`
`35`	`36`	`}`
`36`	`37`
`37`	`38`	`/**`