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.