Skip to content

fix(chore): 🐛 incluir funciones de extensiones#1757

Open
elguitarraverde wants to merge 1 commit intoNeoRazorX:masterfrom
elguitarraverde:incluir-funciones-de-extensiones
Open

fix(chore): 🐛 incluir funciones de extensiones#1757
elguitarraverde wants to merge 1 commit intoNeoRazorX:masterfrom
elguitarraverde:incluir-funciones-de-extensiones

Conversation

@elguitarraverde
Copy link
Copy Markdown
Contributor

Descripción

  • Actualmente no se verifica si existe la funcion en el array de extensiones.
  • Ahora se pueden incluir rows de estadistica sin tener que extender el modelo existente. ahora se puede hacer con extensiones.

Ejemplo de plugin:

<?xml version="1.0" encoding="UTF-8"?>
<view>
    <rows>
        <row type="statistics">
            <datalabel icon="fa-solid fa-person-circle-plus" label="customer-points" function="getPuntos"/>
        </row>
    </rows>
</view>

Extension:

class EditCliente
{
    public function getPuntos(): \Closure
    {
        return function () {
            return '999';
        };
    }
}

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • 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.

@NeoRazorX
Copy link
Copy Markdown
Owner

Añadí a las extensiones los métodos para comprobar si existe una extensión

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.

2 participants