jmueller17 / Aixada

Aixada helps self-managed consumption cooperatives to organize their flow of products, money, and information.
Other
33 stars 37 forks source link

Error al validar comanda: Error al distribuir i validar la comanda #1922 - Incorrecte DATE value: " #308

Open cecilethvnt opened 1 year ago

cecilethvnt commented 1 year ago

Hola! David ens han actualitzat versio servidor i versio aixada fa pocs dies. nomes habia provat el tancar una comanda i funcionava, pero no ens deixa validar i distribuir comandes. adjunto copia pantalla.

si no podem solucionar aixo (a part del envio de mail que tenim que solucionar a banda), haurem de canviar d'eina... image gracies i feliç equinocci! cecile

jorix commented 1 year ago

Per començar necesito el contingut de la taula aixada_version (estic fora de casa i sense PC)

davidromani commented 1 year ago

Per començar necesito el contingut de la taula _aixadaversion (estic fora de casa i sense PC)

mysql> SELECT * FROM aixada_version;
+----+------------------------------------------------------------------------------------------+-----------------+
| id | module_name                                                                              | version         |
+----+------------------------------------------------------------------------------------------+-----------------+
|  1 | main                                                                                     | 2.8             |
|  2 | START dbUpdate_280_c01: 2016-08-23 10:18:52                                              | 2.8             |
|  3 | > temporarily remove: alter table aixada_product drop foreign key aixada_product_ibfk_7; | removeRelatedFk |
|  4 | > temporarily remove: alter table aixada_product drop foreign key aixada_product_ibfk_8; | removeRelatedFk |
|  5 | > CHANGE aixada_unit_measure.id from tinyint to smallint                                 | 2.8             |
|  6 | END dbUpdate_280_c01: 2016-08-23 10:18:53                                                | 2.8             |
|  7 | START dbUpdate_280_c02: 2018-04-27 17:43:53                                              | 2.8             |
|  8 | > ALTER aixada_order_item ADD notes                                                      | 2.8             |
|  9 | > ALTER aixada_provider ADD order_send_format and order_send_prices                      | 2.8             |
| 10 | END dbUpdate_280_c02: 2018-04-27 17:43:53                                                | 2.8             |
| 11 | START dbUpdate_280_c02: 2021-02-28 21:16:31                                              | 2.8             |
| 12 | END dbUpdate_280_c02: 2021-02-28 21:16:31                                                | 2.8             |
| 13 | START dbUpdate_280_c02: 2023-09-13 12:42:27                                              | 2.8             |
| 14 | END dbUpdate_280_c02: 2023-09-13 12:42:27                                                | 2.8             |
+----+------------------------------------------------------------------------------------------+-----------------+
14 rows in set (0.00 sec)
jorix commented 1 year ago
mysql> SELECT * FROM aixada_version;
+----+--------------------------------------------------------------------+----------+
...
14 rows in set (0.00 sec)

Ja correu de fa temps amb les últimes actualitzacions de taules de la BD, a aixada_version res més a mirar.

Hauria de correr en local la vostra base de dades per mirar jo on i què falla.
Però aquí no es bon lloc per penjar-la ( podem parlar per correu )
( aquests dies estic molt liat amb unes obres, miraria esgarrapar alguna estona )

jorix commented 1 year ago

He vist que teniu una definició de la BBDD alterada respecte /sql/aixada.sql.

A la taula aixada_cart hi ha:

CREATE TABLE aixada_cart (
  ...
  ts_validated timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
  ts_last_saved timestamp ...

però hauria de ser:

CREATE TABLE aixada_cart (
  ...
  ts_validated timestamp NOT NULL DEFAULT 0,
  ts_last_saved timestamp ...

Desaconsello fer canvis d'aquesta mena.
I actualment Aixada permet distribuir i validar amb $order_distribution_method = 'distribute_and_validate' , que és la configuració que teniu.

Però això no provoca problema que heu reportat en aquest issue.

jorix commented 1 year ago

He trobat l'error (culpa meva, venia del PR #305) pero no fallava al meu servidor de proves
(jo usava Mysql 5.7 i falla amb Mysql 8.0)

Proveu el canvi jorix:Aixada:Issue_308

davidromani commented 1 year ago

@cecilethvnt he publicat els canvis del codi font que ha creat @jorix per solucionar el problema al nostre servidor de producció on tenim allotjada l'Aixada de La Vianda.

@cecilethvnt pots confirmar si ara ja funciona correctament la validació de comandes?

Captura de pantalla 2023-10-16 a les 10 51 12