[Koha-bugs] [Bug 22551] Stray "//" appears at bottom of opac-detail.tt

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

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22551

Hayley Mapley <hayleymapley at catalyst.net.nz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hayleymapley at catalyst.net.n
                   |                            |z

--- Comment #5 from Hayley Mapley <hayleymapley at catalyst.net.nz> ---
(In reply to Eric Phetteplace from comment #3)

> In any case, I can re-submit a patch that removes both since that seems to
> be the right thing to do. It's not actually clear to me what's wrong with my
> commit message...I can be more descriptive but I thought I stated the bug
> number/name in the subject and outlined the testing plan already.

It doesn't look like the bug number or description is in the first line of your
patches, it should be. Also the patch should be descriptive, the commit you
attached was:

'patch'

It should be:
'Bug 22551: Removed stray characters from opac-detail.tt

There are stray characters, for the reasons outlined in the bug report. This
patch fixes that.

Test plan:
1) Note the extra characters on the template
2) Apply the patch
3) Verify the characters have disappeared' 

or something similar.

Not a dig, just trying to make what Owen said clearer :) I will test this patch
now

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