Includes Database changes. You need to log all terminals out during upgrade

• Tweaks and fixes
  • Honda magician
    • Reworked magician code, something NQR about old code. (Rewrote to use std java http apis instead of apache httpclient)
    • Integration with Honda public deadstock website working again (with their new site addr)
  • Modify job : include button 'view unit', similar to UX design for view customer button
  • Previously reported bug with nullpointerexception on workshop parts positively identified and fixed (bind error view in selCustDailyOrder)
  • Paperwork for dealers in SA, default to SA (previously was defaulting to Vic paperwork)
  • View spares invoice : for spares invoices which are part of a complex graph of multiple invoices because of repeated use of 'merge invoice' function, some invoices would not be accessible / viewable because they did not land in the graph of invoices as expected. Problem only occurs in some very narrow circumstances and structuring of invoices. Fixed in this version so should work for all possible ways invoices could organise.

Something broken with 5.2.16, pc-eftpos integration not working with it.

No actual bug in the code, I suspect some sort of issue with compilation of code into program. Did a full recompile. Okay now.

Includes DB changes, you need to log all terminals out during upgrade

• Tweaks and fixes
  • Bank rec and cash till : tweaked some times to improve database performance. Significantly snappier on systems I tested ( nested SQL procedures do not seem to be optimising well. Rewrote core query as plpgsql function, stored ref/prior rec ids in temporary variables. The resulting plan is essentially the same but it is significantly faster, suggests STABLE flag on prior/ref stored procs is not working as advertised and might be getting executed multiple times. Some more testing strongly suggests this is indeed the case )
  • low level v5 changed to help improve data migration tooling. (allow dbtool to hook into any JDBC database, not just c9 postgres. Improve performance of TrImport using cursors and some logic fixes on job batching not isolating self contained txn runs perfectly)
  • Couple of reports of add part to workshop triggering nullpointerexception. Problem likely caused by a closed table in a view which is triggered when keying in a part on order (pops select customer order screen). Added some guards to prevent tables from being accidently closed on main thread.