-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
fix regression #36276
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
base: develop
Are you sure you want to change the base?
fix regression #36276
Conversation
|
Go toward to |
htdocs/reception/card.php
Outdated
| $alreadysent = array(); | ||
|
|
||
| //$origin = 'commande_fournisseur'; | ||
| $origin = 'commande_fournisseur'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$origin seems set and used previously.
I suggest to set $origin to 'supplier_order' here only is unset or empty.
Then on line 2491, you can replace with
(($origin == 'supplier_order') ? 'commande_fournisseur' : $origin)
Hi @eldy
I've gone through the process of creating a reception from purchase order with option to use standalone reception to off comparing step by step with v. 22 and everything seems fine!
Same result with RECEPTION_STANDALONE = 1
Through the code we can see origin takes different values
order_supplier(most common), commandeFournisseur, supplierorder, supplier_order, CommandeFournisseur, commande_fournisseur
Can you give me your guidance on this?
examples
dolibarr/htdocs/reception/class/reception.class.php
Lines 545 to 546 in bdd8dc0
dolibarr/htdocs/reception/class/reception.class.php
Line 857 in bdd8dc0
dolibarr/htdocs/reception/card.php
Line 153 in bdd8dc0
dolibarr/htdocs/reception/card.php
Line 1380 in bdd8dc0
dolibarr/htdocs/reception/card.php
Line 1920 in bdd8dc0
dolibarr/htdocs/reception/card.php
Line 2480 in bdd8dc0
dolibarr/htdocs/reception/dispatch.php
Line 325 in bdd8dc0