Bug 18621: Added in value attribute to dateexpiry field
Test plan: 1. Create a patron category with the dateexpiry value of 29/9/2017 2. Create a patron user from that patron category (which I'll refer to as patron A) with the date expiry value of 1/10/2017 and submit the form 3. Notice that the manual dateexpiry you have submitted is correctly displayed 4. Create a duplicate patron with the same firstname and surname and patron A, and set the date expiry value of 1/10/2017 and submit the form 5. The form displays a duplicate patron message. Notice that the dateexpiry input box is empty now 6. Select the new member (not a duplicate member) option in the messagebox 7. The form successfully submits and notice that the date expiry value displayed is that of the patron category (i.e. it is 29/9/2017) not the dateexpiry value of 1/10/2017 that you manually set for this patron 8. Apply patch 9. Repeat step 4 10. The form displays a duplicate patron message. Notice the dateexpiry input box still contains the value you entered which is 1/10/2017. Select the new member (not a duplicate member) option in the messagebox 11. The form successfully submits and notice that the date expiry value displayed is 1/10/2017 that you manually set for this patron Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> (cherry picked from commit515e629928) Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> (cherry picked from commit60b6d736ee) Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
This commit is contained in:
parent
7d448f6684
commit
255cf73ae9
|
|
@ -757,7 +757,7 @@ $(document).ready(function() {
|
|||
[% UNLESS ( opadd ) %]
|
||||
<input type="text" id="to" name="dateexpiry" maxlength="10" size="10" value="[% dateexpiry UNLESS opduplicate %]" class="datepickerto" />
|
||||
[% ELSE %]
|
||||
<input type="text" id="to" name="dateexpiry" maxlength="10" size="10" class="datepickerto" />
|
||||
<input type="text" id="to" name="dateexpiry" maxlength="10" size="10" value="[% dateexpiry %]" class="datepickerto" />
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% IF ( mandatorydateexpiry ) %]<span class="required">Required</span>[% END %]
|
||||
|
|
|
|||
Loading…
Reference in New Issue