clientGBP 2.2.0__tar.gz → 2.2.2__tar.gz

{clientgbp-2.2.0 → clientgbp-2.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: clientGBP
-Version: 2.2.0
+Version: 2.2.2
 Summary: Libreria para el uso de GBP mediante HTTP. Sin utilizar interfaz gráfica (Scraping).Usando las credenciales de una cuenta, se obtiene las cookies de sesión y se pueden realizar operaciones. Se realiza todo de manera sincrona.
 License-Expression: MIT
 License-File: LICENSE

{clientgbp-2.2.0 → clientgbp-2.2.2}/gbp/parsers/__init__.py

@@ -1,5 +1,6 @@
 """Paquete de parsers HTML/AJAX y extractores de formularios."""
 
+from gbp.parsers.customers import extract_client_edit_keys, extract_usd_client_ids
 from gbp.parsers.entities import parse_client_form, parse_contact_form
 from gbp.parsers.forms import (
     extract_all_form_fields,
@@ -32,10 +33,12 @@ __all__ = [
     "extract_ajax_tokens",
     "extract_order_detail_dialog_url",
     "extract_all_form_fields",
+    "extract_client_edit_keys",
     "extract_item_code_from_row",
     "extract_order_message",
     "extract_save_order_alert",
     "extract_sku_tail",
+    "extract_usd_client_ids",
     "get_input_value",
     "get_option_value_by_text",
     "get_select_text",

clientgbp-2.2.2/gbp/parsers/customers.py

@@ -0,0 +1,35 @@
+"""Parsers de la grilla de resultados de la búsqueda de clientes (`table#grdData`)."""
+
+import re
+
+from bs4 import BeautifulSoup
+
+# Un CUIT puede tener dos fichas: la real, en dólares, y una espejo en pesos que crea Mercado Libre.
+USD_ACCOUNT = "(USD)"
+EDIT_LINK_PATTERN = re.compile(r"wfmGenericEditNew\.aspx','(\?[^']+)'")
+
+
+def extract_client_edit_keys(soup: BeautifulSoup) -> list[str]:
+    """Extrae la clave del link "Modificar Registro" de cada fila, primero las fichas en dólares.
+
+    Args:
+        soup (BeautifulSoup): Grilla de resultados parseada.
+
+    Returns:
+        list[str]: Claves de edición, una por cliente.
+    """
+    rows = sorted(soup.select("table#grdData tr"), key=lambda row: USD_ACCOUNT not in row.get_text())
+    return [match.group(1) for row in rows if (match := EDIT_LINK_PATTERN.search(str(row)))]
+
+
+def extract_usd_client_ids(soup: BeautifulSoup) -> set[int]:
+    """Extrae los IDs (última columna) de las fichas con cuenta corriente en dólares.
+
+    Args:
+        soup (BeautifulSoup): Grilla de resultados parseada.
+
+    Returns:
+        set[int]: IDs de cliente de las filas en dólares.
+    """
+    ids = {row.find_all("td")[-1].get_text(strip=True) for row in soup.select("table#grdData tr") if USD_ACCOUNT in row.get_text()}
+    return {int(client_id) for client_id in ids if client_id.isdigit()}

{clientgbp-2.2.0 → clientgbp-2.2.2}/gbp/payloads/ws.py

@@ -4,18 +4,18 @@ import re
 
 from gbp.validators.identifiers import get_national_id
 
-def crm_payload(tax_number: str) -> dict[str, str | int]:
+def crm_payload(tax_number: str, prefix_text: str | None = None) -> dict[str, str | int]:
     """Obtener payload para la búsqueda del CRM completo de un cliente para crear un contacto.
 
     Args:
         tax_number (str): CUIT o DNI del cliente objetivo.
+        prefix_text (str | None): Texto a buscar; por defecto el DNI extraido de `tax_number`.
 
     Returns:
         dict[str, str | int]: Payload serializable para enviar al endpoint correspondiente.
     """
-    national_id = get_national_id(tax_number)
     return {
-        "prefixText": national_id,
+        "prefixText": prefix_text or get_national_id(tax_number),
         "count": 20,
         "contextKey": "2/1/1/50031/1/-1/-1/2/-1",
     }

{clientgbp-2.2.0 → clientgbp-2.2.2}/gbp/services/base.py

@@ -1,7 +1,6 @@
 """Base service con estado de sesion y utilidades HTTP/comunes de GBP."""
 
 import logging
-import re
 from typing import Any
 
 from bs4 import BeautifulSoup
@@ -9,12 +8,19 @@ from bs4.element import Tag
 from requests import RequestException, Response, Session
 
 from gbp.decorator.require_login import require_login
-from gbp.http.headers import BASE_HEADERS, HEADERS_JSON_CONTACT
+from gbp.http.headers import BASE_HEADERS, HEADERS_AJAX, HEADERS_JSON_CONTACT
 from gbp.http.http import HttpClient
-from gbp.http.urls import BASE_URL, CUSTOMER_WS_URL, DEFAULT_TIMEOUT, ITEM_WS_URL, MAIN_URL
-from gbp.parsers import get_input_value, match_item_to_requested_sku, parse_html, extract_sku_tail
-from gbp.payloads import crm_payload, item_search_payload
-from gbp.validators import normalize_valid_tax_number
+from gbp.http.urls import BASE_URL, CUSTOMER_WS_URL, DEFAULT_TIMEOUT, ITEM_WS_URL, MAIN_URL, SEARCH_URL
+from gbp.parsers import (
+    extract_client_edit_keys,
+    extract_sku_tail,
+    extract_usd_client_ids,
+    get_input_value,
+    match_item_to_requested_sku,
+    parse_html,
+)
+from gbp.payloads import crm_payload, item_search_payload, search_client_payload
+from gbp.validators import get_national_id, normalize_valid_tax_number
 
 logger = logging.getLogger(__name__)
 logger.addHandler(logging.NullHandler())
@@ -217,7 +223,7 @@ class BaseGBPService:
         return "grdData" in html
 
     def _extract_client_edit_url(self, html: str) -> str | None:
-        """Extrae la URL de edición de un cliente desde los resultados de búsqueda.
+        """Extrae la URL de edición de un cliente desde los resultados de búsqueda (la ficha en dólares si hay).
 
         Args:
             html (str): Contenido HTML a analizar.
@@ -225,11 +231,9 @@ class BaseGBPService:
         Returns:
             str | None: Cadena encontrada o `None` cuando el dato no esta disponible.
         """
-        pattern = r"wfmGenericEditNew\.aspx','(\?[^']+)'"
-        matches = re.findall(pattern, html)
-        if matches:
-            edit_key = matches[0]
-            return f"{BASE_URL}/wfmGenericEditNew.aspx{edit_key}"
+        edit_keys = extract_client_edit_keys(parse_html(html))
+        if edit_keys:
+            return f"{BASE_URL}/wfmGenericEditNew.aspx{edit_keys[0]}"
         return None
 
     def _get_header_page_details(self) -> tuple[BeautifulSoup, str]:
@@ -277,33 +281,59 @@ class BaseGBPService:
             logger.warning("CUIT/DNI invalido para busqueda CRM: %s", tax_number)
             return None
 
-        try:
-            resp_customer = self._send_post_json(CUSTOMER_WS_URL, crm_payload(normalized_tax_number), HEADERS_JSON_CONTACT)
-            logger.debug(resp_customer.text)
-            if resp_customer.status_code != 200:
-                logger.error(f"Error en búsqueda CRM. Status: {resp_customer.status_code}")
-                return None
+        # GBP solo matchea por inicio de palabra: un CUIT guardado sin guiones no aparece buscando por DNI.
+        prefixes = [get_national_id(normalized_tax_number)]
+        if len(normalized_tax_number) == 11:
+            prefixes.append(normalized_tax_number)
 
-            # Obtiene los resultados encontrados y lo parsea a JSON para manejarlo de mejor manera.
-            customer_data = resp_customer.json()
-            clients_crm = customer_data["d"]
-            total_clients = len(clients_crm)
-            logger.info(f"Clientes encontrados para CUIT {normalized_tax_number}: {clients_crm}")
-            if total_clients == 0:
+        try:
+            # Se buscan siempre los dos: la ficha en pesos y la de dólares pueden guardar el CUIT distinto.
+            clients_crm: list[str] = []
+            for prefix_text in prefixes:
+                resp_customer = self._send_post_json(
+                    CUSTOMER_WS_URL, crm_payload(normalized_tax_number, prefix_text), HEADERS_JSON_CONTACT
+                )
+                logger.debug(resp_customer.text)
+                if resp_customer.status_code != 200:
+                    logger.error(f"Error en búsqueda CRM. Status: {resp_customer.status_code}")
+                    return None
+
+                # Obtiene los resultados encontrados y lo parsea a JSON para manejarlo de mejor manera.
+                found = resp_customer.json()["d"]
+                logger.info(f"Clientes encontrados para {prefix_text}: {found}")
+                clients_crm += found
+
+            if not clients_crm:
                 logger.warning("No se encontró cliente con ese CUIT")
                 return None
 
-            if total_clients > 1:
-                usd_crm_clients = [client for client in clients_crm if "PESOS" not in client]
-                if usd_crm_clients:
-                    return usd_crm_clients[0]
-                return clients_crm[0]
+            if len(clients_crm) > 1:
+                # El CRM ("RAZÓN SOCIAL |00006149|CUIT") no trae la moneda: se cruza por ID con la grilla.
+                usd_ids = self._get_usd_client_ids(normalized_tax_number)
+                usd_crm_clients = [client for client in clients_crm if any(f"|{client_id:08d}|" in client for client_id in usd_ids)]
+                return (usd_crm_clients or [client for client in clients_crm if "PESOS" not in client] or clients_crm)[0]
 
-            return customer_data["d"][0]
+            return clients_crm[0]
         except Exception as e:
             logger.error(f"Error inesperado buscando en CRM: {e}")
             return None
 
+    def _get_usd_client_ids(self, tax_number: str) -> set[int]:
+        """Busca el CUIT en la grilla de clientes y devuelve los IDs de sus fichas en dólares.
+
+        Args:
+            tax_number (str): CUIT o DNI del cliente objetivo.
+
+        Returns:
+            set[int]: IDs de las fichas en dólares; vacío si no se pudo buscar.
+        """
+        keys = self._get_page_session_keys(SEARCH_URL)
+        if not keys:
+            return set()
+
+        payload = search_client_payload(keys[0], tax_number=get_national_id(tax_number))
+        return extract_usd_client_ids(parse_html(self._send_post(SEARCH_URL, payload, HEADERS_AJAX).text))
+
     @require_login
     def _search_item(self, sku: str) -> str | None:
         """Busca un producto por SKU en el webservice itemGetText.

{clientgbp-2.2.0 → clientgbp-2.2.2}/gbp/services/customers.py

@@ -11,7 +11,7 @@ from requests import RequestException, Response
 from gbp.domain.models import Client
 from gbp.http.headers import HEADERS_AJAX
 from gbp.http.urls import BASE_URL, EDIT_URL, SEARCH_URL
-from gbp.parsers import get_input_value, get_seller_id, get_state_id, parse_client_form, parse_html
+from gbp.parsers import extract_client_edit_keys, get_input_value, get_seller_id, get_state_id, parse_client_form, parse_html
 from gbp.payloads import clean_client_payload, create_client_payload, search_client_payload, update_client_payload
 from gbp.decorator import require_login
 from gbp.services.base import BaseGBPService
@@ -387,12 +387,9 @@ class CustomerService(BaseGBPService):
             html (str): Contenido HTML a analizar.
 
         Returns:
-            list[str]: Lista de URLs de edición sin duplicados.
+            list[str]: Una URL por cliente, primero las fichas en dólares.
         """
-        pattern = r"wfmGenericEditNew\.aspx','(\?[^']+)'"
-        matches = re.findall(pattern, html)
-        urls = [f"{BASE_URL}/wfmGenericEditNew.aspx{match}" for match in matches]
-        return list(dict.fromkeys(urls))
+        return [f"{BASE_URL}/wfmGenericEditNew.aspx{key}" for key in extract_client_edit_keys(parse_html(html))]
 
     def _is_same_tax_number(self, candidate_tax_number: str, requested_tax_number: str) -> bool:
         """Valida coincidencia por CUIT completo o por DNI derivado del CUIT.

{clientgbp-2.2.0 → clientgbp-2.2.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "clientGBP"
-version = "2.2.0"
+version = "2.2.2"
 description = "Libreria para el uso de GBP mediante HTTP. Sin utilizar interfaz gráfica (Scraping).Usando las credenciales de una cuenta, se obtiene las cookies de sesión y se pueden realizar operaciones. Se realiza todo de manera sincrona."
 authors = [
     {name = "Leandro Franco",email = "leandro00fr@gmail.com"}