Bug 9376

Summary: Problem with dateexpiry display in moremember.tt
Product: Koha Reporter: Vitor Fernandes <vfernandes>
Component: Staff interfaceAssignee: Vitor Fernandes <vfernandes>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: minor    
Priority: P1 - high CC: chris, gmcharlt, jonathan.druart
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: Sponsored
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on:    
Bug Blocks: 9406    
Attachments: Bug 9376: Problem with dateexpiry display in moremember.tt
Bug 9376: Problem with dateexpiry display in moremember.tt
Bug 9376: extend HTML fix to moremember-brief.tt
Bug 9376: Problem with dateexpiry display in moremember.tt
Bug 9376: extend HTML fix to moremember-brief.tt

Description Vitor Fernandes 2013-01-10 16:12:33 UTC
There is a problem that brokens the interface of moremmeber when displaying the date expiry caused by this line:

<strong class="reregistryinfo">[% dateexpiry %]</span>

It should be:

<span class="reregitryinfo">[% dateexpiry %]</span>

It happens in version 3.8.
In version 3.10, I don't know if the problem happens.
Comment 1 Vitor Fernandes 2013-01-10 16:31:41 UTC Comment hidden (obsolete)
Comment 2 Galen Charlton 2013-01-16 17:08:03 UTC
Confirmed that this affect 3.10.x.  It doesn't affect master, but only because of how the patch for bug 6261 reworks the member toolbar to use member/setstatus.pl rather than members/moremember.pl to renew the patron.
Comment 3 Galen Charlton 2013-01-16 17:13:11 UTC Comment hidden (obsolete)
Comment 4 Galen Charlton 2013-01-16 17:13:15 UTC Comment hidden (obsolete)
Comment 5 Galen Charlton 2013-01-16 17:14:02 UTC
Note that I added a follow-up patch along with my signoff.
Comment 6 Jonathan Druart 2013-01-17 13:32:12 UTC
QA comment:

The patch fixes a malformed strong tag.

Marked as Passed QA.
Comment 7 Jonathan Druart 2013-01-17 13:32:43 UTC
Created attachment 14662 [details] [review]
Bug 9376: Problem with dateexpiry display in moremember.tt

There is a problem that brokens the interface of moremmeber when
displaying the date expiry when renewing a patron.
In the first comment I said that the line that causes the error
should be:

<span class="reregitryinfo">[% dateexpiry %]</span>

but the correct way should be:

<strong class="reregitryinfo">[% dateexpiry %]</strong>

This way the date expiry after renew a patron appears in bold.

Test plan:

- before applying the patch try to renew a patron and check
  that almost everything went bold
- apply the patch
- renew a patron and only the expiration date is bolder.
- Note that the bug doesn't manifest in master because
  the patch for bug 6261 causes the invalid HTML to never
  be reached; it *does* affect 3.10.x and earlier.

Sponsored-by: KEEP SOLUTIONS
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 8 Jonathan Druart 2013-01-17 13:32:52 UTC
Created attachment 14663 [details] [review]
Bug 9376: extend HTML fix to moremember-brief.tt

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 9 Jared Camins-Esakov 2013-01-18 02:08:24 UTC
This patch has been pushed to master.
Comment 10 Chris Cormack 2013-01-24 02:24:56 UTC
Pushed to 3.10.x and 3.8.x will be in 3.10.3 and 3.8.10