Open Source Integrated Library System

Evergreen on IRC

#evergreen Logs for Sunday, April 3rd, 2011

< Saturday, April 2nd, 2011Raw Log FileMonday, April 4th, 2011 >
#TimeNickMessage
#00:44:27adbowling-isl has quit IRC
#01:12:37_deXter has joined #evergreen
#01:27:28suho has quit IRC
#01:29:19_deXter has quit IRC
#02:09:04phasefx@later tell tsbere I like the idea of having the labels on the commands, so that multiple widgets referencing the command (say a main menuitem and a context menuitem) can share the label with less duplication of text in the file. Exceptions can be made for when we don't want such sharing
#02:09:04pinesol`phasefx: The operation succeeded.
#02:10:53phasefx@later tell tsbere I didn't really grok the attribute forwarding properties of command when I first started, or I would have done that from the start
#02:10:53pinesol`phasefx: The operation succeeded.
#07:57:32tsbere@later tell phasefx There are issues with putting label/accesskey on commands only, though, meaning some will be on menuitems still. Such as patron by barcode that exists as three "functions": Checkout item, Retrieve Patron by Barcode, Search for Patron by Barcode. Which should be the "primary" on the command node?
#07:57:32pinesol`tsbere: The operation succeeded.
#09:57:43tsbere@later tell phasefx Also, the label (accesskey not being that useful) also may apply to toolbar buttons with the same command. Which in some cases use the same language string, in others use a shorter one?
#09:57:43pinesol`tsbere: The operation succeeded.
#10:02:00flopaul has joined #evergreen
#10:05:43rjackson-isl has joined #evergreen
#10:08:08rjackson-islGood Sunday morning - was wondering if anyone is online that has had experience migrating a library onto 2.0.x?
#10:08:25rjackson-islWHen the copy of bibs starts up it never seems to finish
#10:08:40rjackson-islbeen running for over 15 hours now - about 30K bibs
#10:09:32flopaul has quit IRC
#10:15:51Dyrcona has joined #evergreen
#10:16:55suho has joined #evergreen
#11:10:44DyrconaIs there a good reason that there is no foreign key constraint on asset.copy in action.circulation?
#11:11:04DyrconaI was able to load 325,000+ circulations without any copies in the database.
#11:11:16eeevilDyrcona: yes. serial.unit.id could be used there
#11:12:15Dyrconaok, then. makes sense and you can't make a foreign key that references either of two other tables.
#11:12:24eeevilright
#11:12:30Dyrconanow, i vaguely remember discussing this or something like it months ago.
#11:13:14eeevilwe could use a check constraint of BEFORE trigger, though. the letter being more "correct"
#11:13:49Dyrconayeah. a before trigger....
#11:14:20Dyrconai'll see if i can come up with a good one in between the thousand other things that i'm doing this week/weekend.
#11:14:44eeevil:)
#11:15:27Dyrconajust a had a load blow up because of a typo in a query. we were going to update our demo server with newer code and data.
#11:16:01bjwebb has joined #evergreen
#11:16:24Dyrconathat's how i saw the circulations created with no copies.
#11:17:09tsbere missed a typo and accidently wiped out 20 minutes of work and missed it for 3 git commits
#11:18:56tsbereOh well. Now that I know what I am doing it is a set of 3 or 4 sed commands.
#11:20:23Dyrconased is the answer to which is more powerful: vi or emacs.
#11:22:04eeevilgrabbing 0507
#11:28:32phasefxtsbere: if you anticipate the need for separate labels for widgets using the same command, I say go ahead and use separate DTD's directly on the widget. Is that what you're asking/getting at?
#11:29:06phasefxah, two later's <reads>
#11:29:12tsberephasefx: I am saying there are 8 (1 commented out) cmd_blah IDs being used with multiple labels/accesskeys in the menus.
#11:31:28phasefxand what are you wanting my opinion on?
#11:34:13tsbereHow do I determine which of the conflicting labels goes on the <command> node?
#11:34:27tsbereOr do I not put any on there and leave all of them on the menuitems?
#11:34:32phasefxI think that
#11:34:37phasefxthe latter
#11:35:08phasefxif the entities are different, I probably wanted to give translators an option to distinguish between context
#11:35:37tsbereTake note, if there is something on the menuitem it should override the command node.
#11:35:38phasefxbut I don't really know how much of that is needed, and probably haven't been consistent in that regard
#11:35:49phasefxah
#11:35:54tsbereSo the "default" can be on the command node
#11:36:06tsbereWhich brings me back to "which one should be the default?"
#11:36:39tsbere notes that there are technically 9 commands with multiple menu entries, commented out or otherwise, but cmd_replace_barcode uses the same languge string 3 times
#11:36:56phasefxI think if we want to do that, then any arbitrary rule is probably as good as any other. First occurance is the default? Last occurance?
#11:37:32tsbereOccurance that seems to most closely match the name of the command in the english translation?
#11:37:39phasefxtsbere++ btw, for getting an itch to clean things up
#11:37:48phasefxthat sounds good to me too
#11:38:41tsbereThis is part of an ongoing "prep" for an extension or feature to do major messing with the menus (and maybe toolbar)
#11:38:46phasefxor, most prominent widget
#11:39:27phasefxprominent or easiest widget to get to
#11:39:29tsbereBTW, figured out that for toolbar use you can put the image= on the command too. Think I should move those while I am at it?
#11:40:57phasefxtsbere: a little more iffy on images than labels. Let's say you want to have a dropdown menu on the toolbar that lists all toolbar commands, even if the toolbar is hiding some icons
#11:41:53phasefxbut I'm fine with changing things when you need to, and keeping stuff consistent/easy to find otherwise. So could have images on commands until and if they become an issue being there
#11:41:55tsbere notes that menu items don't show icons at all unless they have menuitem-iconic as a class, and thus ignore the image setting
#11:42:58phasefxtsbere: I think you have more of a grip on this than I do; if you want to trailblaze I say have at it
#11:44:25flopaul has joined #evergreen
#11:45:14tsbereBTW, is there an easy way from the javascript layer to check permissions?
#11:46:09phasefxI never wrapped such checks in a utility function, but berick may have in the opac code
#11:47:41phasefxthe dojo stuff may have it wrapped up nice too, not sure
#11:49:14tsbereI am considering a "assign one or more permissions to each command node in a new custom attribute, then iterate over them on window open and disable any that the user doesn't have the permission for" addition
#11:50:21tsbereMaybe cache somewhere so that it isn't doing it all the time
#11:50:25phasefxthere should be a batch perms check
#11:50:34tsbereWould need to re-run the check on new operator, though
#11:52:36tsbereI don't see much in the way of batch check. Hmmm.
#11:52:52phasefxopen-ils.actor.user.has_work_perm_at.batch
#11:53:02phasefxI think that may do what you need
#11:53:24tsbere would probably do better if he spelled "batch" correctly when grepping
#12:00:33suhothayan has joined #evergreen
#12:03:56suho has quit IRC
#12:04:17flopaul has quit IRC
#12:23:40suhothayan is now known as suho
#12:52:09Dyrconahmm.... apparently postgres doesn't handle "not null default value" the same way that some other databases do.
#12:52:27Dyrconait gives an error when i insert NULL, rather than use the default value.
#12:53:50phasefxyeah, I've noticed that
#12:54:21Dyrconakind of a pita when every other database i use would use the default.
#12:54:30phasefxI think you can insert DEFAULT in psql
#12:56:27tsberephasefx: I have a client_menus branch in my git repo, if you want to see/comment on what I have done so far. http://git.mvlcstaff.org/?p=tsbere/ILS.git;a=shortlog;h=refs/heads/client_menus
#12:57:09phasefxtsbere: cool deal
#12:59:13tsbere is happy he knows sed and vim regex replace because otherwise some of that would have taken days
#13:00:38tsberephasefx: Also, the current state (3 commits on it) should (given a quick test or two) be safe and equiv to what was there from the other side of things. Thus commitable to trunk if you wanted to.
#13:01:20tsberephasefx: Ok, it breaks the label change on operator swap. *grumble*
#13:02:13tsbere takes a much closer look and wonders if he broke labels, or the *slight* upgrade to xulrunner he did in the process changed something
#13:02:41phasefxhotkeys still work after removing @key?
#13:03:51tsberecommand nodes ignore key=. Hotkeys work without them. However, between the two versions of xulrunner command node labels started to *override* other labels, like on toolbar buttons and menu items.
#13:04:03tsbere grumbles
#13:04:23phasefxfun
#13:04:39tsbereLuckily that was a different commit
#13:04:44tsbereand can thus be reverted easily enough
#13:05:01tsbereGiven that, though, should I go the other way and put them on the menuitems?
#13:05:37phasefxfor the multi-label case or for all cases?
#13:06:16tsbereIn all cases, so that you don't run into "I want a shorter one for this toolbar button" and it won't let you.
#13:06:21tsbereOr other similar things
#13:07:47phasefxwell, I'm torn. In either case, if you need a different label, you're going to be editing some xul. But I like things on command for the majority of cases, just because it makes the files smaller, and gives xmlent less work to do. But if everything is on menuitems, etc., then at least it's consistent and more comprehensible for newcomers
#13:08:51tsbereWith them on the items using the label you don't run into "I edited this *other* file (where the toolbars are) and can't set/change a label" issues too.
#13:08:52phasefxmaybe see what the convention is in Firefox, Thunderbird
#13:09:44tsbere notes that he aims to make it so that almost anything in the menus can be given a toolbar button, and thus may want to mess with labels a lot more than is currently done in that regard
#13:11:18phasefxif it makes things easier for you to not have stuff on command, then go for it
#13:16:40tsberePushed another commit to swing things the other way
#13:16:58tsbereAlthough I think I restored a pile of oldaccesskey type settings in the process.
#13:19:48phasefxmy way of commenting out attributes. Probably a bad habit with VCS's
#13:20:39tsbereMaybe. I only removed them because my regex stuff moved them to the command nodes and I was sanitizing the command nodes.
#13:20:49tsbereHence reverting that change putting them back
#13:22:40tsbereI suspect before commit to svn that some squashing is going to be needed
#13:51:16bjwebb has quit IRC
#14:45:33rsinger has quit IRC
#15:40:19suho has quit IRC
#15:57:27rsinger has joined #evergreen
#16:19:58Dyrcona has quit IRC
#16:21:46Dyrcona has joined #evergreen
#16:49:19eeevildenials: curious ... what's authority field 148 from? what cataloging standard, I mean
#17:08:32Dyrconaeeevil: Where should a before trigger on action.circulation go that requires the asset and serials schemas both be loaded first, i.e. what file?
#17:09:46Dyrconaeeevil: Unless I hear otherwise, i'll put it in 800.fkeys.sql.
#17:09:59eeevilDyrcona: you could put it in 800.fkeys.sql, since it's implementing an inherited-table fkey of sorts
#17:10:03eeevilyeah
#17:10:10Dyrcona:)
#17:10:57eeeviland you don't have to check them separately, you can just do SELECT TRUE FROM asset.copy WHERE id = $1 or equiv
#17:11:28eeevilthat will find the id in asset.copy or any child table (only serial.unit for now, but others could be added later)
#17:12:02eeevilthe fkey checks are essentially SELECT TRUE FROM ONLY asset.copy WHERE id = $1
#17:12:08eeevil(note the ONLY keyword)
#17:13:08eeevilnow, if you want it to be deferrable you can use an AFTER trigger and raise an exception from within the function
#17:13:38eeevilthat might be closer to what we want, as we almost always say DEFERRABLE INITIALLY DEFERRED
#17:14:44eeevil(it's not really DEFERRED, but it at least allows BEFORE triggers to fill in the data before checking for it)
#17:15:16Dyrconawill an after trigger that raises an exception prevent the bad data going in?
#17:16:13Dyrconai also see that there was a set null on the foreign key constraint that is commented out. should I try to support that and allow the target copy id to be null?
#17:17:07eeevilDyrcona: well, you can't really delete asset.copy objects, they're protected by a RULE
#17:17:17eeevilso, I don't think that's necessary
#17:17:30Dyrconaok
#17:17:37eeevilis the field currently nullable?
#17:17:43eeevilif so, /that/ should be supported, yes
#17:17:55eeevilbut not the ON CASCADE SET NULL
#17:18:07eeeviler, ON DELETE SET NULL
#17:18:22Dyrconano, it isn't, it turns out
#17:18:31eeevilk
#17:18:53eeevilso, a simple constraint that checks to see if asset.copy and kids has that value in the id field is enough
#17:19:20eeevildenials: I found the 148, btw, no worries
#18:04:30Dyrconaeeevil: git://git.mvlcstaff.org/jason/ILS branch ac_target_copy_trig
#18:16:43suho has joined #evergreen
#18:18:31suho has quit IRC
#18:20:30Dyrconacreated a launchpad bug anyway.
#18:26:30Dyrconathat is more than enough for today.
#18:26:56Dyrcona has quit IRC
#19:08:16Digital_Pioneer creates an Android app for Evergreen with all the functionality of Hello World.
#19:10:04tsbere notes that Digital_Pioneer has more functionality than he does when it comes to an Android app doing anything right now
#19:10:16Digital_Pioneer:D
#19:10:38Digital_PioneerIt seemed reasonable to make something that proves I can at least put something on the device.
#19:10:44tsbereTrue
#19:14:28Digital_Pioneer ships said app to his phone. It worx.
#19:15:02Digital_PioneerLet's see, here... I need an icon.
#19:15:54Digital_PioneerDoes Evergreen have an official logo?
#19:16:27Digital_PioneerAnd does anyone happen to know where I could get a nice SVG of it? :P
#19:27:49tsbereHmmm
#19:28:39tsbereWe have .ico files in the staff client folder
#19:29:44Digital_PioneerEhh. I'll just take the little emblem on the homepage logo.
#19:30:02tsbere had some source files kicking around
#19:30:24bshumThe main Evergreen logo info is here: http://www.georgialibraries.org/lib/pines/evergreenlogo/
#19:30:35bshumBut that icon for the staff client is the usual one, right?
#19:31:15bshumFor an app icon, that seems like the best choice
#19:31:16tsbereThe icons in the staff client folder were created by mrpeters-isl, I think. I have the eps source files and some large png files I just found too.
#19:33:52senator_ has joined #evergreen
#19:35:39finnapz2 has quit IRC
#19:35:43senator has quit IRC
#19:37:14finnapz2 has joined #evergreen
#19:49:33tsbere_ has joined #evergreen
#19:51:41tsbere has quit IRC
#19:51:41rjackson-isl has quit IRC
#19:51:44eeevil has quit IRC
#19:51:46agJohn has quit IRC
#19:51:46Callender has quit IRC
#19:52:38Callender_ has joined #evergreen
#19:52:39eeevil_ has joined #evergreen
#19:52:44JMCraig has joined #evergreen
#19:52:45Callender_ is now known as Callender
#19:52:45JMCraig is now known as agJohn
#19:55:09eeevil_ is now known as eeevil
#19:55:43Jbergy has joined #evergreen
#20:03:52Jbergy has quit IRC
#20:04:17leed-home has quit IRC
#20:07:21Digital_PioneerI made this for the Android launcher icon. If anyone wants to make something else, feel free, and I'll put it in. Here's mine: http://dl.dropbox.com/u/453116/icon.png
#20:12:30leed-home has joined #evergreen
#20:14:21bshumDigital_Pioneer: From trunk, I think this is what tsbere was referring to - http://svn.open-ils.org/trac/ILS/browser/trunk/Open-ILS/xul/staff_client/evergreen-icon.ico
#20:14:27bshumDigital_Pioneer: Nice icon though :)
#20:38:34finnapz has quit IRC
#20:38:42mrpeters-isl has quit IRC
#20:38:54mrpeters-isl has joined #evergreen
#20:38:57finnapz has joined #evergreen
#20:40:00flopaul has joined #evergreen
#20:51:23tsbere_bshum / Digital_Pioneer : I have the eps files from mrpeters-isl for the one in trunk. And the beta variant. Dunno where they may live in a repo, if anywhere.
#21:00:45Dmagick has quit IRC
#21:01:54Dmagick has joined #evergreen
#21:02:10atheos has quit IRC
#21:02:11mjgiarlo has quit IRC
#21:02:37moodaepo has quit IRC
#21:03:39moodaepo has joined #evergreen
#21:07:33atheos has joined #evergreen
#21:25:01flopaul has quit IRC
#21:31:49Digital_Pioneerbshum: Yeah, but mine is more compliant with the Android icon standards (as I made it with them in mind).
#21:32:21Digital_PioneerThat said, an icon is an icon. :P I can use that one if you think it would be better.
#21:36:45tsbere_@later tell phasefx I updated my client_menus branch with the ability to set permissions on command nodes. No permission = command node disabled. Command node disabled = menu/toolbar items/hotkeys auto-disabled.
#21:36:45pinesol`tsbere_: The operation succeeded.
#21:36:50tsbere_ is now known as tsbere
#21:46:46flopaul has joined #evergreen
#21:58:33flopaul has quit IRC
#22:23:36artunit_ has joined #evergreen
#22:27:16artunit has quit IRC
#22:27:28artunit_ is now known as artunit
#22:54:24dbwells_ has joined #evergreen
#22:55:17atz_ has quit IRC
#22:55:17mtate has quit IRC
#22:55:20dbwells has quit IRC
#22:55:21eby has quit IRC
#22:55:21phasefx_ has quit IRC
#22:55:23kbeswick has quit IRC
#22:55:27dbwells_ is now known as dbwells
#22:55:29atz_ has joined #evergreen
#23:00:55tsbere has quit IRC
#23:01:00sylvar has quit IRC
#23:01:01leed has quit IRC
#23:01:03csharp has quit IRC
#23:04:32kbeswick has joined #evergreen
#23:04:35eby has joined #evergreen
#23:04:37phasefx_ has joined #evergreen
#23:04:52tsbere has joined #evergreen
#23:04:52csharp has joined #evergreen
#23:04:52sylvar has joined #evergreen
#23:04:52leed has joined #evergreen
#23:05:02mtate has joined #evergreen
#23:26:54rangi` has joined #evergreen
#23:27:20_bott_1 has joined #evergreen
#23:28:49bshum_ has joined #evergreen
#23:31:16bshum_ has left #evergreen
#23:32:41AaronZ-PLS has quit IRC
#23:32:46_bott_ has quit IRC
#23:32:46bshum has quit IRC
#23:32:46rangi has quit IRC
#23:32:46jeff has quit IRC
#23:36:54jeff has joined #evergreen
#23:36:54jeff has joined #evergreen
#23:38:14bshum has joined #evergreen
#23:55:22mtisi_ has joined #evergreen
#23:55:30mtisi has quit IRC
< Saturday, April 2nd, 2011Raw Log FileMonday, April 4th, 2011 >