amandasaurus / django-template-i18n-lint

Lint tool to find non-trans/blocktrans text in django templates

Django Template i18n lint

A simple script to find non-i18n text in a Django template.

It can also automatically wrap the strings in {% trans "" %} tags, by running it with the -r command-line flag. The translation will be written to a new file, <filename>_translated.html.

For more info see Lint tool to find non-i18n strings in a django template

Code is copyright Rory McCann 2013, and dual licenced under the GNU GPL version3 (or at your option a later version), and the BSD licence. See the files LICENCE.GPLv3 and LICENCE.BSD for more information

django-template-i18n-lint's Issues

Dual Licensing

Hello,

Is the dual licensing meant as 'AND' (you have to fulfill both BSD and GPLv3) or an 'OR' (you either have to fulfill BSD or GPLv3).

If the argument is the AND, the BSD license is not required as it is fully contained within the GPLv3 requirements.
If the argument is the OR, the GPLv3 license is not required as relicensing under GPLv3 can be done at any time by users.

Bug when running from command line

I feel like I'm missing something obvious... But it looks like one should pass overwrite into the replace_strings function.

(oscar)(releases/0.6) T530:templates $ ./django_template_i18n_lint.py -r
Traceback (most recent call last):
  File "./django_template_i18n_lint.py", line 188, in <module>
    main()
  File "./django_template_i18n_lint.py", line 183, in main
    replace_strings(filename)
  File "./django_template_i18n_lint.py", line 125, in replace_strings
    if options.overwrite:
NameError: global name 'options' is not defined

Memoization

It'd be great to have the script could use Memoization to keep track of the answers and doesn't ask the user twice for the same string. This could be made optional so the user can decide. I'd be happy to implement this.

Please provide licence details

Hi Rory

What licence is this code released under, please? I need to know before I can use it properly in a project.

Cheers
Steve

GOOD_STRINGS addition novalidate

novalidate should be added to GOOD_STRINGS

JavaScript - untranslated strings

I'd found this script really helpful, so why not extend it to support scanning for JavaScript untranslated strings? :)

v1.2.1 tag missing

https://pypi.org/project/django-template-i18n-lint/ has v1.2.1 which doesnt appear in the list of GitHub tags.

parsing issues

-  <input type="submit" value="Confirm" class="btn btn-danger" /> 
+  <input type="submit" {% trans "value=\"Confirm" %}" class="btn btn-danger" />  

-              <li><a href="https://twitter.com/localunews" class="icon-twitter" rel="tooltip" title="" data-placement="top" data-original-title="Twitter"><i class="fa fa-twitter"></i></a></li>

+              <li><a href="https://twitter.com/localunews" class="icon-twitter" rel="tooltip" {% trans "title" %}="" data-placement="top" {% trans "data-original-title=\"Twitter" %}"><i class="fa fa-twitter"></i></a></li>

And {% blocktrans %} is not used for multiline text.

-r flag hits "template is not defined" error

Change raw_input to input for the script to work with python3

Change raw_input to input for the script to work with python3