Fixed and a new file can be downloaded here.
Joe
]]>I had to set config.php variable "$no_check_edit_conflicts = 1;" in windows for it to work though - it did not work for earlier commits.
]]>!isset($_POST['stock_id'])does not necessarily mean that the stock_id does not exist - it merely states that it was not chosen in the $_POST variable.
]]>Committed and a new file here.
/Joe
]]>As like @Apmuthu words, it rolls back to the last item sold to the customer. we are not able to change the item to sell. But we can try this option.it may be fessible to work on.
We will put an if condition, if the stock_id is not exist, we can get the last_sales_order_detail,like this, here is code
if ($order->fixed_asset)
stock_disposable_fa_list_cells(null,'stock_id', null, _('[Select item]'), true, $order->line_items);
else{
if(!isset($_POST['stock_id'])){ // Check the stock id not exist
$_POST['stock_id'] = last_sales_order_detail($order, 'stk_code'); //get the llast sales stock id.
}
sales_items_list_cells(null,'stock_id', last_sales_order_detail($order, 'stk_code'), false, true, true);
}
if (list_updated('stock_id')) {
$Ajax->activate('price');
$Ajax->activate('units');
$Ajax->activate('qty');
$Ajax->activate('line_total');
}Hope this may be fixed the problem.
if not, we remove the function from sales_order_d.inc last_sales_order_detail. It was defined there. and keep the sales item list cells like this
sales_items_list_cells(null,'stock_id', null, false, true, true);Did you mean that it was ok to change the
line 505 in /sales/includes/ui/sales_order_ui.inc
from:
sales_items_list_cells(null,'stock_id', last_sales_order_detail($order, 'stk_code'), false, true, true);
to:
sales_items_list_cells(null,'stock_id', null, false, true, true);?
/Joe
]]>@joe ,sorry for my mistake.
the code should be like this.
if ($order->fixed_asset) stock_disposable_fa_list_cells(null,'stock_id', null, _('[Select item]'), true, $order->line_items); else sales_items_list_cells(null,'stock_id', last_sales_order_detail($order, 'stk_code'), false, true, true); if (list_updated('stock_id')) { $Ajax->activate('price'); $Ajax->activate('units'); $Ajax->activate('qty'); $Ajax->activate('line_total'); }By mistake i put in else part of list_updated.
@joe - here I gave the same code exactly what he committed there. So you can update it.
]]>Please make a comment on this. Do you agree with @apmuthu?
Joe
]]>What is the purpose of getting the last_sales_order_detail() field (in this case, 'stk_code' which is the stock_id)? Byusing this commit, we are making the default choice of the Item description select box override the actual selection made in it. Should this be restricted to only those with editable description alone if used at all?
]]>Fixed and a new file can be downloaded here.
Joe
]]>the code should be like this.
if ($order->fixed_asset)
stock_disposable_fa_list_cells(null,'stock_id', null, _('[Select item]'), true, $order->line_items);
else
sales_items_list_cells(null,'stock_id', last_sales_order_detail($order, 'stk_code'), false, true, true);
if (list_updated('stock_id')) {
$Ajax->activate('price');
$Ajax->activate('units');
$Ajax->activate('qty');
$Ajax->activate('line_total');
}
By mistake i put in else part of list_updated.
]]>The fixed file can be downloaded here.
Joe
]]>
if ($order->fixed_asset)
stock_disposable_fa_list_cells(null,'stock_id', null, _('[Select item]'), true, $order->line_items);
else
if (list_updated('stock_id')) {
sales_items_list_cells(null,'stock_id', null, false, true, true);
$Ajax->activate('price');
$Ajax->activate('units');
$Ajax->activate('qty');
$Ajax->activate('line_total');
} else
sales_items_list_cells(null,'stock_id', last_sales_order_detail($order, 'stk_code'), false, true, true);But it should be like this
if ($order->fixed_asset)
stock_disposable_fa_list_cells(null,'stock_id', null, _('[Select item]'), true, $order->line_items);
else
sales_items_list_cells(null,'stock_id', null, false, true, true);
if (list_updated('stock_id')) {
$Ajax->activate('price');
$Ajax->activate('units');
$Ajax->activate('qty');
$Ajax->activate('line_total');
} else
sales_items_list_cells(null,'stock_id', last_sales_order_detail($order, 'stk_code'), false, true, true);