Merge pull request #121 from PyAr/main-dev-jun-2023

Cumulative updates from develop (python3 conversion):

Changelog:
* WSPadron: agrego es_sucesion
* WSRemHarina: ajustes acumulados
* WSFEXv1: actividades (cambios acumulados)
* WSMTX: actividades (cambios acumulados)
* WSFEv1: actividades (cambios acumulados 28-Mar)
* PyRECE: agrego campos WSFEXv1

Commands used to convert from python 2.7 and fix coding style / unicode:
* 2to3 -w file.py
* python -m black file.py
* iconv -f iso-8859-1 -t utf-8 < file_new.py > file.py

Author: reingart

.github/workflows/python-package.yml

@@ -16,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: [2.7, 3.9, 3.11]
+        python-version: [3.9, 3.11]
 
     steps:
     - uses: actions/checkout@v2

.github/workflows/windows-installer.yml

@@ -156,6 +156,7 @@ jobs:
         run: |
           cat dist-64/.env >> $GITHUB_ENV
       - uses: "marvinpinto/action-automatic-releases@latest"
+        if: github.actor != 'dependabot[bot]'
         with:
           repo_token: "${{ secrets.GITHUB_TOKEN }}"
           automatic_release_tag: ${{ (github.ref_name != 'main') && 'beta' || 'latest' }}

formatos/formato_xml.py

@@ -49,6 +49,7 @@
                 "numero_cotizacion": str,
                 "numero_remito": str,
                 "ape": str,
+                "permiso_existente": str,
                 "incoterms": str,
                 "detalleincoterms": str,
                 "destinocmp": int,
@@ -188,6 +189,12 @@
     "fecha_serv_desde": "fechaservdesde",
     "fecha_serv_hasta": "fechaservhasta",
     "fecha_venc_pago": "fechavencpago",
+    "tipo_expo": "concepto",
+    "incoterms": "incoterms",
+    "incoterms_ds": "detalleincoterms",
+    "pais_dst_cmp": "destinocmp",
+    "idioma_cbte": "idioma",
+    "permiso_existente": "permiso_existente",
     "obs_generales": "otrosdatosgenerales",
     "obs_comerciales": "otrosdatoscomerciales",
     "resultado": "resultado",

requirements.txt

@@ -10,4 +10,4 @@ Pillow>=2.0.0
 tabulate==0.8.5
 certifi>=2020.4.5.1
 qrcode==6.1
-future==0.18.2
+future==0.18.3

ws_sr_padron.py

@@ -23,9 +23,9 @@
 from builtins import next
 
 __author__ = "Mariano Reingart <[email protected]>"
-__copyright__ = "Copyright (C) 2017 Mariano Reingart"
-__license__ = "GPL 3.0"
-__version__ = "3.04e"
+__copyright__ = "Copyright (C) 2017-2023 Mariano Reingart"
+__license__ = "LGPL-3.0-or-later"
+__version__ = "3.05a"
 
 import csv
 import datetime
@@ -42,6 +42,7 @@
     abrir_conf,
     norm,
     SoapFault,
+    safe_console,
 )
 from configparser import SafeConfigParser
 from pyafipws.padron import TIPO_CLAVE, PROVINCIAS
@@ -97,6 +98,7 @@ class WSSrPadronA4(BaseWS):
         "nro_doc",
         "tipo_persona",
         "estado",
+        "es_sucesion",
         "impuestos",
         "actividades",
         "direccion",
@@ -125,6 +127,7 @@ def inicializar(self):
         self.tipo_persona = ""  # FISICA o JURIDICA
         self.tipo_doc = self.nro_doc = 0
         self.estado = ""  # ACTIVO
+        self.es_sucesion = ""
         self.denominacion = ""
         self.direccion = self.localidad = self.provincia = self.cod_postal = ""
         self.domicilios = []
@@ -274,6 +277,7 @@ def Consultar(self, id_persona):
         self.nro_doc = data.get("idPersona")
         self.cuit = self.nro_doc
         self.estado = data.get("estadoClave")
+        self.es_sucesion = data.get("esSucesion")
         if not "razonSocial" in data:
             self.denominacion = ", ".join(
                 [data.get("apellido", ""), data.get("nombre", "")]
@@ -320,6 +324,7 @@ def main():
     global CONFIG_FILE
 
     DEBUG = "--debug" in sys.argv
+    safe_console()
 
     if "--constancia" in sys.argv:
         padron = WSSrPadronA5()
@@ -427,6 +432,7 @@ def main():
         print("Denominacion:", padron.denominacion)
         print("Tipo:", padron.tipo_persona, padron.tipo_doc, padron.nro_doc)
         print("Estado:", padron.estado)
+        print("Es Sucesion:", padron.es_sucesion)
         print("Direccion:", padron.direccion)
         print("Localidad:", padron.localidad)
         print("Provincia:", padron.provincia)

wsfev1.py

@@ -29,9 +29,9 @@
 from past.builtins import basestring
 
 __author__ = "Mariano Reingart <[email protected]>"
-__copyright__ = "Copyright (C) 2010-2021 Mariano Reingart"
+__copyright__ = "Copyright (C) 2010-2023 Mariano Reingart"
 __license__ = "LGPL-3.0-or-later"
-__version__ = "3.26a"
+__version__ = "3.27c"
 
 import datetime
 import decimal
@@ -140,9 +140,10 @@ class WSFEv1(BaseWS):
     _reg_class_spec_ = "pyafipws.wsfev1.WSFEv1"
 
     if TYPELIB:
-        _typelib_guid_ = '{8AE2BD1D-A216-4E98-95DB-24A11225EF67}'
+        _typelib_guid_ = "{8AE2BD1D-A216-4E98-95DB-24A11225EF67}"
         _typelib_version_ = 1, 26
-        _com_interfaces_ = ['IWSFEv1']
+        _com_interfaces_ = ["IWSFEv1"]
+        ##_reg_class_spec_ = "wsfev1.WSFEv1"
 
     # Variables globales para BaseWS:
     HOMO = HOMO
@@ -329,8 +330,8 @@ def AgregarComprador(self, doc_tipo=80, doc_nro=0, porcentaje=100.00, **kwarg):
 
     def AgregarActividad(self, actividad_id=0, **kwarg):
         "Agrego actividad a una factura (interna)"
-        op = { 'actividad_id': actividad_id }
-        self.factura['actividades'].append(op)
+        act = {"actividad_id": actividad_id}
+        self.factura["actividades"].append(act)
         return True
 
     def ObtenerCampoFactura(self, *campos):
@@ -381,9 +382,6 @@ def CAESolicitar(self):
                             "FchServDesde": f.get("fecha_serv_desde"),
                             "FchServHasta": f.get("fecha_serv_hasta"),
                             "FchVtoPago": f.get("fecha_venc_pago"),
-                            "FchServDesde": f.get("fecha_serv_desde"),
-                            "FchServHasta": f.get("fecha_serv_hasta"),
-                            "FchVtoPago": f["fecha_venc_pago"],
                             "MonId": f["moneda_id"],
                             "MonCotiz": f["moneda_ctz"],
                             "PeriodoAsoc": {
@@ -456,7 +454,7 @@ def CAESolicitar(self):
                             "Actividades": [
                                 {
                                     "Actividad": {
-                                        'Id': actividad['actividad_id'],
+                                        "Id": actividad["actividad_id"],
                                     }
                                 }
                                 for actividad in f["actividades"]
@@ -638,9 +636,9 @@ def CompConsultar(self, tipo_cbte, punto_vta, cbte_nro, reproceso=False):
                 }
                 copia = resultget.copy()
                 # TODO: ordenar / convertir opcionales (por ahora no se verifican)
-                del verificaciones['Opcionales']
-                if 'Opcionales' in copia:
-                    del copia['Opcionales']
+                del verificaciones["Opcionales"]
+                if "Opcionales" in copia:
+                    del copia["Opcionales"]
                 verifica(verificaciones, copia, difs)
                 if difs:
                     print("Diferencias:", difs)
@@ -802,9 +800,6 @@ def CAESolicitarX(self):
                             "FchServDesde": f.get("fecha_serv_desde"),
                             "FchServHasta": f.get("fecha_serv_hasta"),
                             "FchVtoPago": f.get("fecha_venc_pago"),
-                            "FchServDesde": f.get("fecha_serv_desde"),
-                            "FchServHasta": f.get("fecha_serv_hasta"),
-                            "FchVtoPago": f["fecha_venc_pago"],
                             "MonId": f["moneda_id"],
                             "MonCotiz": f["moneda_ctz"],
                             "PeriodoAsoc": {
@@ -1034,9 +1029,6 @@ def CAEARegInformativo(self):
                             "FchServDesde": f.get("fecha_serv_desde"),
                             "FchServHasta": f.get("fecha_serv_hasta"),
                             "FchVtoPago": f.get("fecha_venc_pago"),
-                            "FchServDesde": f.get("fecha_serv_desde"),
-                            "FchServHasta": f.get("fecha_serv_hasta"),
-                            "FchVtoPago": f["fecha_venc_pago"],
                             "MonId": f["moneda_id"],
                             "MonCotiz": f["moneda_ctz"],
                             "PeriodoAsoc": {
@@ -1095,6 +1087,15 @@ def CAEARegInformativo(self):
                                 for opcional in f["opcionales"]
                             ]
                             or None,
+                            "Actividades": [
+                                {
+                                    "Actividad": {
+                                        "Id": actividad["actividad_id"],
+                                    }
+                                }
+                                for actividad in f["actividades"]
+                            ]
+                            or None,
                             "CAEA": f["caea"],
                             "CbteFchHsGen": f.get("fecha_hs_gen"),
                         }
@@ -1301,13 +1302,15 @@ def ParamGetPtosVenta(self, sep="|"):
 
     @inicializar_y_capturar_excepciones
     def ParamGetActividades(self, sep="|"):
-        "Recuperador de valores referenciales de códigos de Actividades"
+        "Recuperador de valores referenciales de c�digos de Actividades"
         ret = self.client.FEParamGetActividades(
-            Auth={'Token': self.Token, 'Sign': self.Sign, 'Cuit': self.Cuit},
-            )
-        res = ret['FEParamGetActividades']
-        return [(u"%(Id)s\t%(Orden)s\t%(Desc)s" % p['ActividadesTipo']).replace("\t", sep)
-                 for p in res['ResultGet']]
+            Auth={"Token": self.Token, "Sign": self.Sign, "Cuit": self.Cuit},
+        )
+        res = ret["FEParamGetActividadesResult"]
+        return [
+            ("%(Id)s\t%(Orden)s\t%(Desc)s" % p["ActividadesTipo"]).replace("\t", sep)
+            for p in res["ResultGet"]
+        ]
 
 
 def p_assert_eq(a, b):
@@ -1500,7 +1503,7 @@ def main():
             if "--rg4540" in sys.argv:
                 wsfev1.AgregarPeriodoComprobantesAsociados("20200101", "20200131")
 
-            if '--rg5259' in sys.argv:
+            if "--rg5259" in sys.argv:
                 wsfev1.AgregarActividad(960990)
 
             # agregar la factura creada internamente para solicitud múltiple:

wsfexv1.py

@@ -21,9 +21,9 @@
 from builtins import str
 
 __author__ = "Mariano Reingart ([email protected])"
-__copyright__ = "Copyright (C) 2011-2021 Mariano Reingart"
+__copyright__ = "Copyright (C) 2011-2023 Mariano Reingart"
 __license__ = "LGPL-3.0-or-later"
-__version__ = "3.10a"
+__version__ = "3.11a"
 
 import datetime
 import decimal
@@ -44,6 +44,7 @@ class WSFEXv1(BaseWS):
         "GetCMP",
         "AgregarPermiso",
         "AgregarCmpAsoc",
+        "AgregarActividad",
         "GetParamMon",
         "GetParamTipoCbte",
         "GetParamTipoExpo",
@@ -67,6 +68,7 @@ class WSFEXv1(BaseWS):
         "GetParametro",
         "GetLastCMP",
         "GetLastID",
+        "GetParamActividades",
         "Dummy",
         "Conectar",
         "SetTicketAcceso",
@@ -192,6 +194,7 @@ def CrearFactura(
             "cbtes_asoc": [],
             "permisos": [],
             "detalles": [],
+            "actividades": [],
         }
         self.factura = fact
 
@@ -237,6 +240,12 @@ def AgregarCmpAsoc(
         )
         return True
 
+    def AgregarActividad(self, actividad_id=0, **kwarg):
+        "Agrego actividad a una factura (interna)"
+        act = {"actividad_id": actividad_id}
+        self.factura["actividades"].append(act)
+        return True
+
     @inicializar_y_capturar_excepciones
     def Authorize(self, id):
         "Autoriza la factura cargada en memoria"
@@ -304,6 +313,16 @@ def Authorize(self, id):
                     }
                     for d in f["detalles"]
                 ],
+                "Actividades": f["actividades"]
+                and [
+                    {
+                        "Actividad": {
+                            "Id": a["actividad_id"],
+                        }
+                    }
+                    for a in f["actividades"]
+                ]
+                or None,
             },
         )
 
@@ -748,23 +767,57 @@ def GetParamPtosVenta(self, sep="|"):
         res = ret["FEXGetPARAM_PtoVentaResult"].get("FEXResultGet")
         ret = []
         for pu in res:
-            u = pu["ClsFEXResponse_PtoVenta"]
+            p = pu["ClsFEXResponse_PtoVenta"]
             try:
                 r = {
-                    "nro": u.get("Pve_Nro"),
-                    "baja": u.get("Pve_FchBaj"),
-                    "bloqueado": u.get("Pve_Bloqueado"),
+                    "nro": p.get("Pve_Nro"),
+                    "baja": p.get("Pve_FchBaj"),
+                    "bloqueado": p.get("Pve_Bloqueado"),
                 }
             except Exception as e:
                 print(e)
             ret.append(r)
         return [
-            (u"%(nro)s\tBloqueado:%(bloqueado)s\tFchBaja:%(baja)s" % r).replace(
+            ("%(nro)s\tBloqueado:%(bloqueado)s\tFchBaja:%(baja)s" % r).replace(
                 "\t", sep
             )
             for r in ret
         ]
 
+    @inicializar_y_capturar_excepciones
+    def GetParamActividades(self, sep="|"):
+        "Recuperar lista de valores referenciales de códigos de Idiomas"
+        ret = self.client.FEXGetPARAM_Actividades(
+            Auth={
+                "Token": self.Token,
+                "Sign": self.Sign,
+                "Cuit": self.Cuit,
+            }
+        )
+        result = ret["FEXGetPARAM_ActividadesResult"]
+        self.__analizar_errores(result)
+
+        ret = []
+        for u in result.get("FEXResultGet", []):
+            u = u["ClsFEXResponse_ActividadTipo"]
+            try:
+                r = {
+                    "codigo": u.get("Id"),
+                    "ds": u.get("Desc"),
+                    "orden": u.get("Orden"),
+                }
+            except Exception as e:
+                print(e)
+
+            ret.append(r)
+        if sep:
+            return [
+                ("\t%(codigo)s\t%(ds)s\t%(orden)s\t" % it).replace("\t", sep)
+                for it in ret
+            ]
+        else:
+            return ret
+
 
 class WSFEX(WSFEXv1):
     "Wrapper para retrocompatibilidad con WSFEX"
@@ -936,6 +989,8 @@ def main():
                         cbteasoc_tipo, cbteasoc_pto_vta, cbteasoc_nro, cbteasoc_cuit
                     )
 
+                ok = wsfexv1.AgregarActividad(1234)
+
                 ##id = "99000000000100" # número propio de transacción
                 # obtengo el último ID y le adiciono 1
                 # (advertencia: evitar overflow y almacenar!)
@@ -1039,6 +1094,12 @@ def main():
             for r in ret:
                 print("||%(codigo)s||%(ds)s||" % r)
 
+            if "--rg5259" in sys.argv:
+                print("=== Actividades ===")
+                ret = wsfexv1.GetParamActividades(sep=False)
+                for r in ret:
+                    print("||%(codigo)s||%(ds)s||" % r)
+
         if "--ctz" in sys.argv:
             print(wsfexv1.GetParamCtz("DOL"))