Project

General

Profile

Development #43479

connecteur avis-imposition

Added by Benjamin Dauvergne about 1 month ago. Updated about 1 month ago.

Status:
Solution proposée
Priority:
Normal
Target version:
-
Start date:
29 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Pour se passer d'API particulier.

0001-add-avis-imposition.patch View (28.4 KB) Benjamin Dauvergne, 29 May 2020 04:53 PM

0001-avis_imposition-mimic-avis-imposition-API-particulie.patch View (31.1 KB) Benjamin Dauvergne, 01 Jun 2020 09:48 AM

0001-avis_imposition-mimic-avis-imposition-API-particulie.patch View (38.6 KB) Benjamin Dauvergne, 04 Jun 2020 09:41 PM

History

#1 Updated by Benjamin Dauvergne about 1 month ago

Draft.

#2 Updated by Benjamin Dauvergne about 1 month ago

La branche contient une version iso-compatible avec API particulier pour le endpoint avis-imposition, aucun besoin de token ou de raccordement.

#3 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Nouveau to Solution proposée

#5 Updated by Nicolas Roche about 1 month ago

(Je me serais attendu à un parseur HTML, comme par exemple BeautifulSoup,
peut-être que ça simplifierai la simulation du click sur "valider",
et rendrai plus robuste la récupération des champs, actuellement basé sur leur ordre d'apparition.)

Pour se passer d'API particulier.

oui mais peut-être pas tout de suite ?

    class Meta:
        verbose_name = _('API Particulier')

Avec ma fiche d'impôt j'ai un décalage :

Situation de famille     Célibataire
Nombre de personne(s) à charge     0

qui me renvoit :

{"err": 1, "err_class": "NotImplementedError", "err_desc": "0", "data": null}NotImplementedError: 0

Pour info :

196  -> data[field] = ADAPTERS[field](data[field])
(Pdb) data[field]
'0'
(Pdb) index
30
(Pdb) field
'situationFamille_simple'
(Pdb) td_contents[27]
'Situation de famille'
(Pdb) td_contents[28]
td_contents[28]
'Célibataire'
(Pdb) td_contents[29]
'Nombre de personne(s) à charge'
(Pdb) td_contents[30]
'0'

Par ailleurs je n'ai que 41 champs alors que 'revenuFiscalReference' indexe le 42ème champ :

(Pdb) len(td_contents)
41

#8 Updated by Benjamin Dauvergne about 1 month ago

Voilà avec le cas Nicolas Roche en sus. Je suis resté sur une recherche par index des cellules du tableau, je ne trouve pas la technique d'API particulier (recherche par chaîne des lignes puis par cellule) beaucoup plus fiable, si la DGFIP change le formatage ou les chaînes ça pêtera de toute façon. Et vu comment ces gens communiquent bien ensemble je doute que ça soit prévu par API particulier.

#11 Updated by Nicolas Roche about 1 month ago

Pour moi c'est bon (mais je préfère passer la main à quelqu'un d'autre pour valider).

Also available in: Atom PDF