Fix: Current Bills thinks it is Bill History user/jeff/lp855894_current_bills_vs_bill_history
authorJeff Godin <jgodin@tadl.org>
Wed, 21 Sep 2011 20:41:19 +0000 (16:41 -0400)
committerJeff Godin <jgodin@tadl.org>
Wed, 21 Sep 2011 20:41:19 +0000 (16:41 -0400)
commitde8ef373ab90f490ad7943a281b32ab4f6a41f8f
tree1c97c8723272e0925734ba802cdee33cdd1af5a9
parent038c205d146e49ba9727370cc661da9676ff6223
Fix: Current Bills thinks it is Bill History

This fixes LP 855894: Bills interface calls itself Bill History,
uses bills_historical receipt template

https://bugs.launchpad.net/evergreen/+bug/855894

At some point in development, the Current Bills and the Bill History
interface were in a unified file, with an xul_param of "current" used
to switch between the two.

Nothing currently sets the xul_param "current", so the Current Bills
interface (bill2.xul and bill2.js) was improperly:

* not hiding the xact_finish column
* using a caption label of "Bill History"
* using the bills_historical receipt template

The Bill History interface was okay -- "not current" was the default.

This commit removes the tests for xul_param('current') and uses the
appropriate code for each interface.

// this code in both bill2.js and bill_history.js:
var template = 'bills_historical';
if (xul_param('current')) template = 'bills_current';

// becomes:
var template = 'bills_current';    // in bill2.js
var template = 'bills_historical'; // in bill_history.js

Also: added missing staff.patron.bill_interface.caption.label entity

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Open-ILS/web/opac/locale/en-US/lang.dtd
Open-ILS/xul/staff_client/server/patron/bill2.js
Open-ILS/xul/staff_client/server/patron/bill_history.js