Manual order from back office gives error for certain customers

[30knees]

With certain customers, and I haven't been able to figure out which, I'm unable to place a manual back office order. I get this error message when I select any product: An order has already been placed with this cart. This product cannot be added to the cart. Does anyone have an idea how to troubleshoot the issue?

[wakabayashi]

@30knees I had this too! :O Thought this is some issue here... Are you using multistore?

I noticed, that it's working if you try a second time...

[30knees]

@wakabayashi Thanks for confirming!

Multistore is active, but I only have one store.

I've tried to create the order multiple times and it still doesn't work. I've also cleared out any unordered carts, etc.

[wakabayashi]

@30knees you are right. I messed up the errors. But I really have had this too. I checked now my files and I used a very dirty hack.

I changed in AdminCartsController.php the function ajaxProcessUpdateQty()

if ($this->context->cart->OrderExists() && 1==2) {

This is no solution of course. It just skipps the the condition. We should understand from where it comes. Since when do you have this? Does it appear when you don't use multistore?

[wakabayashi]

Which version are you using?

[30knees]

@wakabayashi I'm using 1.0.3. I'll test turning off multistore and also your hack and write again. Thanks! :)

[30knees]

@wakabayashi I turned off multistore but get the same error message.

Your hack works perfectly, though! Thank you!

[30knees]

@wakabayashi I just noticed that while the hack fixes the problem on the order creation page, it pushes it further down the line for the first order attempt. Now the error pops up on the next page after clicking "Create the order". But, when I tried a second time, like you wrote above, the order went through.

Might there be another spot where some code needs to be edited?

[wakabayashi]

@30knees have you fixed this issue somehow? It is still there in 1.0.7.

[dosbiner]

@wakabayashi this issue solved, see here https://github.com/thirtybees/thirtybees/issues/612

[wakabayashi]

@dosbiner thanks a lot. I am already at the same line of code :D but somehow I too dumb to understand the fix. Why was the variable been renamed?

[wakabayashi]

But ok doesn't seem to be important. thx @datakick for fixing it!

[DRMasterChief]

Will this be implemented in one of the next updates? https://github.com/thirtybees/thirtybees/commit/54c54a81e4f109e0efedecfe8e6c5d10bf03cbe1

[datakick]

sure, it will be part of the upcoming release. Here's all the commits that are scheduled: https://github.com/thirtybees/thirtybees/compare/1.0.7...1.0.x