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

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

Open
cecilethvnt opened this issue Sep 23, 2023 · 6 comments

Comments

@cecilethvnt
Copy link

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
Copy link
Contributor

jorix commented Sep 23, 2023

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

@davidromani
Copy link
Contributor

Per començar necesito el contingut de la taula aixada_version (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
Copy link
Contributor

jorix commented Sep 26, 2023

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
Copy link
Contributor

jorix commented Oct 12, 2023

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
Copy link
Contributor

jorix commented Oct 14, 2023

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
Copy link
Contributor

davidromani commented Oct 16, 2023

@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

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

No branches or pull requests

3 participants