gettext_po_exporter refactoring gone wrong
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Critical
|
Jeroen T. Vermeulen |
Bug Description
The recent small refactoring we did for translations has one big problem: it doesn't always work. I am surprised it works at all, but here's the traceback we get (https:/
2010-08-16 09:31:18 DEBUG Exporting objects for Данило Шеган, related to template scinotes in Scilab trunk
2010-08-16 09:32:23 DEBUG Removing lock file: /var/lock/
Traceback (most recent call last):
File "cronscripts/
script.
File "/srv/staging.
implicit_
File "/srv/staging.
self.main()
File "cronscripts/
process_
File "/srv/staging.
process_
File "/srv/staging.
target_
File "/srv/staging.
force_
File "/srv/staging.
file_path, file_extension, encoded_
UnboundLocalError: local variable 'encoded_
The problem seems to be simple: encoded_
Related branches
Changed in rosetta: | |
status: | Triaged → In Progress |
Changed in rosetta: | |
assignee: | nobody → Jeroen T. Vermeulen (jtv) |
Changed in rosetta: | |
status: | In Progress → Fix Released |
I believe the fix is just de-indenting all the code after exported_ file_content assignment until storage.addFile call. I've just tested that on staging.
However, we do need tests for this!