Attachment 16644 Details for Bug 9673 – Bug 9673 - Track when items are marked as lost or withdrawn

[patch] Bug 9673 - Track when items are marked as lost or withdrawn

Bug-9673---Track-when-items-are-marked-as-lost-or-.patch (text/plain), 5.90 KB, created by Kyle M Hall (khall) on 2013-03-21 14:19:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-03-21 14:19:53 UTC

Size: 5.90 KB

patch

obsolete

>From 1e891ec41d910c29e7d749e762416e9f7f720162 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 20 Feb 2013 09:34:38 -0500
>Subject: [PATCH] Bug 9673 - Track when items are marked as lost or withdrawn
>
>Add date fields to track when an item was marked as lost or withdrawn.
>Display those fields on catalogue/moredetail.pl
>
>Test Plan:
>1) Apply patch
>2) Run updatedatabase.pl
>3) Pick a record with items, browse to the 'items' tab ( moredetail.pl )
>4) Mark an item as lost, verify the field "Lost on:" displays below
>   the "Lost status" field with todays date.
>5) Mark the item as not lost, verify the field no longer displays
>6) Repeat steps 4 and 5 with the Withdrawn field.
>
>Signed-off-by: Mathieu Saby <mathieu.saby@univ-rennes2.fr>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> C4/Items.pm                                        |   10 ++++++++++
> installer/data/mysql/kohastructure.sql             |    4 ++++
> installer/data/mysql/updatedatabase.pl             |   10 ++++++++++
> .../prog/en/modules/catalogue/moredetail.tt        |    2 ++
> 4 files changed, 26 insertions(+), 0 deletions(-)
>
>diff --git a/C4/Items.pm b/C4/Items.pm
>index 6533a68..43ea2c1 100644
>--- a/C4/Items.pm
>+++ b/C4/Items.pm
>@@ -535,6 +535,16 @@ sub ModItem {
> 
>     $item->{onloan} = undef if $item->{itemlost};
> 
>+    for my $field ( qw( itemlost wthdrawn ) ) {
>+        if ( defined( $item->{$field} ) ) {
>+            if ( $item->{$field} ) {
>+                $item->{$field . '_on'} = DateTime::Format::MySQL->format_datetime( DateTime->now() );
>+            } else {
>+                $item->{$field . '_on'} = undef;
>+            }
>+        }
>+    }
>+
>     _set_derived_columns_for_mod($item);
>     _do_column_fixes_for_mod($item);
>     # FIXME add checks
>diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index c74c39b..ecd5957 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -760,7 +760,9 @@ CREATE TABLE `deleteditems` (
>   `notforloan` tinyint(1) NOT NULL default 0, -- authorized value defining why this item is not for loan (MARC21 952$7)
>   `damaged` tinyint(1) NOT NULL default 0, -- authorized value defining this item as damaged (MARC21 952$4)
>   `itemlost` tinyint(1) NOT NULL default 0, -- authorized value defining this item as lost (MARC21 952$1)
>+  `itemlost_on` datetime DEFAULT NULL, -- the date and time an item was last marked as lost, NULL if not lost
>   `wthdrawn` tinyint(1) NOT NULL default 0, -- authorized value defining this item as withdrawn (MARC21 952$0)
>+  `wthdrawn_on` datetime DEFAULT NULL, -- the date and time an item was last marked as withdrawn, NULL if not withdrawn
>   `itemcallnumber` varchar(255) default NULL, -- call number for this item (MARC21 952$o)
>   `coded_location_qualifier` varchar(10) default NULL, -- coded location qualifier(MARC21 952$f)
>   `issues` smallint(6) default NULL, -- number of times this item has been checked out
>@@ -1047,7 +1049,9 @@ CREATE TABLE `items` ( -- holdings/item information
>   `notforloan` tinyint(1) NOT NULL default 0, -- authorized value defining why this item is not for loan (MARC21 952$7)
>   `damaged` tinyint(1) NOT NULL default 0, -- authorized value defining this item as damaged (MARC21 952$4)
>   `itemlost` tinyint(1) NOT NULL default 0, -- authorized value defining this item as lost (MARC21 952$1)
>+  `itemlost_on` datetime DEFAULT NULL, -- the date and time an item was last marked as lost, NULL if not lost
>   `wthdrawn` tinyint(1) NOT NULL default 0, -- authorized value defining this item as withdrawn (MARC21 952$0)
>+  `wthdrawn_on` datetime DEFAULT NULL, -- the date and time an item was last marked as withdrawn, NULL if not withdrawn
>   `itemcallnumber` varchar(255) default NULL, -- call number for this item (MARC21 952$o)
>   `coded_location_qualifier` varchar(10) default NULL, -- coded location qualifier(MARC21 952$f)
>   `issues` smallint(6) default NULL, -- number of times this item has been checked out/issued
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 222428d..3e68bc1 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -6655,6 +6655,16 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) {
>     SetVersion($DBversion);
> }
> 
>+$DBversion = "3.11.00.XXX";
>+if ( CheckVersion($DBversion) ) {
>+   $dbh->do("ALTER TABLE items ADD itemlost_on DATETIME NULL AFTER itemlost");
>+   $dbh->do("ALTER TABLE items ADD wthdrawn_on DATETIME NULL AFTER wthdrawn");
>+   $dbh->do("ALTER TABLE deleteditems ADD itemlost_on DATETIME NULL AFTER itemlost");
>+   $dbh->do("ALTER TABLE deleteditems ADD wthdrawn_on DATETIME NULL AFTER wthdrawn");
>+   print "Upgrade to $DBversion done (Bug 9673 - Track when items are marked as lost or withdrawn)\n";
>+   SetVersion ($DBversion);
>+}
>+
> =head1 FUNCTIONS
> 
> =head2 TableExists($table)
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt
>index 5234e28..cbec780 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt
>@@ -116,6 +116,7 @@
>                         &nbsp;
>                     [% END %]
>                 </li>
>+                [% IF ITEM_DAT.itemlost_on %]<li><span class="label">Lost on:</span>[% ITEM_DAT.itemlost_on | $KohaDates %] &nbsp;</li>[% END %]
>             [% END %]
>             [% IF ( ITEM_DAT.itemdamagedloop ) %]
>             <li><span class="label">Damaged status:</span>
>@@ -165,6 +166,7 @@
>                     </form>
>                 [% END %]
>             </li>
>+            [% IF ITEM_DAT.wthdrawn_on %]<li><span class="label">Withdrawn on:</span>[% ITEM_DAT.wthdrawn_on | $KohaDates %] &nbsp;</li>[% END %]
>             </ol></div>
>             <div class="listgroup"><h4>History</h4>
>             <ol class="bibliodetails">
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 9673: 15567 | 15890 | 16002 | 16003 | 16032 | 16033 | 16642 | 16643 | 16644 | 16645 | 16646 | 18031 | 18032 | 18033 | 18034 | 19013 | 19014 | 19079 | 19080 | 19120 | 19121 | 19483 | 19484 | 19485 | 19486 | 19487 | 19488 | 19489 | 19490 | 19492 | 21730 | 21731 | 22690 | 22691