Attachment 165437 Details for Bug 35906 – Bug 35906: (follow-up) rebased it

[patch] Bug 35906: (follow-up) rebased it

Bug-35906-follow-up-rebased-it.patch (text/plain), 1.37 KB, created by ByWater Sandboxes on 2024-04-23 18:48:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ByWater Sandboxes

Created: 2024-04-23 18:48:32 UTC

Size: 1.37 KB

patch

obsolete

>From 0e8fd6421f0ea5d05f7f368e3eeaa58ae13526d8 Mon Sep 17 00:00:00 2001
>From: Thibaud Guillot <thibaud.guillot@biblibre.com>
>Date: Wed, 10 Apr 2024 10:21:54 +0200
>Subject: [PATCH] Bug 35906: (follow-up) rebased it
>
>Signed-off-by: Esther <esther@bywatersolutions.com>
>---
> Koha/Item.pm | 18 ------------------
> 1 file changed, 18 deletions(-)
>
>diff --git a/Koha/Item.pm b/Koha/Item.pm
>index 12f6908e4b..1991548882 100644
>--- a/Koha/Item.pm
>+++ b/Koha/Item.pm
>@@ -1812,24 +1812,6 @@ sub itemtype {
>     return Koha::ItemTypes->find( $self->effective_itemtype );
> }
> 
>-=head3 item_type
>-
>-    my $item_type = $item->item_type;
>-
>-Returns the effective I<Koha::ItemType> for the item.
>-
>-FIXME: it should either return the 'real item type' or undef if no item type
>-defined. And effective_itemtype should return... the effective itemtype. Right
>-now it returns an id... This is all inconsistent. And the API should make it clear
>-if the attribute is part of the resource, or a calculated value i.e. if the item
>-is not linked to an item type on its own, then the API response should contain
>-item_type: null! And the effective item type... be another attribute. I understand
>-that this complicates filtering, but some query trickery could do it in the controller.
>-
>-=cut
>-
>-sub item_type {
>-    return shift->itemtype;
> =head3 effective_bookable
> 
>   my $bookable = $item->effective_bookable;
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 35906: 161397 | 161638 | 161744 | 161961 | 162242 | 162365 | 162465 | 162466 | 162467 | 162468 | 162469 | 162470 | 162472 | 163291 | 163292 | 163293 | 163294 | 163295 | 163547 | 163548 | 163549 | 163550 | 163551 | 164579 | 164580 | 164581 | 164582 | 164583 | 164584 | 164585 | 165437 | 166866 | 166867 | 166868 | 166869 | 166870 | 166871 | 168120 | 168121 | 168122 | 168160 | 168161 | 168162 | 168163 | 168164 | 170187 | 170188 | 170189 | 170190 | 170191 | 170582 | 170583 | 170584 | 170585 | 170586 | 172127 | 172128 | 172129 | 172130 | 172131 | 172132 | 172941 | 172945 | 172950 | 172951 | 172952 | 172953 | 172954 | 172955 | 172956 | 172957 | 172958 | 172959 | 172960 | 172961 | 173324 | 173325 | 173326 | 173327 | 173328 | 173329 | 173330 | 173331 | 173332 | 173333 | 173334 | 173335 | 173893