| Name |
Last commit
|
Last update |
|---|---|---|
| .. | ||
| djangoapps | ||
| envs | ||
| lib | ||
| static | ||
| templates | ||
| .coveragerc | ||
| __init__.py | ||
| startup.py | ||
| urls.py | ||
| urls_dev.py | ||
| wsgi.py |
This resolves an issue where a checklist's long_description being empty caused gettext to dump PO file headers. I had originally made this change to the Stanford-Online/edx-platform repository a few weeks ago [0] to address a failing test on our fork. I didn't take the time to diagnose the root issue; I just fixed it for the single case. After stumbling across the issue elsewhere, @kluo <kevkluo@gmail.com> followed through and tracked down the cause of the behavior. It turns out that `gettext`, by default, translates the empty string into PO file headers; this is apparently a feature, not a bug [1]. This behavior seems counterintuitive; for our use, it seems that translating an empty string should simply return an empty string. We already program defensively around this in platform code [2], but it'd be convenient to resolve this with a centralized solution, instead of a patchwork of fixes. [0] 4b88f54a [1] http://stackoverflow.com/questions/18459387/translation-of-empty-string-shows-po-header [2] https://github.com/edx/edx-platform/blob/master/cms/djangoapps/contentstore/views/checklist.py#L140
| Name |
Last commit
|
Last update |
|---|---|---|
| .. | ||
| djangoapps | Loading commit data... | |
| envs | Loading commit data... | |
| lib | Loading commit data... | |
| static | Loading commit data... | |
| templates | Loading commit data... | |
| .coveragerc | Loading commit data... | |
| __init__.py | Loading commit data... | |
| startup.py | Loading commit data... | |
| urls.py | Loading commit data... | |
| urls_dev.py | Loading commit data... | |
| wsgi.py | Loading commit data... |