FrontAccounting forum — sales_order_ui.inc has a bug

Re: sales_order_ui.inc has a bug

null@example.com (kvvaradha) — Tue, 23 Apr 2019 03:25:28 +0000

For me it works fine. If you face problem with this. find a better solution for it. we will adapt to your solutions. I tested it on Linux and Windows. Both works fine for me

Re: sales_order_ui.inc has a bug

null@example.com (apmuthu) — Mon, 22 Apr 2019 17:58:01 +0000

The new code works for now.
More testing is needed to check for edge conditions stated above.

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.

Re: sales_order_ui.inc has a bug

null@example.com (apmuthu) — Mon, 22 Apr 2019 17:50:09 +0000

!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.

Re: sales_order_ui.inc has a bug

null@example.com (joe) — Mon, 22 Apr 2019 08:11:53 +0000

Ok, thanks @kvvaradha.

Committed and a new file here.

/Joe

Re: sales_order_ui.inc has a bug

null@example.com (kvvaradha) — Mon, 22 Apr 2019 05:22:15 +0000

@joe, Yes you are right.

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);

Re: sales_order_ui.inc has a bug

null@example.com (joe) — Sun, 21 Apr 2019 20:57:04 +0000

@kvvaradha.

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

Re: sales_order_ui.inc has a bug

null@example.com (apmuthu) — Sun, 21 Apr 2019 12:05:19 +0000

@kvvaradha: Your Post #3 is the same as your post #7 which has been incorporated in this official commit which is what I would like to revert and make it as per my commit.

Re: sales_order_ui.inc has a bug

null@example.com (kvvaradha) — Sun, 21 Apr 2019 11:09:16 +0000

kvvaradha wrote:

@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.

Re: sales_order_ui.inc has a bug

null@example.com (joe) — Sat, 20 Apr 2019 15:35:43 +0000

@kvvaradha

Please make a comment on this. Do you agree with @apmuthu?

Joe

Re: sales_order_ui.inc has a bug

null@example.com (apmuthu) — Sat, 20 Apr 2019 09:57:55 +0000

Will need to revert this commit - see this post.

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?

Re: sales_order_ui.inc has a bug

null@example.com (joe) — Fri, 25 Jan 2019 09:27:25 +0000

Ok, @kvvaradha

Fixed and a new file can be downloaded here.

Joe

Re: sales_order_ui.inc has a bug

null@example.com (kvvaradha) — Fri, 25 Jan 2019 05:19:47 +0000

@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.

Re: sales_order_ui.inc has a bug

null@example.com (joe) — Thu, 24 Jan 2019 22:35:34 +0000

Fixed and sent to repo. Thanks @kvvaradha.

The fixed file can be downloaded here.

Joe

sales_order_ui.inc has a bug

null@example.com (kvvaradha) — Thu, 24 Jan 2019 19:05:28 +0000

it looks like this now.

 
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);