Report import POSyntaxWarnings to uploader

Bug #148425 reported by Jeroen T. Vermeulen
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Jeroen T. Vermeulen

Bug Description

The PO parser can issue warnings about minor problems in translation files (see bug 1773 for an example). These warnings are currently logged, which is a nuisance to administrators (see bug 35430), but not reported back to the uploader.

The main source of pointless email to error-reports is "PO file header entry has a bad entry." Just getting rid of that log message would spare us a lot of noise on error-reports.

We should add a mechanism for parsers to return warnings to their callers, and check for and include any warnings in confirmation and error emails. This should replace the logging.warning(POSyntaxWarning(message)) construct.

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

The same goes for actual syntax errors. A TranslationFormatSyntaxError results in a generic message referring the uploader to msgfmt. Not all syntax problems can be detected that way, e.g. http://launchpadlibrarian.net/11577117/eYWndZBCUT3646o1PW63ZEKi0OO.txt

Changed in rosetta:
importance: Undecided → High
status: New → Confirmed
Changed in rosetta:
status: Confirmed → Triaged
summary: - Report import warnings to uploader
+ Report import POSyntaxWarnings to uploader
description: updated
description: updated
tags: added: rosetta-imports
Changed in rosetta:
assignee: nobody → Jeroen T. Vermeulen (jtv)
milestone: none → 2.2.7
status: Triaged → In Progress
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Fixed in devel 8859

Changed in rosetta:
status: In Progress → Fix Committed
Changed in rosetta:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.