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

--- Comment #43 from Frédérick Capovilla <[email protected]> ---
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&amp;where=up&amp;borrowernumber=[% 
> reserveloo.borrowernumber %]&amp;biblionumber=[% reserveloo.biblionumber 
> %]&amp;reserve_id=[% reserveloo.reserve_id %]&amp;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&amp;where=top&amp;borrowernumber=[% 
> reserveloo.borrowernumber %]&amp;biblionumber=[% reserveloo.biblionumber 
> %]&amp;reserve_id=[% reserveloo.reserve_id %]&amp;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&amp;where=bottom&amp;borrowernumber=[% 
> reserveloo.borrowernumber %]&amp;biblionumber=[% reserveloo.biblionumber 
> %]&amp;reserve_id=[% reserveloo.reserve_id %]&amp;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&amp;where=down&amp;borrowernumber=[% 
> reserveloo.borrowernumber %]&amp;biblionumber=[% reserveloo.biblionumber 
> %]&amp;reserve_id=[% reserveloo.reserve_id %]&amp;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&amp;borrowernumber=[% 
> reserveloo.borrowernumber %]&amp;biblionumber=[% reserveloo.biblionumber 
> %]&amp;reserve_id=[% reserveloo.reserve_id %]&amp;date=[% reserveloo.date %]">

Needed in Bug 9394.

@@ +687,4 @@
>      [% END %]
>  
>       <td>
> +           <a title="Cancel Hold" 
> href="request.pl?action=cancel&amp;borrowernumber=[% 
> reserveloo.borrowernumber %]&amp;biblionumber=[% reserveloo.biblionumber 
> %]&amp;reserve_id=[% reserveloo.reserve_id %]&amp;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&amp;reserve_id=[%
>  reserveloo.reserve_id %]&amp;borrowernumber=[% reserveloo.borrowernumber 
> %]&amp;biblionumber=[% reserveloo.biblionumber %]&amp;date=[% reserveloo.date 
> %]&amp;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.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to