Changeset 11123 for trunk/plugins

Timestamp:

May 31, 2011, 12:58:12 AM (13 years ago)

Author:

plg

Message:

refactor language switch PHP code to make it readable

File:

• trunk/plugins/language_switch/language_switch.inc.php (modified) (3 diffs)

trunk/plugins/language_switch/language_switch.inc.php

@@ -21 +21 @@
 // | USA.                                                                  |
 // +-----------------------------------------------------------------------+
+
 if (!defined('LANGUAGE_SWITCH_PATH'))
-define('LANGUAGE_SWITCH_PATH' , PHPWG_PLUGINS_PATH . basename(dirname(__FILE__)) . '/');
-class language_controler {
+{
+  define('LANGUAGE_SWITCH_PATH' , PHPWG_PLUGINS_PATH.basename(dirname(__FILE__)).'/');
+}
+
+class language_controler
+{
   static public function _switch()
   {
     global $user;
-    if (!defined('PHPWG_ROOT_PATH')) { die('Hacking attempt!'); }
+    
+    if (!defined('PHPWG_ROOT_PATH'))
+    {
+      die('Hacking attempt!');
+    }
+    
     $same = $user['language'];
-    if ( isset( $_GET['lang']) ) {
-      if ( !empty($_GET['lang'] ) and file_exists( PHPWG_ROOT_PATH.'language/'
-        . $_GET['lang'].'/common.lang.php') ) {
-        if (is_a_guest() or is_generic()) {
-          pwg_set_session_var( 'lang_switch', $_GET['lang'] );
-        } else {
-          $query = 'UPDATE '.USER_INFOS_TABLE.'
-          SET language = \''.$_GET['lang'].'\'
-          WHERE user_id = '.$user['id'].'
-          ;';
+    
+    if (isset($_GET['lang']))
+    {
+      if (!empty($_GET['lang']) and file_exists(PHPWG_ROOT_PATH.'language/'.$_GET['lang'].'/common.lang.php'))
+      {
+        if (is_a_guest() or is_generic())
+        {
+          pwg_set_session_var('lang_switch', $_GET['lang']);
+        }
+        else
+        {
+          $query = '
+UPDATE '.USER_INFOS_TABLE.'
+  SET language = \''.$_GET['lang'].'\'
+  WHERE user_id = '.$user['id'].'
+;';
           pwg_query($query);
         }
+        
         $user['language'] = $_GET['lang'];
       }
     }
-    elseif ((is_a_guest() or is_generic())) {
-      $user['language'] = pwg_get_session_var( 'lang_switch', $user['language'] );
+    elseif ((is_a_guest() or is_generic()))
+    {
+      $user['language'] = pwg_get_session_var('lang_switch', $user['language']);
     }
-  // Reload language only if it isn't the same one
-    if ( $same !== $user['language']) {
-      load_language('common.lang', '', array('language'=>$user['language']) );
-      load_language('lang', PHPWG_ROOT_PATH.PWG_LOCAL_DIR,
-        array('language'=>$user['language'], 'no_fallback'=>true, 'local'=>true)
-      );
-      if (defined('IN_ADMIN') and IN_ADMIN) { // Never currently
-        load_language('admin.lang', '', array('language'=>$user['language']) );
+    
+    // Reload language only if it isn't the same one
+    if ( $same !== $user['language'])
+    {
+      load_language('common.lang', '', array('language'=>$user['language']));
+      
+      load_language(
+        'lang',
+        PHPWG_ROOT_PATH.PWG_LOCAL_DIR,
+        array(
+          'language' => $user['language'],
+          'no_fallback' => true,
+          'local' => true
+          )
+        );
+      
+      if (defined('IN_ADMIN') and IN_ADMIN)
+      {
+        // Never currently
+        load_language('admin.lang', '', array('language'=>$user['language']));
       }
     }
   }
+  
   static public function _flags()
   {
     global $user, $template, $conf;
+    
     $available_lang = get_languages();
-    if ( isset($conf['no_flag_languages']) ) 
-      $available_lang = 
-        array_diff_key($available_lang, array_flip($conf['no_flag_languages']));
+    
+    if (isset($conf['no_flag_languages']))
+    {
+      $available_lang = array_diff_key($available_lang, array_flip($conf['no_flag_languages']));
+    }
+    
     $url_starting = get_query_string_diff(array('lang'));
-    foreach ( $available_lang as $code => $displayname ) {
+    
+    foreach ($available_lang as $code => $displayname)
+    {
       $qlc = array ( 
-        'url' => str_replace(array('=&','?&'),array('&','?'),
-                 add_url_params( $url_starting, array('lang'=> $code) )),
-        'alt' => ucwords( $displayname ),
+        'url' => str_replace(
+          array('=&','?&'),
+          array('&','?'),
+          add_url_params($url_starting, array('lang'=> $code))
+          ),
+        'alt' => ucwords($displayname),
         'title' => substr($displayname, 0, -4), // remove [FR] or [RU]
-        'img' => get_root_url().'language/' . $code . '/' . $code . '.jpg',
+        'img' => get_root_url().'language/'.$code.'/'.$code.'.jpg',
         );
       
@@ -82 +122 @@
       }
     }
+    
     $template->set_filename('language_flags', dirname(__FILE__) . '/flags.tpl');
+    
     $lsw['side'] = ceil(sqrt(count($available_lang)));
-    $template->assign(array(
-      'lang_switch'=> $lsw,
-      'LANGUAGE_SWITCH_PATH' => LANGUAGE_SWITCH_PATH,
-    ));
+    
+    $template->assign(
+      array(
+        'lang_switch'=> $lsw,
+        'LANGUAGE_SWITCH_PATH' => LANGUAGE_SWITCH_PATH,
+        )
+      );
+    
     $flags = $template->parse('language_flags',true);
     $template->clear_assign('lang_switch');
@@ -93 +139 @@
   }
 }
+
   /* {html_head} usage function */
   /* See flags.tpl for example (due no catenation available) */
-if (!function_exists('Componant_exists')) {
+if (!function_exists('Componant_exists'))
+{
   function Componant_exists($path, $file)
-    { return file_exists( $path . $file); }
+  {
+    return file_exists( $path . $file);
+  }
 }