Changeset 2479

Timestamp:

Aug 20, 2008, 2:35:22 AM (16 years ago)

Author:

rvelices

Message:

• local.lang is loaded without fallback on default language or PHPWG_DEFAULT_LANGUAGE (needed to change the signature of load_language which became a little too big)
• move a function from functions.inc.php to functions_picture.inc.php (included only when necessary)
• removed some css (not as much as I wanted)

Location:

trunk

Files:

• about.php (modified) (1 diff)
• admin/help.php (modified) (1 diff)
• include/common.inc.php (modified) (1 diff)
• include/functions.inc.php (modified) (6 diffs)
• include/functions_mail.inc.php (modified) (1 diff)
• include/functions_picture.inc.php (modified) (1 diff)
• install.php (modified) (1 diff)
• nbm.php (modified) (1 diff)
• plugins/add_index/main.normal.inc.php (modified) (1 diff)
• plugins/language_switch/language_switch.inc.php (modified) (1 diff)
• popuphelp.php (modified) (1 diff)
• template-extension/distributed/samples/my-picture.tpl (modified) (1 diff)
• template/yoga/default-layout.css (modified) (1 diff)
• template/yoga/picture.css (modified) (2 diffs)
• template/yoga/picture.tpl (modified) (1 diff)

trunk/about.php

@@ -64 +64 @@
 }
 
-$template->assign('ABOUT_MESSAGE', load_language('about.html','','',true) );
+$template->assign('ABOUT_MESSAGE', load_language('about.html','', array('return'=>true)) );
 
 $template->pparse('about');

trunk/admin/help.php

@@ -29 +29 @@
 check_status(ACCESS_ADMINISTRATOR);
 
-$template->assign('ADMIN_CONTENT', load_language('help.html','','',true) );
+$template->assign('ADMIN_CONTENT', load_language('help.html','',array('return'=>true)) );
 ?>

trunk/include/common.inc.php

@@ -178 +178 @@
 }
 trigger_action('loading_lang');
-load_language('local.lang');
+load_language('local.lang', '', array('no_fallback'=>true) );
 
 // only now we can set the localized username of the guest user (and not in

trunk/include/functions.inc.php

@@ -238 +238 @@
 
   return $tndir;
-}
-
-// The get_picture_size function return an array containing :
-//      - $picture_size[0] : final width
-//      - $picture_size[1] : final height
-// The final dimensions are calculated thanks to the original dimensions and
-// the maximum dimensions given in parameters.  get_picture_size respects
-// the width/height ratio
-function get_picture_size( $original_width, $original_height,
-                           $max_width, $max_height )
-{
-  $width = $original_width;
-  $height = $original_height;
-  $is_original_size = true;
-
-  if ( $max_width != "" )
-  {
-    if ( $original_width > $max_width )
-    {
-      $width = $max_width;
-      $height = floor( ( $width * $original_height ) / $original_width );
-    }
-  }
-  if ( $max_height != "" )
-  {
-    if ( $original_height > $max_height )
-    {
-      $height = $max_height;
-      $width = floor( ( $height * $original_width ) / $original_height );
-      $is_original_size = false;
-    }
-  }
-  if ( is_numeric( $max_width ) and is_numeric( $max_height )
-       and $max_width != 0 and $max_height != 0 )
-  {
-    $ratioWidth = $original_width / $max_width;
-    $ratioHeight = $original_height / $max_height;
-    if ( ( $ratioWidth > 1 ) or ( $ratioHeight > 1 ) )
-    {
-      if ( $ratioWidth < $ratioHeight )
-      {
-        $width = floor( $original_width / $ratioHeight );
-        $height = $max_height;
-      }
-      else
-      {
-        $width = $max_width;
-        $height = floor( $original_height / $ratioWidth );
-      }
-      $is_original_size = false;
-    }
-  }
-  $picture_size = array();
-  $picture_size[0] = $width;
-  $picture_size[1] = $height;
-  return $picture_size;
 }
 
@@ -753 +697 @@
     load_language('common.lang');
     trigger_action('loading_lang');
-    load_language('local.lang');
+    load_language('local.lang', '', array('no_fallback'=>true) );
     list($tmpl, $thm) = explode('/', get_default_template());
     $template = new Template(PHPWG_ROOT_PATH.'template/'.$tmpl, $thm);
@@ -1395 +1339 @@
  * @param string filename
  * @param string dirname
- * @param string language
- * @param bool return_content - if true the file content is returned otherwise
- *  the file is evaluated as php
- * @return boolean success status or a string if return_content is true
- */
-function load_language($filename, $dirname = '', $language = '',
-    $return_content=false, $target_charset=null)
+ * @param mixed options can contain
+ *     language - language to load (if empty uses user language)
+ *     return - if true the file content is returned otherwise the file is evaluated as php
+ *     target_charset -
+ *     no_fallback - the language must be respected
+ * @return boolean success status or a string if options['return'] is true
+ */
+function load_language($filename, $dirname = '',
+    $options = array() )
 {
   global $user;
 
-  if (!$return_content)
+  if (! @$options['return'] )
   {
     $filename .= '.php'; //MAYBE to do .. load .po and .mo localization files
@@ -1416 +1362 @@
 
   $languages = array();
-  if ( !empty($language) )
-  {
-    $languages[] = $language;
+  if ( !empty($options['language']) )
+  {
+    $languages[] = $options['language'];
   }
   if ( !empty($user['language']) )
@@ -1424 +1370 @@
     $languages[] = $user['language'];
   }
-  if ( defined('PHPWG_INSTALLED') )
-  {
-    $languages[] = get_default_language();
-  }
-  $languages[] = PHPWG_DEFAULT_LANGUAGE;
+  if ( ! @$options['no_fallback'] )
+  {
+    if ( defined('PHPWG_INSTALLED') )
+    {
+      $languages[] = get_default_language();
+    }
+    $languages[] = PHPWG_DEFAULT_LANGUAGE;
+  }
+
   $languages = array_unique($languages);
 
-  if ( empty($target_charset) )
+  if ( empty($options['target_charset']) )
   {
     $target_charset = get_pwg_charset();
+  }
+  else
+  {
+    $target_charset = $options['target_charset'];
   }
   $target_charset = strtolower($target_charset);
@@ -1473 +1427 @@
   if ( !empty($source_file) )
   {
-    if (!$return_content)
+    if (! @$options['return'] )
     {
       @include($source_file);

trunk/include/functions_mail.inc.php

@@ -247 +247 @@
 
       // language files
-      load_language('common.lang', '', $language);
+      load_language('common.lang', '', array('language'=>$language) );
       // No test admin because script is checked admin (user selected no)
       // Translations are in admin file too
-      load_language('admin.lang', '', $language);
+      load_language('admin.lang', '', array('language'=>$language) );
       trigger_action('loading_lang');
-      load_language('local.lang', '', $language);
+      load_language('local.lang', '', array('language'=>$language, 'no_fallback'=>true));
 
       $switch_lang[$language]['lang_info'] = $lang_info;

trunk/include/functions_picture.inc.php

@@ -328 +328 @@
 }
 
+// The get_picture_size function return an array containing :
+//      - $picture_size[0] : final width
+//      - $picture_size[1] : final height
+// The final dimensions are calculated thanks to the original dimensions and
+// the maximum dimensions given in parameters.  get_picture_size respects
+// the width/height ratio
+function get_picture_size( $original_width, $original_height,
+                           $max_width, $max_height )
+{
+  $width = $original_width;
+  $height = $original_height;
+  $is_original_size = true;
+
+  if ( $max_width != "" )
+  {
+    if ( $original_width > $max_width )
+    {
+      $width = $max_width;
+      $height = floor( ( $width * $original_height ) / $original_width );
+    }
+  }
+  if ( $max_height != "" )
+  {
+    if ( $original_height > $max_height )
+    {
+      $height = $max_height;
+      $width = floor( ( $height * $original_width ) / $original_height );
+      $is_original_size = false;
+    }
+  }
+  if ( is_numeric( $max_width ) and is_numeric( $max_height )
+       and $max_width != 0 and $max_height != 0 )
+  {
+    $ratioWidth = $original_width / $max_width;
+    $ratioHeight = $original_height / $max_height;
+    if ( ( $ratioWidth > 1 ) or ( $ratioHeight > 1 ) )
+    {
+      if ( $ratioWidth < $ratioHeight )
+      {
+        $width = floor( $original_width / $ratioHeight );
+        $height = $max_height;
+      }
+      else
+      {
+        $width = $max_width;
+        $height = floor( $original_height / $ratioWidth );
+      }
+      $is_original_size = false;
+    }
+  }
+  $picture_size = array();
+  $picture_size[0] = $width;
+  $picture_size[1] = $height;
+  return $picture_size;
+}
+
 ?>

trunk/install.php

@@ -207 +207 @@
 }
 
-load_language( 'common.lang', '', $language, false, 'utf-8' );
-load_language( 'admin.lang', '', $language, false, 'utf-8' );
-load_language( 'install.lang', '', $language, false, 'utf-8' );
+load_language( 'common.lang', '', array('language'=>$language, 'target_charset'=>'utf-8') );
+load_language( 'admin.lang', '', array('language'=>$language, 'target_charset'=>'utf-8') );
+load_language( 'install.lang', '', array('language'=>$language, 'target_charset'=>'utf-8') );
 
 //----------------------------------------------------- template initialization

trunk/nbm.php

@@ -35 +35 @@
 // Need to update a second time
 trigger_action('loading_lang');
-load_language('local.lang');
+load_language('local.lang', '', array('no_fallback'=>true) );
 
 

trunk/plugins/add_index/main.normal.inc.php

@@ -34 +34 @@
     {
       $help_content =
-        load_language('help/'.$page.'.html', $this->path, '', true);
+        load_language('help/'.$page.'.html', $this->path, array('return'=>true) );
     }
     else

trunk/plugins/language_switch/language_switch.inc.php

@@ -58 +58 @@
   if ( $same !== $user['language'])
   {
-    load_language('common.lang', '', $user['language']);
-    load_language('local.lang', '', $user['language']);
+    load_language('common.lang', '', array('language'=>$user['language']) );
+    load_language('local.lang', '', array('language'=>$user['language'], 'no_fallback'=>true) );
     if (defined('IN_ADMIN') and IN_ADMIN)
     {
-      load_language('admin.lang', '', $user['language']);
+      load_language('admin.lang', '', array('language'=>$user['language']) );
     }
   }

trunk/popuphelp.php

@@ -47 +47 @@
 {
   $help_content =
-    load_language('help/'.$_GET['page'].'.html', '', '', true);
+    load_language('help/'.$_GET['page'].'.html', '', array('return'=>true) );
 
   if ($help_content == false)

trunk/template-extension/distributed/samples/my-picture.tpl

@@ -23 +23 @@
 
 {if isset($previous) }
-<a class="navThumb" id="thumbPrev" href="{$previous.U_IMG}" title="{'previous_page'|@translate} : {$previous.TITLE}" rel="prev">
-  <img src="{$previous.THUMB_SRC}" class="thumbLink" id="linkPrev" alt="{$previous.TITLE}">
+<a class="navThumb" id="linkPrev" href="{$previous.U_IMG}" title="{'previous_page'|@translate} : {$previous.TITLE}" rel="prev">
+  <img src="{$previous.THUMB_SRC}" alt="{$previous.TITLE}">
 </a>
 {/if}
 {if isset($next) }
-<a class="navThumb" id="thumbNext" href="{$next.U_IMG}" title="{'next_page'|@translate} : {$next.TITLE}" rel="next">
-  <img src="{$next.THUMB_SRC}" class="thumbLink" id="linkNext" alt="{$next.TITLE}">
+<a class="navThumb" id="linkNext" href="{$next.U_IMG}" title="{'next_page'|@translate} : {$next.TITLE}" rel="next">
+  <img src="{$next.THUMB_SRC}" alt="{$next.TITLE}">
 </a>
 {/if}

trunk/template/yoga/default-layout.css

@@ -164 +164 @@
 FORM.filter FIELDSET P
 {
-  clear: left;
-  display: block;
+        clear: left;
+        display: block;
 }
 
 FORM.filter INPUT[type="submit"] {
-  margin-top: 1em;
-}
-
-FORM.properties UL, FORM#update UL {
-  list-style-type: none;
-  margin: 0;
-  padding: 0;
-}
-
-FORM.properties LI, FORM#update UL  {
-  margin-bottom: 0.5em;
-  padding: 0;
-  line-height: 1.8em;
-  clear: left;
+        margin-top: 1em;
+}
+
+FORM.properties UL {
+        list-style-type: none;
+        margin: 0;
+        padding: 0;
+}
+
+FORM.properties LI {
+        margin-bottom: 0.5em;
+        padding: 0;
+        line-height: 1.8em;
+        clear: left;
 }
 

trunk/template/yoga/picture.css

@@ -59 +59 @@
 }
 
-.navThumb {
-    margin-top: 2px;
-}
-#thumbPrev {
-    float: left;
-}
-
-#thumbNext {
-    float: right;
-}
-
 #linkPrev {
-    margin-right: 10px;
-    margin-left: 5px;
+        float: left;
 }
 
 #linkNext {
-    margin-right: 5px;
-    margin-left: 10px;
+        float: right;
+}
+
+#linkPrev IMG {
+        margin-right: 10px;
+        margin-left: 5px;
+}
+
+#linkNext IMG {
+        margin-right: 5px;
+        margin-left: 10px;
 }
 
@@ -97 +94 @@
 
 .rateButton, .rateButtonSelected, .rateButtonStarFull, .rateButtonStarEmpty  {
-  padding:0;
-  border:0;
+        padding:0;
+        border:0;
 }
 
 .rateButton, .rateButtonStarFull, .rateButtonStarEmpty {
-  cursor: pointer;
+        cursor: pointer;
 }
 
 .rateButtonSelected {
-  font-weight:bold;
-  font-size:120%;
+        font-weight:bold;
+        font-size:120%;
 }
 

trunk/template/yoga/picture.tpl

@@ -94 +94 @@
 
 {if isset($previous) }
-<a class="navThumb" id="thumbPrev" href="{$previous.U_IMG}" title="{'previous_page'|@translate} : {$previous.TITLE}" rel="prev">
-  <img src="{$previous.THUMB_SRC}" class="thumbLink" id="linkPrev" alt="{$previous.TITLE}">
+<a class="navThumb" id="linkPrev" href="{$previous.U_IMG}" title="{'previous_page'|@translate} : {$previous.TITLE}" rel="prev">
+  <img src="{$previous.THUMB_SRC}" alt="{$previous.TITLE}">
 </a>
 {/if}
 {if isset($next) }
-<a class="navThumb" id="thumbNext" href="{$next.U_IMG}" title="{'next_page'|@translate} : {$next.TITLE}" rel="next">
-  <img src="{$next.THUMB_SRC}" class="thumbLink" id="linkNext" alt="{$next.TITLE}">
+<a class="navThumb" id="linkNext" href="{$next.U_IMG}" title="{'next_page'|@translate} : {$next.TITLE}" rel="next">
+  <img src="{$next.THUMB_SRC}" alt="{$next.TITLE}">
 </a>
 {/if}