Skip to content

Commit

Permalink
fix: phpcs ignorando erros conhecidos
Browse files Browse the repository at this point in the history 
PHPCS para correção da complexidade ciclomática no futuro
  • Loading branch information
@caduvieira
caduvieira committed Oct 30, 2023
1 parent 32dfca2 commit 2d4eed1
Show file tree
Hide file tree
Showing 7 changed files with 99 additions and 64 deletions.
61 changes: 31 additions & 30 deletions src/PENIntegracao.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -575,6 +575,7 @@ private static function formatarXMLAjax($str)
return $str;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
public function processarControlador($strAcao)
{
//Configuração de páginas do contexto da Árvore do processo para apresentação de erro de forma correta
Expand Down Expand Up @@ -608,27 +609,27 @@ public function processarControlador($strAcao)
require_once dirname(__FILE__) . '/pen_procedimento_expedido_listar.php';
break;

case 'pen_map_tipo_documento_envio_listar';
case 'pen_map_tipo_documento_envio_excluir';
case 'pen_map_tipo_documento_envio_desativar';
case 'pen_map_tipo_documento_envio_listar':
case 'pen_map_tipo_documento_envio_excluir':
case 'pen_map_tipo_documento_envio_desativar':
case 'pen_map_tipo_documento_envio_ativar':
require_once dirname(__FILE__) . '/pen_map_tipo_documento_envio_listar.php';
break;
break;

case 'pen_map_tipo_documento_envio_cadastrar';
case 'pen_map_tipo_documento_envio_cadastrar':
case 'pen_map_tipo_documento_envio_visualizar':
require_once dirname(__FILE__) . '/pen_map_tipo_documento_envio_cadastrar.php';
break;
break;

case 'pen_map_tipo_documento_recebimento_listar';
case 'pen_map_tipo_documento_recebimento_listar':
case 'pen_map_tipo_documento_recebimento_excluir':
require_once dirname(__FILE__) . '/pen_map_tipo_documento_recebimento_listar.php';
break;
break;

case 'pen_map_tipo_documento_recebimento_cadastrar';
case 'pen_map_tipo_documento_recebimento_cadastrar':
case 'pen_map_tipo_documento_recebimento_visualizar':
require_once dirname(__FILE__) . '/pen_map_tipo_documento_recebimento_cadastrar.php';
break;
break;

case 'pen_apensados_selecionar_expedir_procedimento':
require_once dirname(__FILE__) . '/apensados_selecionar_expedir_procedimento.php';
Expand All @@ -648,56 +649,56 @@ public function processarControlador($strAcao)
break;

// Mapeamento de Hipóteses Legais de Envio
case 'pen_map_hipotese_legal_envio_cadastrar';
case 'pen_map_hipotese_legal_envio_cadastrar':
case 'pen_map_hipotese_legal_envio_visualizar':
require_once dirname(__FILE__) . '/pen_map_hipotese_legal_envio_cadastrar.php';
break;
break;

case 'pen_map_hipotese_legal_envio_listar';
case 'pen_map_hipotese_legal_envio_listar':
case 'pen_map_hipotese_legal_envio_excluir':
require_once dirname(__FILE__) . '/pen_map_hipotese_legal_envio_listar.php';
break;
break;

// Mapeamento de Hipóteses Legais de Recebimento
case 'pen_map_hipotese_legal_recebimento_cadastrar';
case 'pen_map_hipotese_legal_recebimento_cadastrar':
case 'pen_map_hipotese_legal_recebimento_visualizar':
require_once dirname(__FILE__) . '/pen_map_hipotese_legal_recebimento_cadastrar.php';
break;
break;

case 'pen_map_hipotese_legal_recebimento_listar';
case 'pen_map_hipotese_legal_recebimento_listar':
case 'pen_map_hipotese_legal_recebimento_excluir':
require_once dirname(__FILE__) . '/pen_map_hipotese_legal_recebimento_listar.php';
break;
break;

case 'pen_map_hipotese_legal_padrao_cadastrar';
case 'pen_map_hipotese_legal_padrao_cadastrar':
case 'pen_map_hipotese_legal_padrao_visualizar':
require_once dirname(__FILE__) . '/pen_map_hipotese_legal_padrao_cadastrar.php';
break;
break;

case 'pen_map_unidade_cadastrar';
case 'pen_map_unidade_cadastrar':
case 'pen_map_unidade_visualizar':
require_once dirname(__FILE__) . '/pen_map_unidade_cadastrar.php';
break;
break;

case 'pen_map_unidade_listar';
case 'pen_map_unidade_listar':
case 'pen_map_unidade_excluir':
require_once dirname(__FILE__) . '/pen_map_unidade_listar.php';
break;
break;

case 'pen_parametros_configuracao';
case 'pen_parametros_configuracao':
case 'pen_parametros_configuracao_salvar':
require_once dirname(__FILE__) . '/pen_parametros_configuracao.php';
break;
break;

case 'pen_map_tipo_documento_envio_padrao_atribuir';
case 'pen_map_tipo_documento_envio_padrao_atribuir':
case 'pen_map_tipo_documento_envio_padrao_consultar':
require_once dirname(__FILE__) . '/pen_map_tipo_documento_envio_padrao.php';
break;
break;

case 'pen_map_tipo_doc_recebimento_padrao_atribuir';
case 'pen_map_tipo_doc_recebimento_padrao_atribuir':
case 'pen_map_tipo_doc_recebimento_padrao_consultar':
require_once dirname(__FILE__) . '/pen_map_tipo_doc_recebimento_padrao.php';
break;
break;

case 'pen_envio_processo_lote_cadastrar':
require_once dirname(__FILE__) . '/pen_envio_processo_lote_cadastrar.php';
Expand Down
8 changes: 8 additions & 0 deletions src/rn/Editor3011RN.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ protected function inicializarObjInfraIBanco()
return BancoSEI::getInstance();
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function montarSimplesConectado(EditorDTO $objEditorDTO)
{

Expand Down Expand Up @@ -224,6 +225,7 @@ private function montarBarraFerramentas($bolAdicionarTextoPadrao, $bolBtnWSC, $b
return $arrRetorno;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function montarControlado(EditorDTO $parObjEditorDTO)
{
try {
Expand Down Expand Up @@ -539,6 +541,7 @@ public function filtrarTags($strConteudo)
return str_replace(array('<o:p>', '</o:p>'), '', $strConteudo);
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function gerarVersaoInicialControlado(EditorDTO $parObjEditorDTO)
{
try {
Expand Down Expand Up @@ -915,6 +918,7 @@ protected function gerarVersaoInicialControlado(EditorDTO $parObjEditorDTO)
}
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function adicionarVersaoControlado(EditorDTO $parObjEditorDTO)
{
try {
Expand Down Expand Up @@ -1574,6 +1578,7 @@ private function consultarHtmlIdentificacaoVersao3015(EditorDTO $parObjEditorDTO
return $html;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function consultarHtmlVersaoConectado($dados)
{
$parObjEditorDTO=$dados["parObjEditorDTO"];
Expand Down Expand Up @@ -2055,6 +2060,8 @@ private function substituirTagsInterno(EditorDTO $parObjEditorDTO, $strConteudo)
}, $strConteudo);
return $strConteudo;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
private function obterParametros(EditorDTO $parObjEditorDTO, $parStrTag){

global $SEI_MODULOS;
Expand Down Expand Up @@ -2958,6 +2965,7 @@ protected function recuperarLinkAcessoExternoControlado(DocumentoDTO $parObjDocu

}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
private function montarTagsContato(ContatoDTO $objContatoDTO, $strTipo){

if ($objContatoDTO->getStrStaGenero()==ContatoRN::$TG_MASCULINO){
Expand Down
34 changes: 21 additions & 13 deletions src/rn/EditorSEI4RN.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ protected function montarBasicoConectado(EditorDTO $objEditorDTO)
}

$ie = PaginaSEI::getInstance()->isBolNavegadorIE();
if ($ie) { $ie = PaginaSEI::getInstance()->getNumVersaoInternetExplorer();
if ($ie) { $ie = PaginaSEI::getInstance()->getNumVersaoInternetExplorer();
}
if (InfraArray::contar($arrImagemPermitida) > 0 && !PaginaSEI::getInstance()->isBolNavegadorSafariIpad() && (!$ie || $ie > 7)) {
$includePlugins[] = 'base64image';
Expand Down Expand Up @@ -132,7 +132,7 @@ protected function montarBasicoConectado(EditorDTO $objEditorDTO)


$temp = array('Cut', 'Copy', 'PasteFromWord', 'PasteText', '-', 'Undo', 'Redo');
if ($scayt != '') { $temp[] = 'SpellChecker';
if ($scayt != '') { $temp[] = 'SpellChecker';
}
$temp[] = 'Scayt';
$arrBotoes[] = $temp;
Expand All @@ -150,7 +150,8 @@ protected function montarBasicoConectado(EditorDTO $objEditorDTO)

return $ret;
}


// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function montarSimplesConectado(EditorDTO $objEditorDTO)
{

Expand Down Expand Up @@ -192,7 +193,7 @@ protected function montarSimplesConectado(EditorDTO $objEditorDTO)
$arrImagemPermitida = EditorINT::getArrImagensPermitidas();

$includePlugins = array('simpleLink', 'notification', 'extenso', 'maiuscula', 'stylesheetparser', 'tableresize', 'tableclean', 'symbol');
if ($bolAutotexto) { $includePlugins[] = 'autotexto';
if ($bolAutotexto) { $includePlugins[] = 'autotexto';
}
$removePlugins = array('resize', 'maximize', 'link', 'wsc', 'assinatura', 'save', 'autogrow');

Expand All @@ -216,7 +217,7 @@ protected function montarSimplesConectado(EditorDTO $objEditorDTO)
}

$ie = PaginaSEI::getInstance()->isBolNavegadorIE();
if ($ie) { $ie = PaginaSEI::getInstance()->getNumVersaoInternetExplorer();
if ($ie) { $ie = PaginaSEI::getInstance()->getNumVersaoInternetExplorer();
}
if ($bolImagens && InfraArray::contar($arrImagemPermitida) > 0 && !PaginaSEI::getInstance()->isBolNavegadorSafariIpad() && (!$ie || $ie > 7)) {
$includePlugins[] = 'base64image';
Expand Down Expand Up @@ -314,9 +315,9 @@ private function montarBarraFerramentas($bolAdicionarTextoPadrao, $bolBtnWSC, $b


$temp = array('Cut', 'Copy', 'CopyFormatting', 'PasteFromWord', 'PasteText', '-', 'Undo', 'Redo', 'ShowBlocks', 'Symbol');
if ($bolBtnWSC) { $temp[] = 'SpellChecker';
if ($bolBtnWSC) { $temp[] = 'SpellChecker';
}
if ($bolBtnScayt) { $temp[] = 'Scayt';
if ($bolBtnScayt) { $temp[] = 'Scayt';
}
$arrRetorno[] = $temp;

Expand All @@ -331,6 +332,7 @@ private function montarBarraFerramentas($bolAdicionarTextoPadrao, $bolBtnWSC, $b
return $arrRetorno;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function montarControlado(EditorDTO $parObjEditorDTO)
{
try {
Expand Down Expand Up @@ -816,6 +818,7 @@ public function filtrarTags($strConteudo)
return str_replace(array('<o:p>', '</o:p>'), '', $strConteudo);
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function gerarVersaoInicialControlado(EditorDTO $parObjEditorDTO)
{
try {
Expand Down Expand Up @@ -1206,6 +1209,7 @@ public function adicionarVersao(EditorDTO $parObjEditorDTO){
}
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function adicionarVersaoInternoControlado(EditorDTO $parObjEditorDTO)
{
try {
Expand Down Expand Up @@ -1606,7 +1610,7 @@ private function comparaEstilo($arrEstilos, $strEstilo)
if (InfraArray::contar(array_diff_assoc($value, $arrEstilos2))===0 &&
InfraArray::contar(array_diff_assoc($arrEstilos2, $value))===0
) {
return $key;
return $key;
}
}
} else {
Expand Down Expand Up @@ -1639,7 +1643,7 @@ private function converterDocumento(EditorDTO $parObjEditorDTO)
$objConjuntoEstilosDTO->setStrSinUltimo('S');
$objConjuntoEstilosDTO->retNumIdConjuntoEstilos();
$objConjuntoEstilosDTO = $objConjuntoEstilosRN->consultar($objConjuntoEstilosDTO);
if ($objConjuntoEstilosDTO==null) { throw new InfraException('Erro consultando conjunto de estilos.');
if ($objConjuntoEstilosDTO==null) { throw new InfraException('Erro consultando conjunto de estilos.');
}
$arrEstilos = $this->getArrayCss($objConjuntoEstilosDTO->getNumIdConjuntoEstilos());
$parObjEditorDTO->setNumIdConjuntoEstilos($objConjuntoEstilosDTO->getNumIdConjuntoEstilos());
Expand Down Expand Up @@ -1838,6 +1842,7 @@ private function consultarHtmlIdentificacaoVersao(EditorDTO $parObjEditorDTO)
return $html;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
protected function consultarHtmlVersaoConectado($dados)
{
$parObjEditorDTO=$dados["parObjEditorDTO"];
Expand Down Expand Up @@ -2030,7 +2035,7 @@ protected function consultarHtmlVersaoConectado($dados)
}

$numVersaoAtual = explode('.', SEI_VERSAO);
$numVersaoAtual = array_map(function($item){ return str_pad($item, 2, '0', STR_PAD_LEFT);
$numVersaoAtual = array_map(function($item){ return str_pad($item, 2, '0', STR_PAD_LEFT);
}, $numVersaoAtual);
$numVersaoAtual = intval(join($numVersaoAtual));

Expand Down Expand Up @@ -2320,7 +2325,7 @@ private function substituirTagsInterno(EditorDTO $parObjEditorDTO, $strConteudo,
}

$strConteudo=preg_replace_callback('/@([a-zA-Z0-9_-]+)@/', function($match) use ($parObjEditorDTO, $bolTesteModelo) {
if (isset(self::$arrTags[$match[1]])) { return self::$arrTags[$match[1]];
if (isset(self::$arrTags[$match[1]])) { return self::$arrTags[$match[1]];
}
$this->obterParametros($parObjEditorDTO, $match[1], $bolTesteModelo);
if (isset(self::$arrTags[$match[1]])) {
Expand Down Expand Up @@ -2492,6 +2497,8 @@ public function obterListaTags(){
}
return $arr;
}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
private function obterParametros(EditorDTO $parObjEditorDTO, $parStrTag, $bolTesteModelo = false){

global $SEI_MODULOS;
Expand Down Expand Up @@ -3157,7 +3164,7 @@ public function buscarImagemUpload($nomeArquivo)
$ext = pathinfo(DIR_SEI_TEMP . '/' . $nomeArquivo);

$ret = print_r($ext, true);
if (!in_array($ext['extension'], $arrImagemPermitida)) { return 'Tipo de Arquivo não permitido.';
if (!in_array($ext['extension'], $arrImagemPermitida)) { return 'Tipo de Arquivo não permitido.';
}

return 'data:image/' . $ext['extension'] . ';base64,' . base64_encode(file_get_contents(DIR_SEI_TEMP . '/' . $nomeArquivo));
Expand Down Expand Up @@ -3292,7 +3299,7 @@ public function validarTagsCriticas($arrImagemPermitida, $str)

public function processarLinksSei($str)
{
if ($str==null) { return null;
if ($str==null) { return null;
}
//remover sujeira scayt
$ret=preg_replace(self::$REGEX_SPAN_SCAYT, '$4', $str);
Expand Down Expand Up @@ -3610,6 +3617,7 @@ protected function recuperarLinkAcessoExternoControlado(DocumentoDTO $parObjDocu

}

// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
private function montarTagsContato(ContatoDTO $objContatoDTO, $strTipo){

if ($objContatoDTO->getStrStaGenero()==ContatoRN::$TG_MASCULINO){
Expand Down
4 changes: 2 additions & 2 deletions src/rn/ExpedirProcedimentoRN.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -739,7 +739,7 @@ public function registrarAndamentoExpedicaoAbortada($dblIdProtocolo)
$objAtividadeRN->gerarInternaRN0727($objAtividadeDTO);
}

public static function receberRecusaProcedimento($motivo, $unidade_destino, $numUnidadeDestino = null, $idProtocolo)
public static function receberRecusaProcedimento($motivo, $unidade_destino, $idProtocolo, $numUnidadeDestino = null)
{
try{
//Muda o status do protocolo para "Normal"
Expand Down Expand Up @@ -1283,7 +1283,7 @@ private function consultarComponenteDigital($parDblIdDocumento)
return (count($arrObjComponenteDigitalDTO) > 0) ? $arrObjComponenteDigitalDTO[0] : null;
}


// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
private function obterDadosArquivo(DocumentoDTO $objDocumentoDTO, $paramStrStaAssociacao, $bolMultiplosComponentes = false, $numOrdemComponenteDigital = 1)
{
if(!isset($objDocumentoDTO)){
Expand Down
5 changes: 3 additions & 2 deletions src/rn/ReceberProcedimentoRN.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ public function receberProcedimento($parNumIdentificacaoTramite)
}
}


protected function receberProcedimentoInternoControlado($parObjMetadadosProcedimento)
{
try {
Expand Down Expand Up @@ -1460,7 +1460,8 @@ private function atribuirDadosUnidade(ProcedimentoDTO $objProcedimentoDTO, $objD

return $objUnidadeDTOEnvio;
}


// phpcs:ignore Generic.Metrics.CyclomaticComplexity.MaxExceeded
private function atribuirDocumentos($parObjProcedimentoDTO, $parObjProtocolo, $objUnidadeDTO, $parObjMetadadosProcedimento, $parStrNumeroRegistro, $parDblIdProcedimentoAnexado = null)
{
if(!isset($parObjProtocolo)) {
Expand Down
Loading

0 comments on commit 2d4eed1

Please sign in to comment.