Version: D6
Project link: http://drupal.org/sandbox/tomotomo/1825768
Git: git clone --recursive --branch 6.x-1.x http://git.drupal.org/sandbox/tomotomo/1825768.git translate_filter

This is an input filter which can be configured, say for the default Filtered HTML input, to do automatic translation of strings when the default language is different from English but strings are being translated from English.

Instead of sprinkling through your content calls like print t('...');

Instead you can just do "[t]...[/t]" and you don't need to use the PHP filter.

Description:
Allows content editors to have strings translated to the editor's language instead of embedding t() calls by wrapping them with "[t]" and "[/t]".

Similar to gist_filter and mathfilter but the purpose is to translate strings.

Comments

ymakux’s picture

ymakux commented
Status: Needs review » Needs work

Welcome

README.txt is missing, see the guidelines for in-project documentation.
Drupal Code Sniffer has found some issues with your code (please check the Drupal coding standards).

Review by Coder module

FILE: translate_filter.module
--------------------------------------------------------------------------------
FOUND 15 ERROR(S) AND 4 WARNING(S) AFFECTING 18 LINE(S)
--------------------------------------------------------------------------------
2 | ERROR | Missing file doc comment
5 | WARNING | Format should be "* Implements hook_foo()." or "Implements
| | hook_foo_BAR_ID_bar() for xyz_bar()."
14 | WARNING | Format should be "* Implements hook_foo()." or "Implements
| | hook_foo_BAR_ID_bar() for xyz_bar()."
21 | WARNING | Format should be "* Implements hook_foo()." or "Implements
| | hook_foo_BAR_ID_bar() for xyz_bar()."
23 | ERROR | Whitespace found at end of line
24 | ERROR | Whitespace found at end of line
25 | ERROR | Whitespace found at end of line
26 | ERROR | Whitespace found at end of line
28 | ERROR | Whitespace found at end of line
29 | WARNING | Avoid backslash escaping in translatable strings when possible,
| | use "" quotes instead
29 | ERROR | Whitespace found at end of line
31 | ERROR | Whitespace found at end of line
34 | ERROR | Whitespace found at end of line
39 | ERROR | Inline comments must start with a capital letter
46 | ERROR | Whitespace found at end of line
47 | ERROR | Whitespace found at end of line
48 | ERROR | Whitespace found at end of line
49 | ERROR | Whitespace found at end of line
51 | ERROR | Files must end in a single new line character

udaksh’s picture

udaksh commented

Master Branch is still present in your repository. Try to make 6.x-1.x as your default branch and remove master branch.
http://drupal.org/node/1127732

udaksh’s picture

udaksh commented

Also add installation and usage instruction in your project page .

udaksh’s picture

udaksh commented

Please have a look at section 2.3 on this link http://drupal.org/node/1587704 .Your project is only 50 lines of code and everything with less than 120 lines of code or less than 5 functions cannot be seriously reviewed, that means we can't make sure you're able to write secure code and use Drupal's APIs correctly.

Thanks

adnasa’s picture

adnasa commented

Please change your default repo branch to a specific module version rather than master.
read this page carefully and apply it accordingly: http://drupal.org/node/1127732

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

If you reopen this please keep in mind that we are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)