chore: correção automática usando rector-1.2

composer.json

@@ -16,6 +16,7 @@
     "squizlabs/php_codesniffer": "*",
     "mockery/mockery": "^1.5",
     "phpcompatibility/php-compatibility": "^9.3",
-    "pheromone/phpcs-security-audit": "*"
+    "pheromone/phpcs-security-audit": "*",
+    "rector/rector": "^1.2"
   }
 }

rector.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Set\ValueObject\LevelSetList;
+
+return RectorConfig::configure()
+    ->withPaths([
+        __DIR__ . '/src',
+    ])
+
+    ->withPreparedSets(deadCode: true)
+    ->withSets([LevelSetList::UP_TO_PHP_82])
+    ->withTypeCoverageLevel(0);

src/LogPenWs.php

@@ -23,9 +23,9 @@ class LogPenWs {
      * Construtor
      */
   // phpcs:ignore PEAR.Functions.ValidDefaultValue.NotAtEnd
-  public function __construct($config = array(), $wsdl, $options)
+  public function __construct($wsdl, $options, $config = [])
     {
-      $this->arrListaMetodos = is_array($config) ? $config : array();
+      $this->arrListaMetodos = is_array($config) ? $config : [];
       $this->objSoapClient = new \BeSimple\SoapClient\SoapClient($wsdl, $options);
   }
 
@@ -35,7 +35,7 @@ public function __construct($config = array(), $wsdl, $options)
      */
   public function __call($method, $arguments)
     {
-      $mixResultado = call_user_func_array(array($this->objSoapClient, $method), $arguments);
+      $mixResultado = call_user_func_array([$this->objSoapClient, $method], $arguments);
 
     if(in_array($method, $this->arrListaMetodos)) {
 

src/bd/PenMetaBD.php

@@ -120,7 +120,7 @@ public function isVersaoSuportada($strRegexVersaoSistema, $strVerMinRequirida){
       return $this;
   }
 
-  public function adicionarChaveUnica($strNomeTabela = '', $arrNomeChave = array()){
+  public function adicionarChaveUnica($strNomeTabela = '', $arrNomeChave = []){
 
       $this->getObjInfraIBanco()
               ->executarSql('ALTER TABLE '.$strNomeTabela.' ADD CONSTRAINT UK_'.$strNomeTabela.' UNIQUE('.implode(', ', $arrNomeChave).')');
@@ -255,7 +255,7 @@ public function isChaveExiste($strNomeTabela = '', $strNomeChave = ''){
      * @throws InfraException
      * @return PenMetaBD
      */
-  public function criarTabela($arrSchema = array()){
+  public function criarTabela($arrSchema = []){
 
       $strNomeTabela = $arrSchema['tabela'];
 
@@ -264,12 +264,12 @@ public function criarTabela($arrSchema = array()){
     }
 
       $objInfraBanco = $this->getObjInfraIBanco();
-      $arrColunas = array();
-      $arrStrQuery = array();
+      $arrColunas = [];
+      $arrStrQuery = [];
 
     foreach($arrSchema['cols'] as $strNomeColuna => $arrColunaConfig) {
 
-        list($strTipoDado, $strValorPadrao) = $arrColunaConfig;
+        [$strTipoDado, $strValorPadrao] = $arrColunaConfig;
 
       if($strValorPadrao != self::SNULLO && $strValorPadrao != self::NNULLO) {
 
@@ -315,11 +315,8 @@ public function criarTabela($arrSchema = array()){
       }
     }
 
-    if(!empty($arrStrQuery)) {
-
-      foreach($arrStrQuery as $strQuery) {
-          $objInfraBanco->executarSql($strQuery);
-      }
+    foreach($arrStrQuery as $strQuery) {
+        $objInfraBanco->executarSql($strQuery);
     }
 
       return $this;

src/bd/PenParametroBD.php

@@ -36,9 +36,7 @@ public function setValor($strNome, $strValor) {
         $sql .= ' WHERE nome = ' . $this->getObjInfraIBanco()->formatarGravacaoStr($strNome);
     }
 
-      $ret = $this->getObjInfraIBanco()->executarSql($sql);
-
-      return $ret;
+      return $this->getObjInfraIBanco()->executarSql($sql);
   }
 
   public function isSetValor($strNome) {

src/bd/TramiteBD.php

@@ -9,11 +9,10 @@ public function __construct(InfraIBanco $objInfraIBanco){
   }
 
     /**
-     * Recupera os dados do último trâmite válido realizado para determinado número de processo eletrônico
-     *
-     * @param ProcessoEletronicoDTO $parObjProcessoEletronicoDTO
-     * @return void
-     */
+   * Recupera os dados do último trâmite válido realizado para determinado número de processo eletrônico
+   *
+   * @return void
+   */
   public function consultarUltimoTramite(ProcessoEletronicoDTO $parObjProcessoEletronicoDTO, $parStrStaTipoTramite = null)
     {
     if(is_null($parObjProcessoEletronicoDTO)){
@@ -29,10 +28,7 @@ public function consultarUltimoTramite(ProcessoEletronicoDTO $parObjProcessoElet
       $objTramiteDTOPesquisa->setNumMaxRegistrosRetorno(1);
       $objTramiteDTOPesquisa->setOrdNumIdTramite(InfraDTO::$TIPO_ORDENACAO_DESC);
       $objTramiteDTOPesquisa->setStrStaTipoProtocolo(
-          array(
-              ProcessoEletronicoRN::$STA_TIPO_PROTOCOLO_PROCESSO,
-              ProcessoEletronicoRN::$STA_TIPO_PROTOCOLO_DOCUMENTO_AVULSO
-          ),
+          [ProcessoEletronicoRN::$STA_TIPO_PROTOCOLO_PROCESSO, ProcessoEletronicoRN::$STA_TIPO_PROTOCOLO_DOCUMENTO_AVULSO],
           InfraDTO::$OPER_IN
       );
 

src/console.php

@@ -2,10 +2,7 @@
 
 set_time_limit(0);
 
-set_include_path(implode(PATH_SEPARATOR, array(
-    realpath(__DIR__.'/../../../infra/infra_php'),
-    get_include_path(),
-)));
+set_include_path(implode(PATH_SEPARATOR, [realpath(__DIR__.'/../../../infra/infra_php'), get_include_path()]));
 
 require_once DIR_SEI_WEB.'/SEI.php';
 require_once DIR_SEI_WEB.'/SEI.php';

src/dto/ProcessoExpedidoDTO.php

@@ -6,7 +6,8 @@
  */
 class ProcessoExpedidoDTO extends InfraDTO {
 
-  public function __construct($bolComposto = false) {
+  public function __construct()
+  {
       // Força o JOIN com todas as tabelas
       parent::__construct(true);
   }

src/int/ProcessoEletronicoINT.php

@@ -18,7 +18,7 @@ class ProcessoEletronicoINT extends InfraINT {
      * @param array(EstruturaDTO) $estruturas
      * @return array
      */
-  public static function gerarHierarquiaEstruturas($estruturas = array()){
+  public static function gerarHierarquiaEstruturas($estruturas = []){
 
     if(empty($estruturas)) {
         return $estruturas;
@@ -29,9 +29,9 @@ public static function gerarHierarquiaEstruturas($estruturas = array()){
           $nome  = $estrutura->getStrNome();
           $nome .= ' - ';
 
-          $array = array($estrutura->getStrSigla());
+          $array = [$estrutura->getStrSigla()];
         foreach($estrutura->getArrHierarquia() as $sigla) {
-          if(trim($sigla) !== '' && !in_array($sigla, array('PR', 'PE', 'UNIAO'))) {
+          if(trim($sigla) !== '' && !in_array($sigla, ['PR', 'PE', 'UNIAO'])) {
                 $array[] = $sigla;
           }
         }
@@ -50,7 +50,7 @@ public static function gerarHierarquiaEstruturas($estruturas = array()){
    * @param array(EstruturaDTO) $estruturas
    * @return array
    */
-  public static function gerarHierarquiaEstruturasAutoCompletar($estruturas = array())
+  public static function gerarHierarquiaEstruturasAutoCompletar($estruturas = [])
   {
 
     if (empty($estruturas['itens'])) {
@@ -62,11 +62,9 @@ public static function gerarHierarquiaEstruturasAutoCompletar($estruturas = arra
         $nome  = $estrutura->getStrNome();
         $nome .= ' - ';
 
-        $array = array($estrutura->getStrSigla());
+        $array = [$estrutura->getStrSigla()];
         foreach ($estrutura->getArrHierarquia() as $sigla) {
-          if (trim($sigla) !== '' && !in_array($sigla, array(
-            'PR', 'PE', 'UNIAO'
-          ))) {
+          if (trim($sigla) !== '' && !in_array($sigla, ['PR', 'PE', 'UNIAO'])) {
             $array[] = $sigla;
           }
         }
@@ -121,7 +119,7 @@ public static function autoCompletarRepositorioEstruturas($strPalavrasPesquisa)
   {
     $objProcessoEletronicoRN = new ProcessoEletronicoRN();
     $arrObjRepositorioDTO = (array) $objProcessoEletronicoRN->listarRepositoriosDeEstruturas();
-    $arrayRepositorioEstruturas = array();
+    $arrayRepositorioEstruturas = [];
     foreach ($arrObjRepositorioDTO as $value) {
       if (strpos(strtoupper($value->getStrNome()), strtoupper($strPalavrasPesquisa)) !== false) {
         $arrayRepositorioEstruturas[] = $value;
@@ -144,7 +142,7 @@ public static function formatarHierarquia($ObjEstrutura)
 
       $arrObjNivel = $ObjEstrutura->hierarquia;
 
-        $siglasUnidades = array();
+        $siglasUnidades = [];
         $siglasUnidades[] = $ObjEstrutura->sigla;
 
       foreach($arrObjNivel as $key => $objNivel){
@@ -168,9 +166,8 @@ public static function formatarHierarquia($ObjEstrutura)
         $objNivel=current($arrObjNivel);
 
     }
-      $dados=["nome"=>$nome,"objNivel"=>$objNivel];
 
-      return $dados;
+      return ["nome"=>$nome,"objNivel"=>$objNivel];
 
   }
 
@@ -184,40 +181,29 @@ public static function getCaminhoIcone($imagem, $relPath = null) {
       switch ($imagem) {
         case 'imagens/consultar.gif':
             return '/infra_css/svg/consultar.svg';
-            break;
         case 'imagens/alterar.gif':
             return '/infra_css/svg/alterar.svg';
-            break;
         case 'imagens/excluir.gif':
             return '/infra_css/svg/excluir.svg';
-            break;
         case '/pen_expedir_procedimento.gif':
             return 'modulos/' . $strModulo . '/imagens/pen_expedir_procedimento.png';
-            break;
         case '/pen_consultar_recibos.png':
             return 'modulos/' . $strModulo . '/imagens/consultar_recibo.png';
-            break;
         case '/pen_cancelar_tramite.gif':
             return 'modulos/' . $strModulo . '/imagens/pen_cancelar_envio.svg';
-            break;
         case '/infra_js/arvore/plus.gif':
             return '/infra_css/svg/mais.svg';
-            break;
         case '/infra_js/arvore/minus.gif':
             return '/infra_css/svg/menos.svg';
-            break;
         case 'imagens/anexos.gif':
             return '/infra_css/imagens/anexos.gif';
-            break;
         case 'imagens/sei_erro.png':
             return 'modulos/' . $strModulo . '/imagens/sei_erro.png';
-          break;
         default:
           if($relPath==null){
                 return $imagem;
           }
             return $relPath . $imagem;
-            break;
       }
     }
 
@@ -235,22 +221,18 @@ public static function getCssCompatibilidadeSEI4($arquivo)
       switch ($arquivo) {
         case 'pen_procedimento_expedir.css':
             return 'pen_procedimento_expedir_sei4.css';
-            break;
 
         default:
             return $arquivo;
-            break;
       }
     }elseif (InfraUtil::compararVersoes(SEI_VERSAO, ">", "4.0.1")) {
 
       switch ($arquivo) {
         case 'pen_procedimento_expedir.css':
             return 'pen_procedimento_expedir_sei402.css';
-              break;
 
         default:
             return $arquivo;
-              break;
       }
     }
 
@@ -276,22 +258,22 @@ public static function montarRestricaoTramitaGovBr($idUnidade, &$strCss, &$strHt
 
       $objPenUnidadeRestricaoRN = new PenUnidadeRestricaoRN();
       $arrObjPenUnidadeRestricaoDTO = $objPenUnidadeRestricaoRN->listar($objPenUnidadeRestricaoDTO);
-      $items = array();
-      $arrayKeys = array();
-      $arrObjPenUnidadeDTO = array();
-      $itemsUnidades = array();
-      $hdnRepoEstruturas = array();
+      $items = [];
+      $arrayKeys = [];
+      $arrObjPenUnidadeDTO = [];
+      $itemsUnidades = [];
+      $hdnRepoEstruturas = [];
       $strHtmlRepoEstruturasUnidades = "";
       foreach ($arrObjPenUnidadeRestricaoDTO as $item) {
         if (!in_array($item->getNumIdUnidadeRestricao(), $arrayKeys)) {
           //IdUnidadeRestricao NomeUnidadeRestricao
           $arrayKeys[] = $item->getNumIdUnidadeRestricao();
-          $items[] = array($item->getNumIdUnidadeRestricao(), $item->getStrNomeUnidadeRestricao());
-          $hdnRepoEstruturas[$item->getNumIdUnidadeRestricao()] = array();
+          $items[] = [$item->getNumIdUnidadeRestricao(), $item->getStrNomeUnidadeRestricao()];
+          $hdnRepoEstruturas[$item->getNumIdUnidadeRestricao()] = [];
         }
         if ($item->getNumIdUnidadeRHRestricao() != null) {
           $arrObjPenUnidadeDTO[] = $item;
-          $itemsUnidades[] = array($item->getNumIdUnidadeRHRestricao(), $item->getStrNomeUnidadeRHRestricao());
+          $itemsUnidades[] = [$item->getNumIdUnidadeRHRestricao(), $item->getStrNomeUnidadeRHRestricao()];
           $hdnRepoEstruturas[$item->getNumIdUnidadeRestricao()][] = $item->getNumIdUnidadeRHRestricao() . '±' . $item->getStrNomeUnidadeRHRestricao();
         }
       }

src/pen_expedir_bloco.php

@@ -13,12 +13,12 @@
   $strParametros = '';
   $bolErrosValidacao = false;
   $executarExpedicao = false;
-  $arrComandos = array();
+  $arrComandos = [];
 
   $strDiretorioModulo = PENIntegracao::getDiretorio();
 
-  $arrProtocolosOrigem = array();
-  $tramiteEmBloco = isset($_GET['tramite_em_bloco']) ? $_GET['tramite_em_bloco'] : null;
+  $arrProtocolosOrigem = [];
+  $tramiteEmBloco = $_GET['tramite_em_bloco'] ?? null;
   if ($tramiteEmBloco == 1) {
     $objPenBlocoProcessoDTO = new PenBlocoProcessoDTO();
     $objPenBlocoProcessoDTO->retDblIdProtocolo();
@@ -35,7 +35,7 @@
     $arrPenBlocoProcesso = $objPenBlocoProcessoRN->listar($objPenBlocoProcessoDTO);
     $idTramiteEmBloco = $arrPenBlocoProcesso[0]->getNumIdBloco();
     $strParametros .= '&id_bloco=' . $idTramiteEmBloco;
-    foreach ($arrPenBlocoProcesso as $i => $PenBlocoProcesso) {
+    foreach ($arrPenBlocoProcesso as $PenBlocoProcesso) {
       $arrProtocolosOrigem[] = $PenBlocoProcesso->getDblIdProtocolo();
     }
   } else {
@@ -98,7 +98,7 @@
 
         //Preparação dos dados para montagem da tela de expedição de processos
         if ($arrIdUnidadeRestricao != null) {
-          $repositorios = array();
+          $repositorios = [];
           foreach ($arrIdUnidadeRestricao as $value) {
             $repositorios[$value->getNumIdUnidadeRestricao()] = $value->getStrNomeUnidadeRestricao();
           }
@@ -109,7 +109,7 @@
         $repositorios = $objExpedirProcedimentoRN->listarRepositoriosDeEstruturas();
       }
 
-      $idRepositorioSelecionado = (isset($numIdRepositorio)) ? $numIdRepositorio : '';
+      $idRepositorioSelecionado = $numIdRepositorio ?? '';
       $strItensSelRepositorioEstruturas = InfraINT::montarSelectArray('', 'Selecione', $idRepositorioSelecionado, $repositorios);
 
       $strLinkAjaxUnidade = $objSessaoSEI->assinarLink('controlador_ajax.php?acao_ajax=pen_unidade_auto_completar_expedir_procedimento');
@@ -151,7 +151,7 @@
 
           $objPenBlocoProcessoRN = new PenBlocoProcessoRN();
           $arrPenBlocoProcesso = $objPenBlocoProcessoRN->listar($objPenBlocoProcessoDTO);
-          $arrProcedimentos = array();
+          $arrProcedimentos = [];
           foreach ($arrPenBlocoProcesso as $objDTO) {
             $objDTO->setNumIdRepositorioOrigem($numIdRepositorioOrigem);
             $objDTO->setNumIdUnidadeOrigem($numIdUnidadeOrigem);