By defining your settings/variables to the Variable API they become translatable. The file attached does this for the more link URI and text, but maybe some other settings might be translatable as well.

Just add this file to your module directory and that's it. No need to include it from your .module, nor define it in your .info. No performance penalty, no warnings when the variable module is not installed, simple and non-intrusive.

Note that this issue is complementary to #2476801: Ensure all strings are translatable that handles hard coded strings in your module's code.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fietserwin created an issue. See original summary.

fietserwin’s picture

fietserwin’s picture

I had to change the extension before i could upload it. Rename the attached file to instagram_social_feed.variable.inc.

DamienMcKenna’s picture

Status: Needs review » Needs work

A minor nitpick, but there shouldn't be a t() call on an empty string.

fietserwin’s picture

Status: Needs work » Needs review
FileSize
714 bytes

You are right. Copy, paste, no(t enough) edit.

  • fietserwin authored def4651 on 7.x-1.x
    Issue #2602444 by fietserwin: Support variable API (makes settings...
dysproseum’s picture

Status: Needs review » Fixed

Committed patch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.