Changeset 18629


Ignore:
Timestamp:
10/12/12 22:00:16 (7 years ago)
Author:
rvelices
Message:
  • remove unsed code; shorten existing code and improve readability ...
Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/admin/extend_for_templates.php

    r12922 r18629  
    114114$flip_templates = array_flip($eligible_templates); 
    115115 
     116/** 
     117 * returns an array contening sub-directories, excluding ".svn" 
     118 * 
     119 * @param string $dir 
     120 * @return array 
     121 */ 
     122function get_dirs($directory) 
     123{ 
     124  $sub_dirs = array(); 
     125  if ($opendir = opendir($directory)) 
     126  { 
     127    while ($file = readdir($opendir)) 
     128    { 
     129      if ($file != '.' 
     130          and $file != '..' 
     131          and is_dir($directory.'/'.$file) 
     132          and $file != '.svn') 
     133      { 
     134        array_push($sub_dirs, $file); 
     135      } 
     136    } 
     137    closedir($opendir); 
     138  } 
     139  return $sub_dirs; 
     140} 
     141 
    116142$available_templates = array_merge( 
    117143  array('N/A' => '----------'), 
  • trunk/include/functions.inc.php

    r18165 r18629  
    124124  $pos = strrpos( $filename, '.' ); 
    125125  return ($pos===false) ? $filename : substr( $filename, 0, $pos); 
    126 } 
    127  
    128 /** 
    129  * returns an array contening sub-directories, excluding ".svn" 
    130  * 
    131  * @param string $dir 
    132  * @return array 
    133  */ 
    134 function get_dirs($directory) 
    135 { 
    136   $sub_dirs = array(); 
    137   if ($opendir = opendir($directory)) 
    138   { 
    139     while ($file = readdir($opendir)) 
    140     { 
    141       if ($file != '.' 
    142           and $file != '..' 
    143           and is_dir($directory.'/'.$file) 
    144           and $file != '.svn') 
    145       { 
    146         array_push($sub_dirs, $file); 
    147       } 
    148     } 
    149     closedir($opendir); 
    150   } 
    151   return $sub_dirs; 
    152126} 
    153127 
     
    12751249  $languages = array_unique($languages); 
    12761250 
     1251  /*Note: target charset is always utf-8 
    12771252  if ( empty($options['target_charset']) ) 
    12781253  { 
     
    12831258    $target_charset = $options['target_charset']; 
    12841259  } 
    1285   $target_charset = strtolower($target_charset); 
     1260  $target_charset = strtolower($target_charset);*/ 
    12861261  $source_file    = ''; 
    12871262  foreach ($languages as $language) 
     
    13101285      if ( !isset($lang_info) ) $lang_info=array(); 
    13111286 
     1287      /* Note: target charset is always utf-8 
    13121288      if ( 'utf-8'!=$target_charset) 
    13131289      { 
     
    13341310      } 
    13351311      else 
    1336       { 
     1312      {*/ 
    13371313        $lang = array_merge( $lang, (array)$load_lang ); 
    13381314        $lang_info = array_merge( $lang_info, (array)$load_lang_info ); 
    1339       } 
     1315      //} 
    13401316      return true; 
    13411317    } 
     
    13431319    { 
    13441320      $content = @file_get_contents($source_file); 
    1345       $content = convert_charset($content, 'utf-8', $target_charset); 
     1321      //Note: target charset is always utf-8 $content = convert_charset($content, 'utf-8', $target_charset); 
    13461322      return $content; 
    13471323    } 
     
    16051581 
    16061582  $options = array(); 
     1583  $label = ''; 
    16071584  foreach (array_reverse($conf['available_permission_levels']) as $level) 
    16081585  { 
    1609     $label = null; 
    1610  
    16111586    if (0 == $level) 
    16121587    { 
     
    16151590    else 
    16161591    { 
    1617       $labels = array(); 
    1618       $sub_levels = array_reverse($conf['available_permission_levels']); 
    1619       foreach ($sub_levels as $sub_level) 
     1592      if (strlen($label)) 
    16201593      { 
    1621         if ($sub_level == 0 or $sub_level < $level) 
    1622         { 
    1623           break; 
    1624         } 
    1625         array_push( 
    1626           $labels, 
    1627           l10n( 
    1628             sprintf( 
    1629               'Level %d', 
    1630               $sub_level 
    1631               ) 
    1632             ) 
    1633           ); 
     1594        $label .= ', '; 
    16341595      } 
    1635  
    1636       $label = implode(', ', $labels); 
     1596      $label .= l10n( sprintf('Level %d',$level) ); 
    16371597    } 
    16381598    $options[$level] = $label; 
  • trunk/include/functions_user.inc.php

    r18164 r18629  
    288288  global $conf; 
    289289 
    290   $userdata = array(); 
    291  
    292290  // retrieve basic user data 
    293291  $query = ' 
     
    348346 
    349347  // then merge basic + additional user data 
    350   $row = array_merge($row, $user_infos_row); 
    351  
    352   foreach ($row as $key => $value) 
    353   { 
    354     if (!is_numeric($key)) 
    355     { 
    356       // If the field is true or false, the variable is transformed into a 
    357       // boolean value. 
    358       if ($value == 'true' or $value == 'false') 
    359       { 
    360         $userdata[$key] = get_boolean($value); 
    361       } 
    362       else 
    363       { 
    364         $userdata[$key] = $value; 
    365       } 
    366     } 
    367   } 
     348  $userdata = array_merge($row, $user_infos_row); 
     349 
     350  foreach ($userdata as &$value) 
     351  { 
     352      // If the field is true or false, the variable is transformed into a boolean value. 
     353      if ($value == 'true') 
     354      { 
     355        $value = true; 
     356      } 
     357      elseif ($value == 'false') 
     358      { 
     359        $value = false; 
     360      } 
     361  } 
     362  unset($value); 
    368363 
    369364  if ($use_cache) 
     
    644639  if ($ref_level <> 0) 
    645640  { 
    646     array_push($list_cat_id, $cat_id); 
     641    $list_cat_id[] = $cat_id; 
    647642  } 
    648643} 
     
    663658    if ($level > $ref_level) 
    664659    { 
    665       array_push($list_cat_id, $id); 
     660      $list_cat_id[] = $id; 
    666661    } 
    667662    else 
    668663    { 
    669664      compute_branch_cat_data($cats, $list_cat_id, $level, $ref_level); 
    670       array_push($list_cat_id, $id); 
     665      $list_cat_id[] = $id; 
    671666    } 
    672667    $ref_level = $level; 
     
    830825  if (is_array($cache['default_user']) and $convert_str) 
    831826  { 
    832     $default_user = array(); 
    833     foreach ($cache['default_user'] as $name => $value) 
    834     { 
    835       // If the field is true or false, the variable is transformed into a 
    836       // boolean value. 
    837       if ($value == 'true' or $value == 'false') 
    838       { 
    839         $default_user[$name] = get_boolean($value); 
    840       } 
    841       else 
    842       { 
    843         $default_user[$name] = $value; 
     827    $default_user = $cache['default_user']; 
     828    foreach ($default_user as &$value) 
     829    { 
     830      // If the field is true or false, the variable is transformed into a boolean value. 
     831      if ($value == 'true') 
     832      { 
     833        $value = true; 
     834      } 
     835      elseif ($value == 'false') 
     836      { 
     837        $value = false; 
    844838      } 
    845839    } 
     
    985979          )); 
    986980 
    987       array_push($inserts, $insert); 
     981      $inserts[] = $insert; 
    988982    } 
    989983 
Note: See TracChangeset for help on using the changeset viewer.