Project

General

Profile

Actions

Defect #18605

closed

Wrong usage of logger.info to test log level

Added by Jérôme BATAILLE about 10 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Normal
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

logger.info (instead of logger.info?) can trigger an error

Line 185 :

logger.info "time entry could not be updated: #{time_entry.errors.full_messages}" if logger && logger.info

replace by :

logger.info "time entry could not be updated: #{time_entry.errors.full_messages}" if logger && logger.info?

Actions

Also available in: Atom PDF