Closed (fixed)
Project:
Auto Assign Role
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Oct 2011 at 10:37 UTC
Updated:
18 Nov 2015 at 19:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dkliewer commentedI've hacked a quick solution which makes the fields translatable (variables module required).
Add the following hook at the end of autoassignrole.module:
This enables the three fields for translation via Configuration > Multilingual Settings > Variables
They can then be translated by using Configuration > Variables > Modules > Auto Assign Role
Comment #2
Hess commentedThanks!
Comment #3
skaughthere a fix for some of the strings...they just shouldn't be in the variable table.
and a little uninstall of variables too.
EDIT:
sorry i now see the little logic fix won't really help. i don't' take the time to work it out right now, sorry--gotta focus on my own task at hand.
Comment #4
damienmckennaThe strings_and_minor_logic file in patches in #3 changes some logic that didn't need to be changed, and we already have a working uninstall script per another issue.
This patch reformats the field descriptions from autoassignrole.admin.inc to make translation easier.
Comment #5
damienmckennaCommitted.
Comment #6
damienmckennaClosing this to clean up the issue queue as 7.x-1.0-beta1 was released.
Comment #7
muschpusch commentedSorry for reopening this but the reformatting of the descriptions isn't the issue here. We want the actual values to be translated which #1 fixes. Please have a look at the variable project page
Comment #8
muschpusch commentedhere #1 as a patch but in a seperate file since variable seems to support some lazy loading. Please review!
Comment #10
mike.davis commentedI have updated this patch with the changes to the description field to be a text_format and updated the variables to be in a module group.
Comment #12
mike.davis commentedThis has been committed to 7.x-1.x. Need to apply to 7.x-2.x now
Comment #14
mike.davis commentedAll committed now