Bugzilla – Attachment 146369 Details for
Bug 31212
Datelastseen should be a datetime
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 31212: Change datelastseen from date to datetime field
Bug-31212-Change-datelastseen-from-date-to-datetim.patch (text/plain), 11.08 KB, created by
Lucas Gass
on 2023-02-07 23:17:19 UTC
(
hide
)
Description:
Bug 31212: Change datelastseen from date to datetime field
Filename:
MIME Type:
Creator:
Lucas Gass
Created:
2023-02-07 23:17:19 UTC
Size:
11.08 KB
patch
obsolete
>From eefade3b393b18372736d19e6c54d17fe8f03bb3 Mon Sep 17 00:00:00 2001 >From: Lucas Gass <lucas@bywatersolutions.com> >Date: Tue, 7 Feb 2023 23:14:43 +0000 >Subject: [PATCH] Bug 31212: Change datelastseen from date to datetime field > >--- > C4/Items.pm | 2 ++ > Koha/Schema/Result/Deleteditem.pm | 4 ++-- > Koha/Schema/Result/Item.pm | 4 ++-- > installer/data/mysql/kohastructure.sql | 4 ++-- > koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 2 +- > .../intranet-tmpl/prog/en/modules/catalogue/moredetail.tt | 2 +- > .../intranet-tmpl/prog/en/modules/cataloguing/additem.tt | 6 +++++- > .../intranet-tmpl/prog/en/modules/reports/itemslost.tt | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt | 4 ++-- > 9 files changed, 18 insertions(+), 12 deletions(-) > >diff --git a/C4/Items.pm b/C4/Items.pm >index de776d5dfe5..b96b7a56bcb 100644 >--- a/C4/Items.pm >+++ b/C4/Items.pm >@@ -403,6 +403,7 @@ The last optional parameter allows for passing skip_record_index through to the > > sub ModDateLastSeen { > my ( $itemnumber, $leave_item_lost, $params ) = @_; >+ my $today = output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 0 }); > > my $item = Koha::Items->find($itemnumber); > $item->datelastseen(dt_from_string->ymd); >@@ -587,6 +588,7 @@ sub GetItemsForInventory { > } > > if ($datelastseen) { >+ $datelastseen = output_pref({ str => $datelastseen, dateformat => 'iso', dateonly => 0 }); > push @where_strings, '(datelastseen < ? OR datelastseen IS NULL)'; > push @bind_params, $datelastseen; > } >diff --git a/Koha/Schema/Result/Deleteditem.pm b/Koha/Schema/Result/Deleteditem.pm >index 8ccb4d4499c..436629af5c7 100644 >--- a/Koha/Schema/Result/Deleteditem.pm >+++ b/Koha/Schema/Result/Deleteditem.pm >@@ -112,7 +112,7 @@ the date the item was last checked out > > =head2 datelastseen > >- data_type: 'date' >+ data_type: 'datetime' > datetime_undef_if_invalid: 1 > is_nullable: 1 > >@@ -404,7 +404,7 @@ __PACKAGE__->add_columns( > "datelastborrowed", > { data_type => "date", datetime_undef_if_invalid => 1, is_nullable => 1 }, > "datelastseen", >- { data_type => "date", datetime_undef_if_invalid => 1, is_nullable => 1 }, >+ { data_type => "datetime", datetime_undef_if_invalid => 1, is_nullable => 1 }, > "stack", > { data_type => "tinyint", is_nullable => 1 }, > "notforloan", >diff --git a/Koha/Schema/Result/Item.pm b/Koha/Schema/Result/Item.pm >index 55d33e4a198..6da9361ecd8 100644 >--- a/Koha/Schema/Result/Item.pm >+++ b/Koha/Schema/Result/Item.pm >@@ -115,7 +115,7 @@ the date the item was last checked out/issued > > =head2 datelastseen > >- data_type: 'date' >+ data_type: 'datetime' > datetime_undef_if_invalid: 1 > is_nullable: 1 > >@@ -418,7 +418,7 @@ __PACKAGE__->add_columns( > "datelastborrowed", > { data_type => "date", datetime_undef_if_invalid => 1, is_nullable => 1 }, > "datelastseen", >- { data_type => "date", datetime_undef_if_invalid => 1, is_nullable => 1 }, >+ { data_type => "datetime", datetime_undef_if_invalid => 1, is_nullable => 1 }, > "stack", > { data_type => "tinyint", is_nullable => 1 }, > "notforloan", >diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql >index df66cee1750..9107aa34b45 100644 >--- a/installer/data/mysql/kohastructure.sql >+++ b/installer/data/mysql/kohastructure.sql >@@ -2622,7 +2622,7 @@ CREATE TABLE `deleteditems` ( > `replacementprice` decimal(8,2) DEFAULT NULL COMMENT 'cost the library charges to replace the item if it has been marked lost (MARC21 952$v)', > `replacementpricedate` date DEFAULT NULL COMMENT 'the date the price is effective from (MARC21 952$w)', > `datelastborrowed` date DEFAULT NULL COMMENT 'the date the item was last checked out', >- `datelastseen` date DEFAULT NULL COMMENT 'the date the item was last see (usually the last time the barcode was scanned or inventory was done)', >+ `datelastseen` datetime DEFAULT NULL COMMENT 'the date the item was last see (usually the last time the barcode was scanned or inventory was done)', > `stack` tinyint(1) DEFAULT NULL, > `notforloan` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'authorized value defining why this item is not for loan (MARC21 952$7)', > `damaged` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'authorized value defining this item as damaged (MARC21 952$4)', >@@ -3590,7 +3590,7 @@ CREATE TABLE `items` ( > `replacementprice` decimal(8,2) DEFAULT NULL COMMENT 'cost the library charges to replace the item if it has been marked lost (MARC21 952$v)', > `replacementpricedate` date DEFAULT NULL COMMENT 'the date the price is effective from (MARC21 952$w)', > `datelastborrowed` date DEFAULT NULL COMMENT 'the date the item was last checked out/issued', >- `datelastseen` date DEFAULT NULL COMMENT 'the date the item was last see (usually the last time the barcode was scanned or inventory was done)', >+ `datelastseen` datetime DEFAULT NULL COMMENT 'the date the item was last see (usually the last time the barcode was scanned or inventory was done)', > `stack` tinyint(1) DEFAULT NULL, > `notforloan` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'authorized value defining why this item is not for loan (MARC21 952$7)', > `damaged` tinyint(1) NOT NULL DEFAULT 0 COMMENT 'authorized value defining this item as damaged (MARC21 952$4)', >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >index 58ca8e1a703..af87f0972f4 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >@@ -567,7 +567,7 @@ Note that permanent location is a code, and location may be an authval. > [% END %] > > </td> >- <td class="datelastseen" data-order="[% item.datelastseen | html %]">[% item.datelastseen | $KohaDates %]</td> >+ <td class="datelastseen" data-order="[% item.datelastseen | html %]">[% item.datelastseen | $KohaDates with_hours => 1 %]</td> > <td class="issues" data-order="[% item.issues || 0 | html %]">[% item.issues || 0 | html %]</td> > <td class="renewals" data-order="[% item.renewals || 0 | html %]">[% item.renewals || 0 | html %]</td> > <td class="dateaccessioned" data-order="[% item.dateaccessioned | html %]">[% item.dateaccessioned | $KohaDates %]</td> >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 5ba3e48b08c..1abd28a7b8d 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt >@@ -393,7 +393,7 @@ > <li> > <span class="label">Last seen:</span> > [% IF ( ITEM_DAT.datelastseen ) %] >- [% ITEM_DAT.datelastseen | $KohaDates %] >+ [% ITEM_DAT.datelastseen | $KohaDates with_hours => 1 %] > [%END %] > </li> > <li> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt >index 38fdd8995a3..728373b0419 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt >@@ -135,7 +135,11 @@ > [% SET attribute = header.attribute %] > [% SET can_mod = item.nomod ? "nomod" : "canmod" %] > [% IF header.attribute AND date_fields.grep('^' _ attribute _ '$').size %] >- <td class="[% can_mod | html %]" data-order="[% item.$attribute | html %]">[% item.$attribute | $KohaDates %]</td> >+ [% IF header.attribute == 'datelastseen' %] >+ <td class="[% can_mod | html %]" data-order="[% item.$attribute | html %]">[% item.$attribute | $KohaDates with_hours => 1 %]</td> >+ [% ELSE %] >+ <td class="[% can_mod | html %]" data-order="[% item.$attribute | html %]">[% item.$attribute | $KohaDates %]</td> >+ [% END %] > [% ELSIF ( item.$attribute && ( attribute == 'price' || attribute == 'replacementprice' ) ) %] > <td class="[% can_mod | html %]" data-order="[% item.$attribute | html %]">[% item.$attribute | $Price %]</td> > [% ELSE %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/itemslost.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/itemslost.tt >index 7251adb3976..2d01b8e5373 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/itemslost.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/itemslost.tt >@@ -129,7 +129,7 @@ > <a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% item.biblionumber | uri %]">[% item.barcode | html %]</a> > </td> > <td>[% item.itemcallnumber | html %]</td> >- <td data-order="[% item.datelastseen | html %]">[% item.datelastseen | $KohaDates %]</td> >+ <td data-order="[% item.datelastseen | html %]">[% item.datelastseen | $KohaDates with_hours => 1 %]</td> > <td>[% item.price | $Price %]</td> > <td>[% item.replacementprice | $Price %]</td> > <td>[% Branches.GetName(item.homebranch) | html %]</td> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt >index 1571f6a3ee7..40557b4d95b 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt >@@ -50,7 +50,7 @@ > <main> > > <h1>Inventory</h1> >- [% IF (moddatecount) %]<div class="dialog message">[% moddatecount | html %] items modified : datelastseen set to [% date | $KohaDates %]</div> >+ [% IF (moddatecount) %]<div class="dialog message">[% moddatecount | html %] items modified : datelastseen set to [% date | $KohaDates with_hours => 1 %]</div> > <div class="dialog alert">Number of potential barcodes read: [% LinesRead | html %]</div>[% END %] > [% IF (errorfile) %]<div class="dialog alert">[% errorfile | html %] can't be opened</div>[% END %] > [% IF (err_length && err_length==1) %]<div class="dialog alert">There was 1 barcode that was too long.</div>[% END %] >@@ -312,7 +312,7 @@ > [% result.withdrawn | html %] > </td> > <td data-order="[% result.datelastseen | html %]"> >- [% result.datelastseen | $KohaDates%] >+ [% result.datelastseen | $KohaDates with_hours => 1 %] > </td> > <td> > [% FOREACH problem IN result.problems %] >-- >2.30.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 31212
:
138053
|
139510
|
142262
|
142263
|
142264
|
142282
|
142283
|
142284
|
142286
|
142287
|
142288
|
143595
|
146369
|
146370
|
146371
|
146372
|
147664
|
147665
|
147666
|
147667
|
148939
|
150378
|
150379
|
150380
|
150381
|
150382
|
150383
|
151189
|
151190
|
151194