Bug 22421: (QA follow-up) Update API mapping and specs

We missed some cases where the API specs and mappings were missing for
the new old_issue_id field.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>

Koha/Account/Credit.pm

@@ -48,6 +48,7 @@ sub to_api_mapping {
         borrowernumber    => 'patron_id',
         branchcode        => 'library_id',
         issue_id          => undef,
+        old_issue_id      => undef,
         itemnumber        => undef,
         manager_id        => 'user_id',
         note              => 'internal_note',

Koha/Account/Debit.pm

@@ -48,6 +48,7 @@ sub to_api_mapping {
         borrowernumber    => 'patron_id',
         branchcode        => 'library_id',
         issue_id          => 'checkout_id',
+        old_issue_id      => 'old_checkout_id',
         itemnumber        => 'item_id',
         manager_id        => 'user_id',
         note              => 'internal_note',

Koha/Account/Line.pm

@@ -982,6 +982,7 @@ sub to_api_mapping {
         borrowernumber    => 'patron_id',
         branchcode        => 'library_id',
         issue_id          => 'checkout_id',
+        old_issue_id      => 'old_checkout_id',
         itemnumber        => 'item_id',
         manager_id        => 'user_id',
         note              => 'internal_note',

api/v1/swagger/definitions/account_line.yaml

@@ -24,6 +24,11 @@ properties:
       - integer
       - "null"
     description: Internal identifier for the checkout the account line is related to
+  old_checkout_id:
+    type:
+      - integer
+      - "null"
+    description: Internal identifier for the old checkout the account line is related to
   credit_number:
     type:
       - string

api/v1/swagger/definitions/debit.yaml

@@ -23,6 +23,11 @@ properties:
       - integer
       - "null"
     description: Internal identifier for the checkout the account line is related to
+  old_checkout_id:
+    type:
+      - integer
+      - "null"
+    description: Internal identifier for the old checkout the account line is related to
   date:
     type: string
     format: date-time

t/db_dependent/api/v1/patrons_accounts.t

@@ -90,7 +90,7 @@ subtest 'get_balance() tests' => sub {
     $debit_2->discard_changes;
 
     $t->get_ok("//$userid:$password@/api/v1/patrons/$patron_id/account")
-      ->status_is(200)->json_is(
+      ->status_is(200)->or(sub { diag $t->tx->res->body } )->json_is(
         {
             balance            => 100.01,
             outstanding_debits => {