[Koha-bugs] [Bug 7710] multiple holds per title

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7710

--- Comment #43 from Frédérick Capovilla <fcapovilla at live.ca> ---
Comment on attachment 17448
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17448
Bug 7710 - multiple holds per title

Review of attachment 17448:
 --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=7710&attachment=17448)
-----------------------------------------------------------------

Some lines in this patch are needed in bug 9394 so it can pass QA as a
standalone patch.

::: koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ +564,4 @@
>    [% FOREACH reserveloo IN biblioloo.reserveloop %]
>    [% UNLESS ( loop.odd ) %]<tr class="highlight">[% ELSE %]<tr>[% END %]
>          <td>
> +          <input type="hidden" name="reserve_id" value="[% reserveloo.reserve_id %]" />

Needed in Bug 9394.

@@ +584,4 @@
>  
>       [% IF ( CAN_user_reserveforothers_modify_holds_priority ) %]
>          <td style="white-space:nowrap;">
> +            <a title="Move Hold Up" href="request.pl?action=move&where=up&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&reserve_id=[% reserveloo.reserve_id %]&date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +588,4 @@
>              <img src="/intranet-tmpl/[% theme %]/img/go-up.png" border="0" alt="Go up" />
>                  </a>
>  
> +                <a title="Move Hold To Top" href="request.pl?action=move&where=top&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&reserve_id=[% reserveloo.reserve_id %]&date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +592,4 @@
>                      <img src="/intranet-tmpl/[% theme %]/img/go-top.png" border="0" alt="Go top" />
>                  </a>
>  
> +                <a title="Move Hold To Bottom" href="request.pl?action=move&where=bottom&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&reserve_id=[% reserveloo.reserve_id %]&date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +596,4 @@
>                      <img src="/intranet-tmpl/[% theme %]/img/go-bottom.png" border="0" alt="Go bottom" />
>                  </a>
>  
> +                <a title="Move Hold Down" href="request.pl?action=move&where=down&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&reserve_id=[% reserveloo.reserve_id %]&date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +676,4 @@
>  
>      [% IF ( CAN_user_reserveforothers_modify_holds_priority ) %]
>  	<td>
> +                <a title="Toggle Lowest Priority" href="request.pl?action=setLowestPriority&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&reserve_id=[% reserveloo.reserve_id %]&date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +687,4 @@
>      [% END %]
>  
>  	<td>
> +           <a title="Cancel Hold" href="request.pl?action=cancel&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&reserve_id=[% reserveloo.reserve_id %]&date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +695,4 @@
>          [% IF SuspendHoldsIntranet %]
>  	<td>
>  	[% UNLESS ( reserveloo.wait ) %]
> +            <input type="button" value="[% IF ( reserveloo.suspend ) %]Unsuspend[% ELSE %]Suspend[% END %]" onclick="window.location.href='request.pl?action=toggleSuspend&reserve_id=[% reserveloo.reserve_id %]&borrowernumber=[% reserveloo.borrowernumber %]&biblionumber=[% reserveloo.biblionumber %]&date=[% reserveloo.date %]&suspend_until=' + $('#suspend_until_[% reserveloo.reserve_id %]').val()" />

Needed in Bug 9394.

@@ +699,3 @@
>  
>              [% IF AutoResumeSuspendedHolds %]
> +        <label for="suspend_until_[% reserveloo.reserve_id %]">[% IF ( reserveloo.suspend ) %] on [% ELSE %] until [% END %]</label>

Needed in Bug 9394.

@@ +699,4 @@
>  
>              [% IF AutoResumeSuspendedHolds %]
> +        <label for="suspend_until_[% reserveloo.reserve_id %]">[% IF ( reserveloo.suspend ) %] on [% ELSE %] until [% END %]</label>
> +            <input name="suspend_until" id="suspend_until_[% reserveloo.reserve_id %]" size="10" readonly="readonly" value="[% reserveloo.suspend_until | $KohaDates %]" class="datepicker" />

Needed in Bug 9394.

@@ +699,5 @@
>  
>              [% IF AutoResumeSuspendedHolds %]
> +        <label for="suspend_until_[% reserveloo.reserve_id %]">[% IF ( reserveloo.suspend ) %] on [% ELSE %] until [% END %]</label>
> +            <input name="suspend_until" id="suspend_until_[% reserveloo.reserve_id %]" size="10" readonly="readonly" value="[% reserveloo.suspend_until | $KohaDates %]" class="datepicker" />
> +            <a href='#' onclick="document.getElementById('suspend_until_[% reserveloo.reserve_id %]').value='';">Clear Date</a>

Needed in Bug 9394.

-- 
You are receiving this mail because:
You are watching all bug changes.