[jifty-devel] [Jifty-commit] r7630 - jifty/trunk/lib/Jifty

[Jesse Vincent]

This change feels like it's giving I18N too much knowledge of the
internals of a current_user->user_object.

Maybe we should give current_user some sort of API for this? But I'd
like I18N not to go poking around in user objects.

Author: sunnavy
Date: Wed Nov 18 03:16:00 2009
New Revision: 7630
jifty/trunk/lib/Jifty/I18N.pm
try to find user's lang pref from db too
Modified: jifty/trunk/lib/Jifty/I18N.pm
==============================================================================
--- jifty/trunk/lib/Jifty/I18N.pm (original)
+++ jifty/trunk/lib/Jifty/I18N.pm Wed Nov 18 03:16:00 2009
@@ -184,6 +184,20 @@
# optional argument makes it easy to disable I18N
# while comparing test strings (without loading session)
my $lang = shift || Jifty->web->session->get('jifty_lang');
+
+ if ( !$lang
+ && Jifty->web->current_user
+ && Jifty->web->current_user->id )
+ {
+ my $user = Jifty->web->current_user->user_object;
+ for my $column (qw/language lang/) {
+ if ( $user->can($column) ) {
+ $lang = $user->$column;
+ last;
+ }
+ }
+ }
+
$$DynamicLH = $self->get_handle($lang ? $lang : ()) if $DynamicLH;
}
_______________________________________________
Jifty-commit mailing list
http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-commit