Time Nick Message 21:13 chris hi debra 20:51 gmcharlt yeah, that was nice 20:51 chris feel good story for the day http://www.orlandosentinel.com/news/opinion/orl-ed08308may08,0,3680260.story 20:47 acmoore good afternoon, chris. 20:46 acmoore I'll probably eventually at least fix the pod usage problems and check in something to run Pod::Test on all files. 20:46 gmcharlt hi chris 20:46 gmcharlt yeah, such things would definitely have to be advisory 20:46 chris morning 20:46 acmoore I did that in a previous code base and it was pretty nice. I eventually made it so that if the file used to pass at one level, and now it doesn't that that was a faling test. That's a little extreme for now, I think. 20:43 gmcharlt maybe do it passively - have the eventual buildbot also run perlcritic and put up the report, and see what happens 20:41 gmcharlt heh - that was certainly one of the best technical books I've read recently 20:41 acmoore I don't think you can go that far, but it's a good guide. 20:40 acmoore I like Andy Lester's quote about Perl Best Practices. It was something about how he writes "coding policy manual" on the front of it with a marker and settles all of those wars. 20:39 gmcharlt we can haz war^W debate about indentation and block style? 20:39 acmoore no. I wouldn't worry about that part. ;) 20:39 atz acmoore: but then somebody might use it! 20:38 acmoore I wouldn't mind seeing one in git eventually. 20:38 acmoore do you guys use perltidy or perlcritic on your code? I'm wondering if anyone has an .rc file for either that I can use for consistency. 20:28 acmoore I guess. but I'm OK with it not being fixed as long as we agree on some things that we can do to make it better. 20:27 atz that's just the manic phase before the depression hits that despite all these other ppl agreeing, it's still unfixed. :) 20:26 acmoore I'm so glad that I'm not the only one that seems to think this way. 20:25 atz acmoore: it's true OO would reduce the prominence of this problem. 20:25 gmcharlt and acmoore, I'm with you re using more classes 20:25 acmoore gmcharlt, I'm with you. 20:25 acmoore I would like to steer the battleship towards more objects and classes, but I recognize that's a long, uphill battle. 20:24 gmcharlt acmoore: in the long run, better sore fingers than subtle namespace bugs :) 20:24 acmoore but it does mean sometimes writing C4::Some::Namespace::Somewhere::Far::Far::Away::DoFoo() 20:24 gmcharlt saorge: hopefully to be made available soon 20:23 gmcharlt saorge: beyond what's available on the wiki - not much. nengard is working on a manual, which iwll include authorities 20:23 acmoore gmcharlt, Oh yeah, I meant going forward. 20:23 atz of course... cleaning up almost every C4 use statement was never quite assigned priority 20:23 gmcharlt acmoore: for new stuff we write, yes - worth refactoring C4 as well, but obviously doing that will take time and discussion 20:22 atz actually, that was one thing I insisted in my first week 20:21 acmoore can we make a rule-of-thumb to stop exporting everying by default? 20:21 atz EXPORT_OK is fine. bulk EXPORT is ass. 20:20 gmcharlt I have learned to hate it with a passion during the previous few months 20:20 saorge is there a documentation about authorities? 20:20 atz this is the lameness of all the C4 modules EXPORT=qw(everything)'; 20:20 acmoore reason #496. 20:20 acmoore I hate Exporter. 20:20 gmcharlt which doesn't work, of course 20:20 gmcharlt and an eval "KohaTest::Acq::NewOrder->runtests" ends up trying to run C4::Acq::NewOrder()->runtests 20:20 acmoore oh yeah, that's a drag. 20:19 gmcharlt because KohaTest::Acquisitions uses C4::Acquisitions, C4::Acq::NewOrder() gets imported into namespace of KohaTest::Acq 20:18 acmoore I'll make a note about tools/import_borrowers.pl. 20:18 gmcharlt say you have KohaTest::Acquisition::NewOrder 20:18 acmoore I don't anticipate having to alter either of the borrower_attributes or borrower_attribute_types tables. I'll add a few rows to the b_a_types table, though, to describe the preferences I'm holding in there. 20:18 gmcharlt acmoore: btw, found gotcha in using eval to run a particular test class 20:17 gmcharlt otherwise updating an alternate student ID via import would scribble over prefs 20:17 gmcharlt for prefs that don't require secondary columns to be added to borrower_attributes, should be fine - but one thing: tools/import_borrowers.pl needs to be adjusted so that you can supply a patron_attributes column in the CSV, update certain atts, and leave others alone 20:15 acmoore I really think I can use this to hold other borrower preferences. Do you think that will be a prolem? 20:15 gmcharlt cool 20:14 acmoore I'm adding an UpdateBorrowerAttributes method that lets you update one type without disturbing the other attributes that this borrower has. 20:14 acmoore yeah, no worries. 20:13 gmcharlt acmoore: checks of those conditions undoubtedly could be tightened up - alas, at moment has to be enforced at app level not DB due to lack of check constrainsts in mysql 20:13 acmoore f 20:12 acmoore that's not a commentary on design or implementation. I just confused mysel or something. 20:11 gmcharlt eek 20:11 acmoore This is one of those thigns that I understood better before I started using it for a few hours. ;) 20:11 gmcharlt right 20:10 acmoore and borrower_attribute_types.unique_id determines if more than one borrower can have the same value for this attribute? 20:10 acmoore thanks, 20:10 acmoore yeah, that's what I thought that was for, I just couldn't see how it was working. 20:09 gmcharlt yes, if borrower_attribute_type.repeatable == 1 20:09 acmoore gmcharlt, is a borrower supposed to be able to have more than one borrower_attributes with the same code? 20:07 saorge yes 20:07 ryan so, for a subfield you want controlled, you have selected an authority type in the 'thesaurus' pulldown in the framework editor ? 20:06 saorge and, in the MARC framework, i can see the link 20:05 saorge i can see some MARC authorities (subject, etc) 20:04 saorge during the install, i've asked to install some data 20:04 saorge i guess 20:03 ryan saorge: do you have your marc framework set up with authority control ? 20:01 saorge i've tried but I must miss something 19:52 ryan otherwise, you should immediately be able to edit a record using authorities. 19:51 ryan saorge: if you have bib records in the catalog, you'll want to run misc/link_bibs_to_authorities.pl to link the authority records to the bibs. 19:42 saorge is there another script to execute to be able to use authorities? 19:35 saorge Koha is used as an illustration 19:35 saorge well, not exactly on Koha, but on ILS 19:24 saorge I have to give a course on it 19:23 saorge actually, I only test Koha 19:23 ryan If you have over 10k records, I highly recommend using Zebra. 19:22 ryan how large is your collection? 19:22 ryan ah, yes. 19:21 saorge or rebuild_nozebra.pl in my case 19:21 saorge rebuild_zebra.pl ? 19:20 ryan also a script in misc/ dir for that 19:20 ryan you'll then need to link the records 19:20 ryan you should just be able to run bulkauthimport, then rebuild_zebra with -a option 19:19 saorge I'm from Belgium 19:19 saorge no, Unimarc 19:17 ryan saorge: marc21 ? 19:17 saorge bulkauthimport ? OK, i'll check this script 19:16 saorge s/tt/t/ 19:16 saorge yes, i'm ttrying to work with 3.0 19:16 owen ryan, I'll need to do some reorganization if that gets added: I made opac-userupdate.tmpl the default screen instead of having the user click from opac-userdetails.tmpl 19:16 saorge for another one, the possibilities are too big, so, the librarian will create the record 19:16 ryan saorge: are you working with 3.0 ? 19:15 ryan works just like bulkmarcimport. 19:15 ryan saorge: there's a bulkauthimport script 19:15 ryan owen: hrmm. i'm not seeing it though. 19:14 saorge for one of them, i have to upload all the authority records into Koha 19:13 saorge ryan: actually, it depends of the thesauri ;) 19:13 atz ryan: i know I've seen that syspref 19:12 ryan saorge: do you already have authority records or do you want to create them from your biblio records ? 19:12 ryan oh. am i making that up ? :) 19:12 saorge hi, is there anybody who can explain me how to add a thesaurus into koha ? 19:10 owen ryan, what preference is that? 19:01 acmoore In some ways, sending patron change requests off in an email somewhere is pretty much the same thing as disabling them... 19:00 ryan you can disable the ability of patrons to change their info 19:00 gmcharlt acmoore: it certainly doesn't scale 18:59 acmoore I suspect that people would like to be able to turn off this "feature". 18:58 acmoore wait, so if you (a staff member) get one of these emails, does it have the new information in it, and you type that into the patron's record? 18:58 acmoore oh, man. It sounds like that feature needs to be fleshed out a little more at some point. 18:54 ryan lol 18:50 gmcharlt or as atz said more succently, versioning 18:49 gmcharlt for a public library, I could also see wanting to keep address history - i.e., if you let patrons change addresses online, don't necessarily have an approval mechanism, but keep older addresses in case somebody tries to "disappear" after checking out the library's copy of the Necronomicon 18:49 atz or maybe DB support for versioning 18:44 atz you'd need either a shadow borrowers table or the ability to create a temp borrower w/ updated info 18:43 owen But no one has ever built a framework for managing approvals via a web interface. I'm sure that email method goes back to 1.x 18:42 owen I don't know that anyone considers it a feature that the info is emailed, per se 18:37 atz for whatever reason, tiny libraries seem to think this is a feature 18:37 atz it emails the admin w/ the updated info 18:36 atz acmoore: it's dumber than that, last I saw 17:26 acmoore when a patron udpates their information, a staff member has to approve it or whatever. Where is that tool in the staff interface? 17:25 acmoore I swear, I don't ask these questions without trying to answer them myself first, but... 17:16 acmoore yeah, that's where that kind of thing belongs. 17:15 atz it would be obsolete if our "extended attributes" code ever gets integrated into reports 17:14 acmoore hunh. 17:13 atz could be a homeroom number, teacher, subscription level, etc. 17:12 atz just arbitrary fields, used for sorting on reports 17:12 atz acmoore: those are what I was thinking of, actually. 17:11 frederic atz: thanks. I was trying to discover why biblio level call number isn't displayed at all on an UNIMARC DB. I was thinking it was displaying in MARC21 but I realized now that it isn't the case anymore... I was thinking that cn_ was linking to biblio classification field for result/detail page display. 17:10 acmoore while we're on the topic, what are the borrowers.sort1 and borrowers.sort2 fields? I've looked in few script to try to figure out their point, but with no success. 17:07 atz actually, nevermind that. not sure about unimarc vs. marc21 17:06 atz frederic: i think those are arbitrary sorting fields 17:05 frederic And is there anywhere an explanation of the purpose of those cn_xxx fields? 17:05 frederic And is there anywhere an explanation of the purpose of those cn_xxx fields? 17:05 frederic Anyone knows why there are differences between MARC21 and UNIMARC in MARC to biblioitems links? There are links in MARC21 which don't exist for UNIMARC in those fields: cn_source, cn_class, cn_item, cn_suffix, cn_sort and totalissues... 16:32 gmcharlt done via t/test-config.txt that Makefile.PL creates 16:31 atz nice! 16:31 gmcharlt atz: yes 16:31 atz are responses to the installer regarding test DB persistent? 16:29 acmoore or, if you figure them out yourself, be bold and update! 16:29 acmoore atz, if you actually try to follow that page, please let me know what questions it doesn't answer and I'll work on it a bit. 16:27 frederic text cut. Forget it 16:27 frederic get wrong somewhere. But who knows if there aren't other issues like that which will 'explode' later. 16:25 gmcharlt http://wiki.koha.org/doku.php?id=en:development:unit_testing, specifically 16:25 atz cool 16:25 gmcharlt atz: yeah, acmoore put something up on the koha wiki a couple days ago 16:24 atz gmcharlt: is there an intro to usage for KohaTest ? 16:22 gmcharlt yeah - I wonder if there was some weird sequencing issue where running a version updatedatabase.pl taken from git HEAD on the wrong date caused the 075 SQL to be skipped even as KOHAVESION was set past 075 16:21 frederic If I alter the table by hand it works. The question remains now why this upgrade hasn't been applied? 16:20 frederic version 075 16:20 frederic I see in updatedatabase.pl that an upgrage is supposed to insert imageurl new field in authorised_values table 16:18 frederic gmcharlt: I'm up-to-date to last version from git 16:18 gmcharlt frederic: this was fixed recently - are you running git HEAD or something a bit earlier than that? 16:17 liz @gmcharlt: yea, for sure. I don't have time to rebuild it all the time ;) 16:14 frederic /admin/authorised_values.pl page doesn't work anymore for me on several installs. In log file, I can see that this issue come from authorised_values table lacking imageurl column. Does anyone else got this? 15:49 owen Must be something weird about my system. 15:49 owen I've got the cookie settings set appropriately, I think 15:49 gmcharlt is it accepting the session cookies? 15:48 gmcharlt owen: just tested in both staff and OPAC - not seeing that 15:45 owen Every time I use it, I'm forced to log in for every link I click or form I submit. Have you encountered that problem? 15:44 gmcharlt owen: yes 15:44 owen gmcharlt: do you have access to Internet Explorer 7? 15:27 gmcharlt ah, ok - hopefully the next time he does it ;) he'll take better notes so that we can fix the bug and prevent it from happening again 15:24 liz thanks, I just gave up and dropped the database, it's only a test server he can redo it since he broke it 15:22 gmcharlt Apache log, that is 15:22 gmcharlt liz: are there any error messages in the Apache when you try to log in? 15:20 liz and his description is "I didn't do anything but change the code!" 15:20 liz i wasn't there I don' tknow what he did to it >.< 15:19 liz one of my users seems to have changed something in the branch definitions that has locked out all of our users, even kohadmin. I've tried changing the branch codes back to their original settings but we're still locked out, any ideas? 15:18 gmcharlt ok 15:18 owen No, not yet 15:16 gmcharlt owen: quick question - have you had a chance to test fix for bug 2105? 15:01 owen Thanks, I'll see what I can find out 15:00 ccatalfo owen: fyi in my brief troubleshooting yesterday i thought that behavior was starting in git rev 9717926 14:58 ccatalfo heh i'm glad it's in koha and not biblios :/ 14:57 owen It has to do with the display properties of the patron search box when autocomplete is turned on 14:57 owen Yeah 14:57 ryan ah, so it's koha, and not biblios ? 14:56 owen Yeah, I'm seeing it too. I've been trying to figure out what's going on myself. 14:49 ccatalfo rch: i'm seeing that happen in biblios; i started to track down yesterday but didn't get it resolved yet 14:39 ryan or is that local to my install? 14:39 ryan owen: are you seeing the more dropdown in toplevelnav appearing behind the resident search box ? 14:25 masonj sorry about the duped patches i just sent guys... 14:16 mc moin moin danny 13:58 danny good morning #koha 13:45 masonj wow, thanks galen - ill try that out 13:40 gmcharlt masonj: one approach: http://pastebin.com/d67f19ba0 13:34 masonj if i can grok that, then theres no need for the GetMemberDetails() call 13:33 masonj 9 = editcatalogue 13:33 masonj whats the math to work out if she has bit 9 set? 13:33 masonj if sally has a flag value of 524802 13:32 masonj so , i just need a little clue-stick at the mo.. 13:31 masonj ta, just mailed you the db update... 13:29 gmcharlt I can look at your patch if you want 13:29 gmcharlt mysql does have bitwise operators and functions, so shouldn't be too bad (http://dev.mysql.com/doc/refman/5.0/en/other-functions.html) 13:27 gmcharlt sorry ;) 13:27 masonj i was dreading you suggesting that... 13:25 gmcharlt masonj: i.e., if this is regarding the new userflag, you should be able to use SQL functions to do the arithmetic 13:24 gmcharlt masonj: although, if you can do what you want directly with SQL, it might be better (to avoid an extra dependency and to avoid breaking the DB upgrade script of GetMemberDetails changes) 13:24 masonj thanks galen 13:23 masonj ok, that was my work-around too 13:22 owen lloyd_,I hadn't had a chance to look. That's great. 13:22 gmcharlt masonj: you would need to put in a 'require C4::Members;' 13:22 masonj but i get an error saying 'sub not found...' 13:21 masonj C4::Members::GetMemberDetails() 13:21 lloyd_ Owen - as you might have seen, I got it working ;)) 13:20 lloyd_ Owen - thanks for changing that google-jackets.js path yesterday 13:20 masonj wernumber'} ); 13:20 masonj my ($bor1, $flags) = C4::Members::GetMemberDetails($bor->{'borro 13:20 masonj ok , in installer/data/mysql/updatedatabase.pl - i want to call a sub from C4::Members .. 13:19 gmcharlt masonj: shoot 13:19 masonj heya galen, me ask a perl question? 13:18 gmcharlt hi owen and #koha 13:18 owen Hey #kohers.