PrestaShop/PrestaShop-1.5 — pull request #3 by PBo34 (2014-03-03 18:06:31+00:00)
Title: [-] FO : Modification to save carriercompare choice to page 4

Following this thread : http://www.prestashop.com/forums/topic/309699-impossible-de-valider-un-transporteur-dans-le-panier-carriercompare/
this is the modification I've made on several files in order to be able to validate the carrier "Paquet prioritaire" in carriercompare page 1 of the checkout.This validation ist still kept in page 4 of the checkout.
It works here. Please check that it won't create bugs somewhere else as I have little knowledge of the code and ...
NB : This part of the code that manages carrier validation, shipping cost makes repetitive calls both directions (classes cart et carrier). You have to put traces in to believe it. Dozens or even hundreds of the same calls or sequences of calls. It takes very little for the snake to bite his tail and enter infinite recursion. There's too many functionalities processed along the code that would deserve to be taken out into separate methods conditionned by parameters. As it is now, it's heavy, long, and hard to maintain.
Last but not least, repeating dozens of times the same calls impacts response times badly.

Kind regards.
PBo