-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add autoselect optional parameter to select_produits #11496
Conversation
@@ -1891,9 +1891,10 @@ public function select_dolusers_forevent($action = '', $htmlname = 'userid', $sh | |||
* 'warehouseclosed' = select products from closed warehouses, | |||
* 'warehouseinternal' = select products from warehouses for internal correct/transfer only | |||
* @param array $selected_combinations Selected combinations. Format: array([attrid] => attrval, [...]) | |||
* @param int $autoselect Autoselect option for Ajax Product select |
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.
Can you confirm this feature means "Automatic selection if just one value" ? If yes, please change comment so we can understand difference between param "autoselect" and "selected"
@@ -187,9 +188,9 @@ | |||
|
|||
$form = new Form($db); | |||
if (empty($mode) || $mode == 1) { // mode=1: customer | |||
$arrayresult = $form->select_produits_list("", $htmlname, $type, 0, $price_level, $searchkey, $status, $finished, $outjson, $socid, '1', 0, '', $hidepriceinlabel, $warehouseStatus); | |||
$arrayresult = $form->select_produits_list("", $htmlname, $type, $limit, $price_level, $searchkey, $status, $finished, $outjson, $socid, '1', 0, '', $hidepriceinlabel, $warehouseStatus); |
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.
It seems the PR contains change from another PR.
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.
Umm. I'll correct that
It seems this PR was set with "Discussion" or "PR to fix" tag, but no answer nor correction was provided since a long time. So request is automatically closed. |
New Add optional Autoselect parameter for Select_produits
Add optional Autoselect parameter for Select_produits for used in ajax product select call.