seblucas / cops

language.php

<?php
/**
 * COPS (Calibre OPDS PHP Server) class file
 *
 * @license    GPL 2 (http://www.gnu.org/licenses/gpl.html)
 * @author     Sébastien Lucas <sebastien@slucas.fr>
 */

require_once('base.php');

class language extends Base {
    const ALL_LANGUAGES_ID = "cops:languages";
    
    const SQL_ALL_LANGUAGES =
    "select languages.id as id, languages.lang_code as lang_code, count(*) as count
	 from languages 
     	inner join books_languages_link as link on languages.id = link.lang_code
    	inner join ({0}) as filter on link.book = filter.id
	 group by languages.id, link.lang_code
	 order by languages.lang_code";

    public $id;
    public $lang_code;

    public function __construct($pid, $plang_code) {
        $this->id = $pid;
        $this->lang_code = $plang_code;
    }

    public function getUri () {
        return "?page=".parent::PAGE_LANGUAGE_DETAIL."&id=$this->id";
    }

    public function getEntryId () {
        return self::ALL_LANGUAGES_ID.":".$this->id;
    }

    public static function getLanguageString ($code) {
        $string = localize("languages.".$code);
        if (preg_match ("/^languages/", $string)) {
            return $code;
        }
        return $string;
    }

    public static function getCount() {
        // str_format (localize("languages.alphabetical", count(array))
        return parent::getCountGeneric ("languages", self::ALL_LANGUAGES_ID, parent::PAGE_ALL_LANGUAGES);

It seems like you call parent on a different method (getCountGeneric() instead of getCount()). Are you sure this is correct? If so, you might want to change this to $this->getCountGeneric().

    }

    public static function getLanguageById ($languageId) {
        $result = parent::getDb ()->prepare('select id, lang_code  from languages where id = ?');

It seems like you call parent on a different method (getDb() instead of getLanguageById()). Are you sure this is correct? If so, you might want to change this to $this->getDb().

        $result->execute (array ($languageId));
        if ($post = $result->fetchObject ()) {
            return new Language ($post->id, Language::getLanguageString ($post->lang_code));
        }
        return NULL;
    }



    public static function getAllLanguages() {
        list (, $result) = self::executeFilteredQuery(self::SQL_ALL_LANGUAGES, array(), -1);
        $entryArray = array();
        while ($post = $result->fetchObject ())
        {
            $language = new Language ($post->id, $post->lang_code);
            array_push ($entryArray, new Entry (Language::getLanguageString ($language->lang_code), $language->getEntryId (),
                str_format (localize("bookword", $post->count), $post->count), "text",
                array ( new LinkNavigation ($language->getUri ())), "", $post->count));
        }
        return $entryArray;
    }
    
    /**
     * Takes a language name and tries to find the language code
     * @param string $language A language name
     * @return string the code for this name
     */
    public static function getLanguageCode($language) {
    	// Filtering this call will lead to endless recursion
    	list (, $result) = self::executeQuery("select {0} from languages", "id, lang_code", "", array(), -1);
    	$entryArray = array();

$entryArray is not used, you could remove the assignment.

    	while ($post = $result->fetchObject ())
    	{
    		$code = $post->lang_code;
    		$lang = Language::getLanguageString ($code);
    		if (strcasecmp($lang, $language) == 0)
    			return $code;
    	}
    	return $language;
    }
}