IRC log for #koha, 2007-12-14

← Previous day | Today | Next day → | Search | Index

All times shown according to UTC.

Time Nick Message
12:33 fbcit g'morning koha
13:15 gmcharlt good morning fbcit
13:55 paul hello fbcit & gmcharlt
13:55 and good morning to you
14:08 gmcharlt good morning paul
16:13 fbcit I notice that when I save a marc record imported from a z3950 search w/o a DDN...
16:13 but to which I have added a DDN...
16:13 that the DDN does not show up on the Add Item view....
16:13 however,
16:13 if the z3950 search returns a marc record w/ DDN,
16:14 it shows up on the Add Item view....
16:14 ?
16:39 kados very strange fbcit
16:41 fbcit try it... :)
16:53 kados: here is a book to try it on:
16:53 Author: Garland, Hamlin,
16:53 Title:A son of the middle border,
16:53 lccn:   17022272
16:54 run the search against loc.gov
16:54 it should come back w/o DDN
16:54 add a DDN and save
16:57 bbiab
17:47 kados: were you able to verify what I'm seeing?
18:19 masonj morning #koha
18:42 fbcit hdl: u around?
18:42 hi masonj
19:25 gmcharlt: u around
19:26 gmcharlt hi fbcit
19:30 fbcit hi gmcharlt, do you have any understanding of how the adv search code works?
19:30 gmcharlt not much, I'm afraid
19:31 if you send me some info, I can take a look with fresher eyes
19:32 fbcit I chased down a small bug earlier which cause a malformed "No results..." message
19:32 so I thought I would try to figure out why the adv search was broke...
19:33 but i don't understand how the queries are built
19:33 syntax wise, that is...
19:33 so I think maybe I should leave it for hdl
19:35 to me it looks like all words entered are treated as keywords rather than set to title, author or whatever.
19:35 gmcharlt hmmm
19:36 fbcit when I do this: opac-search.pl?idx=kw&idx=kw&idx=ti%2Cw​rdl&q=alexander&sort_by=relevance&do=OK
19:36 for titles I have which contain 'alexander'
19:37 gmcharlt does it make a difference if you manually do opac-search.pl?idx=ti%2Cwrdl&q=a​lexander&sort_by=relevance&do=OK
19:37 fbcit $query_desc ends up eq "kw: alexander"
19:37 gmcharlt i.e., drop the two idx=kw parms?
19:38 fbcit still no results, but $query_desc is now eq "ti,wrdl: alexander" like I would expect...
19:39 gmcharlt if you drop the ,wrdl, does a plain ti work?
19:39 also, looking at Zebra's own log can be useful
19:39 fbcit yep...
19:39 appears to be a misformed query cgi?
19:40 gmcharlt partly -- the extract idx=kw don't seem right
19:40 but also I would assume that ti,wrdl should work
19:40 extra idx=kw, rather
19:41 fbcit Here is some debug from search.pm
19:41 [Thu Dec 13 15:28:10 2007] opac-search.pl: string :ti,wrdl= alexander  at /usr/share/kohaclone1/C4/Search.pm line 1249.
19:41 [Thu Dec 13 15:28:10 2007] opac-search.pl: leaf : ti wrdl= alexander
19:41 [Thu Dec 13 15:28:10 2007] opac-search.pl: ti wrdl / = /  alexander
19:41 would the comma need to carry through?
19:42 gmcharlt yes, I'm pretty it's required
19:42 fbcit so maybe that's it...
19:42 I'll see about making a few changes.
19:46 ok, the first problem (with the form generated query cgi) is due to the user (me) entering a term in the last of three boxes && nothing in the first two.
19:47 the script probably needs to handle that as there will be users who will make that seemingly obvious mistake. :-)
19:48 gmcharlt yep
19:48 fbcit If I enter a term into the first box, the second error comes into play.
19:49 tnx gmcharlt
19:49 gmcharlt you're welcome
20:07 fbcit gmcharlt: question - just before the line in question, the comma is s/,/ /g
20:07 gmcharlt sounds like it  -- is this in C4/Search.pm, or elsewhere?
20:08 fbcit I wonder if the operand should really be 'ti' rather than 'ti,wrdl' or if the first regex in and then mod the above regex to be ^ti,wrdl
20:08 Search.pm ~line 1300
20:08 I see two fixes, but one is probably more correct
20:10 see also line 1314 or thereabouts
20:10 fyi... I'm running nozebra
20:11 I am guessing that leaving a comma in could mess up some other things...
20:13 gmcharlt fbcit: not sure -- definitely check with kados, hdl, etc
20:14 fbcit either way, that seems to fix adv search on nozebra.
20:15 I saw kados around earlier.
20:49 gmcharlt: I took the easy way out... $left='title' if $left =~ '^ti\b';
21:01 gmcharlt fbcit: probably ok for now
21:03 fbcit I think the final solution is to get rid of the comma, but I'll wait to hear from kados && hdl
21:10 some java script magic seems to be the best cure for fixing the user who wants to skip the first two 'Search for:' boxes...
21:14 gmcharlt: are 'ti,wrdl', 'kw', etc properly referred to as operands?
21:15 the commenting in Search.pm seems to be a bit confused in this regard.
21:37 kados fbcit: ti and wrdl are not operands, the could be part of an operand though
21:37 fbcit: they are 'parameters'
21:37 fbcit: but the nozebra code confuses the terms quite badly unfortunately
21:37 fbcit: in variable naming, etc.
21:37 so it's pretty confusing unfortunately :/
21:38 hdl hi
21:38 fbcit kados && hdl hi
21:38 both have a minute?
21:39 I think I've isolated a problem with opac adv search
21:40 it appears that Search.pm removes the comma from ti,wrdl and then only allows for ti when converting to long parameter/operands whatever...
21:41 there are several ways to fix it
21:41 but what is the best way?
21:41 kados fbcit: you're running nozebra?
21:41 fbcit right
21:42 for now, I have changed $left='title' if $left =~ '^ti$'; to $left='title' if $left =~ '^ti\b';
21:42 kados I don't know why ti,wrdl is being used, where are you seeing that syntax?
21:42 fbcit which allows it to keep stripping the comma
21:42 enable debug on Search.pm and watch the log while doing a title search
21:43 opac-search.pl?idx=ti%2Cwrdl&q=alexande​r&idx=kw&idx=kw&sort_by=relevance&do=OK
21:43 kados is this just in the OPAC?
21:43 fbcit note idx=ti%2Cwrdl&q=alexander
21:43 not sure... hold on
21:43 hdl fbcit : with nozebra any index modifier should be dropped.
21:43 kados            <option value="ti,wrdl">Title</option>
21:43 hdl such as wrdl or expr.
21:44 kados wrdl should be removed from all templates
21:44 fbcit:  I'm guessing that will fix it
21:44 fbcit yep
21:44 only on opac
21:44 that answers my question.
21:45 hdl since nozebra search donot manage phrase search.
21:45 fbcit I also found a small bug which causes a no result search to return an empty set of quotes
21:45 kados fbcit: file it in bugzilla, I'll have a look at it
21:45 fbcit k
21:45 I'll fix the template and push a patch...
21:47 kados fbcit++
21:51 fbcit kados: try this: http://71.0.121.24:8181/cgi-bi[…]y=relevance&do=OK
21:51 to see the no result problem
22:15 kados fbcit++
22:16 fbcit kados: I also submitted patches to add lccn to the z3950 search
22:17 btw, the link in my bug report will not show the error now that I have fixed it... sorry
23:03 [K] *** join #koha@FreeNode: phasefx__ n=phasefx@adsl-215-217-176.aep.bellsouth.net
00:08 kados thd-away`: hi
02:16 CGI034 03
02:16 hallo
02:17 masonj hello cgi034
02:18 CGI034 is this a meeting?
02:19 masonj hmm, not now
02:19 were you expecting a meeting now?
02:22 CGI034 ahh, I just see many people on the IRC,
02:22 Seeking information about new koha release, ready now?
02:23 masonj veeery soon
02:25 CGI034 ahhh, tried to install, some progress, not quite yet
02:25 could you know sir, the date of soon?
02:25 masonj galen has been doing so new work on the installer
02:26 s/so/some/
02:26 [K] <ru55el@FreeNode> there is not specific date for a release - it is tied to when all the work is completed  - but it should be very soon (weeks not months)
02:26 masonj hmm, sorry im not sure of dates
02:27 CGI034 ahhh
02:27 i see
02:28 when should i attempt install?
02:29 [K] <ru55el@FreeNode> are you a member of the koha-devel mailing list? i suggest watching that for announcements
02:30 *** part FreeNode!#koha: ru55el n=russel@203-118-134-114.netspace.net.nz
02:31 masonj good idea russ
02:32 CGI034 thanks yous
02:32 masonj install now, and get familiar with koha3, before its offical release
04:00 [K] *** join #koha@FreeNode: ru55el n=russel@203-118-134-114.netspace.net.nz

← Previous day | Today | Next day → | Search | Index

koha1