[Bug 23463] New: Move C4::Items CRUD subroutines to Koha::Item

classic Classic list List threaded Threaded
235 messages Options
1234 ... 8
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] New: Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

            Bug ID: 23463
           Summary: Move C4::Items CRUD subroutines to Koha::Item
 Change sponsored?: ---
           Product: Koha
           Version: unspecified
          Hardware: All
                OS: All
            Status: ASSIGNED
          Severity: enhancement
          Priority: P5 - low
         Component: Architecture, internals, and plumbing
          Assignee: [hidden email]
          Reporter: [hidden email]
        QA Contact: [hidden email]

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #1 from Jonathan Druart <[hidden email]> ---
This patchset will be quite big and will move the following C4::Items
subroutines to the Koha namespace:

AddItem
ModItem
DelItem
ItemSafeToDelete
DelItemCheck

SearchItemsByField
UpdateHoldingbranch

_koha_new_item
_koha_modify_item
_set_defaults_for_add
_set_derived_columns_for_add
_do_column_fixes_for_mod
_set_derived_columns_for_mod
_build_default_values_for_mod_marc
_get_single_item_column
_calc_items_cn_sort
_mod_item_dates
_after_item_action_hooks

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

Jonathan Druart <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.renvoize@ptfs-europe
                   |                            |.com,
                   |                            |[hidden email],
                   |                            |[hidden email]

--- Comment #2 from Jonathan Druart <[hidden email]> ---
I have pushed a WIP branch to: https://gitlab.com/joubu/Koha/commits/bug_23463

The branch is ready for testing (but still not clean enough to be integrated).

Early feedbacks welcomed!

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

Jonathan Druart <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |18252


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18252
[Bug 18252] Move C4::Items code to the Koha namespace
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #3 from Jonathan Druart <[hidden email]> ---
Remote branch has been rebased.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

Martin Renvoize <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rel_20_05_target

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #4 from Martin Renvoize <[hidden email]> ---
Lets hit this one hard beginning of next cycle :)

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

Jonathan Druart <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |24107


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24107
[Bug 24107] Returns.pl fetches the same item from the database multiple times
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

Jonathan Druart <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |Needs Signoff

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #5 from Jonathan Druart <[hidden email]> ---
Created attachment 95899
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95899&action=edit
Bug 23463: Replace SearchItemsByField by SearchItems

The subroutine SearchItemsByField is only called once and can be
replaced easily with SearchItems

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #6 from Jonathan Druart <[hidden email]> ---
Created attachment 95900
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95900&action=edit
Bug 23463: Remove uneeded SearchItemsByField subroutine

It is no longer used.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #7 from Jonathan Druart <[hidden email]> ---
Created attachment 95901
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95901&action=edit
Bug 23463: fix for testbuilder - set default value items.damaged=0

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #8 from Jonathan Druart <[hidden email]> ---
Created attachment 95902
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95902&action=edit
Bug 23463: Replace AddItem calls with Koha::Item->store

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #9 from Jonathan Druart <[hidden email]> ---
Created attachment 95903
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95903&action=edit
Bug 23463: Remove _set_defaults_for_add

_set_defaults_for_add is no longer needed as it is handled by dbic and
koha::object->store

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #10 from Jonathan Druart <[hidden email]> ---
Created attachment 95904
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95904&action=edit
Bug 23463: Replace _set_derived_columns_for_add

Remove the unique call to _set_derived_columns_for_add
Even if design to deal with other values it only dealts with cn_sort

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #11 from Jonathan Druart <[hidden email]> ---
Created attachment 95905
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95905&action=edit
Bug 23463: AddItem removing

We are done with AddItem, only need to log and index.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #12 from Jonathan Druart <[hidden email]> ---
Created attachment 95906
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95906&action=edit
Bug 23463: Replace ModItem with Koha::Item->store

Starting to replace the ModItem calls with Koha::Item->store

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #13 from Jonathan Druart <[hidden email]> ---
Created attachment 95907
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95907&action=edit
Bug 23463: remove trivial UpdateHoldingbranch sub

We do not need this subroutine, moving it where it is used.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #14 from Jonathan Druart <[hidden email]> ---
Created attachment 95908
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95908&action=edit
Bug 23463: Move plugin hook

From C4::Items to Koha::Item

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #15 from Jonathan Druart <[hidden email]> ---
Created attachment 95909
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95909&action=edit
Bug 23463: _set_derived_columns_for_mod was only used for cn_sort

Same as _set_defaults_for_add

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #16 from Jonathan Druart <[hidden email]> ---
Created attachment 95910
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95910&action=edit
Bug 23463: _do_column_fixes_for_mod dealt with integer location and timestamp
moved

That's done in Koha::Item->store, and Koha::Object->store for integers.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #17 from Jonathan Druart <[hidden email]> ---
Created attachment 95911
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95911&action=edit
Bug 23463: Remove _mod_item_dates

We did not do anything useful here.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #18 from Jonathan Druart <[hidden email]> ---
Created attachment 95912
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95912&action=edit
Bug 23463: Remove ModItem

There should not be any calls left

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #19 from Jonathan Druart <[hidden email]> ---
Created attachment 95913
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95913&action=edit
Bug 23463: Remove DelItem

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #20 from Jonathan Druart <[hidden email]> ---
Created attachment 95914
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95914&action=edit
Bug 23463: Remove DelItemCheck and ItemSafeToDelete

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #21 from Jonathan Druart <[hidden email]> ---
Created attachment 95915
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95915&action=edit
Bug 23463: Remove no longer needed subs related to default values

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #22 from Jonathan Druart <[hidden email]> ---
Created attachment 95916
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95916&action=edit
Bug 23463: Fix CancelReceipt.t

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #23 from Jonathan Druart <[hidden email]> ---
Created attachment 95917
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95917&action=edit
Bug 23463: Fix failing tests

And adding a FIXME about discard_changes

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #24 from Jonathan Druart <[hidden email]> ---
Created attachment 95918
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95918&action=edit
Bug 23463: Getting less item-level_itypes warnings (?)

So, we get less warnings. I do not really understand why...

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|

[Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23463

--- Comment #25 from Jonathan Druart <[hidden email]> ---
Created attachment 95919
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95919&action=edit
Bug 23463: Fix AutomaticItemModificationByAge.t

We do not want to log if nothing changed

How could we do that better?

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
1234 ... 8