[OPEN-ILS-DEV] [RFC] Price (or percent thereof) as max fine amount.

Mike Rylander mrylander at gmail.com
Sun Jul 27 11:24:49 EDT 2008


On Sun, Jul 27, 2008 at 11:21 AM, Mike Rylander <mrylander at gmail.com> wrote:
> On Sun, Jul 27, 2008 at 8:52 AM, Bill Erickson <erickson at esilibrary.com> wrote:
>> On Saturday 26 July 2008 10:52 Mike Rylander wrote:
> [snip]
>>>
>>> * The config.rule_max_fine table gets a new boolean column called
>>> is_percent which defaults to false
>>> * The IDL (and, for good measure, the storage server's ancient
>>> equivelant) is taught about this new field
>>> * OpenILS::Circ::Circulate::build_checkout_circ_object inspects this
>>> field on the chosen rule for truthiness, and finding a positive
>>> result:
>>>   - gets the item price from the copy and
>>>   - if that's null (not allowed yet, but will be in 1.4) attempts to
>>> find an org_unit-appropriate default item price (also used for LOST
>>> fee)
>>>   - or, if price == 0 and the "charge default on 0-price" org unit
>>> setting is true, attempts to find an org_unit-appropriate default item
>>> price (also used for LOST fee)
>>>   - and now, having found an appropriate full price for the item,
>>> interprets the max fine amount as a percentage value, scaling the full
>>> price as specified
>>>   - and finally, uses that amount as the max fine amount, instead of a
>>> specific value
>>>
> [snip]
>>
>>
>> I think it's a great addition.  I eyeballed the patch and it looks good.
>> (comment:  instead of redefining isTrue, we can use $U->isTrue).

I don't see the definition for isTrue anywhere in the OpenILS:: namespace ...

--miker

>>
>> One addition to consider is adding an org-setting cache (just a local data
>> structure) for default item price and charge-on-0 to reduce network calls.
>> That can come later, though.
>>
>
> Next question is ... how far back do we backport this.  I would lobby
> for 1.2.3.0, but not to strongly.
>
> --
> Mike Rylander
>  | VP, Research and Design
>  | Equinox Software, Inc. / The Evergreen Experts
>  | phone: 1-877-OPEN-ILS (673-6457)
>  | email: miker at esilibrary.com
>  | web: http://www.esilibrary.com
>



-- 
Mike Rylander
 | VP, Research and Design
 | Equinox Software, Inc. / The Evergreen Experts
 | phone: 1-877-OPEN-ILS (673-6457)
 | email: miker at esilibrary.com
 | web: http://www.esilibrary.com


More information about the Open-ils-dev mailing list