As per code:

  // Use user's preferred language to build the url (e.g. reset language prefix to user's preferred language).
  // @todo Maybe this should be a configurable option.

This is very confusing. User installs multilink, creates a configuration and tries first links. Because his preferred language is probably different from the language of the node he is trying, he will get his language links instead of "expected" links. This took me 30 minutes to figure out.

Let's make it configurable. In filter settings, have configuration option:

Preferred language:
- Use users preferred language
- Use global languages

With a default to user's preferred language to keep backwards compatibility?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Andy Inman’s picture

Category: bug » feature

Agreed, and as per the todo note, it should be configurable, and will be in the next release, but its not a bug, just lack of a feature :) See http://drupal.org/node/796822 also.

plach’s picture

Status: Active » Needs review
FileSize
2.01 KB

And here it is :)

plach’s picture

FileSize
2.01 KB

Fixed a typo. The setting defaults to TRUE to preserve BC.

Andy Inman’s picture

Version 2.5 is out - see http://drupal.org/node/1277026 - allows some configuration of language preference order via settings.php, so hopefully patching will no longer be needed for most use cases. No UI currently to configure it, so you may still prefer 2.4 plus plach's patch.