Bug 11802 - corrects for the patron purchase suggestions page
This patch corrects some errors with the staff client's patron purchase suggestions page (members/purchase-suggestions.pl). To test, apply the patch and make sure the patronimages system preference is enabled. - View the purchase suggestions page for a patron whose record has a patron image attached. The image should appear in the left-hand sidebar. - View the page for a patron who has submitted no purchase suggestions. The message should be styled correctly. - The toolbar on the page should look correct and work correctly. Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
parent
e9903d760e
commit
2453b991b8
2 changed files with 9 additions and 4 deletions
|
@ -33,8 +33,8 @@
|
||||||
<div class="yui-g">
|
<div class="yui-g">
|
||||||
<h2>Purchase suggestions</h2>
|
<h2>Purchase suggestions</h2>
|
||||||
|
|
||||||
<div id="toolbar">
|
<div id="toolbar" class="btn-toolbar">
|
||||||
<a class="btn" id="newsuggestion" href="/cgi-bin/koha/suggestion/suggestion.pl?op=add&suggestedby=[% borrowernumber %]&redirect=purchase_suggestions&borrowernumber=[% borrowernumber %]"><i class="icon-plus"></i> New purchase suggestion</a>
|
<a class="btn btn-small" id="newsuggestion" href="/cgi-bin/koha/suggestion/suggestion.pl?op=add&suggestedby=[% borrowernumber %]&redirect=purchase_suggestions&borrowernumber=[% borrowernumber %]"><i class="icon-plus"></i> New purchase suggestion</a>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
[% IF suggestions %]
|
[% IF suggestions %]
|
||||||
|
@ -55,7 +55,7 @@
|
||||||
<p>
|
<p>
|
||||||
<strong>
|
<strong>
|
||||||
[% IF ( CAN_user_catalogue ) %]
|
[% IF ( CAN_user_catalogue ) %]
|
||||||
<a href="/cgi-bin/koha/suggestion/suggestion.pl?suggestionid=[% s.suggestionid %]&op=show">[% s.title |html %]</a>
|
<a href="/cgi-bin/koha/suggestion/suggestion.pl?suggestionid=[% s.suggestionid %]&op=show">[% s.title |html %]</a>
|
||||||
[% ELSE %]
|
[% ELSE %]
|
||||||
[% s.title |html %]
|
[% s.title |html %]
|
||||||
[% END %]
|
[% END %]
|
||||||
|
@ -94,7 +94,9 @@
|
||||||
</tbody>
|
</tbody>
|
||||||
</table>
|
</table>
|
||||||
[% ELSE %]
|
[% ELSE %]
|
||||||
There is no purchase suggestion for this patron.
|
<div class="dialog message">
|
||||||
|
<p>This patron has not submitted any purchase suggestions</p>
|
||||||
|
</div>
|
||||||
[% END %]
|
[% END %]
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
|
@ -51,6 +51,9 @@ $template->param(
|
||||||
branchname => GetBranchName( $borrower->{'branchcode'} ),
|
branchname => GetBranchName( $borrower->{'branchcode'} ),
|
||||||
);
|
);
|
||||||
|
|
||||||
|
my ($picture, $dberror) = GetPatronImage($borrowernumber);
|
||||||
|
$template->param( picture => 1 ) if $picture;
|
||||||
|
|
||||||
my $suggestions = SearchSuggestion( { suggestedby => $borrowernumber } );
|
my $suggestions = SearchSuggestion( { suggestedby => $borrowernumber } );
|
||||||
|
|
||||||
$template->param( suggestions => $suggestions );
|
$template->param( suggestions => $suggestions );
|
||||||
|
|
Loading…
Reference in a new issue