From bugzilla-daemon@bugs.koha-community.org Wed Mar 20 17:34:06 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] New: Stray "//" appears at bottom of	opac-detail.tt
Date: Wed, 20 Mar 2019 16:33:36 +0000
Message-ID: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3091897931963119012=="

--===============3091897931963119012==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

            Bug ID: 22551
           Summary: Stray "//" appears at bottom of opac-detail.tt
 Change sponsored?: ---
           Product: Koha
           Version: 18.11
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P5 - low
         Component: OPAC
          Assignee: oleonard@myacpl.org
          Reporter: ephetteplace@cca.edu
        QA Contact: testopia@bugs.koha-community.org
  Target Milestone: ---

Created attachment 86822
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3D86822&action=3D=
edit
screenshot of stray characters

In the "SocialNetworks" JavaScript area of the OPAC display, there's this code
(currently lines 1390-1393 of opac-detail.tt):

//<![CDATA[
{lang: '[% lang | html %]'}
//]]>
</script>

It seems like an opening <script> tag is missing (the closing one is orphaned)
which results in the "//" characters displaying on the page towards the very
bottom (see screenshot). It's not 100% clear to me what this code was intended
to do but I think it was meant to be part of the old Google+ (removed from Ko=
ha
earlier this year) script tag like this:

<script src=3D"https://apis.google.com/js/plusone.js" type=3D"text/javascript=
">
//<![CDATA[
{"lang":"en-US"}
//]]>
</script>

However, right now it's only effect is the stray "//"=E2=80=94the JS code is =
commented
out and even if it wasn't I don't think declaring {"lang": "en-US"} as an
object literal unassigned to any variable has any effect.

I can make a patch for this shortly.

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

--===============3091897931963119012==--


From bugzilla-daemon@bugs.koha-community.org Wed Mar 20 17:34:35 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Wed, 20 Mar 2019 16:34:33 +0000
Message-ID: <bug-22551-70-YihyIC1ooW@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============5710560196773890333=="

--===============5710560196773890333==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Eric Phetteplace <ephetteplace@cca.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|oleonard@myacpl.org         |ephetteplace@cca.edu
   Patch complexity|---                         |Trivial patch
                 CC|                            |ephetteplace@cca.edu

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

--===============5710560196773890333==--


From bugzilla-daemon@bugs.koha-community.org Wed Mar 20 18:05:21 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Wed, 20 Mar 2019 17:05:19 +0000
Message-ID: <bug-22551-70-5BXBVBnwM9@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4327457811172140928=="

--===============4327457811172140928==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

--- Comment #1 from Eric Phetteplace <ephetteplace@cca.edu> ---
Created attachment 86824
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=86824&action=edit
patch

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

--===============4327457811172140928==--


From bugzilla-daemon@bugs.koha-community.org Wed Mar 20 18:05:51 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Wed, 20 Mar 2019 17:05:45 +0000
Message-ID: <bug-22551-70-TBsDjBtKqT@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7226661709514060714=="

--===============7226661709514060714==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Eric Phetteplace <ephetteplace@cca.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |Needs Signoff

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

--===============7226661709514060714==--


From bugzilla-daemon@bugs.koha-community.org Sat Apr  6 17:21:56 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Sat, 06 Apr 2019 15:21:26 +0000
Message-ID: <bug-22551-70-da5fJ3D0Dt@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0038162406543621564=="

--===============0038162406543621564==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Katrin Fischer <katrin.fischer@bsz-bw.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |katrin.fischer@bsz-bw.de
           Severity|enhancement                 |normal

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

--===============0038162406543621564==--


From bugzilla-daemon@bugs.koha-community.org Mon Apr  8 15:07:42 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Mon, 08 Apr 2019 13:07:39 +0000
Message-ID: <bug-22551-70-pQcjxgjg7Y@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4462249794117819729=="

--===============4462249794117819729==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

Owen Leonard <oleonard@myacpl.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #2 from Owen Leonard <oleonard@myacpl.org> ---
I think you're right to identify that section as obsolete, but I don't think
that's what's causing the problem. There is a stray "//]]>" further down in t=
he
template. A "//<![CDATA[" marker was removed without cleaning up the ending
marker.

Please also take a look at our guidelines for commit messages:

https://wiki.koha-community.org/wiki/Commit_messages

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

--===============4462249794117819729==--


From bugzilla-daemon@bugs.koha-community.org Tue Apr  9 00:59:30 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Mon, 08 Apr 2019 22:59:27 +0000
Message-ID: <bug-22551-70-qVbCwtIh1N@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7744695483782988401=="

--===============7744695483782988401==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

--- Comment #3 from Eric Phetteplace <ephetteplace@cca.edu> ---
I do see that other stray "//]]>" around line 1720 of opac-detail.tt but it is
not what's causing the text to display=E2=80=94inspect an opac-detail page in=
 your web
browser, the segment I removed (just above an orphaned closing </script> tag
and the Twitter script) falls outside any <script> tags which is why the
browser renders it as a text node. That second stray "//]]>" is inside a very
long <script> tag with several functions (OpacHighlightedWords,
OPACShelfBrowser,IDreamBooksReviews, etc.) and thus not rendered as text.

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.

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

--===============7744695483782988401==--


From bugzilla-daemon@bugs.koha-community.org Tue Apr  9 01:08:23 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Mon, 08 Apr 2019 23:08:20 +0000
Message-ID: <bug-22551-70-OjX8anXQjy@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2725771308465754285=="

--===============2725771308465754285==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Eric Phetteplace <ephetteplace@cca.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #86824|0                           |1
        is obsolete|                            |

--- Comment #4 from Eric Phetteplace <ephetteplace@cca.edu> ---
Created attachment 87568
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87568&action=edit
patch

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

--===============2725771308465754285==--


From bugzilla-daemon@bugs.koha-community.org Mon Apr 15 23:27:54 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Mon, 15 Apr 2019 21:27:51 +0000
Message-ID: <bug-22551-70-q14AdXWwy4@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8359589531436344174=="

--===============8359589531436344174==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Eric Phetteplace <ephetteplace@cca.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Needs Signoff

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

--===============8359589531436344174==--


From bugzilla-daemon@bugs.koha-community.org Tue Apr 16 05:44:55 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Tue, 16 Apr 2019 03:44:25 +0000
Message-ID: <bug-22551-70-gSPlIqAusI@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2253950419873194806=="

--===============2253950419873194806==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

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

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

--- Comment #5 from Hayley Mapley <hayleymapley@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 yo=
ur
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'=20

or something similar.

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

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

--===============2253950419873194806==--


From bugzilla-daemon@bugs.koha-community.org Tue Apr 16 06:23:28 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Tue, 16 Apr 2019 04:23:25 +0000
Message-ID: <bug-22551-70-e5rsJscLeU@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1422774658003046306=="

--===============1422774658003046306==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

--- Comment #6 from Hayley Mapley <hayleymapley@catalyst.net.nz> ---
(In reply to Hayley Mapley from comment #5)
> (In reply to Eric Phetteplace from comment #3)
>=20
> > 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.
>=20
> 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:
>=20
> 'patch'
>=20
> It should be:
> 'Bug 22551: Removed stray characters from opac-detail.tt
>=20
> There are stray characters, for the reasons outlined in the bug report. This
> patch fixes that.
>=20
> Test plan:
> 1) Note the extra characters on the template
> 2) Apply the patch
> 3) Verify the characters have disappeared'=20
>=20
> or something similar.
>=20
> Not a dig, just trying to make what Owen said clearer :) I will test this
> patch now
(In reply to Eric Phetteplace from comment #3)
> I do see that other stray "//]]>" around line 1720 of opac-detail.tt but it
> is not what's causing the text to display=E2=80=94inspect an opac-detail pa=
ge in
> your web browser, the segment I removed (just above an orphaned closing
> </script> tag and the Twitter script) falls outside any <script> tags which
> is why the browser renders it as a text node. That second stray "//]]>" is
> inside a very long <script> tag with several functions
> (OpacHighlightedWords, OPACShelfBrowser,IDreamBooksReviews, etc.) and thus
> not rendered as text.
>=20
> 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.

(In reply to Hayley Mapley from comment #5)
> (In reply to Eric Phetteplace from comment #3)
>=20
> > 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.
>=20
> 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:
>=20
> 'patch'
>=20
> It should be:
> 'Bug 22551: Removed stray characters from opac-detail.tt
>=20
> There are stray characters, for the reasons outlined in the bug report. This
> patch fixes that.
>=20
> Test plan:
> 1) Note the extra characters on the template
> 2) Apply the patch
> 3) Verify the characters have disappeared'=20
>=20
> or something similar.
>=20
> Not a dig, just trying to make what Owen said clearer :) I will test this
> patch now

Ahh I see. Did you remove the description when you attached the commit? I see
it now that I am amending to your commit that your commit message is perfect.

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

--===============1422774658003046306==--


From bugzilla-daemon@bugs.koha-community.org Tue Apr 16 06:24:28 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Tue, 16 Apr 2019 04:24:25 +0000
Message-ID: <bug-22551-70-S0XVkh0g9K@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3193333073590194916=="

--===============3193333073590194916==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off

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

--===============3193333073590194916==--


From bugzilla-daemon@bugs.koha-community.org Tue Apr 16 06:24:31 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Tue, 16 Apr 2019 04:24:28 +0000
Message-ID: <bug-22551-70-twVr3n9zhK@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2938384139342593989=="

--===============2938384139342593989==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

--- Comment #7 from Hayley Mapley <hayleymapley@catalyst.net.nz> ---
Created attachment 88041
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3D88041&action=3D=
edit
Bug 22551: Stray "//" appears at bottom of opac-detail.tt

Removes a couple stray "//<![CDATA[" like comments which are artifacts of oth=
er
code that since been removed. One of the comments causes two forward slashes =
to
appear at the bottom of each OPAC detail page.

Testing plan:

- enable SocialNetworks
- visit any OPAC detail page, observe "//" at bottom of page
- apply patch
- revisit page, slashes should be gone

Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>

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

--===============2938384139342593989==--


From bugzilla-daemon@bugs.koha-community.org Wed Apr 17 00:25:33 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Tue, 16 Apr 2019 22:25:30 +0000
Message-ID: <bug-22551-70-rZ0BX7we6b@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============7409568395543913236=="

--===============7409568395543913236==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

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

--- Comment #8 from Eric Phetteplace <ephetteplace@cca.edu> ---
Thank you, Hayley, that helps me understand what I did wrong. You're exactly
right—I didn't enter a description or "comment" when I selected "add an
attachment" in bugzilla because I thought the patch file itself was the only
thing that mattered; my patch looks the same as yours it's just I didn't fill
out the Bugzilla form correctly. I'll do that next time!

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

--===============7409568395543913236==--


From bugzilla-daemon@bugs.koha-community.org Wed Apr 17 12:45:03 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Wed, 17 Apr 2019 10:45:01 +0000
Message-ID: <bug-22551-70-yY2c1tQIN0@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0505027614620917401=="

--===============0505027614620917401==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Martin Renvoize <martin.renvoize@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.renvoize@ptfs-europe
                   |                            |.com
  Attachment #87568|0                           |1
        is obsolete|                            |

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

--===============0505027614620917401==--


From bugzilla-daemon@bugs.koha-community.org Thu Apr 25 08:55:48 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Thu, 25 Apr 2019 06:55:44 +0000
Message-ID: <bug-22551-70-AuGHFjPMNr@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4015253043591994234=="

--===============4015253043591994234==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

Martin Renvoize <martin.renvoize@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #88041|0                           |1
        is obsolete|                            |

--- Comment #9 from Martin Renvoize <martin.renvoize@ptfs-europe.com> ---
Created attachment 88688
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3D88688&action=3D=
edit
Bug 22551: Stray "//" appears at bottom of opac-detail.tt

Removes a couple stray "//<![CDATA[" like comments which are artifacts of oth=
er
code that since been removed. One of the comments causes two forward slashes =
to
appear at the bottom of each OPAC detail page.

Testing plan:

- enable SocialNetworks
- visit any OPAC detail page, observe "//" at bottom of page
- apply patch
- revisit page, slashes should be gone

Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

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

--===============4015253043591994234==--


From bugzilla-daemon@bugs.koha-community.org Thu Apr 25 08:56:22 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Thu, 25 Apr 2019 06:56:19 +0000
Message-ID: <bug-22551-70-6xhuE0o9zZ@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1728034116375652214=="

--===============1728034116375652214==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Martin Renvoize <martin.renvoize@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Passed QA

--- Comment #10 from Martin Renvoize <martin.renvoize@ptfs-europe.com> ---
It's great to see such teamwork going on here guys, well done.

Patch works well and causes no regressions.. Passing QA

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

--===============1728034116375652214==--


From bugzilla-daemon@bugs.koha-community.org Thu Apr 25 08:57:49 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Thu, 25 Apr 2019 06:57:46 +0000
Message-ID: <bug-22551-70-pjp1tS6TiI@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3384022522139434598=="

--===============3384022522139434598==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

--- Comment #11 from Martin Renvoize <martin.renvoize@ptfs-europe.com> ---
As an addendum to the advice given here, you might find git-bz helpful Eric.
https://wiki.koha-community.org/wiki/Git_bz_configuration

It's automates the creation and upload of patches to bugzilla (as well as
making the application of such patches to your development environment very
simple too)

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

--===============3384022522139434598==--


From bugzilla-daemon@bugs.koha-community.org Thu Apr 25 13:56:50 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Thu, 25 Apr 2019 11:56:47 +0000
Message-ID: <bug-22551-70-Pf7mpVlUaS@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3643330704116772750=="

--===============3643330704116772750==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Nick Clemens <nick@bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nick@bywatersolutions.com
             Status|Passed QA                   |Pushed to Master

--- Comment #12 from Nick Clemens <nick@bywatersolutions.com> ---
Awesome work all!

Pushed to master for 19.05

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

--===============3643330704116772750==--


From bugzilla-daemon@bugs.koha-community.org Fri Apr 26 17:25:29 2019
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject:
 [Koha-bugs] [Bug 22551] Stray "//" appears at bottom of	opac-detail.tt
Date: Fri, 26 Apr 2019 15:25:26 +0000
Message-ID: <bug-22551-70-cx7fLPDQ0f@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-22551-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0923662540330605974=="

--===============0923662540330605974==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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

Martin Renvoize <martin.renvoize@ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Version(s)|                            |19.05.00, 18.11.05
        released in|                            |
             Status|Pushed to Master            |Pushed to Stable

--- Comment #13 from Martin Renvoize <martin.renvoize@ptfs-europe.com> ---
Pushed to 18.11.x for 18.11.05

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

--===============0923662540330605974==--