Jump to content

This is insane!


Recommended Posts

Oh, Prestashop... I have such a love/hate relationship with you. Some of your features are sooo good and others are just plain crazy.

 

Take this for example:

Every time I make a change to a Carrier in the Carrier Module...even a small one such as in the 'Time Delay' field, the ID for the carrier is deleted and a new one is assigned.

 

This is CRAZY! This is causing so many broken connections to other modules within Prestashop.

 

Why oh why does it work this way... :wacko: :wacko: :wacko: :wacko: :wacko::angry: :angry: :angry: :angry: :huh: :huh: :huh: :huh: :o :o :o :o

  • Like 1
Link to comment
Share on other sites

it has to work this way.  carriers as used in carts and orders, and once an order is created, the carrier is associated to that order.

 

Now if you go change the carrier, it would affect all of the historical orders that are associated to it.  For that reason, a new carrier is created anytime you change the carrier.  The 'old' carrier record is just marked 'hidden' so that old orders can still reference it.

 

Unfortunately the modules that are 'broken' are poorly designed.

  • Like 1
Link to comment
Share on other sites

Thanks for your reply :)

 

To me, it would be so much better, and make more sense, to just delete a carrier and create a new one, if you needed to.

your response tells me you did not understand what I wrote.  if you deleted the old carrier, and added a new one, then all the old orders that have referenced that carrier would break. 

Link to comment
Share on other sites

there is a hook that is executed whenever a carrier is changed, that the modules should have registered if they were designed properly.

 

this way, the module will be informed when this scenario occurs, and they can update themselves.  you should have the author of these modules update them so they work properly

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...