koha-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Koha-devel] pm structure : Acquisition.pm modifications


From: Pat Eyler
Subject: Re: [Koha-devel] pm structure : Acquisition.pm modifications
Date: Tue Jun 11 06:05:02 2002

(pate puts on his Unit Testing hat.)

On Mon, 10 Jun 2002, Alan Millar wrote:

> On Mon, Jun 10, 2002 at 10:35:58PM +0200, paul POULAIN wrote:
> > the Acquisition.pm (aka : Catalogue.pm for MARC version) contains too
> > much subs imho. It's hard to read, and not always very coherent :
>
> > I propose to divide the script in 2 differents ones.
> > the first would be Biblio.pm (used only for biblio management :
> > biblio/biblioitems/items/additionalauthor/subtitle tables), the second
> > would remain Acquisition.pm (Catalogue.pm for MARC)
>
> I like the idea of separating the routines for biblio handling
> from the routines for order processing.   If the proposed Catalogue.pm
> is going to handle Marc record processing, which is biblio information,
> doesn't that mean that Catalogue.pm would replace Biblio.pm, not
> Acquisition.pm?
>

As we split subroutines out into new files, let's please try to write
tests for them using the Test::Harness framework.  Being able to run a
testsuite automagically and verify that we've not added new bugs will help
all of us sleep better at night.

-pate


> Better yet, instead of replacing Biblio.pm later, it could perhaps turn
> into a higher-level abstraction of Catalogue.pm
>
> - Alan
>
> ----
> Alan Millar     --==> address@hidden <==--
>
> _______________________________________________________________
>
> Don't miss the 2002 Sprint PCS Application Developer's Conference
> August 25-28 in Las Vegas - 
> http://devcon.sprintpcs.com/adp/index.cfm?source=osdntextlink
>
> _______________________________________________
> Koha-devel mailing list
> address@hidden
> https://lists.sourceforge.net/lists/listinfo/koha-devel
>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]