Bug 23425: (QA follow-up) Keep it generic

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

diff --git a/Koha/Biblio/Metadata.pm b/Koha/Biblio/Metadata.pm
index bbc11d9..4dc99a2 100644 (file)
--- a/Koha/Biblio/Metadata.pm
+++ b/Koha/Biblio/Metadata.pm
@@ -75,7 +75,7 @@ sub record {
                 id     => $self->id,
                 format => $self->format,
                 schema => $self->schema,
-                marcxml_error => $marcxml_error,
+                decoding_error => $marcxml_error,
             );
         }
     }

diff --git a/Koha/Exceptions/Metadata.pm b/Koha/Exceptions/Metadata.pm
index 887a49e..3930b86 100644 (file)
--- a/Koha/Exceptions/Metadata.pm
+++ b/Koha/Exceptions/Metadata.pm
@@ -25,7 +25,7 @@ use Exception::Class (
     'Koha::Exceptions::Metadata::Invalid' => {
         isa => 'Koha::Exceptions::Metadata',
         description => 'Invalid data',
-        fields => ['id','format','schema', 'marcxml_error']
+        fields => ['id','format','schema', 'decoding_error']
     }
 );
 
@@ -36,8 +36,8 @@ sub full_message {
 
     unless ($msg) {
         if ( $self->isa('Koha::Exceptions::Metadata::Invalid') ) {
-            $msg = sprintf( "Invalid data, cannot decode object (id=%s, format=%s, schema=%s, MARCXML error='%s')",
-                $self->id, $self->format, $self->schema, $self->marcxml_error );
+            $msg = sprintf( "Invalid data, cannot decode object (id=%s, format=%s, schema=%s, decoding_error='%s')",
+                $self->id, $self->format, $self->schema, $self->decoding_error );
         }
     }
 

diff --git a/t/Koha/Exceptions.t b/t/Koha/Exceptions.t
index a5a504e..bb36df3 100644 (file)
--- a/t/Koha/Exceptions.t
+++ b/t/Koha/Exceptions.t
@@ -115,12 +115,14 @@ subtest 'Koha::Exceptions::Metadata tests' => sub {
     $object->mock( 'schema', 'a_schema' );
 
     throws_ok
-        { Koha::Exceptions::Metadata::Invalid->throw( id => 'an_id', format => 'a_format', schema => 'a_schema' ); }
+        { Koha::Exceptions::Metadata::Invalid->throw(
+            id => 'an_id', format => 'a_format',
+            schema => 'a_schema', decoding_error => 'a_nasty_error' ); }
         'Koha::Exceptions::Metadata::Invalid',
         'Exception is thrown :-D';
 
     # stringify the exception
-    is( "$@", 'Invalid data, cannot decode object (id=an_id, format=a_format, schema=a_schema)', 'Exception stringified correctly' );
+    is( "$@", 'Invalid data, cannot decode object (id=an_id, format=a_format, schema=a_schema, decoding_error=\'a_nasty_error\')', 'Exception stringified correctly' );
 
     throws_ok
         { Koha::Exceptions::Metadata::Invalid->throw( "Manual message exception" ) }