Bug 16017

Summary: Currency constraints and poorly named prices
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: AcquisitionsAssignee: Jonathan Druart <jonathan.druart>
Status: NEW --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: veron
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15084
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 13323, 13726    
Bug Blocks:    

Description Marcel de Rooy 2016-03-09 12:53:52 UTC
Remainder from bug 15084:
Note that another report should deal with adding missing constraints on
the currency code in suggestions and aqbooksellers.
Also note that the aqorder fields listprice and invoiceprice refer to
currency. Imo these are very poor names for currency codes; you should
never call something a price when you mean a currency code!
Comment 1 Marcel de Rooy 2016-03-09 12:54:40 UTC
Jonathan: Can I please leave this one for you? :)
Comment 2 Jonathan Druart 2016-03-09 13:30:22 UTC
I can, but I won't now.
There are bug 13323 and bug 13726 in the queue.
Comment 3 Marcel de Rooy 2016-03-09 13:42:00 UTC
Just to abuse this report, to add something slightly related:

AqordersItem -- what should be the primary key? 
And why is itemnumber no FK ?
Comment 4 Jonathan Druart 2016-03-11 11:23:46 UTC
(In reply to Marcel de Rooy from comment #3)
> Just to abuse this report, to add something slightly related:
> 
> AqordersItem -- what should be the primary key? 

The PK is correct, it's itemnumber

> And why is itemnumber no FK ?

See bug 12555