Skip to content

Fix: remove dotted regimen suffix ( CAFETALERA S.P.R. DE R.L. | CAFET… - #134

Open
devpolamx wants to merge 1 commit into
eclipxe13:masterfrom
devpolamx:fix-regimen-capital-dotted-suffix
Open

Fix: remove dotted regimen suffix ( CAFETALERA S.P.R. DE R.L. | CAFET…#134
devpolamx wants to merge 1 commit into
eclipxe13:masterfrom
devpolamx:fix-regimen-capital-dotted-suffix

Conversation

@devpolamx

Copy link
Copy Markdown

ste PR corrige un caso donde el RegimenCapitalRemover no elimina sufijos cuando vienen con puntos intermedios, como:
S.P.R. DE R.L.
Caso Real que encontré en un certificado:
CONCORDIA SUPERIOR COMPAÑIA CAFETALERA S.P.R. DE R.L.

Ejemplos de normalización

Entrada (del certificado) Normalizado Sufijo removido
EMPRESA X S.A. DE C.V. EMPRESA X SA DE CV SA DE CV
CAFETALERA S.P.R. DE R.L. CAFETALERA SPR DE RL SPR DE RL
COOPERATIVA S.C. DE R.L. DE C.V. COOPERATIVA SC DE RL DE CV SC DE RL DE CV
EMPRESA NORMAL SA DE CV EMPRESA NORMAL SA DE CV ✅ sin cambios, ya funciona
SOCIEDAD COOPERATIVA SOCIEDAD COOPERATIVA ✅ sin cambios, no hay puntos

Ventajas de este enfoque

  1. No modifica RegimenCapitalRemover — la lista de sufijos permanece intacta
  2. No duplica entradas — no necesitamos agregar variantes con puntos
  3. Es retrocompatible — nombres sin puntos no se ven afectados
  4. Un solo punto de cambio — la normalización ocurre en el método que consume el nombre
Captura de pantalla 2026-05-29 130558

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant