fix deprecation notices for php8.2

ovos · Oct 15, 2022 · 9f3c896 · 9f3c896
1 parent 3c91fa4
commit 9f3c896
Show file tree

Hide file tree

Showing 32 changed files with 175 additions and 21 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -13,14 +13,11 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: ['5.3', '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1']
-        include:
-          - php: '8.2'
-            allow-failure: true
+        php: ['5.3', '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
 
     steps:
       - name: Checkout Code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       - name: Setup PHP ${{ matrix.php }}
         uses: shivammathur/setup-php@v2

diff --git a/lib/Doctrine/Cli/AnsiColorFormatter.php b/lib/Doctrine/Cli/AnsiColorFormatter.php
@@ -115,7 +115,7 @@ public function formatSection($section, $text, $size = null)
     {
         $width = 9 + strlen($this->format('', 'INFO'));
 
-        return sprintf(">> %-${width}s %s", $this->format($section, 'INFO'), $this->excerpt($text, $size));
+        return sprintf(">> %-{$width}s %s", $this->format($section, 'INFO'), $this->excerpt($text, $size));
     }
 
     /**

diff --git a/lib/Doctrine/Connection.php b/lib/Doctrine/Connection.php
@@ -1255,7 +1255,6 @@ public function clear()
     public function evictTables()
     {
         $this->tables = array();
-        $this->exported = array();
     }
 
     /**

diff --git a/lib/Doctrine/Hydrator/Graph.php b/lib/Doctrine/Hydrator/Graph.php
@@ -36,6 +36,7 @@
 abstract class Doctrine_Hydrator_Graph extends Doctrine_Hydrator_Abstract
 {
     protected $_tables = array();
+    protected $_rootAlias;
 
     /**
      * Gets the custom field used for indexing for the specified component alias.

diff --git a/lib/Doctrine/Import/Builder.php b/lib/Doctrine/Import/Builder.php
@@ -87,6 +87,11 @@ class Doctrine_Import_Builder extends Doctrine_Builder
      */
     protected $_generateTableClasses = false;
 
+    /**
+     * @var boolean
+     */
+    protected $_generateAccessors = false;
+
     /**
      * Prefix to use for generated base classes
      *

diff --git a/lib/Doctrine/Query.php b/lib/Doctrine/Query.php
@@ -168,11 +168,6 @@ class Doctrine_Query extends Doctrine_Query_Abstract implements Countable
      */
     protected $_parsers = array();
 
-    /**
-     * @var array $_pendingJoinConditions    an array containing pending joins
-     */
-    protected $_pendingJoinConditions = array();
-
     /**
      * @var array
      */

diff --git a/lib/Doctrine/Query/Abstract.php b/lib/Doctrine/Query/Abstract.php
@@ -279,6 +279,11 @@ abstract class Doctrine_Query_Abstract
      */
     protected $_preQueried = false;
 
+    /**
+     * @var array $_pendingJoinConditions    an array containing pending joins
+     */
+    protected $_pendingJoinConditions = array();
+
     /**
      * Fix for http://www.doctrine-project.org/jira/browse/DC-701
      *

diff --git a/lib/Doctrine/RawSql.php b/lib/Doctrine/RawSql.php
@@ -59,7 +59,7 @@ function __construct(Doctrine_Connection $connection = null, Doctrine_Hydrator_A
 
     protected function clear()
     {
-        $this->_preQuery = false;
+        $this->_preQueried = false;
         $this->_pendingJoinConditions = array();
     }
 

diff --git a/lib/Doctrine/Sequence.php b/lib/Doctrine/Sequence.php
@@ -33,6 +33,11 @@
  */
 class Doctrine_Sequence extends Doctrine_Connection_Module
 {
+    /**
+     * @var array
+     */
+    protected $warnings = array();
+
     /**
      * Returns the next free id of a sequence
      *

diff --git a/lib/Doctrine/Task/BuildAll.php b/lib/Doctrine/Task/BuildAll.php
@@ -38,7 +38,12 @@ class Doctrine_Task_BuildAll extends Doctrine_Task
 
     protected $models,
               $tables;
-
+
+    /**
+     * @var Doctrine_Task_CreateDb
+     */
+    protected $createDb;
+
     public function __construct($dispatcher = null)
     {
         parent::__construct($dispatcher);

diff --git a/lib/Doctrine/Task/BuildAllLoad.php b/lib/Doctrine/Task/BuildAllLoad.php
@@ -35,7 +35,17 @@ class Doctrine_Task_BuildAllLoad extends Doctrine_Task
     public $description          =   'Calls build-all, and load-data',
            $requiredArguments    =   array(),
            $optionalArguments    =   array();
-
+
+    /**
+     * @var Doctrine_Task_BuildAll
+     */
+    protected $buildAll;
+
+    /**
+     * @var Doctrine_Task_LoadData
+     */
+    protected $loadData;
+
     public function __construct($dispatcher = null)
     {
         parent::__construct($dispatcher);

diff --git a/lib/Doctrine/Task/BuildAllReload.php b/lib/Doctrine/Task/BuildAllReload.php
@@ -35,7 +35,17 @@ class Doctrine_Task_BuildAllReload extends Doctrine_Task
     public $description          =   'Calls rebuild-db and load-data',
            $requiredArguments    =   array(),
            $optionalArguments    =   array();
-
+
+    /**
+     * @var Doctrine_Task_RebuildDb
+     */
+    protected $rebuildDb;
+
+    /**
+     * @var Doctrine_Task_LoadData
+     */
+    protected $loadData;
+
     public function __construct($dispatcher = null)
     {
         parent::__construct($dispatcher);

diff --git a/lib/Doctrine/Task/RebuildDb.php b/lib/Doctrine/Task/RebuildDb.php
@@ -35,7 +35,22 @@ class Doctrine_Task_RebuildDb extends Doctrine_Task
     public $description          =   'Drops and re-creates databases',
            $requiredArguments    =   array(),
            $optionalArguments    =   array();
-
+
+    /**
+     * @var Doctrine_Task_DropDb
+     */
+    protected $dropDb;
+
+    /**
+     * @var Doctrine_Task_CreateDb
+     */
+    protected $createDb;
+
+    /**
+     * @var Doctrine_Task_CreateTables
+     */
+    protected $createTables;
+
     public function __construct($dispatcher = null)
     {
         parent::__construct($dispatcher);

diff --git a/tests/AuditLogTestCase.php b/tests/AuditLogTestCase.php
@@ -32,6 +32,10 @@
  */
 class Doctrine_AuditLog_TestCase extends Doctrine_UnitTestCase
 {
+    /**
+     * @var Doctrine_Connection_Profiler
+     */
+    private $profiler;
 
     public function prepareData()
     { }

diff --git a/tests/Cache/DbTestCase.php b/tests/Cache/DbTestCase.php
@@ -33,6 +33,11 @@
  */
 class Doctrine_Cache_Db_TestCase extends Doctrine_Cache_Abstract_TestCase
 {
+    /**
+     * @var Doctrine_Cache_Db
+     */
+    private $cache;
+
     public function setUp()
     {
         parent::setUp();

diff --git a/tests/Connection/CustomTestCase.php b/tests/Connection/CustomTestCase.php
@@ -32,6 +32,16 @@
  */
 class Doctrine_Connection_Custom_TestCase extends Doctrine_UnitTestCase 
 {
+    /**
+     * @var Doctrine_Connection
+     */
+    protected $_conn;
+
+    /**
+     * @var Doctrine_Adapter_Interface
+     */
+    protected $_dbh;
+
     public function setUp()
     {
         $manager = Doctrine_Manager::getInstance();

diff --git a/tests/Connection/ProfilerTestCase.php b/tests/Connection/ProfilerTestCase.php
@@ -33,6 +33,11 @@
  */
 class Doctrine_Connection_Profiler_TestCase extends Doctrine_UnitTestCase 
 {
+    /**
+     * @var Doctrine_Connection_Profiler
+     */
+    private $profiler;
+
     public function prepareTables()
     {}
     public function prepareData() 

diff --git a/tests/Db/ProfilerTestCase.php b/tests/Db/ProfilerTestCase.php
@@ -33,6 +33,11 @@
  */
 class Doctrine_Connection_Profiler_TestCase extends Doctrine_UnitTestCase 
 {
+    /**
+     * @var Doctrine_Connection_Profiler
+     */
+    protected $profiler;
+
     public function prepareTables()
     {}
     public function prepareData() 

diff --git a/tests/DoctrineTest/Doctrine_UnitTestCase.php b/tests/DoctrineTest/Doctrine_UnitTestCase.php
@@ -54,9 +54,13 @@ class Doctrine_UnitTestCase extends UnitTestCase
     protected $transaction;
     protected $_name;
 
-
     protected $init = false;
 
+    protected $query;
+    protected $exc;
+    protected $sequence;
+    protected $import;
+
     public function getName()
     {
         return $this->_name;

diff --git a/tests/DoctrineTest/GroupTest.php b/tests/DoctrineTest/GroupTest.php
@@ -5,6 +5,7 @@ class GroupTest extends UnitTestCase
     protected $_name;
     protected $_title;
     protected $_onlyRunFailed = false;
+    protected $_formatter;
 
     public function __construct($title, $name)
     {

diff --git a/tests/ManagerTestCase.php b/tests/ManagerTestCase.php
@@ -31,6 +31,26 @@
  * @version     $Revision$
  */
 class Doctrine_Manager_TestCase extends Doctrine_UnitTestCase {
+    /**
+     * @var string
+     */
+    protected $conn1_database;
+
+    /**
+     * @var string
+     */
+    protected $conn2_database;
+
+    /**
+     * @var Doctrine_Connection
+     */
+    protected $conn1;
+
+    /**
+     * @var Doctrine_Connection
+     */
+    protected $conn2;
+
     public function testGetInstance() {
         $this->assertTrue(Doctrine_Manager::getInstance() instanceOf Doctrine_Manager);
     }

diff --git a/tests/Record/FromArrayTestCase.php b/tests/Record/FromArrayTestCase.php
@@ -32,6 +32,8 @@
  */
 class Doctrine_Record_FromArray_TestCase extends Doctrine_UnitTestCase
 {
+    private $previous_group;
+
     public function prepareTables()
     {
         parent::prepareTables();

diff --git a/tests/Record/SynchronizeTestCase.php b/tests/Record/SynchronizeTestCase.php
@@ -32,6 +32,8 @@
  */
 class Doctrine_Record_Synchronize_TestCase extends Doctrine_UnitTestCase
 {
+    private $previous_group;
+
     public function prepareTables()
     {
         parent::prepareTables();

diff --git a/tests/Relation/OrderByTestCase.php b/tests/Relation/OrderByTestCase.php
@@ -32,6 +32,11 @@
  */
 class Doctrine_Relation_OrderBy_TestCase extends Doctrine_UnitTestCase 
 {
+    /**
+     * @var Doctrine_Connection_Profiler
+     */
+    private $profiler;
+
     public function prepareTables()
     {
         $this->profiler = new Doctrine_Connection_Profiler();

diff --git a/tests/Search/FileTestCase.php b/tests/Search/FileTestCase.php
@@ -32,6 +32,11 @@
  */
 class Doctrine_Search_File_TestCase extends Doctrine_UnitTestCase
 {
+    /**
+     * @var Doctrine_Search_File
+     */
+    private $_search;
+
     public function prepareData()
     { }
     public function prepareTables()

diff --git a/tests/TableTestCase.php b/tests/TableTestCase.php
@@ -142,9 +142,9 @@ public function testGetData()
 
     public function testSetSequenceName()
     {
-        $this->objTable->sequenceName = 'test-seq';
+        $this->objTable->setOption('sequenceName', 'test-seq');
         $this->assertEqual($this->objTable->sequenceName, 'test-seq');
-        $this->objTable->sequenceName = null;
+        $this->objTable->setOption('sequenceName', null);
     }
 
     public function testCreate() 

diff --git a/tests/Ticket/1106TestCase.php b/tests/Ticket/1106TestCase.php
@@ -32,6 +32,8 @@
  */
 class Doctrine_Ticket_1106_TestCase extends Doctrine_UnitTestCase
 {
+    private $user_id;
+
     public function prepareTables()
     {
         parent::prepareTables();

diff --git a/tests/Ticket/1131TestCase.php b/tests/Ticket/1131TestCase.php
@@ -32,6 +32,9 @@
  */
 class Doctrine_Ticket_1131_TestCase extends Doctrine_UnitTestCase
 {
+    private $role_one;
+    private $role_two;
+
     public function prepareTables()
     {
         //$this->tables = array();

diff --git a/tests/Ticket/1436TestCase.php b/tests/Ticket/1436TestCase.php
@@ -32,6 +32,10 @@
  */
 class Doctrine_Ticket_1436_TestCase extends Doctrine_UnitTestCase
 {
+    private $group_one;
+    private $group_two;
+    private $group_three;
+
     public function prepareTables()
     {
         parent::prepareTables();

diff --git a/tests/Ticket/1992TestCase.php b/tests/Ticket/1992TestCase.php
@@ -32,6 +32,21 @@
  */
 class Doctrine_Ticket_1992_TestCase extends Doctrine_UnitTestCase 
 {
+    /**
+     * @var Ticket_1992_Person
+     */
+    private $person;
+
+    /**
+     * @var Ticket_1992_Profile
+     */
+    private $profile1;
+
+    /**
+     * @var Ticket_1992_Profile
+     */
+    private $profile2;
+
     public function prepareTables()
     {
         $this->tables[] = 'Ticket_1992_Person';