[Koha-bugs] [Bug 11703] Convert checkouts table to ajax datatable

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

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

Katrin Fischer <katrin.fischer at bsz-bw.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #79 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
Hi Kyle,

as promised, here my first test results:

GENERAL
- New jQuery plugin: jquery.dataTables.grouping.js
  Should we add a note to the about page about this?
- Translation:
    +    var HOLD_IS = _("Hold is");
    +    var SUSPENDED = _("suspended");
    +    var UNTIL = _("until");
    +    var ITEM_IS = _("Item is");
    +    var WAITING = _("waiting");
    +    var AT = _("at");
    +    var FROM = _("from");
   Would be much nicer as complete sentences with placeholders - this is
    really hard to translate grammatically correct. 

CHECKOUTS
- Sorting table
  > OK. Sorting the table was not possible before patch. Sorting
    will remove the Today's and Previous checkouts differentiation.
    Works ok.
- Column price and charge
  > Bug: columns price and charge are not sortable.
- Column Due date
  > Ok, only shows time now when it's an hourly loan. Nice!
- Column Item type
  > Shows the code instead of description, something for another day.
- Column Checked out on
  > Bug: Shows seconds, but those are always 00 so I guess we don't record
    the issue by second?

Checking out - still to be tested
- 'normal' check out
  > OK
- hourly check out (with specify due date)
  > OK
- with rental fee
- notforloan item
- too many items
  > OK, override possible.
- item on hold for someone else
- item with material note

Renewing items
- Check out with an already checked out barcode again
  > OK, item is renewed
- renewal using checkboxes
  > Concern: It's only possible to do one renewal at a time.
    After renewing once the checkbox is gone and then you will have
    to reload the page to bring the checkboxes back.
- renewal override
  > OK. If the item is on hold, the renewal will fail. I wonder if that's 
    always been the case, but it's behaviour pre-patches.
- too early renewal (bug 6779)
  > Bug: Shows too_soon instead of a nice message for why the renewal
    is not yet allowed. Should show the date - compare to master.

System preferences
- UseTableSortForCirc
  > Is removed, sorting will be always available
  > Bug: Should be removed properly with updatedatabase.pl as well
- [prefs for sorting by checkout date] - still to be tested

RELATIVE's CHECKOUTS
- releative's checkouts tab is only shown for the guarantee (child),
  but not for the parent - something seems messed up here?
  > Bug: My patron has 2 children, one of them has an item checked out.
    It doesn't show up, the relative's checkouts tab is empty.

HOLDS
- Sorting table
  > Sorting the holds table was not possible pre-patches.
    Concern: The sorting and page load seems a bit slow for 4 holds
- Columns: Title, Hold date, expiration date, Delete
  > OK
- Barcode column
  > Ok, now offers a link to the item
- Delete column
- Suspending and unsuspending holds
  > OK
- Cancelling holds
  > OK, possible enhancement: Redirect back to the holds tab after 
    cancelling a hold instead of sending user to the checkouts tab. 
    Exists pre-patch.

Concern: Performance testing / benchmarking
- I would be really interested to see a comparison on how fast the
  complete page loads in comparison to before and similar. Is this possible?

Concern: Accessibility
- We currently have no feedback about accessibility issues this might
  cause in circulation. I want to note that there is no non-Javascript
  fallback here. In case the Ajax proves to be a problem, we'd have
  a regression as currenct circulation works nicely.

I really like he idea of this, but I think I'd like it a bit better if
it had been implemented as an alternative at first - but that sure is
debatable :)

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