[Koha-bugs] [Bug 8215] Add Course Reserves

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8215

--- Comment #32 from Kyle M Hall <kyle at bywatersolutions.com> ---
> - can't add instructors (tried placing barcode number, and borrowoer id in
> both add and edit course forms).  I'm suggesting you make this a button with
> popup form allowing selection from the list of "teachers".  Maybe you can
> leverage patron search. (is that a template yet?)
> (if you clear you dev database course tables, maybe this would show up. or
> am i missing something?)

This was an ajax based search, but it seems to have broken at some point. I
think I will replace it with a more traditional popup search box.

> - the "Course Reserves" column in the OPAC does show at the end of the
> holdings tab row (Nice!).   But, not in the staff client holdings.  For
> staff holdings, it might need to be near the column showing status (Not for
> Loan), so you can see if its already checked out and highlight in red if its
> a course reserve and "On Loan" conflict.  If its course reserved, should we
> disallow loaning it?

I'll have to add that.

> Note: If the course is made inactive, it should allow it to be for loan. (in
> item table)

That is actually something you can choose by defining the 'standard' item type
an item has, and the new item type it has while on reserve!

> - Are you integrating with the "holdings" subsystem?  If an item is "course
> reserved" what does that mean?  Is the On Loan flag set in the biblo item?
> etc

When an item is on "course reserve" it means that the the item how has the new
holding branch, category code, 

> - the "Course reserves" column inthe OPAC, shows even when the course is
> InActive.  Your logic to decide to show the column should check the course
> active flag as well.

The course reserves column should only appear when one or more items on the
record have course reserves. I'll have to check on this.

> - mod_course.pl  has template_name  => "about.tmpl"    (is this what you
> want?)

Yes, it doesn't use a template, but it needs to call get_template in order to
check permissions. It seems to be standard practice to use about.tmpl in these
cases.

> - when adding an item for "course resserve", you have "LEAVE UNCHANGED" and
> show "Unchanged" in the course_details page, for "Item Type", "Collection",
> "Location", "Library".  Can you add the current data underneath the 
> italicized "Unchanged" for easier usability.  You already have the code t
> show it.

I removed it for readability, but I can re-add it.

> - I checked out an item that was a "course reserve"  is that allowed? 
> Should a staff get a popup warning it is a course reserve at checkout time? 
> Also, in OPAC item view I see the status as checked out, when I click on the
> "cousre reserve item link" I get the course-details which show the item is
> available, so maybe you need to supply the model with those fields in that
> item-status.inc.

It is allowed if you've defined the item type to allow it. The system is
designed for flexibility. Putting an item on course reserve does not in itself
change the behavior of an item. The behavior as altered by setting new item
types, holding branches, etc.

> Note; Hopefully in the future, each column would have its own tt template,
> and one would compose a row (for holdings, etc).  The logic to determine
> sophisticated "Availability" might be moved to the service/manager  (ie.
> ItemManager.GetStatus.GetAvailabilityDescription or such)

This is an interesting notion!

-- 
You are receiving this mail because:
You are watching all bug changes.