Due Date Time Not Shown in Staff Catalogue

Bug #1998133 reported by Jennifer Pringle
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
New
Undecided
Galen Charlton

Bug Description

Evergreen 3.9

In the angular staff catalogue it's not possible to see the due date time, just the date. This is an issue when working with hourly loans.

Only the Due Date (not the time) displays in More Details on the search results and in the record on the Item Table and Holdings View tabs. To find the due date time you have to go out of the catalogue to Item Status or the patron's account.

Ideally the due date time would show just for hourly loans, but for libraries with hourly loans showing for all loans is better than showing for none.

Revision history for this message
Bill Erickson (berick) wrote :

Just noting as of EG 3.10 there is a new Angular egDueDate filter for addressing these scenarios.

Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: bitesize
Revision history for this message
Gina Monti (gmonti90) wrote :

Made a commit with the datatype Terran suggested:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=461a5746cee5c93a83734131a718559dc89d1be0

I did test it but not entirely jazzed with the formatting. Attached is a screenshot.

tags: added: pullrequest
Revision history for this message
Terran McCanna (tmccanna) wrote :

Hmm, it doesn't look to me like the screenshot is reflecting the change. At least, it's not how it formatted the time in the Carousel refresh time column.

Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

I agree that the formatting is not the best, but it's nice to see the time displaying there.

Revision history for this message
Joan Kranich (jkranich) wrote :

I'm looking at the due date in the catalog and in Item Status on the MOBIUS server for Bug Squashing. The format of the time in the staff catalog is 2023-03-08T23:59:59-0500. The item status screen does not display the time in the view details. Only the date displays.

Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

Hello, I was just trying to test this out also.

The AngularJS interfaces seem to be making use of the library setting "Format Date+Time with this pattern
" if it is set.

Open-ILS/src/templates/staff/cat/item/t_summary_pane.tt2
{{circ.due_date() | egDueDate:egDateAndTimeFormat:circ.circ_lib():circ.duration()}}

If it isn't set then it is just the date in my testing when looking at /eg/ interfaces.

I was trying to change a checkout to being hourly, but I wasn't able to get that to work. No matter what time I set in the change due date interface, it kept forcing it to 11:59pm.

So maybe the issue is that the Angular interfaces need to check the wanted format and use that? Rather than just showing the raw format?

At first I was thinking there was some logic that was saying ( If the hour = 11:59am then only show the date, else show timestamp.) or something like that. Only show the hours when it is significant. But I'm having a hard time confirming that without being able to set the hours in a due date.

Is there a switch to enable hourly checkouts so I can change a due date?

Revision history for this message
Christine Burns (christine-burns) wrote :

You must setup or use an hourly loan duration in the circulation policy.

I have tested this code and consent to signing off on it with my name, Christine Burns and my email address, <email address hidden>.

tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

Noting upon reviewing this that there's a bit of a mess going on here:

- egDueDate is not currently an option for formatting grid values in rel_3_10. It expects to be passed a whole circ IDL object, which is not currently available in to the Angular item table. Also, there's some code that looks like this that cannot work at present:

src/app/staff/circ/patron/billing-history.component.html: label="Due Date" datefilter="egDueDate" [hidden]="true"></eg-grid-column>

The problem? The Angular grid has no notion of a datefilter attribute; only the AngularJS grid does.

- Now, the Angular eg-grid *does* have a way to get at a similar effect: use the timezoneContextOrg and dateOnlyIntervalField attributes of eg-grid-column. However, changes to open-ils.search.bib.copies.staff would be needed to supply the _loan_'s circ_lib and duration values.

tags: added: needswork
removed: pullrequest signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

So, marking this as "needswork". I'll make a stab at sorting this out.

Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.