Attachment 124669 Details for Bug 6796 – Bug 6796: Add branch_hours table and set opening hours for library

[patch] Bug 6796: Add branch_hours table and set opening hours for library

Bug-6796-Add-branchhours-table-and-set-opening-hou.patch (text/plain), 12.56 KB, created by Aleisha Amohia on 2021-09-09 03:51:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2021-09-09 03:51:15 UTC

Size: 12.56 KB

patch

obsolete

>From 405ca9f1dd66170350223ab472eb8d1a67787c97 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Tue, 7 Sep 2021 15:22:25 +1200
>Subject: [PATCH] Bug 6796: Add branch_hours table and set opening hours for
> library
>
>---
> Koha/Library/Hour.pm                               | 44 +++++++++++++
> Koha/Library/Hours.pm                              | 53 ++++++++++++++++
> admin/branches.pl                                  | 35 ++++++++++-
> .../bug_6796_-_add_branch_hours_table.perl         | 20 ++++++
> installer/data/mysql/kohastructure.sql             | 16 +++++
> .../prog/en/modules/admin/branches.tt              | 73 ++++++++++++++++++++++
> 6 files changed, 240 insertions(+), 1 deletion(-)
> create mode 100644 Koha/Library/Hour.pm
> create mode 100644 Koha/Library/Hours.pm
> create mode 100644 installer/data/mysql/atomicupdate/bug_6796_-_add_branch_hours_table.perl
>
>diff --git a/Koha/Library/Hour.pm b/Koha/Library/Hour.pm
>new file mode 100644
>index 0000000000..f2565bc769
>--- /dev/null
>+++ b/Koha/Library/Hour.pm
>@@ -0,0 +1,44 @@
>+package Koha::Library::Hour;
>+
>+# Copyright 2021 Aleisha Amohia <aleisha@catalyst.net.nz>
>+#
>+# This file is part of Koha.
>+#
>+# Koha is free software; you can redistribute it and/or modify it under the
>+# terms of the GNU General Public License as published by the Free Software
>+# Foundation; either version 3 of the License, or (at your option) any later
>+# version.
>+#
>+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
>+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
>+# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
>+#
>+# You should have received a copy of the GNU General Public License
>+# along with Koha; if not, see <http://www.gnu.org/licenses>.
>+
>+use Modern::Perl;
>+
>+use Koha::Database;
>+
>+use base qw(Koha::Object);
>+
>+=head1 NAME
>+
>+Koha::Library::Hour - Koha Library Hour Object class
>+
>+=head1 API
>+
>+=head2 Class Methods
>+
>+=cut
>+
>+=head3 _type
>+
>+=cut
>+
>+sub _type {
>+    return 'BranchHour';
>+}
>+
>+1;
>+
>diff --git a/Koha/Library/Hours.pm b/Koha/Library/Hours.pm
>new file mode 100644
>index 0000000000..4e97eff01c
>--- /dev/null
>+++ b/Koha/Library/Hours.pm
>@@ -0,0 +1,53 @@
>+package Koha::Library::Hours;
>+
>+# Copyright 2021 Aleisha Amohia <aleisha@catalyst.net.nz>
>+#
>+# This file is part of Koha.
>+#
>+# Koha is free software; you can redistribute it and/or modify it under the
>+# terms of the GNU General Public License as published by the Free Software
>+# Foundation; either version 3 of the License, or (at your option) any later
>+# version.
>+#
>+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
>+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
>+# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
>+#
>+# You should have received a copy of the GNU General Public License
>+# along with Koha; if not, see <http://www.gnu.org/licenses>.
>+
>+use Modern::Perl;
>+
>+use Koha::Database;
>+use Koha::Library::Hour;
>+
>+use base qw(Koha::Objects);
>+
>+=head1 NAME
>+
>+Koha::Library::Hours - Koha Library Hours Object set class
>+
>+=head1 API
>+
>+=head2 Class Methods
>+
>+=cut
>+
>+=head3 _type
>+
>+=cut
>+
>+sub _type {
>+    return 'BranchHour';
>+}
>+
>+=head3 object_class
>+
>+=cut
>+
>+sub object_class {
>+    return 'Koha::Library::Hour';
>+}
>+
>+1;
>+
>diff --git a/admin/branches.pl b/admin/branches.pl
>index 66b69868f0..496cb57b1f 100755
>--- a/admin/branches.pl
>+++ b/admin/branches.pl
>@@ -33,6 +33,7 @@ use Koha::Patrons;
> use Koha::Items;
> use Koha::Libraries;
> use Koha::SMTP::Servers;
>+use Koha::Library::Hours;
> 
> my $input        = CGI->new;
> my $branchcode   = $input->param('branchcode');
>@@ -56,10 +57,12 @@ if ( $op eq 'add_form' ) {
>     }
> 
>     my @smtp_servers = Koha::SMTP::Servers->search;
>+    my @opening_hours = Koha::Library::Hours->search({ branchcode => $branchcode }, { order_by => { -asc => 'day' } });
> 
>     $template->param(
>         library      => $library,
>-        smtp_servers => \@smtp_servers
>+        smtp_servers => \@smtp_servers,
>+        opening_hours => \@opening_hours
>     );
> } elsif ( $op eq 'add_validate' ) {
>     my @fields = qw(
>@@ -117,6 +120,21 @@ if ( $op eq 'add_form' ) {
>                         }
>                     }
> 
>+                    my @days = $input->multi_param("day");
>+                    my @open_times = $input->multi_param("open_time");
>+                    my @close_times = $input->multi_param("close_time");
>+
>+                    foreach my $day ( @days ) {
>+                        if ( $open_times[$day] and $open_times[$day] eq '' ) {
>+                            $open_times[$day] = undef;
>+                        }
>+                        if ( $close_times[$day] and $close_times[$day] eq '' ) {
>+                            $close_times[$day] = undef;
>+                        }
>+
>+                        my $openday = Koha::Library::Hours->find({ branchcode => $branchcode, day => $day })->update({ open_time => $open_times[$day], close_time => $close_times[$day] });
>+                    }
>+
>                     push @messages, { type => 'message', code => 'success_on_update' };
>                 }
>             );
>@@ -155,6 +173,21 @@ if ( $op eq 'add_form' ) {
>                         }
>                     }
> 
>+                    my @days = $input->multi_param("day");
>+                    my @open_times = $input->multi_param("open_time");
>+                    my @close_times = $input->multi_param("close_time");
>+
>+                    foreach my $day ( @days ) {
>+                        if ( $open_times[$day] and $open_times[$day] eq '' ) {
>+                            $open_times[$day] = undef;
>+                        }
>+                        if ( $close_times[$day] and $close_times[$day] eq '' ) {
>+                            $close_times[$day] = undef;
>+                        }
>+
>+                        my $openday = Koha::Library::Hour->new({ branchcode => $branchcode, day => $day, open_time => $open_times[$day], close_time => $close_times[$day] })->store;
>+                    }
>+
>                     push @messages, { type => 'message', code => 'success_on_insert' };
>                 }
>             );
>diff --git a/installer/data/mysql/atomicupdate/bug_6796_-_add_branch_hours_table.perl b/installer/data/mysql/atomicupdate/bug_6796_-_add_branch_hours_table.perl
>new file mode 100644
>index 0000000000..964da04c61
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug_6796_-_add_branch_hours_table.perl
>@@ -0,0 +1,20 @@
>+$DBversion = 'XXX';
>+if ( CheckVersion( $DBversion ) ) {
>+    $dbh->do(q{ DROP TABLE IF EXISTS branch_hours });
>+    $dbh->do(q{
>+        CREATE TABLE branch_hours (
>+            branchcode varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL,
>+            day int(1) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT 0,
>+            open_time time COLLATE utf8mb4_unicode_ci DEFAULT NULL,
>+            close_time time COLLATE utf8mb4_unicode_ci DEFAULT NULL,
>+            PRIMARY KEY (branchcode, day),
>+            CONSTRAINT branch_hours_ibfk_1 FOREIGN KEY (branchcode) REFERENCES branches (branchcode) ON DELETE CASCADE ON UPDATE CASCADE
>+        ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;
>+    });
>+
>+    my @indexes = (0..6);
>+    for my $i (@indexes) {
>+        $dbh->do(q{ INSERT INTO branch_hours (branchcode, day) SELECT branchcode, ? FROM branches }, undef, $i);
>+    }
>+    NewVersion( $DBversion, 6796, "Add branch_hours table" );
>+}
>diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index 02cf864ddd..9ad2e79e29 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -1488,6 +1488,22 @@ CREATE TABLE `branches` (
> /*!40101 SET character_set_client = @saved_cs_client */;
> 
> --
>+-- Table structure for table `branch_hours`
>+--
>+DROP TABLE IF EXISTS branch_hours;
>+/*!40101 SET @saved_cs_client     = @@character_set_client */;
>+/*!40101 SET character_set_client = utf8 */;
>+CREATE TABLE branch_hours (
>+    branchcode varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL,
>+    day int(1) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT 0,
>+    open_time time COLLATE utf8mb4_unicode_ci DEFAULT NULL,
>+    close_time time COLLATE utf8mb4_unicode_ci DEFAULT NULL,
>+    PRIMARY KEY (branchcode, day),
>+    CONSTRAINT branch_hours_ibfk_1 FOREIGN KEY (branchcode) REFERENCES branches (branchcode) ON DELETE CASCADE ON UPDATE CASCADE
>+) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;
>+/*!40101 SET character_set_client = @saved_cs_client */;
>+
>+--
> -- Table structure for table `branches_overdrive`
> --
> 
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
>index 01b2c0f455..a8a67e1490 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
>@@ -265,6 +265,61 @@ Libraries &rsaquo; Administration &rsaquo; Koha
>                         [% END %]
>                     </select>
>                 </li>
>+                <li>
>+                    <label for="opening_hours">Opening hours: </label>
>+                    <table id="opening_hours_table">
>+                        <thead>
>+                            <tr>
>+                                <th>Day</th>
>+                                <th>Open time</th>
>+                                <th>Close time</th>
>+                            </tr>
>+                        </thead>
>+                        <tbody>
>+                            [% IF opening_hours # Existing library %]
>+                                [% daycount = 0 %]
>+                                [% FOREACH hr IN opening_hours %]
>+                                <tr id="hours_[% daycount %]">
>+                                    <td>
>+                                        [% PROCESS dayname day=daycount %]
>+                                        <input type="hidden" value="[% daycount %]" name="day">
>+                                    </td>
>+                                    <td>
>+                                        [% IF hr.day == daycount && hr.open_time %]
>+                                        <input type="time" value="[% hr.open_time %]" name="open_time">
>+                                        [% ELSE %]
>+                                        <input type="time" value="" name="open_time">
>+                                        [% END %]
>+                                    </td>
>+                                    <td>
>+                                        [% IF hr.day == daycount && hr.close_time %]
>+                                        <input type="time" value="[% hr.close_time %]" name="close_time">
>+                                        [% ELSE %]
>+                                        <input type="time" value="" name="close_time">
>+                                        [% END %]
>+                                    </td>
>+                                </tr>
>+                                [% daycount = daycount+1 %]
>+                                [% END %]
>+                            [% ELSE # New library %]
>+                                [% FOREACH daycount IN [0..6] %]
>+                                <tr id="hours_[% day %]">
>+                                    <td>
>+                                        [% PROCESS dayname day=daycount %]
>+                                        <input type="hidden" value="[% daycount %]" name="day">
>+                                    </td>
>+                                    <td>
>+                                        <input type="time" value="" name="open_time">
>+                                    </td>
>+                                    <td>
>+                                        <input type="time" value="" name="close_time">
>+                                    </td>
>+                                </tr>
>+                                [% END %]
>+                            [% END %]
>+                        </tbody>
>+                    </table>
>+                </li>
>             </ol>
>         </fieldset>
>         <fieldset class="action">
>@@ -274,6 +329,24 @@ Libraries &rsaquo; Administration &rsaquo; Koha
>     </form>
> [% END %]
> 
>+[% BLOCK dayname %]
>+    [% IF day == 0 %]
>+    Monday
>+    [% ELSIF day == 1 %]
>+    Tuesday
>+    [% ELSIF day == 2 %]
>+    Wednesday
>+    [% ELSIF day == 3 %]
>+    Thursday
>+    [% ELSIF day == 4 %]
>+    Friday
>+    [% ELSIF day == 5 %]
>+    Saturday
>+    [% ELSE %]
>+    Sunday
>+    [% END %]
>+[% END %]
>+
> [% IF op == 'delete_confirm' and not ( items_count or patrons_count )%]
>     <div class="dialog alert">
>         <form action="/cgi-bin/koha/admin/branches.pl" method="post">
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 6796: 124669 | 124670 | 124671 | 124672 | 134323 | 134324 | 134325 | 134326 | 134327 | 145669 | 145670 | 145671 | 145672 | 145673 | 146988 | 146989 | 146990 | 146991 | 146992 | 150442 | 150443 | 150444 | 150445 | 150446 | 150447 | 150448 | 150477 | 150478 | 153252 | 153253 | 153254 | 153255 | 153256 | 153257 | 153258 | 153259 | 153260 | 153261 | 153271 | 153272 | 153273 | 153274 | 153275 | 153276 | 153277 | 153278 | 153279 | 153280 | 156772 | 156773 | 156774 | 156775 | 156776 | 156777 | 156778 | 156779 | 156780 | 161970 | 161971 | 161972 | 161973 | 161974 | 161975 | 161976 | 161977 | 161978 | 161979 | 161980 | 161986 | 161987 | 161988 | 161989 | 162043 | 162044 | 162045 | 162046 | 162047 | 162048 | 162049 | 162050 | 162051 | 162052 | 162053 | 162054 | 162055 | 162056 | 162090 | 162091 | 162092 | 162093 | 162094 | 162095 | 162096 | 162097 | 162098 | 162099 | 162100 | 162101 | 162102 | 162103 | 162104 | 164511 | 164512 | 164513 | 164514 | 164515 | 164516 | 164517 | 164518 | 164538 | 164539 | 164540 | 164541 | 164542 | 164543 | 164544 | 164545 | 164546 | 164547 | 164548 | 164549 | 164550 | 164551 | 164552