Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Customer shipping deletion #2927

Closed
all3kcis opened this issue May 29, 2015 · 6 comments
Closed

Customer shipping deletion #2927

all3kcis opened this issue May 29, 2015 · 6 comments
Labels
Bug This is a bug (something does not work as expected) Priority - Low This is a minor bug | feature request : No feature are blocked or altered.

Comments

@all3kcis
Copy link
Contributor

Develop version :
ERROR: "ErrorBadValueForParamNotAString"
By removing a shipment containing batch products.

In versions 3.6 and 3.7 :
shipping is deleted but not well re-increment for batch products.

01

starting quantity
lot-001 5
lot-002 5

After shipping
02

After delete in 3.6 or 3.7
03

@BebZ
Copy link

BebZ commented May 29, 2015

double posted issue but your post is better, so i will close mine ;)
#2907

@rdoursenaud rdoursenaud added version 3.6.2 Bug This is a bug (something does not work as expected) Priority - Low This is a minor bug | feature request : No feature are blocked or altered. labels Jun 1, 2015
@fappels
Copy link
Contributor

fappels commented Jun 2, 2015

Something to check when debugging:
Warning: array_merge(): Argument #1 is not an array in ...\dolibarr\htdocs\expedition\class\expedition.class.php on line 1193

@marcosgdf
Copy link
Contributor

AFAIK, ProductBatch module is marked as experimental and therefore not supported...

@fappels
Copy link
Contributor

fappels commented Jul 11, 2015

In 3.8 it's out of experimental. So if fix in 3.7 we can merge to 3.8.

@all3kcis
Copy link
Contributor Author

I just edit my first post.
Without considering the problem on version 3.6 and 3.7 (where the productbatch module is experimental).

Bug "ErrorBadValueForParamNotAString" in develop version is still present.

As said by fappels :
Warning: array_merge(): Argument #1 is not an array in ...\dolibarr\htdocs\expedition\class\expedition.class.php on line 1193

@eldy eldy closed this as completed in a9289de Jul 14, 2015
@all3kcis
Copy link
Contributor Author

For me this bug is still there.
"ErrorBadValueForParamNotAString" after shipping deletion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug (something does not work as expected) Priority - Low This is a minor bug | feature request : No feature are blocked or altered.
Projects
None yet
Development

No branches or pull requests

5 participants