Tarea #1875 - Implementar funcion para poder copiar un producto. #1400
Conversation
|
@daniel89fg échale un vistazo a esto a ver que tal lo ves. es que no te puedo marcar como Reviewer en Github por eso te menciono en este mensaje. |
Core/Controller/EditProducto.php
Outdated
| /** | ||
| * Copy Product. | ||
| */ | ||
| protected function copyProductAction(): void |
There was a problem hiding this comment.
Toda esta parte se movería al controlador CopyModel, y también se debería poner la parte visual del twig. La idea es copiar todo el producto incluidas sus variantes, solo permitiendo cambiar algunos datos generales como la descripción por ejemplo. Échale un ojo al copiar un asiento o un documento de presupuestos, es lo mismo, es aprovechar eso mismo.
Core/Controller/CopyModel.php
Outdated
| $subject = new Proveedor(); | ||
| if (false === $subject->loadFromCode($this->request->request->get('codproveedor'))) { | ||
| $this->dataBase->beginTransaction(); | ||
|
|
There was a problem hiding this comment.
No entiendo que paso con la función savePurchaseDocument() desapareció entera. ¿por qué?
Core/Controller/CopyModel.php
Outdated
| @@ -241,7 +277,7 @@ protected function savePurchaseDocument() | |||
| $this->saveDocumentEnd($newDoc); | |||
| } | |||
|
|
|||
| protected function saveSalesDocument() | |||
| protected function saveProduct() | |||
There was a problem hiding this comment.
esto está mal, has cambiado el nombre a la función para copiar documentos de venta y la pusiste para copiar el producto.
|
Quizás se envío mal, por que es raro, el controlador CopyModel lo veo bastante mal.... |
|
Sin querer borre un método y puse el código nuevo en ese método en lugar de donde correspondía. un lio, pero ya está corregido. Sigo picando codigo, ahora me falta la lógica de copiar el modelo y sus variantes. faltan también muchas cosas en la view(textos traducidos,....) |
|
@daniel89fg ya está implementada la lógica de copiar el producto y sus variantes. Problemas:
|
elguitarraverde
changed the title
|
Quizás lo que yo haría es dejar que las referencias del nuevo producto y sus variantes, si el usuario no pone nada, que las genera de forma automática. La variante que se crea por defecto al crear el producto hay que eliminarla. |
|
Solo me queda configurar el botón para que solo salga en la vista editar pero no sé como hacerlo. En la vista editar sale el botón abajo y no debería salir ahí. En la vista para crear un producto también sale y no debería salir ahí. |
| return; | ||
| } | ||
|
|
||
| // TODO ACL |
There was a problem hiding this comment.
También me queda ver si hay que aplicar alguna restricción de permisos para que no se pueda crear según qué usuario o dejarlo sin ninguna restricción.
There was a problem hiding this comment.
Por el momento no hace falta, igual que los documentos.
El botón aparece arriba y abajo, es un fallo del core pendiente de solucionar, no le des importancia a eso. Para hacer que no aparezca el botón cuando el producto es nuevo y aún no tiene idproducto, el botón se debe añadir en el |
elguitarraverde
deleted the
Tarea-#1875---Implementar-función-para-poder-copiar-un-producto
branch
Descripción
¿Cómo has probado los cambios?
Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado: