Attachment 111414 Details for Bug 26635 – Bug 26635: Expand authorised values in to_api method

[patch] Bug 26635: Expand authorised values in to_api method

Bug-26635-Expand-authorised-values-in-toapi-method.patch (text/plain), 5.07 KB, created by Agustín Moyano on 2020-10-09 15:01:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Agustín Moyano

Created: 2020-10-09 15:01:13 UTC

Size: 5.07 KB

patch

obsolete

>From e4096847dbdcebcda881e14628dfe3c852f574db Mon Sep 17 00:00:00 2001
>From: Agustin Moyano <agustinmoyano@theke.io>
>Date: Fri, 9 Oct 2020 09:59:41 -0300
>Subject: [PATCH] Bug 26635: Expand authorised values in to_api method
>
>This patch adds the posibility to expand authorised values when to_api
>method is called.
>
>The classes where authorised values should expand must implememnt the
>_fetch_authorised_values method, and must return a hash like the
>following
>
>{
>  column_name => Koha::AuthorisedValue->unblessed
>  ...
>}
>
>This patch will be used in bug 8179, so please test there.
>---
> Koha/Object.pm              | 69 ++++++++++++++++++++++++-------------
> Koha/REST/Plugin/Objects.pm |  2 +-
> 2 files changed, 47 insertions(+), 24 deletions(-)
>
>diff --git a/Koha/Object.pm b/Koha/Object.pm
>index bb2a5d8ee8..b5c4493f53 100644
>--- a/Koha/Object.pm
>+++ b/Koha/Object.pm
>@@ -505,28 +505,10 @@ sub to_api {
>     my ( $self, $params ) = @_;
>     my $json_object = $self->TO_JSON;
> 
>-    my $to_api_mapping = $self->to_api_mapping;
>-
>-    # Rename attributes if there's a mapping
>-    if ( $self->can('to_api_mapping') ) {
>-        foreach my $column ( keys %{ $self->to_api_mapping } ) {
>-            my $mapped_column = $self->to_api_mapping->{$column};
>-            if ( exists $json_object->{$column}
>-                && defined $mapped_column )
>-            {
>-                # key != undef
>-                $json_object->{$mapped_column} = delete $json_object->{$column};
>-            }
>-            elsif ( exists $json_object->{$column}
>-                && !defined $mapped_column )
>-            {
>-                # key == undef
>-                delete $json_object->{$column};
>-            }
>-        }
>-    }
>+    $json_object = $self->_do_api_mapping($json_object);
> 
>     my $embeds = $params->{embed};
>+    my $av_expand = $params->{av_expand};
> 
>     if ($embeds) {
>         foreach my $embed ( keys %{$embeds} ) {
>@@ -545,20 +527,60 @@ sub to_api {
>                 if ( defined $children and ref($children) eq 'ARRAY' ) {
>                     my @list = map {
>                         $self->_handle_to_api_child(
>-                            { child => $_, next => $next, curr => $curr } )
>+                            { child => $_, next => $next, curr => $curr, av_expand => $av_expand } )
>                     } @{$children};
>                     $json_object->{$curr} = \@list;
>                 }
>                 else {
>                     $json_object->{$curr} = $self->_handle_to_api_child(
>-                        { child => $children, next => $next, curr => $curr } );
>+                        { child => $children, next => $next, curr => $curr, av_expand => $av_expand } );
>                 }
>             }
>         }
>     }
> 
>+    if($av_expand && $self->can('_fetch_authorised_values')) {
>+        # _fetch_authorised_values should return a hash as the following
>+        # {
>+        #  column_name => <authorised_value>->unblessed
>+        #  ...
>+        # }
>+        my $avs = $self->_fetch_authorised_values($av_expand);
>+
>+        # Language selection will be implemented when lang overlay for av is ready
>+        # Now we will just fetch plain authorised values from the Koha::AuthorisedValues
>+        $avs = $self->_do_api_mapping($avs);
>+
>+        $json_object->{_authorised_values} = $avs || {};
>+    }
>+
>+    return $json_object;
>+}
>+
>+=head3 _do_api_mapping
> 
>+=cut
> 
>+sub _do_api_mapping {
>+    my ($self, $json_object) = @_;
>+    # Rename attributes if there's a mapping
>+    if ( $self->can('to_api_mapping') ) {
>+        foreach my $column ( keys %{ $self->to_api_mapping } ) {
>+            my $mapped_column = $self->to_api_mapping->{$column};
>+            if ( exists $json_object->{$column}
>+                && defined $mapped_column )
>+            {
>+                # key != undef
>+                $json_object->{$mapped_column} = delete $json_object->{$column};
>+            }
>+            elsif ( exists $json_object->{$column}
>+                && !defined $mapped_column )
>+            {
>+                # key == undef
>+                delete $json_object->{$column};
>+            }
>+        }
>+    }
>     return $json_object;
> }
> 
>@@ -806,6 +828,7 @@ sub _handle_to_api_child {
>     my $child = $args->{child};
>     my $next  = $args->{next};
>     my $curr  = $args->{curr};
>+    my $av_expand = $args->{av_expand};
> 
>     my $res;
> 
>@@ -815,7 +838,7 @@ sub _handle_to_api_child {
>             if defined $next and blessed $child and !$child->can('to_api');
> 
>         if ( blessed $child ) {
>-            $res = $child->to_api({ embed => $next });
>+            $res = $child->to_api({ embed => $next, av_expand => $av_expand });
>         }
>         else {
>             $res = $child;
>diff --git a/Koha/REST/Plugin/Objects.pm b/Koha/REST/Plugin/Objects.pm
>index 0f08a2d37a..7c9ef75555 100644
>--- a/Koha/REST/Plugin/Objects.pm
>+++ b/Koha/REST/Plugin/Objects.pm
>@@ -138,7 +138,7 @@ sub register {
>                 });
>             }
> 
>-            return $objects->to_api({ embed => $embed });
>+            return $objects->to_api({ embed => $embed, av_expand => $c->req->headers->header('x-koha-av') });
>         }
>     );
> }
>-- 
>2.25.0

Actions: View | Diff | Splinter Review

Attachments on bug 26635: 111400 | 111401 | 111402 | 111411 | 111412 | 111413 | 111414 | 112538 | 112539 | 116232 | 116233 | 116234 | 116235 | 116592 | 116593 | 120698 | 120699 | 120700 | 124268 | 142761 | 142762 | 142763 | 142776 | 142792 | 142793 | 142794 | 142795 | 142796 | 142801 | 142802 | 142803 | 142804 | 142805 | 143039 | 143040 | 143041 | 143042 | 143043 | 143044 | 143073 | 143074 | 143075 | 143076 | 143077 | 143078 | 143079 | 143080 | 143127 | 143128 | 143129 | 143130 | 143131 | 143132 | 143133 | 143134 | 143276 | 143354 | 143355 | 143356 | 143357 | 143358 | 143359 | 143360 | 143361 | 143362