[Koha-bugs] [Bug 5881] CSV export : deal with control fields to avoid an error (C4/Record.pm)

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5881

Paul Poulain <paul.poulain at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #4089|0                           |1
        is obsolete|                            |

--- Comment #4 from Paul Poulain <paul.poulain at biblibre.com> 2011-08-09 14:17:28 UTC ---
Created attachment 4907
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4907
proposed patch (passed QA)

QA comment

This patch is critical in fact : if you try to export a field <010 in CSV
export (like 001, or 009), it results in a nasty Perl error, as fields <010
have a specific accessor
The only 'usefull' part of this patch is :
+               # If it is a control field
+               if ($_->is_control_field) {
+                   $value = defined $authvalues->{$_->as_string} ?
$authvalues->{$_->as_string} : $_->as_string;
+               } else {
(the rest is reindenting)

I'm not the real author of this patch, it's critical, marking passed QA.

Chris, if you object, switch to signed off and bug Ian for another QA checking
!

-- 
Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.