Ottimizzazione importazione
Ottimizzate update_categories e manufacturerSave
Merge request reports
Activity
assigned to @c.tranquilli
463 481 } 464 482 } 465 483 466 $ps_manufacturer->name = $name; 467 $ps_manufacturer->active = 1; 484 if($ps_manufacturer->name == $name) 485 $brandChanged = false; 486 487 if($brandChanged){ 488 //$this->io6_write_log(date('Y-m-d H:i:s') . " Manufatcuter changed" , IO6_LOG_WARNING); 489 $ps_manufacturer->name = $name; 490 $ps_manufacturer->active = 1; 491 } - Comment on lines +484 to +491
Ho dei dubbi su questo punto, in particolare sul flag
activeche non viene controllato nella condizione del brandChanged, ma viene poi impostato a 1 solo se il nome del brand è cambiato.Se prima l'intenzione era di forza l'attivazione dei brand esportati, adesso in questo modo si andrebbe a riattivare un brand solo nel caso in cui abbia cambiato il nome. Se vuoi parliamone.
Ho dato un occhiata al codice, in nessun punto noi andiamo ad impostare brand o supplier a 0 (disattivati), quindi vista la possibilità del cliente di disattivarsi i brand o supplier non so quanto sia sensato che poi noi andiamo a riattivarglielo. Questo magari mi sfugge a me perchè non so come funziona l'applicativo lato cliente, immagino siano loro ad impostare quali brand o supplier estrarre, quindi l'atto di disattivarlo da Prestashop non è molto corretto se poi lo fai esportare ugualmente.
507 535 return false; 508 536 } 509 537 } 510 $ps_supplier->name = $name; 511 $ps_supplier->active = 1; 538 539 if($ps_supplier->name == $name) 540 $supplierChanged = false; 541 542 if($supplierChanged){ 543 //$this->io6_write_log(date('Y-m-d H:i:s') . " Supplier changed" , IO6_LOG_WARNING); 544 545 $ps_supplier->name = $name; 546 $ps_supplier->active = 1; 547 } added 1 commit
- 3d4be2ff - Aggiunto controller per donwloadImmagini in differita
mentioned in commit 44bb8374
