[Koha-bugs] [Bug 7180] Order from staged file improvements

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7180

--- Comment #85 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to Katrin Fischer from comment #82)
> Hi Jonathan,
Hi Katrin,

> one of the new tests doesn't pass for me:
> not ok 4 - GetBudgetPeriod(0) returned undef : noactive BudgetPeriod
Will be fixed (there is no transaction for this ut file, we should remove
existing funds/budgets).

(In reply to Katrin Fischer from comment #83)
> Question: The new example has this line: 
> price: 947$a|947$c 
> What does happen in this case? Is the program using the first field that is
> available in the record?
Yes:
If 947$a is defined, we take this value, else we take 947$c if defined.

(In reply to Katrin Fischer from comment #84)
> When selecting a batch to import from, there are lots of warns in the logs:
Will be fixed.

> When I import a single record, by only updating the quantity from 1 to 2,
> not changing any other value (relying on the fact that it might take the
> default from the bottom?) - no warning is issued, but the order line is not
> added to my basket. The logs show lots of unhappiness with me trying that.

Ho, good catch!
I don't see the logs but there was a js bug.

> Should the selections at the bottom be taken into account if you don't
> select anything for the single record? I think we need to warn that you have
> to select a budget on the single record or take the global into account?

I change the ergonomic of the page in order to be much more easy to understand.
I will add 3 tabs: 
    Select to import
    Items information
    Default accounting details
In the first one, the user has to select the records to import and to define
specific values.
The second one allow to add some field values for *all* selected record.
The third one offers to define "default" values for *all* selected record. In
fact a defined value in this tab is used only if no one is filled for the
record.

I hope it will be more understandable like that.

-- 
You are receiving this mail because:
You are watching all bug changes.