Skip to content

Fix #3763: avisa de conflictos de nombres al extender clases - #2018

Open
AdelantiaDev wants to merge 2 commits into
NeoRazorX:masterfrom
AdelantiaDev:fix/3763-extension-name-conflict
Open

Fix #3763: avisa de conflictos de nombres al extender clases#2018
AdelantiaDev wants to merge 2 commits into
NeoRazorX:masterfrom
AdelantiaDev:fix/3763-extension-name-conflict

Conversation

@AdelantiaDev

Copy link
Copy Markdown

Descripción

Implementa la tarea #3763: avisa al desarrollador cuando dos extensiones registran un método con el mismo nombre en un modelo o controlador.

Hasta ahora, si dos plugins añadían por extensión un método homónimo sin usar pipe(), __call() ejecutaba solo el primero y el segundo se ignoraba en silencio. Ahora, cuando __call() detecta más de una extensión para el mismo nombre, registra un warning (una sola vez por clase+método) indicando el conflicto y sugiriendo usar pipe(). No se modifica el comportamiento de ejecución ni pipe().

Archivos:

  • Core/Template/ExtensionsTrait.php: detección y aviso del conflicto.
  • Test/Core/Template/ExtensionsTraitTest.php: pruebas del aviso.

¿Cómo has probado los cambios?

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

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