n7consulting / Incipio

ERP / CRM for Junior-Entreprises.
http://jeyser-crm.n7consulting.fr
GNU Affero General Public License v3.0
42 stars 17 forks source link

erreur import BDD SIAJE #297

Closed Yirmandias closed 5 years ago

Yirmandias commented 5 years ago

Lorsque j'essaie d'importer une BDD SIAJE depuis un fichier CSV, j'obtiens cette erreur.

image

request.CRITICAL: Uncaught PHP Exception Symfony\Component\Debug\Exception\FatalThrowableError: "Undefined class constant 'EXPECTED_FORMAT'" at /app/src/Mgate/PubliBundle/Manager/CsvImporter.php line 28 {"exception":"[object] (Symfony\Component\Debug\Exception\FatalThrowableError(code: 0): Undefined class constant 'EXPECTED_FORMAT' at /app/src/Mgate/PubliBundle/Manager/CsvImporter.php:28)"} {"url":"/Documents/import","ip":"172.18.0.2","http_method":"POST","server":"crm.juniorinsaservices.fr","referrer":"https://crm.juniorinsaservices.fr/Documents/import"}

Est-ce que vous avez déjà rencontré cette erreur?

Stoakes commented 5 years ago

Bonjour,

Le module d'import est très peu utilisé. Pour ne rien cacher, j'avais oublié son existence jusqu'à ton message. Du coup il n'est pas maintenu /testé et c'est un bug.

Je vais corriger ça.

Yirmandias commented 5 years ago

Merci beaucoup! Est-ce que tu as une idée de quand ça sera réglé?

Stoakes commented 5 years ago

Je ne sais pas, je suis partagé entre:

Vous comptez dessus pour importer des données ?

D'ailleurs, est-ce que tu as toujours le squelette du fichier fourni par siaje (avec des fausses données) ? Je n'ai pas réussi à le retrouver dans mes archives.

Yirmandias commented 5 years ago

Effectivement on compte dessus pour importer nos données. On a 280 lignes donc à la main c'est un enfer... Mi-Mai c'est un peu tard pour nous. Je vais essayer de trouver ça, je n'en ai pas à te fournir pour le moment.

Yirmandias commented 5 years ago

squelette_siaje_etudes.xlsx Voilà ce que j'ai pu trouver.

Stoakes commented 5 years ago

Merci.

Est-ce que tu pourrais me donner le fichier squelette avec au moins une ligne de données s'il te plait ? Ca me permettrait de reproduire le problème, car actuellement comme le squelette ne contient aucune donnée je n'arrive pas à reproduire l'erreur (par exemple sur la démo, l'import fonctionne correctement alors qu'il ne devrait pas)

Si ça vous impacte, ok pour patcher ça et sortir une version 2.8.1 rapidement.

Yirmandias commented 5 years ago

Ok, je vais créer une ligne fictive!

Yirmandias commented 5 years ago

Je n'arrive pas à importer un fichier csv sur github.

Yirmandias commented 5 years ago

squelette_siaje_etudes.xlsx Voilà une ligne dans un format .xlsx J'espère que ça te suffira!

Yirmandias commented 5 years ago

Je viens d'essayer qqch : en format xlsx, il n'y a plus d'erreurs avec les fichiers. Par contre il n'importe aucune donnée image

Stoakes commented 5 years ago

Salut,

299 contient le patch pour le problème + quelques améliorations sur l'import depuis Siaje.

Je l'ai testé avec le fichier siaje.zip (le zip contient juste un fichier csv car Github ne supporte pas l'upload de fichier csv).

J'aimerais bien que vous testiez de votre coté avant de merger et de faire une release avec. Est-ce que vous avez moyen de tester le patch de votre coté ?

Stoakes commented 5 years ago

Finalement la version 2.8.1 va sortir aujourd'hui et contiendra le patch #299.

Il y a une série de patch de sécurité qui viennent de sortir sur Symfony, donc je vais produire la version 2.8.1 pour les rendre disponibles à toutes les installations de Jeyser et ne pas attendre votre retour.

Yirmandias commented 5 years ago

Salut,

Merci bcp pour le patch. Je ne vois pas encore que l'on peut mettre à jour jeyser, mais j'imagine qu'il faut que le pull request soit validé par d'autres? Nous n'avons qu'un seul serveur jeyser dans notre JE, il aurait fallu que je l'installe sur un autre PC mais je n'ai pas accès à ces ressources pour le moment.

Stoakes commented 5 years ago

Ok, je viens de release la version 2.8.1. Vous pouvez mettre à jour.