Bug 5480 - Some usual UNIMARC cataloguing plugins doesn't work anymore
Summary: Some usual UNIMARC cataloguing plugins doesn't work anymore
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: 3.4
Hardware: All All
: PATCH-Sent (DO NOT USE) major
Assignee: Frédéric Demians
QA Contact: Bugs List
URL:
Keywords:
: 5564 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-12-09 14:29 UTC by Frédéric Demians
Modified: 2012-10-25 23:04 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Demians 2010-12-09 14:29:30 UTC
An Owen's patch:

git show 69c8b06a9641292b5ea1e71d73c40b06cfb32638

commited by chris_n Oct 25th, broke two UNIMARC cataloguing plugins by
modifying some CGI scripts parameters names:

  - unimarc_field_210c.pl
  - unimarc_field_4xx.pl
Comment 2 Owen Leonard 2010-12-30 19:30:25 UTC
*** Bug 5564 has been marked as a duplicate of this bug. ***
Comment 3 MJ Ray (software.coop) 2011-01-06 10:17:54 UTC
This bug is mentioned in:
Bug 5480 Some usual UNIMARC cataloguing	plugins doesn't work anymore http://lists.koha-community.org/pipermail/koha-patches/2010-December/013434.html
Bug 5480 Some usual UNIMARC	cataloguing plugins doesn't work anymore http://lists.koha-community.org/pipermail/koha-patches/2010-December/013438.html
Comment 4 Chris Nighswonger 2011-01-08 18:55:52 UTC
(In reply to comment #0)
> An Owen's patch:
> 
> git show 69c8b06a9641292b5ea1e71d73c40b06cfb32638
> 
> commited by chris_n Oct 25th, broke two UNIMARC cataloguing plugins ...

Perhaps in the future we should flag patches which touch MARC code to indicate which flavor they touch upon so that those doing QA/signoff will be aware of the need to focus particular attention on that aspect of testing the patch.
Comment 5 Chris Cormack 2011-01-09 15:51:56 UTC
Pushed, please test