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

on corrige le schéma de compte pour pouvoir importer le journal #1561

Merged
merged 1 commit into from
Dec 1, 2024

Conversation

agallou
Copy link
Member

@agallou agallou commented Dec 1, 2024

suite au #1560

le fichier peux bien être uploadé, mais on a cette erreur :

Field 'idclef' doesn't have a default value" string(43) "Field 'idclef' doesn't have a default value" string(43) "Field 'idclef' doesn't have a default value"

Cela car on a pas de valeur par défaut sur le champ.

Ce champ est tout le temps vide :

MariaDB [afup]> select distinct idclef from compta;
+--------+
| idclef |
+--------+
|        |
+--------+
1 row in set (0.05 sec)

dans l'idée on pourrait potentiellement le supprimer, mais le plus simple dans l'immédiat vu qu'on doit importer le journal est de reproduire le comportement que l'on avait avant en ajoutant une valeur par défaut sur le champ.

suite au #1560

le fichier peux bien être uploadé, mais on a cette erreur :

```
Field 'idclef' doesn't have a default value" string(43) "Field 'idclef' doesn't have a default value" string(43) "Field 'idclef' doesn't have a default value"
```

Cela car on a pas de valeur par défaut sur le champ.

Ce champ est tout le temps vide :
```
MariaDB [afup]> select distinct idclef from compta;
+--------+
| idclef |
+--------+
|        |
+--------+
1 row in set (0.05 sec)
```

dans l'idée on pourrait potentiellement le supprimer, mais le plus simple
dans l'immédiat vu qu'on doit importer le journal est de reproduire le comportement que
l'on avait avant en ajoutant une valeur par défaut sur le champ.
@agallou agallou merged commit c5d83ad into master Dec 1, 2024
8 checks passed
@agallou agallou deleted the fix_import_compta branch December 1, 2024 13:59
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