Bug 13850

Summary:	C4::Ris does not manage correctly 660 and 661 - UNIMARC
Product:	Koha	Reporter:	Jonathan Druart <jonathan.druart>
Component:	Architecture, internals, and plumbing	Assignee:	Jonathan Druart <jonathan.druart>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	normal
Priority:	P5 - low	CC:	chris, tomascohen
Version:	Main
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:
Circulation function:
Attachments:	Bug 13850: Fix typo in C4::Ris Bug 13850: Fix typo in C4::Ris [PASSED QA] Bug 13850: Fix typo in C4::Ris

Description Jonathan Druart 2015-03-17 10:57:17 UTC

In C4/Ris.pm:
  @field_list = ('600', '601', '602', '604', '605', '606','607', '608', '610', '615', '620', '660'. '661', '670', '675', '676', '680', '686');

Note the 
  '660'. '661'
instead of
  '660', '661'

Comment 1 Jonathan Druart 2015-03-17 10:58:41 UTC Comment hidden (obsolete)

Created attachment 36943 [details] [review]
Bug 13850: Fix typo in C4::Ris

In C4/Ris.pm:
  @field_list = ('600', '601', '602', '604', '605', '606','607', '608',
  '610', '615', '620', '660'. '661', '670', '675', '676', '680', '686');

Note the
  '660'. '661'
instead of
  '660', '661'

Test plan:
Add a 660 and 661 field to a record and export it in RIS
The 660 and 661 field should be displayed (after "KW").

Comment 2 Mark Tompsett 2015-03-30 17:32:43 UTC Comment hidden (obsolete)

Created attachment 37384 [details] [review]
Bug 13850: Fix typo in C4::Ris

In C4/Ris.pm:
  @field_list = ('600', '601', '602', '604', '605', '606','607', '608',
  '610', '615', '620', '660'. '661', '670', '675', '676', '680', '686');

Note the
  '660'. '661'
instead of
  '660', '661'

Test plan:
Add a 660 and 661 field to a record and export it in RIS
The 660 and 661 field should be displayed (after "KW").

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Comment 3 Katrin Fischer 2015-04-06 09:46:08 UTC

Created attachment 37521 [details] [review]
[PASSED QA] Bug 13850: Fix typo in C4::Ris

In C4/Ris.pm:
  @field_list = ('600', '601', '602', '604', '605', '606','607', '608',
  '610', '615', '620', '660'. '661', '670', '675', '676', '680', '686');

Note the
  '660'. '661'
instead of
  '660', '661'

Test plan:
Add a 660 and 661 field to a record and export it in RIS
The 660 and 661 field should be displayed (after "KW").

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Comment 4 Tomás Cohen Arazi (tcohen) 2015-04-08 14:00:48 UTC

Patch pushed to master.

Good catch Jonathan!

Comment 5 Chris Cormack 2015-04-11 03:03:02 UTC

Pushed to 3.18.x will be in 3.18.6