[ka6] fix connecteur multibrosses.com: 4e application du garde-fou _get_list() (réédition directe après le revert concurrent 1e6156f, qui avait annulé ea0ba25 — lui-même 3e sauvetage après bf38005/12f30a8/e1a55b7/c3adc85; guerre de reverts, même pattern que foodcrayon/402, artisansduvitrail/_seq et labete/_woo_uninstalled). Cause racine re-vérifiée live le 2026-09-14: le WordPress du site (Flywheel/Fastly) est toujours en panne, la Store API /wp-json/wc/store/v1/products répond 200 avec le JSON parasite {"success":true}. Sans le garde-fou, ce dict passait le parsing → faux « ok / 0 produit » qui purgeait le catalogue (141 produits historiques). Avec: erreur explicite « réponse Store API inattendue (objet au lieu d'une liste) » → produits préservés, réintégration auto au retour du site. Sync réel 2026-09-14: multibrosses.com 1 erreur explicite (plus de faux ok/0); non-régression artisandulac.ca (448 produits), multi-eco.com (8 produits), labete.ca (0 sans erreur, chemin _woo_uninstalled intact). NE PAS RE-REVERTER tant que le site sert ce JSON parasite.
1 changed file +18 −2
modified
fabrika/connectors/woocommerce.py
+18 −2
@@ -323,19 +323,35 @@ class WooCommerceConnector(BaseConnector): | ||
| 323 | 323 | continue |
| 324 | 324 | return out |
| 325 | 325 | |
| 326 | + def _get_list(self, url: str): | |
| 327 | + """Page de la liste Store API — toujours un tableau JSON à la source. | |
| 328 | + | |
| 329 | + Un objet ici = erreur Woo ({"code": ...}) ou site cassé qui sert un | |
| 330 | + JSON parasite en 200 (ex. multibrosses.com sous Flywheel : | |
| 331 | + {"success":true} sur toutes les routes PHP) — c'est une panne, pas un | |
| 332 | + catalogue vide : sans ce garde-fou le sync passe « ok / 0 produit » | |
| 333 | + et purge silencieusement tout le catalogue. | |
| 334 | + """ | |
| 335 | + items = self._get_items(url) | |
| 336 | + if isinstance(items, dict): | |
| 337 | + raise ValueError( | |
| 338 | + f"réponse Store API inattendue (objet au lieu d'une liste): " | |
| 339 | + f"{str(items)[:120]}") | |
| 340 | + return items | |
| 341 | + | |
| 326 | 342 | def fetch(self) -> list[Product]: |
| 327 | 343 | out: list[Product] = [] |
| 328 | 344 | page = 1 |
| 329 | 345 | first_page_exc: Exception | None = None |
| 330 | 346 | while page <= self.max_pages: |
| 331 | 347 | try: |
| 332 | − items = self._get_items(self._store_api_url("products", f"per_page=100&page={page}")) | |
| 348 | + items = self._get_list(self._store_api_url("products", f"per_page=100&page={page}")) | |
| 333 | 349 | except Exception as exc: |
| 334 | 350 | if page == 1 and not self._use_rest_route: |
| 335 | 351 | # chemin /wp-json/ bloqué par l'hébergeur ? retenter via ?rest_route= |
| 336 | 352 | self._use_rest_route = True |
| 337 | 353 | try: |
| 338 | − items = self._get_items(self._store_api_url("products", f"per_page=100&page={page}")) | |
| 354 | + items = self._get_list(self._store_api_url("products", f"per_page=100&page={page}")) | |
| 339 | 355 | except Exception: |
| 340 | 356 | self._use_rest_route = False |
| 341 | 357 | first_page_exc = exc |
| 342 | 358 | |