Aggiunta Funzionalità #9
Conversation
- src/Zendcart/Controller/Plugin/ZendCart.php::insert: nel caso in cui il prodotto che si vuole aggiungere abbia lo stesso id di un prodotto già esistente nel carrello, viene chiamata la update anzichè la insert. - config/zendcart.global.php.dist: Aggiunta varibile di configurazione per gestire la nuova feature modalità 'on_insert_update_existing_item', di default è false questo consente di mantenere l'attuale comportamento della libreria, se settata a true viene utilizzata la nuova modalità. - src/Zendcart/Factory/ZendCartFactory.php: reasonable default
|
Ciao, rimango a disposizione per chiarimenti e/o modifiche. |
|
@vikey89 up |
|
Perchè hai bisogno di questa funzionalità? |
|
Mi sono basato sul funzionamento degli altri carrelli, ad esempio amazon.it o prestashop.com Il caso d'uso è il seguente: La modifica proposta serve a coprire questa funzionalità. |
|
mmm ok mi hai convinto 👍 hai fatto dei test? purtroppo non ho tempo di testarla in questi giorni, quindi faccio il merge alla cieca. |
|
ho fatto i test nel mio ambiente, prima di committarla. |
|
se vuoi puoi aspettare a fare il merge, io attualmente utilizzo il mio fork. |
|
Ok questo weekend vedo se posso fare qualche test cosi faccio il merge. |
|
sinceramente avevo anche pensato di inserire una test suite, ma non ho avuto tempo e mi sono un pò incasinato con il test del controller plugin :) |
|
Ciao Vincenzo, |
altri moduli tipo ZendCartDoctrineORM ZendCartCookie
nel caso in cui il prodotto che si vuole aggiungere abbia lo stesso id di un prodotto già esistente nel carrello, viene chiamata la update anzichè la insert.
Aggiunta varibile di configurazione per gestire la nuova feature modalità 'on_insert_update_existing_item', di default è false questo consente di mantenere l'attuale comportamento della libreria, se settata a true viene utilizzata la nuova modalità.