Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tarea #3301 agregar direccion a documentos compra #1575

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

Descripción

  • Al añadir documentos de compra también deberíamos guardar la dirección en el documento, como hacemos con las ventas.

image

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

@@ -104,6 +114,19 @@ public static function assets()
}
}

private static function addressField(Translator $i18n, PurchaseDocument $model, string $field, string $label, int $size, int $maxlength): string
{
$attributes = $model->editable && (empty($model->idcontactofact) || empty($model->direccion)) ?
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NeoRazorX aquí tengo dudas si en los proveedores se tienen que deshabilitar los inputs en algún momento o siempre se pueden quedar habilitados.

como he copiado varios metodos para pintar los inputs desde el SalesHeader, me he traido esta propiedad y no existe en los proveedores: $model->idcontactofact. deberia borrarla pero me deshabilita entonces los inputs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tampoco sé que es lo que hay que modificar al generar los PDF que se menciona en la tarea pero no he logrado ver que es lo que afecta a los nuevos campos de direcció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
1 participant