imagecms / ImageCMS

application/libraries/MY_Lang.php

<?php

if (!defined('BASEPATH')) {
    exit('No direct script access allowed');
}

/**
 * Code Igniter Gettext Extension library
 *
 * This Library overides the original CI's language class. Needs the  $config['language'] variable set as it_IT or en_EN or fr_FR ...
 *
 * @package       Gettext Extension
 * @author        wokamoto
 * @copyright     Copyright (c) 2012
 * @license       http://www.gnu.org/licenses/lgpl.txt
 * @link
 * @version       Version 0.1
 * @since         2012 January, 27th
 */
// ------------------------------------------------------------------------

class MY_Lang extends CI_Lang
{

    private $ci;

    private $gettext_language;

    private $gettext_codeset;

    private $gettext_domain;

    private $gettext_path;

    /**
     * The constructor initialize the library
     *
     * @return MY_Lang

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct() {
        parent::__construct();
    }

    private function _language() {
        static $language;

        if (!isset($this->ci)) {
            $this->ci = & get_instance();
        }

        if (!isset($language)) {
            $language = $this->ci->config->item('language');
        }

        if (!isset($language) || !in_array($language, $this->ci->config->item('selectable_languages'))) {
            $language = 'english';
        }

        if ($language != $this->ci->config->item('language')) {
            $this->ci->config->set_item('language', $language);
        }

        return empty($language) ? 'english' : $language;
    }

    /**
     * Load a language file
     *
     * @access	public
     * @param	mixed	the name of the language file to be loaded. Can be an array
     * @param	string	the language (english, etc.)
     * @return	mixed
     */
    public function load($langfile = '', $idiom = '', $return = FALSE, $add_suffix = TRUE, $alt_path = '') {
        return parent::load($langfile, !empty($idiom) ? $idiom : $this->_language(), $return, $add_suffix, $alt_path);
    }

    /**
     * This method overides the original load method. Its duty is loading the domain files by config or by default internal settings.
     *
     * @access	public
     * @param	string	$userlang	the language, set as ja_JP or it_IT or en_EN or fr_FR ...
     * @param	string	$codeset	the codeset, set as UTF-8 or EUC ...
     * @return	bool
     */
    public function load_gettext($userlang = '', $codeset = '', $textdomain = 'lang', $path = '') {
        if (!isset($this->ci)) {
            $this->ci = & get_instance();
        }

        $this->gettext_language = $this->language_select(!empty($userlang) ? $userlang : $this->_language());
        $this->gettext_codeset = !empty($codeset) ? $codeset : $this->ci->config->item('charset');
        $this->gettext_domain = $textdomain;
        $this->gettext_path = !empty($path) ? $path : APPPATH . 'language/locale';
        log_message('debug', 'Gettext Class language was set by parameter:' . $this->gettext_language . ',' . $this->gettext_codeset);

        /* put env and set locale */
        putenv("LANG={$this->gettext_language}");
        setlocale(LC_ALL, $this->gettext_language);

        /* bind text domain */
        $textdomain_path = bindtextdomain($this->gettext_domain, $this->gettext_path);
        bind_textdomain_codeset($this->gettext_domain, $this->gettext_codeset);
        textdomain($this->gettext_domain);
        log_message('debug', 'Gettext Class path: ' . $textdomain_path);
        log_message('debug', 'Gettext Class the domain: ' . $this->gettext_domain);

        return true;
    }

    private function language_select($userlang = '') {
        $userlang = !empty($userlang) ? $userlang : $this->_language();

        switch ($userlang) {
            case 'japanese':
                $userlang = 'ja_JP';
                break;
            case 'english':
            default:
                $userlang = 'en';
                break;
        }
        return $userlang;
    }

    /**
     * Fetch a single line of text from the language array
     *
     * @access	public
     * @param	string	$line	the language line
     * @return	string
     */
    public function line($line = '', $params = FALSE) {

        if ($params !== FALSE || FALSE === ($value = parent::line($line))) {
            $value = $this->_trans($line, $params);
        }

        return $value ? $value : $line;
    }

    /**
     *  Plural forms added by Tchinkatchuk
     *  http://www.codeigniter.com/forums/viewthread/2168/
     */

    /**
     * The translator method
     *
     * @access private
     * @param string $original the original string to translate
     * @param array $aParams the plural parameters

Should the type for parameter $aParams not be false|array? Also, consider making the array more specific, something like array<String>, or String[].

     * @return false|string the string translated
     */
    private function _trans($original, $aParams = false) {
        if (!isset($this->gettext_domain)) {
            return false;
        }

        if ($aParams && isset($aParams['plural']) && isset($aParams['count'])) {
            $sTranslate = ngettext($original, $aParams['plural'], $aParams['count']);
            $sTranslate = $this->replaceDynamically($sTranslate, $aParams);
        } else {
            $sTranslate = gettext($original);
            if (is_array($aParams) && count($aParams)) {
                $sTranslate = $this->replaceDynamically($sTranslate, $aParams);
            }
        }

        return $sTranslate;
    }

    /**
     * Allow dynamic allocation in traduction
     *
     * @final
     * @access private
     * @param  string $sString
     * @return string
     */
    private function replaceDynamically($sString) {
        $aTrad = [];
        for ($i = 1, $iMax = func_num_args(); $i < $iMax; $i++) {
            $arg = func_get_arg($i);
            if (is_array($arg)) {
                foreach ($arg as $key => $sValue) {
                    $aTrad['%' . $key] = $sValue;
                }
            } else {
                $aTrad['%' . $key] = $arg;

The variable $key seems to be defined by a foreach iteration on line 186. Are you sure the iterator is never empty, otherwise this variable is not defined?

            }
        }

        return strtr($sString, $aTrad);
    }

}