- Adding title, subtitle, and author to output
- Reworking display of shelving location selection
Patch does not address the contents of 'overdue status' and 'notified by'
---
C4/Overdues.pm | 2 +
circ/branchoverdues.pl | 9 ++++++-
.../prog/en/modules/circ/branchoverdues.tmpl | 27 +++++--------------
3 files changed, 17 insertions(+), 21 deletions(-)
diff --git a/C4/Overdues.pm b/C4/Overdues.pm
index 032ca93..adba979 100644
--- a/C4/Overdues.pm
+++ b/C4/Overdues.pm
@@ -1176,12 +1176,14 @@ sub GetOverduesForBranch {
borrowers.phone,
borrowers.email,
biblio.title,
+ biblio.author,
biblio.biblionumber,
issues.date_due,
issues.returndate,
issues.branchcode,
branches.branchname,
items.barcode,
+ items.homebranch,
items.itemcallnumber,
items.location,
items.itemnumber,
diff --git a/circ/branchoverdues.pl b/circ/branchoverdues.pl
index 905af94..5c1dcaf 100755
--- a/circ/branchoverdues.pl
+++ b/circ/branchoverdues.pl
@@ -27,6 +27,7 @@ use C4::Overdues; # AddNotifyLine
use C4::Biblio;
use C4::Koha;
use C4::Debug;
+use C4::Branch;
=head1 branchoverdues.pl
@@ -103,21 +104,27 @@ my ($tag,$subfield) =
GetMarcFromKohaField('items.location','');
my $tagslib = &GetMarcStructure(1,'');
if ($tagslib->{$tag}->{$subfield}->{authorised_value}) {
my $values=
GetAuthorisedValues($tagslib->{$tag}->{$subfield}->{authorised_value});
+ for (@$values) { $_->{selected} = 1 if $location eq $_->{authorised_value}
}
$template->param(locationsloop => $values);
}
# now display infos
foreach my $num (@getoverdues) {
-
my %overdueforbranch;
+ my $record = GetMarcBiblio($num->{biblionumber});
+ if ($record){
+ $overdueforbranch{'subtitle'} =
GetRecordValue('subtitle',$record,'')->[0]->{subfield};
+ }
$overdueforbranch{'date_due'} = format_date( $num->{'date_due'} );
$overdueforbranch{'title'} = $num->{'title'};
$overdueforbranch{'description'} = $num->{'description'};
$overdueforbranch{'barcode'} = $num->{'barcode'};
$overdueforbranch{'biblionumber'} = $num->{'biblionumber'};
+ $overdueforbranch{'author'} = $num->{'author'};
$overdueforbranch{'borrowersurname'} = $num->{'surname'};
$overdueforbranch{'borrowerfirstname'} = $num->{'firstname'};
$overdueforbranch{'borrowerphone'} = $num->{'phone'};
$overdueforbranch{'borroweremail'} = $num->{'email'};
+ $overdueforbranch{'homebranch'} =
GetBranchName($num->{'homebranch'});
$overdueforbranch{'itemcallnumber'} = $num->{'itemcallnumber'};
$overdueforbranch{'borrowernumber'} = $num->{'borrowernumber'};
$overdueforbranch{'itemnumber'} = $num->{'itemnumber'};
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tmpl
b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tmpl
index 8458208..a4ee6d2 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tmpl
@@ -17,29 +17,16 @@
<h1>Circulation: Overdues at <!-- TMPL_VAR NAME="LoginBranchname" --></h1>
-<!-- selector of location -->
-<!-- TMPL_IF NAME="location" -->
-<h4>Location selected</h4>
-<form name="changelocation" action="branchoverdues.pl">
- <ul>
- <li>
- You are working actually on the overdues for the location :
<b><!-- TMPL_VAR NAME="location" --></b>
- <input type="submit" name="changelocation" class="submit" value="change
location" />
- </li>
- </ul>
-</form>
-<!-- TMPL_ELSE -->
<form name="selectlocation" action="branchoverdues.pl" method="post">
- <label for="location">Select a location: </label><select id="location"
name="location">
- <option value="">All locations</option>
+ <label for="location">Shelving location selected: </label><select
id="location" name="location">
+ <option value="">All shelving locations</option>
<!-- TMPL_LOOP NAME="locationsloop" -->
- <option value="<!-- TMPL_VAR NAME="authorised_value" -->"><!--
TMPL_VAR NAME="lib" --></option>
+ <!-- TMPL_IF NAME="selected" --><option value="<!-- TMPL_VAR
NAME="authorised_value" -->" selected="selected"><!-- TMPL_VAR NAME="lib"
--></option><!-- TMPL_ELSE --><option value="<!-- TMPL_VAR
NAME="authorised_value" -->"><!-- TMPL_VAR NAME="lib" --></option><!-- /TMPL_IF
-->
<!-- /TMPL_LOOP -->
</select>
<input type="submit" name="location" class="submit" value="OK" />
</form>
-<!-- /TMPL_IF -->
<!-- TMPL_IF name="overduesloop" -->
<table style="width:100%;">
<tr>
@@ -56,12 +43,12 @@
<!-- TMPL_VAR NAME="date_due" -->
</td>
<td>
- <!-- TMPL_INCLUDE NAME="biblio-default-view.inc" -->
- (<b><!-- TMPL_VAR NAME="description"
--></b>)
+ <!-- TMPL_INCLUDE NAME="biblio-default-view.inc" --><!--
TMPL_VAR NAME="title" escape="html" --> <!-- TMPL_IF NAME="subtitle" --><!--
TMPL_VAR NAME="subtitle" --></a> <!-- /TMPL_IF --> <!-- TMPL_IF NAME="author"
--> by <!-- TMPL_VAR NAME="author" --><!-- /TMPL_IF -->
+
<br />Barcode : <!-- TMPL_VAR NAME="barcode" -->
</td>
<td>
- <a
href="/cgi-bin/koha/members/moremember.pl?borrowernumber=<!--TMPL_VAR
Name="borrowernumber"-->"><!-- TMPL_VAR NAME="borrowersurname" --> <!--
TMPL_VAR NAME="borrowerfirstname" --></a><br /><!-- TMPL_VAR
NAME="borrowerphone" --><br />
+ <a
href="/cgi-bin/koha/members/moremember.pl?borrowernumber=<!--TMPL_VAR
Name="borrowernumber"-->"><!-- TMPL_VAR NAME="borrowersurname" -->, <!--
TMPL_VAR NAME="borrowerfirstname" --></a><br /><!-- TMPL_VAR
NAME="borrowerphone" --><br />
<!-- TMPL_IF NAME="borroweremail" --><a href="mailto:<!--
TMPL_VAR NAME="borroweremail" -->?subject=Overdue: <!-- TMPL_VAR NAME="title"
ESCAPE="html" -->">
<!-- TMPL_VAR NAME="borroweremail" --></a><!--/TMPL_IF-->
</td>
@@ -109,7 +96,7 @@
<!-- /TMPL_LOOP -->
</table>
<!-- TMPL_ELSE -->
- <div class="dialog message">There are no overdues for today.</div>
+ <div class="dialog message">There are no overdues for today<!--
TMPL_IF NAME="location" --> at the selected location<!-- /TMPL_IF -->.</div>
<!-- /TMPL_IF -->
<!-- TMPL_IF name="todayoverduesloop" -->
<div id="branch_odues_today_odues">
--
1.7.3
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-patches
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/