• Resolved David Decker

    (@daveshine)


    Hi there!
    I was excited that you finally added internationalization with v1.2.9 – and that is really important! So, a big “Thank you!” 🙂

    However, there are some errors/typos in the latest version which I found out with testing:

    In file “ninja_forms.php” you have:
    load_plugin_textdomain( 'ninja-forms', false, $plugin_dir );

    it should go this way, otherwise translations will not load:
    load_plugin_textdomain( 'ninja-forms', false, $lang_dir );

    Then in both files “includes/body_settings.php” and “includes/body_subs.php” you have typo errors – you used this (9 times each file!):
    e_(...)

    but it should be this way:
    _e(...)

    –> This error leads to fatal errors on the forms and settings page!

    Otherwise you have a lot of gettext strings without textdomain – you should always only use “ninja-forms” as this is the one you’ve defined and are loading!

    __( 'Your string', 'ninja-forms' );
    _e( 'Your string', 'ninja-forms' );

    Also please use single quotes for all in all gettext strings – it’s the recommended way!

    Thank you very much!
    -Dave from Germany

    P.S. Currently translating your plugin into German 🙂

    http://wordpress.org/extend/plugins/ninja-forms/

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author James Laws

    (@jameslaws)

    Thanks for the updates. When you fat finger and then copy and paste a lot it causes all kinds of chaos. Working on fixing them all as I type this. Which will hopefully not cause additional errors. 🙂

    Plugin Author James Laws

    (@jameslaws)

    I actually caught the lang_dir error but totally missed the others. Sigh. Coding in the wee hours…we should stop that. 🙂

    Thread Starter David Decker

    (@daveshine)

    Thanx! No problem, James – I know these things myself happening way too often sometimes…

    Thanx for your hard work, Dave 🙂

    Plugin Author James Laws

    (@jameslaws)

    David,

    All should be corrected in version 1.2.9.1.

    Thanks again for the heads up and for translating it.

    Plugin Author Kevin Stover

    (@kstover)

    Hey David,

    Thanks for the information and the help with translation. If you’re ok with it, we’d love to include your translation in the ninja-forms package. When you get finished, let me know via email: kevin [a t] wpninjas.net.

    Thanks,

    Kevin

    Thread Starter David Decker

    (@daveshine)

    Hi James & Kevin!

    Thanx for your superfast update – all ok now, great! 🙂

    Of course I’ll send you my language files, I guess I could finish this weekend 🙂

    -Dave.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘[Plugin: Ninja Forms Lite] Translation errors in v1.2.9’ is closed to new replies.