[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Needs Signoff   |Failed QA

--- Comment #9 from Marcel de Rooy  ---
(In reply to Marcel de Rooy from comment #8)
> It seems to be another general issue?

No. I refreshed my container. Could git attach again.
But after applying this patch, we see our friend again:

/usr/local/bin/git-bz:1908: UnicodeWarning: Unicode equal comparison failed to
convert both arguments to Unicode - interpreting them as being unequal
  obsoleted = (initial_description == patch.description)
Traceback (most recent call last):
  File "/usr/local/bin/git-bz", line 2722, in 
do_attach(*args)
  File "/usr/local/bin/git-bz", line 2074, in do_attach
attach_commits(bug, commits, edit_comments=global_options.edit)
  File "/usr/local/bin/git-bz", line 1988, in attach_commits
description, body, obsoletes, statuses, patch_complexities, depends =
edit_attachment_comment(bug, commit.subject, body)
  File "/usr/local/bin/git-bz", line 1940, in edit_attachment_comment
lines = edit_template(template.getvalue())
  File "/usr/lib/python2.7/StringIO.py", line 271, in getvalue
self.buf += ''.join(self.buflist)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 3: ordinal
not in range(128)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

--- Comment #8 from Marcel de Rooy  ---
It seems to be another general issue?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Marcel de Rooy  changed:

   What|Removed |Added

 Status|Needs Signoff   |Failed QA

--- Comment #7 from Marcel de Rooy  ---
Not sure whats going on here.
Please resubmit a new version.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Marcel de Rooy  changed:

   What|Removed |Added

   Severity|critical|normal

--- Comment #6 from Marcel de Rooy  ---
Lowering severity. Critical is really another category :)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Marcel de Rooy  changed:

   What|Removed |Added

 CC||m.de.r...@rijksmuseum.nl

--- Comment #5 from Marcel de Rooy  ---
Same here

UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 3: ordinal
not in range(128)

Will edit commit message now.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

--- Comment #4 from David Nind  ---
I can sign off other bugs, so maybe something wrong with this one?

This is the message I'm getting when I go git bz attach -e 31883 HEAD (after a
git so 1):

git bz attach -e 31883 HEAD
/usr/bin/git-bz:1908: UnicodeWarning: Unicode equal comparison failed to
convert both arguments to Unicode - interpreting them as being unequal
  obsoleted = (initial_description == patch.description)
Traceback (most recent call last):
  File "/usr/bin/git-bz", line 2722, in 
do_attach(*args)
  File "/usr/bin/git-bz", line 2074, in do_attach
attach_commits(bug, commits, edit_comments=global_options.edit)
  File "/usr/bin/git-bz", line 1988, in attach_commits
description, body, obsoletes, statuses, patch_complexities, depends =
edit_attachment_comment(bug, commit.subject, body)
  File "/usr/bin/git-bz", line 1940, in edit_attachment_comment
lines = edit_template(template.getvalue())
  File "/usr/lib/python2.7/StringIO.py", line 271, in getvalue
self.buf += ''.join(self.buflist)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 3: ordinal
not in range(128)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-20 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

David Nind  changed:

   What|Removed |Added

 CC||da...@davidnind.com

--- Comment #3 from David Nind  ---
I followed the test plan and everything works. For some reason I can't sign off
like I normally do (git bz attach -e 31883 HEAD).

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

--- Comment #2 from Tomás Cohen Arazi  ---
Mason, thanks for the patch! I thought of a similar patch, but kept thinking a
variable assignment shouldn't be forced to have a filter, and so was in doubt
about it. I'm happy to go with this regardless.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Mason James  changed:

   What|Removed |Added

   Assignee|ch...@bigballofwax.co.nz|m...@kohaaloha.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

Mason James  changed:

   What|Removed |Added

 Status|NEW |Needs Signoff
   Priority|P5 - low|P1 - high
 CC||tomasco...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/


[Koha-bugs] [Bug 31883] t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes false warnings

2022-10-19 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31883

--- Comment #1 from Mason James  ---
Created attachment 142218
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=142218&action=edit
Bug 31883: t::lib::QA::TemplateFilters missing built-in TT filter "trim" causes
false warnings

to test...

1/ checkout latest master

2/ prove xt/find-missing-filters.t
xt/find-missing-filters.t .. 1/1
#   Failed test 'Template variables should be correctly escaped'
#   at xt/find-missing-filters.t line 59.
#  got: '1'
# expected: '0'
# $VAR1 = {
#   'errors' => [
# {
#   'error' => 'missing_filter',
#   'line' => '[% messages
= INCLUDE \'patron_messages.inc\' moremember = 1 | trim %]',
#   'line_number' => 97
# }
#   ],
#   'file' =>
'koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt'
# };
# Looks like you failed 1 test of 1.

3/ apply patch

4/ prove xt/find-missing-filters.t

xt/find-missing-filters.t .. ok
All tests successful.
Files=1, Tests=1,  2 wallclock secs ( 0.03 usr  0.01 sys +  2.15 cusr  0.07
csys =  2.26 CPU)
Result: PASS <<<

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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/