Active
Project:
LoginToboggan
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
30 Dec 2010 at 10:02 UTC
Updated:
12 Sep 2012 at 12:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
hunmonk commentednot supported, you'll have to write your own module for that.
Comment #2
pfrenssenThis is possible in D7 with the Internationalization module and an implementation of hook_variable_info(). This informs the Internationalization module that the redirection path variables are localize-able. The actual localization action is then handled transparently by the i18n_variable module.
I've attached a patch that adds this functionality to LoginToboggan.
To use it:
Comment #3
hunmonk commentedhook_variable_info() isn't a core hook, and i'm not keen on putting module-specific code in LT. if somebody wants to roll up this patch into a logintoboggan_variable integration module, i'd toss it into the contrib folder.
Comment #4
pfrenssenThis is a very good idea, it would keep the core functionality clean, and would still provide this functionality for whoever needs it. I would suggest calling this submodule logintoboggan_i18n though.
Comment #5
samundra commentedYour problem looks similar to the http://drupal.org/node/1269640#comment-4981732, I have also given piece of code that does the redirection thing. I think you can modify that to suit your need.
that was a piece of code form my custom module.
Comment #6
hunmonk commented@pfrenssen: as far as i can tell, http://drupal.org/project/i18n leverages http://drupal.org/project/variable in order to make this work, so what we'd be building here is compatibility with that module, not i18n. that also would leave open the possibility to export other variables that other modules leveraging the variable module might be interested in.
Comment #7
setvik commentedRolled up pfrenssen's patch in #2 as a contrib module that provides Variable module integration.
Comment #8
hunmonk commentedlooks good. the one thing that would be nice is to get all current, relevant LT variables exported (for a complete implementation of the bridge). i won't hold this up if nobody wants to do this, but i do at least want a yay/nay on the idea.
Comment #9
pfrenssenI've added support for the other variables. I have not tested this code so it might not work. I've basically copied all variable descriptions over from logintoboggan.admin.inc and shortened the descriptions where necessary.
Comment #10
hunmonk commentedwould be great if somebody who's interested in this functionality could run some quick tests to confirm the module is working correctly.
Comment #11
hunmonk commentedCommitted the module to contrib on 7.x-1.x-dev
Comment #13
jose reyero commentedI've just seen this new module and wondering why do we need another module.
All these variable definitions could very well be in logintoboggan.variable.inc in the root folder, and the hook renamed to logintoboggan_variable_info().
This way we'd save one more module, and the file is only included by variable.module when needed anyway.
(Maybe I should explain this feature better somewhere in the Variable documentation?)