Bug 13530

Summary: Typo in bulkmarcimport
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: MARC Bibliographic record staging/importAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: chris, jonathan.druart, the.invinnysible.one, tomascohen
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: bug 13530: Typo in bulkmarcimport
k
[PASSED QA] bug 13530: Typo in bulkmarcimport
[SIGNED-OFF] bug 13530: Typo in bulkmarcimport
bug 13530: Typo in bulkmarcimport
Bug 13530: Typo in bulkmarcimport

Description Martin Renvoize 2015-01-07 13:27:13 UTC
A typo seems to have crept in bulkmarcimport causing GetMarcFrameworkCode to fail.
Comment 1 Martin Renvoize 2015-01-07 13:35:31 UTC
Hint: Line 401 the GetFrameworkcode function is has one wrong case character ;)
Comment 2 the.invinnysible.one 2015-01-13 02:33:15 UTC
Will attempt to correct problem.
Comment 3 the.invinnysible.one 2015-01-13 03:31:05 UTC Comment hidden (obsolete)
Comment 4 the.invinnysible.one 2015-01-13 03:35:13 UTC
Changed GetFrameworkcode to GetFrameworkCode on line 401. Needs signoff.
Comment 5 Katrin Fischer 2015-01-13 07:23:11 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2015-01-13 07:24:01 UTC Comment hidden (obsolete)
Comment 7 Katrin Fischer 2015-01-13 07:25:05 UTC Comment hidden (obsolete)
Comment 8 Martin Renvoize 2015-01-13 09:49:12 UTC
My colleague who uses this script often is now very happy.. thanks :)

I'll try to poke someone in to QA this today, as as the reporter I think that rules me out.

Cheers
Comment 9 Jonathan Druart 2015-01-13 10:26:07 UTC
(In reply to the.invinnysible.one from comment #4)
> Changed GetFrameworkcode to GetFrameworkCode on line 401. Needs signoff.

Hi mysterious author,
Is there a chance to get a patch with your correct name?
The patch's author is Some <some.name@example.org>
Comment 10 the.invinnysible.one 2015-01-13 22:27:26 UTC Comment hidden (obsolete)
Comment 11 the.invinnysible.one 2015-01-13 22:28:21 UTC
(In reply to Jonathan Druart from comment #9)
> (In reply to the.invinnysible.one from comment #4)
> > Changed GetFrameworkcode to GetFrameworkCode on line 401. Needs signoff.
> 
> Hi mysterious author,
> Is there a chance to get a patch with your correct name?
> The patch's author is Some <some.name@example.org>

My apologies, I have fixed the author name on the patch now. :-)
Comment 12 Jonathan Druart 2015-01-14 08:20:56 UTC
Created attachment 35174 [details] [review]
Bug 13530: Typo in bulkmarcimport

GetFrameworkCode was incorrectly spelt as GetFrameworkcode on line 401.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 13 Jonathan Druart 2015-01-14 08:21:33 UTC
Note that I did not test this patch, because I never user bulkmarcimport.
But the fix is trivial and makes sense.
Comment 14 Jonathan Druart 2015-01-14 08:22:28 UTC
(In reply to Martin Renvoize from comment #8)
> I'll try to poke someone in to QA this today, as as the reporter I think
> that rules me out.

Where did you read that?
I am not sure at all this rules exists :)
Comment 15 Martin Renvoize 2015-01-14 08:25:13 UTC
(In reply to Jonathan Druart from comment #14)
> (In reply to Martin Renvoize from comment #8)
> > I'll try to poke someone in to QA this today, as as the reporter I think
> > that rules me out.
> 
> Where did you read that?
> I am not sure at all this rules exists :)

Just had a hunch it might be one.. if not then I could have QA'd it.. oops.  Thnaks anyway Jonathan
Comment 16 Tomás Cohen Arazi 2015-01-14 15:44:33 UTC
Patch pushed to master.

Thanks Morag, and congratulations!
Comment 17 Chris Cormack 2015-01-18 01:55:32 UTC
Pushed to 3.18.x, will be in the 3.18.3 release so your code will be out in use by libraries around the world by the end of January, Morag