Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

info.tt2 generates some noise in Apache error log #688

Closed
dpc22 opened this issue Jul 12, 2019 · 1 comment · Fixed by #696
Closed

info.tt2 generates some noise in Apache error log #688

dpc22 opened this issue Jul 12, 2019 · 1 comment · Fixed by #696
Assignees
Labels
Milestone

Comments

@dpc22
Copy link
Contributor

dpc22 commented Jul 12, 2019

Version

6.2.44

Installation method

rpm from https://copr.fedorainfracloud.org/coprs/xavierb/sympa/

RHEL 7.6

Expected behavior

The list "info" page should not generate entries in the Apache error log

Actual behavior

Each page view generates an error:

[Fri Jul 12 10:41:21.156711 2019] [proxy_fcgi:error] [pid 13008] [client 131.111.56.48:45994] AH01071: Got error 'Argument "" isn't numeric in numeric gt (>) at /usr/share/sympa/default/web_tt2/info.tt2 line 32.\n'

Additional information

Appears to be down to the following in web_tt2/info.tt2:

            [% IF mod_message > 0 %]
           ...
            [% IF mod_total_shared > 0 %]

If I replace that with:

           [% IF mod_message && (mod_message > 0) %]
          ...
           [% IF mod_total_shared && (mod_total_shared > 0) %]

The errors disappear.

@ikedas ikedas added the bug label Jul 12, 2019
@ikedas ikedas self-assigned this Jul 12, 2019
@ikedas ikedas added this to the 6.2.46 milestone Jul 12, 2019
@ikedas
Copy link
Member

ikedas commented Jul 18, 2019

@dpc22, thanks for reporting bug! Your suggestion will be included in the next beta released in this weelend. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants