[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

wajasu <matted-34813 at mypacks.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #35 from wajasu <matted-34813 at mypacks.net> ---
This is working now!

Everything seems to work now!!!
CRUD - courses, adding instructors(teachers), to courses, 
adding reserves to sources, fields, and menus hiding when appropriate.

Except: 
Permissions related:
- if one wanted to delegate ADD/DEL reserves permissions without also the
manage(ADE/EDIT?DEL) courses permission, that can't be done.  Without manage
courses, course_reserves is not an accessible page, so one can't get to the
links to add/del  reserves.
- Also, while i had permissions to delete reserves, I copied the url of the
remove link to the clipboard.  then i removed the staff permssion allowing
delete reserves.  i logged in as staff and saw only the edit link (no remove),
BUT I pasted the remove action url from my clipboard and the delete worked. So
a check needs to be in the script/controller/security layer to see what is an
accepted action.  The same probably goes for courses.

Ideas to consider - take em or leave em.
- I can add multiple courses all with the SAME fields.  Maybe a check to
dissalow duplicates for (name, term, dept, course_number, section).
- Maybe the TERM and DEPARTMENT could be added to the optional data that folks
choose to load. The future testers would be helped.
- I don't know the repercussion of having no library set. (NULL in columns?),
You might want a check and popup to prevent adding rows unless a library is
set.
- I know after this is signed off, you will get other enhancements.  A
course_schedule ( Days/Hours) and course_location ( North Campus/Bldg A/Edgars
Hall/Room 202)   is something the "students" could use. Lots more work for
future effort.


Otherwise:
Greate work!!!  A lot to test.

wajasu

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