Evergreen ILS Website

IRC log for #evergreen, 2015-04-13

| Channels | #evergreen index | Today | | Search | Google Search | Plain-Text | summary | Join Webchat

All times shown according to the server's local time.

Time Nick Message
04:53 pinesol_green Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html>
07:19 graced joined #evergreen
07:35 sarabee joined #evergreen
07:50 mrpeters joined #evergreen
08:00 Newziky joined #evergreen
08:14 rjackson_isl joined #evergreen
08:18 RoganH joined #evergreen
08:19 jboyer-isl joined #evergreen
08:25 mrpeters joined #evergreen
08:28 mrpeters does auditor.actor_usr_history keep any record of which user "created" another user?  would it just be the minimum id value in the audit log for a particular actor.usr.id, with the action of "U"?
08:31 collum joined #evergreen
08:32 Shae joined #evergreen
08:47 mmorgan joined #evergreen
08:49 ericar joined #evergreen
09:01 csharp mrpeters: iirc, there's an 'audit user' or similar field that should be what you're after
09:02 csharp yeah, audit_user
09:02 mrpeters does it distinctly tell you the user was created?  or does it just show as an "Update" or "U"
09:02 csharp I think it should be 'C' for creation, but I've never dug deep
09:03 csharp the requests we usually get are along the lines of "Who changed user X?" (removed note, updated address, etc.)
09:03 jeff record creation is not audited. the state of the record as it was created will be recorded on first update.
09:03 csharp jeff: good to know
09:04 csharp mrpeters: so you'd probably have to go to the logs for who created it
09:04 mrpeters jeff: yep -- thats what i thought
09:04 jeff and actor.usr does not have a creator column or similar, so i think you're going to be looking at something that is only discernable from logs.
09:05 jeff unless you find that there's a part of the standard process of creating a user that results in an immediate update.
09:05 mrpeters they want a report on who created a particular user (obviously migrated data will not qualify) but i was thinking the minimum id in the auditor log for the user in question would get me that info
09:05 mrpeters as it was the first "update"
09:05 jeff but even in that case, you may find that the action isn't one of the actions that sets audit_user / audit_ws
09:06 mrpeters true
09:07 csharp it's possible the "work log" within the staff client might do what they're after
09:08 csharp depends on whether they're after stats or are trying to track/resolve a staff problem
09:08 mrpeters its a stats thing
09:08 csharp k
09:10 jeff interesting: looking at a user created yesterday, they have two entries in auditor.actor_usr_history with audit_time that matches their create_date (and last_update_time). both have audit_user and audit_ws, so it looks like my musing above was on-target.
09:11 * jeff throws that on the pile of things to look into later
09:12 jeff but just off the top of my head, you can probably rely on "auditor.actor_usr_history with audit_time matching actor.usr.create_date for the user will contain the audit_user/audit_ws that created the user"
09:13 jeff test, of course.
09:14 mrpeters that may be a good start, and enough for what they want
09:15 csharp so... I'm investigating another see also from tracing issue with OPAC browse search: http://gapines.org/eg/opac/browse?​blimit=20&amp;qtype=author&amp;bte​rm=poznanski%2C+ursula&amp;locg=1 is the main heading and "Archer, Ursula" is listed as a see also, but there is no authority record for "Archer, Ursula" for it to link to
09:16 csharp in that case, is there a way to have see also/from tracings work - that is, when there is no auth record for the reference?
09:16 * csharp just doesn't know enough about authorities to really know
09:19 Newziky1 joined #evergreen
09:21 * collum just created a user in his test Evergreen and two identical update rows appeared in auditor.actor_usr_history.
09:27 mrpeters thanks for the test collum
09:28 csharp @blame authorities
09:28 pinesol_green csharp: It's all authorities's fault!
09:28 csharp @who has a problem with authority?
09:28 pinesol_green paxed has a problem with authority.
09:30 yboston joined #evergreen
09:32 dbs csharp: so you have this auth record: http://id.loc.gov/authoriti​es/names/no2012027363.html
09:32 Newziky1 left #evergreen
09:34 csharp dbs: yes, I do
09:35 dbs but not http://id.loc.gov/authoriti​es/names/no2012027363.html ?
09:36 dbs argh
09:36 dbs http://id.loc.gov/authorit​ies/names/n2014061399.html
09:36 dbs #$*@(*! chrome highlight-no-copy-to-clipboard
09:36 csharp dbs: correct
09:37 dbs so the first auth says "See also" and points to a deprecated authority record; maybe your cataloguers deleted the deprecated auth record or never had it
09:38 dbs (well, it doesn't even really point to the deprecated auth record because it just uses string literals)
09:40 dbs I assume the reason the valid record says "see also this deprecated form" is because LoC also assumes that there will be bib records out there that had been authorized against the deprecated form, thus a search for that might turn up results you wouldn't otherwise see for Poznanski
09:40 dbs but I'm doing a lot of assuming.
09:44 csharp looks like we never had the deprecated record
09:45 csharp the n2014061399 number is in the 010 $z field (canceled or invalid number), but that may be where you saw it in the first place ;-)
09:45 dbs hey wait, 400 is "see from"
09:47 csharp so should a "see from" show up in the list (btw, I have been doing authority_authority_linker.pl on that record, but seeing no change)
09:47 csharp now I'm trying to disentangle what that script does
09:48 dbs I think only "See also"s (5xx) are supposed to show up in the auth browse
09:49 csharp dbs: "auth browse" = "OPAC browse" or auth browse in the staff client?
09:50 csharp (it isn't showing either place, btw, just wanting to clarify)
09:50 dbs csharp: opac browse
09:51 csharp ok - good - then this may be a case of unfulfilled expectations rather than an actual technical issue
09:52 dbs yeah. perhaps the thought was that you don't want to lead people _to_ an unauthorized from, and all of your entries will of course have been updated to use the preferred, authorized form :)
09:52 dbs Although I've certainly seen auth browses in OPACs that had 4xx entries that said something like: "Archer, Ursula _see Poznanski, Ursula_"
09:53 csharp that is what's expected here
09:54 csharp I think the originating issue was "user searched for Archer, Ursula in the OPAC author browse and didn't see an entry"
09:57 csharp dbs++ # troubleshooting assistance
09:59 bshum mceraso++ # Bibliomation is smoothly upgraded to 2.8.
09:59 jeff bshum: congrats!
09:59 jeff mceraso++
10:00 dbs http://docs.evergreen-ils.org/2.5/_opac_2.htm​l#_bib_record_browser_with_linked_authorities was the feature note; http://docs.evergreen-ils.​org/2.8/_authorities.html has the technical docs that don't really put the pieces together
10:03 csharp dbs: thanks
10:06 csharp "If the browse heading is linked to any authority records, and if any other authority records point to those with "See also" or other non-main entry headings, those alternative headings are displayed linked to a search results page showing related bib records related to the alternate heading."
10:06 csharp hmmmm
10:19 mllewellyn joined #evergreen
10:29 pinesol_green [evergreen|Bill Erickson] LP#1380803 Include direct charges in PO esimated price - <http://git.evergreen-ils.org/?p=​Evergreen.git;a=commit;h=caa6a95>
10:29 pinesol_green [evergreen|Bill Erickson] LP#1380803 Update PO summary amounts - <http://git.evergreen-ils.org/?p=​Evergreen.git;a=commit;h=a91a15d>
10:36 mllewellyn joined #evergreen
10:43 mllewellyn joined #evergreen
10:46 krvmga joined #evergreen
10:49 mllewellyn joined #evergreen
10:51 sandbergja joined #evergreen
11:02 dbs csharp: looks like find_authority_headings_and_notes() in OpenILS::WWW::EGCatLoader::Browse is where the magic happens
11:03 csharp dbs: excellent - I'll take a look
11:04 dbs it's still pretty convoluted in there
11:04 dbs but there are some good comments reflecting the confusion of the author :)
11:11 dreuther joined #evergreen
11:27 krvmga is there a limit on the number of titles that can be included in a bucket?
11:27 eeevil krvmga: nope
11:27 krvmga eeevil: thanks :)
11:29 bshum There isn't, but I feel like there are limits to the number of things retrievable in the XUL client from a bucket before it can blow up.
11:29 * bshum remembers the library that scanned over 1000 things into a copy bucket and then tried to "retrieve it"
11:32 bshum Huh....... so overdue reinstatement of old fines, plus newly generated fines up to today, appears to have added up to fines greater than the max fines for a given circulation.  On multiple circs since we upgraded.
11:32 * bshum wonders what's changed... and double checks all the surrounding details.
11:32 tsbere bshum: How much over the max fines?
11:32 bshum tsbere: Well, it's $6.00 is maxfines, and it's like $9.75 atm
11:33 bshum What I might have normally expected to see is reinstated overdues, and then new fines generated up to the maxfines
11:33 bshum It's a lost item that came back, and they checked it in today.
11:33 bshum So the original overdues were voided
11:33 bshum When the lost bill was added
11:33 bshum And when the item was returned, the lost bill was voided, and overdues reinstated
11:34 bshum But now new overdues added onto it, up to today due to not having returned it on time.
11:34 bshum But it's definitely beyond the maxfine setting that should have applied to the circ based on the rule used and what was listed in the circulation entry
11:34 bshum So it seems like something is super wrong.
11:35 tsbere Is it all overdues, or did other billing types get thrown into the mix?
11:35 * tsbere answers "why do they owe more than X?" or "why didn't it auto-void that fine?" with "because some staff member added a billing manually" way too often
11:36 bshum It looks like just overdue billing type
11:37 csharp bshum: does the max_fine match the max_fine_rule on each circ?
11:39 bshum csharp: Yes, the circulation entry shows the right maxfine
11:39 csharp k - just checking
11:41 bbqben joined #evergreen
11:42 bshum I just wonder if it's a consequence of commits pushed to master as part of https://bugs.launchpad.net/evergreen/+bug/1198465
11:42 pinesol_green Launchpad bug 1198465 in Evergreen "Support for Conditional Negative Balances" (affected: 16, heat: 74) [Wishlist,Confirmed]
11:43 bshum Changes like 54cd1b8c38ffab5cd48572a68b3544c88e7cd3eb
11:43 pinesol_green [evergreen|Dan Wells] LP#1198465 lost overdues generated in main xact - <http://git.evergreen-ils.org/?p=​Evergreen.git;a=commit;h=54cd1b8>
11:43 mrpeters1 joined #evergreen
11:44 tsbere If the new overdues are being generated outside of an xact, and the previously voided ones are unvoided in a different xact, the two combined could result in the issue in question
11:45 tsbere Or, to clear up what I was saying, if the fine generator still sees the old ones as voided it will keep adding on to max fines, it looks like. Then when the un-voiding happens it goes beyond the max.
11:46 * tsbere wonders if voided overdues should count towards the max in the first place or not...
11:46 tsbere I can come up with arguments both ways. <_<
11:46 bshum Well, it seems like this might be a bug then.
11:47 bshum To me, I think maxfine ought to be respected, and if we're there already, don't add more fines
11:47 mllewellyn joined #evergreen
11:47 tsbere I suppose the alternate workflow is "we generated overdues, then un-voided old ones"
11:48 Bmagic I am having issues finding subroutines in the perl code associated with editor. Any insight? AKA: $editor->search_action_circulation where is search_action_circulation defined? Somehow the subroutine is parsed in the CStoreEditor.pm ?
11:48 tsbere Bmagic: Some of that is auto-generated functions based on the IDL
11:48 dbwells bshum: I'd say there is a good chance this is a regression from those changes you linked to.
11:49 bshum dbwells: Ugh
11:49 Bmagic tsbere: ah
11:49 * bshum wonders how hard it'll be to unroll those from his production system before it breaks more patrons... hmm
11:50 Bmagic tsbere: so, somewhere in the code I should* find sub search_action_circulation{  ?
11:50 bshum dbwells: At present, I see a batch of about 6 commits that were merged related to those areas.  I'm checking further to see what else altered and how.
11:52 tsbere Bmagic: I imagine there is a "loop over fieldmapper/the IDL and generate functions that have a cstore handler" routine somewhere that makes the sub dynamically.
11:52 Bmagic tsbere: I see, thanks!
11:53 dbwells bshum: It's probably something which works in the "complete" branch for that bug, and we just didn't get the lines drawn quite right when splitting it into bite-sized pieces.  I will start looking now, too, to see where I went wrong.
11:54 tsbere dbwells / bshum: I think I see what is going on
11:56 tsbere checkin_handle_circ does the "handle lost or longoverdue" bit, which eventually unvoids fines. do_checkin calls that.....but before then it does a handle_fines call that may ignore stop_fines.
11:58 tsbere The "generate fines on checkin" call itself needs to be moved to after checkin_handle_circ, basically...even if some of the things it looks up (ignore_stop_fines) happen before then?
12:00 dbs csharp: if it makes you feel any better, yeah, http://www.loc.gov/marc/authority/adtracing.html 'TRACING FIELDS - SIMPLE CROSS REFERENCES' suggests that browse should have a "Archer, Ursula -- see under Poznanski, Ursula' entry
12:02 * dbs is wondering if the linking_subfield = 0 where tag IN (410, 510) is perhaps the issue
12:02 dbs SELECT * FROM authority.control_set_authority_field WHERE tag IN ('410', '510', '110');
12:03 csharp dbs: linking_subfield is 0 for 510, but not for 410
12:03 csharp and not for 400 either
12:04 dbs that is, maybe the 500 should have a $0 that links to the id of the auth record that has the 100? Damn it would help if the docs actually had a MARC21-centric example to help that tiny subset of libraries that use MARC21 LoC authorities.
12:05 dbwells tsbere: Here is an unpushed commit from the complete branch which I think addresses what you are saying: http://git.evergreen-ils.org/?p=worki​ng/Evergreen.git;a=commitdiff;h=54205​4fce727c7bd09eeb4e1fe479b919aebf3a8
12:06 dbwells There might be a simpler way to do it, but I'm still trying to get all the pieces back in my mind.
12:06 dbs stock seed data says UPDATE authority.control_set_authority_field SET linking_subfield = '0' WHERE tag LIKE ANY (ARRAY['5%','7%']);
12:06 tsbere dbwells: That looks like it deals with some of it. Can't say if it is the best way as I would have to start digging in a lot more.
12:07 dbs csharp: right, sorry, I meant WHERE tag IN (500, 510)
12:07 dbs <-- unhelpful
12:07 jeff Bmagic: The subs you seek are generated by OpenILS::Utils::CStoreEditor::init()
12:07 csharp dbs: yeah - 500 has "0" too
12:08 dbwells tsbere: are you planning to look into it more?  It like to take a crack at it, but also don't want to duplicate any other efforts.
12:08 jeff Bmagic: look around line 913 of OpenILS/Utils/CStoreEditor.pm
12:10 dbs csharp: so given that http://id.loc.gov/authorities/​names/no2012027363.marcxml.xml has a 400, maybe that blows that theory to hell
12:10 dbs (and I'm pretty sure I've been looking at local examples that have 5xx without $0 that aren't resulting in entries either)
12:10 dbs E_TOO_ABSTRACT
12:11 buzzy joined #evergreen
12:12 dbs I guess this needs some serious TPAC updating too: http://docs.evergreen-ils.org/2.8/_​opac_searching_of_authorities.html
12:12 yboston dbs: I was planning on just pullign that section out
12:12 mrpeters joined #evergreen
12:13 yboston dbs: since that particular feature is not found in TPAC
12:16 jihpringle joined #evergreen
12:18 dbwells bshum: are you in a position to test a fix?  If so, I can suggest one to try.
12:19 bshum dbwells: I can give it a shot.
12:19 bshum dbwells: I was putting together a revert branch to remove everything and put it back to the earlier way, if you guys weren't ready to suggest anything.
12:20 bshum Fine bugs make me feel weird.
12:20 dbwells bshum: The fix is cherry-picking in two commits, 76ed91eaf8e2b followed by 542054fce72
12:20 jeff Every Good Bug Does Fines
12:21 bshum dbwells: I'll go give that a quick whirl on our test server.
12:21 bshum Lucky for me I have lots of examples of broken circs from today to try with :\
12:22 jeff and perhaos Fines Always Cause Evil?
12:22 dbs yboston: well, the functionality is there, it's just part of the regular catalogue UI now accessed by "Browse the catalog" instead of under "Advanced search" right?
12:29 yboston dbs: no, that doc is for an older simpler feature that Berklee paid for that only worked on JSpac
12:29 dbwells bshum: I have a lunch appt in a few minutes.  I'll check back and also do some testing of my own when I get back.
12:29 yboston dbs: it only grabbed data from authoritites; and for example the only title search it could do was on uniform titles
12:30 bshum dbwells: Thanks man, I'll see what I can find out.
12:32 bshum dbwells: Checked one circ so far, and it applied up to the right max fines this time 'round
12:32 bshum Testing another circ that's lost to be absolutely sure.
12:35 Dyrcona joined #evergreen
12:39 bshum Well, it takes its sweet time calculating (retrieving, retrieving) but it does eventually give me the right amount of overdues stopped at the maxfine as expected.
12:39 bshum dbwells++
12:39 bshum We'll test further but I think we should slate those fixes for merging to master and rel_2_8
12:40 dbwells bshum++
12:40 dbwells I'll also test further this afternoon.  Don't want to take one step forward, two back.
12:40 * dbwells steps away for a bit
12:45 bbqben joined #evergreen
13:04 bmills joined #evergreen
13:30 collum joined #evergreen
13:38 tsbere dbwells: Sorry, I got pulled away. I have stopped looking at things at this point. (and I see you stepped away too)
13:45 jeffdavis Is anyone using Summon (the discovery layer) with EG?
13:55 Dyrcona @summon Dire Weasel
13:55 pinesol_green Dyrcona: The horror... The horror...
14:00 dbs yboston: yeah, commit e710ecbee519 seems to have the core description of the TPAC version
14:00 pinesol_green [evergreen|Lebbeous Fogle-Weekley] Bib record browser with 'see also', etc from linked authority headings - <http://git.evergreen-ils.org/?p=​Evergreen.git;a=commit;h=e710ecb>
14:01 dbs And bug 1177810 should have the full discussion
14:01 pinesol_green Launchpad bug 1177810 in Evergreen "Bib record browser with 'see also', etc from linked authority headings" (affected: 4, heat: 26) [Wishlist,Fix released] https://launchpad.net/bugs/1177810
14:01 maryj joined #evergreen
14:03 dbs csharp: https://bugs.launchpad.net/ever​green/+bug/1177810/comments/30 is the smoking gun for you. We should get a summary of that whole freaking bug into the docs.
14:03 pinesol_green Launchpad bug 1177810 in Evergreen "Bib record browser with 'see also', etc from linked authority headings" (affected: 4, heat: 26) [Wishlist,Fix released]
14:04 dbs So I guess if you have auth record # 123 with field 100 establishing an author's name, as well as a 500 establishing a "See from" heading, it's really unlikely that that 500 is going to have a $0 pointing to its own record ID
14:05 dbs but if that 500 had $0 (OCLoC)123 or whatever, you'd be golden
14:05 dbs See from: oooookay
14:09 csharp dbs: reading now
14:09 csharp and I'll test too
14:16 * csharp highlights https://bugs.launchpad.net/ever​green/+bug/1438136/comments/12 on behalf of eeevil who is soliciting feedback from people with expertise with/opinions about the query parser's functionality
14:16 pinesol_green Launchpad bug 1438136 in Evergreen "OPAC searching significantly slowed by adding format filters" (affected: 1, heat: 6) [Undecided,New]
14:20 Dyrcona My comment this morning was intended to say, "Dunno. Sounds good to me. Give it a try."
14:20 Dyrcona 'Cause, I don't feel qualified to comment otherwise.
15:05 ldw afk lunch
15:06 ldw mt
16:37 buzzy joined #evergreen
16:51 buzzy joined #evergreen
17:10 Bmagic jeff: Thanks, I figured it out. It's neat how that works.
17:18 mmorgan left #evergreen
18:13 dbwells bshum: Did a bunch of testing, and most things worked fine.  The biggest issue was that it no longer generated new overdues on lost item return due to a thinko in one of those commits.  I also found one other thinko which would clobber a MAX_FINES stop_fines_reason in some cases.  Here is a paste of the diff: http://paste.evergreen-ils.org/49
18:17 dbwells (To clarify, the above lost item return issue only applies if that particular setting is turned on, which is not OOTB behavior.)
18:45 * kmlussier notes that a lot of libs in these parts make use of that setting.
19:51 jeffdavis question about bug 1074096
19:51 pinesol_green Launchpad bug 1074096 in Evergreen "Advanced Search by Bib Call Number Returns 0 Results" (affected: 3, heat: 18) [Low,Fix released] https://launchpad.net/bugs/1074096
19:52 jeffdavis we removed Bib Call Number as a search option in numeric search. However, it still appears as an option in the basic and advanced search interfaces. Was the intent to remove it completely?
19:53 jeffdavis (If so, I think we need to remove the relevant line from qtype_selector.tt2.)
20:24 kmlussier jeffdavis: Oops. Our sites had customized that option away so long ago, I didn't realize that it lived anywhere other than the numeric search. Total oversight on my part.
20:26 * kmlussier can whip up a branch tomorrow unless somebody beats me to it.
21:08 jeff 21 seconds to sync offline patron auth for ezproxy while we're down for upgrade.
21:46 buzzy joined #evergreen
23:00 jeff_ joined #evergreen
23:13 paxed joined #evergreen
23:32 bshum dbwells: Thanks for investigating.  I applied the paste you suggested to our systems too.  We do make use of that setting in our environment.
23:33 bshum I'll coordinate with you tomorrow on getting things into some sort of branch to get pushed along to fix things before the next cuts.
23:33 bshum dbwells++

| Channels | #evergreen index | Today | | Search | Google Search | Plain-Text | summary | Join Webchat