Php-82 (#341)

* Preparing for PHP 8.2

* fixing psalm in php 8.0 and 8.1

* allowing coverage upload to fail

* adding fix from #340

* more restrictive php versions

* php-cs-fixer fixes

* bumping minimum php-cs-fixer version

Author: jaydiablo

.github/workflows/build.yml

@@ -13,7 +13,7 @@ jobs:
             fail-fast: false
             matrix:
                 operating-system: [ubuntu-latest]
-                php-versions: ['7.4', '8.0', '8.1']
+                php-versions: ['7.4', '8.0', '8.1', '8.2']
                 composer-args: ['', '--prefer-lowest']
         runs-on: ${{ matrix.operating-system }}
         steps:
@@ -43,5 +43,4 @@ jobs:
               uses: codecov/codecov-action@v3
               with:
                 files: ./tests/coverage/clover.xml
-                fail_ci_if_error: true
                 verbose: true

.github/workflows/dependabot-auto-merge.yml

@@ -0,0 +1,22 @@
+name: Dependabot auto-merge
+on: pull_request
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  dependabot:
+    runs-on: ubuntu-latest
+    if: ${{ github.actor == 'dependabot[bot]' }}
+    steps:
+      - name: Dependabot metadata
+        id: metadata
+        uses: dependabot/fetch-metadata@v1
+        with:
+          github-token: "${{ secrets.GITHUB_TOKEN }}"
+      - name: Enable auto-merge for Dependabot PRs
+        run: gh pr merge --auto --merge "$PR_URL"
+        env:
+          PR_URL: ${{github.event.pull_request.html_url}}
+          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

.github/workflows/merge-me.yml

@@ -37,15 +37,15 @@
         }
     ],
     "require": {
-        "php": "^7.4 || ~8.0.0 || ^8.1.0",
+        "php": "^7.4 || ~8.0.0 || ~8.1.0 || ~8.2.0",
         "ext-pdo": "*",
         "symfony/yaml": "^3.4 || ^4.1 || ^5.0"
     },
     "require-dev": {
         "phpunit/php-code-coverage": "^9.1.10",
-        "friendsofphp/php-cs-fixer": "3.6.0",
-        "phpstan/phpstan": "1.4.2",
-        "vimeo/psalm": "4.17.0"
+        "friendsofphp/php-cs-fixer": "^3.51.0",
+        "phpstan/phpstan": "1.10.14",
+        "vimeo/psalm": "5.9.0"
     },
     "scripts": {
         "test": "cd tests && php run.php",

composer.json

@@ -255,7 +255,7 @@ public function setAttribute($attribute, $value)
             case Doctrine_Core::ATTR_DRIVER_NAME:
                 //TODO throw an error since driver name can not be changed
             case Doctrine_Core::ATTR_ERRMODE:
-            break;
+                break;
             case Doctrine_Core::ATTR_CASE:
                 if ($value == Doctrine_Core::CASE_NATURAL) {
                     break;

lib/Doctrine/Adapter/Oracle.php

@@ -163,7 +163,7 @@ public function bindParam($column, &$variable, $type = null, $length = null, $dr
         switch ($type) {
             case Doctrine_Core::PARAM_STR:
                 $oci_type = SQLT_CHR;
-            break;
+                break;
         }
 
         if (is_integer($column)) {
@@ -317,25 +317,25 @@ public function fetch($fetchStyle = Doctrine_Core::FETCH_BOTH, $cursorOrientatio
                 return oci_fetch_object($this->statement);
             default:
                 throw new Doctrine_Adapter_Exception('This type of fetch is not supported: ' . $fetchStyle);
-/*
-            case Doctrine_Core::FETCH_BOUND:
-            case Doctrine_Core::FETCH_CLASS:
-            case FETCH_CLASSTYPE:
-            case FETCH_COLUMN:
-            case FETCH_FUNC:
-            case FETCH_GROUP:
-            case FETCH_INTO:
-            case FETCH_LAZY:
-            case FETCH_NAMED:
-            case FETCH_SERIALIZE:
-            case FETCH_UNIQUE:
-               case FETCH_ORI_ABS:
-            case FETCH_ORI_FIRST:
-            case FETCH_ORI_LAST:
-            case FETCH_ORI_NEXT:
-            case FETCH_ORI_PRIOR:
-            case FETCH_ORI_REL:
-*/
+                /*
+                            case Doctrine_Core::FETCH_BOUND:
+                            case Doctrine_Core::FETCH_CLASS:
+                            case FETCH_CLASSTYPE:
+                            case FETCH_COLUMN:
+                            case FETCH_FUNC:
+                            case FETCH_GROUP:
+                            case FETCH_INTO:
+                            case FETCH_LAZY:
+                            case FETCH_NAMED:
+                            case FETCH_SERIALIZE:
+                            case FETCH_UNIQUE:
+                               case FETCH_ORI_ABS:
+                            case FETCH_ORI_FIRST:
+                            case FETCH_ORI_LAST:
+                            case FETCH_ORI_NEXT:
+                            case FETCH_ORI_PRIOR:
+                            case FETCH_ORI_REL:
+                */
         }
     }
 
@@ -524,7 +524,7 @@ public function setAttribute($attribute, $value)
     {
         switch ($attribute) {
             case Doctrine_Core::ATTR_ERRMODE:
-            break;
+                break;
             default:
                 throw new Doctrine_Adapter_Exception("Unsupported Attribute: $attribute");
         }
@@ -572,7 +572,7 @@ private function handleError($params = array())
                 // no break
             case Doctrine_Core::ERRMODE_WARNING:
             case Doctrine_Core::ERRMODE_SILENT:
-            break;
+                break;
         }
     }
 

lib/Doctrine/Adapter/Statement/Oracle.php

@@ -134,7 +134,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));
     }
 
     /**

lib/Doctrine/Cli/AnsiColorFormatter.php

@@ -465,7 +465,7 @@ public function __get($name)
                     /** @psalm-var class-string $class */
                     $class                = 'Doctrine_' . ucwords($name) . '_' . $this->getDriverName();
                     $this->modules[$name] = new $class($this);
-                }
+            }
         }
 
         return $this->modules[$name];

lib/Doctrine/Connection.php

@@ -447,7 +447,7 @@ protected function replaceBoundParamsWithInlineValuesInQuery($query, array $para
              * @return string
              */
             function ($m) use ($value, $params) {
-                return is_null($value) ? 'NULL' : $this->quote($params[$m[1]]);
+                return is_null($value) ? 'NULL' : (string) $this->quote($params[$m[1]]);
             },
             $query
         );

lib/Doctrine/Connection/Mssql.php

@@ -542,7 +542,7 @@ public function update(Doctrine_Record $record)
                 if ($table->getOption('joinedParents')) {
                     // currrently just for bc!
                     $this->_updateCTIRecord($table, $record);
-                //--
+                    //--
                 } else {
                     $array = $record->getPrepared();
                     $this->conn->update($table, $array, $identifier);
@@ -580,7 +580,7 @@ public function insert(Doctrine_Record $record)
                 if ($table->getOption('joinedParents')) {
                     // just for bc!
                     $this->_insertCTIRecord($table, $record);
-                //--
+                    //--
                 } else {
                     $this->processSingleInsert($record);
                 }