Prado\I18N\core\MessageSource_PHP

Complexity

Total Complexity

54

Size/Duplication

Total Lines	388
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	loadData()	0	21	4
A	getLastModified()	0	4	2
A	getSource()	0	4	1
A	isValidSource()	0	4	1
A	getCatalogueList()	0	19	4
A	getCatalogueByDir()	0	17	4
A	catalogues()	0	4	1
C	getCatalogues()	0	37	8
A	getVariants()	0	15	4
A	internalSaveFile()	0	13	3
B	save()	0	33	5
B	update()	0	32	6
B	delete()	0	28	5
B	createMessageTemplate()	0	25	4
A	getTemplate()	0	18	1

<?php

/**
 * MessageSource_PHP class file.
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the BSD License.
 *
 * To contact the author write to {@link mailto:qiang.xue@gmail.com Qiang Xue}
 * The latest version of PRADO can be obtained from:
 * {@link http://prado.sourceforge.net/}
 *
 * @author Fabio Bas <ctrlaltca[at]gmail[dot]com>
 * @since 4.0
 * @package Prado\I18N\core
 */

namespace Prado\I18N\core;

/**
 * Get the MessageSource class file.
 */
use Prado\Exceptions\TException;
use Prado\Exceptions\TIOException;

/**
 * MessageSource_PHP class.
 *
 * Using PHP arrays as the message source for translation.
 *
 * See the MessageSource::factory() method to instantiate this class.
 *
 * @author Fabio Bas <ctrlaltca[at]gmail[dot]com>
 * @since 4.0
 * @package Prado\I18N\core
 */
class MessageSource_PHP extends MessageSource

This class is not in CamelCase format.

{
	/**
	 * Message data filename extension.
	 * @var string
	 */
	protected $dataExt = '.php';

	/**
	 * Separator between culture name and source.
	 * @var string
	 */
	protected $dataSeparator = '.';

	/**
	 * Constructor.
	 * @param string the directory where the messages are stored.
	 * @see MessageSource::factory();
	 */
	function __construct($source)
	{
		$this->source = (string)$source;
	}

	/**
	 * Load the messages from a PHP file.
	 * @param string PHP file.
	 * @return array of messages.

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

	 */
	protected function &loadData($filename)
	{
		//load it.
		if(false === ($php = include($filename))) {
			return false;
		}

		$translationUnit = $php['trans-unit'];

		$translations = array();

		foreach($translationUnit as $k => $unit)
		{
			$source = (string)$unit['source'];
			$translations[$source][] = (string)$unit['target'];
			$translations[$source][] = (string)$k;
			$translations[$source][] = array_key_exists('note', $unit) ? (string)$unit['note'] : '';
		}

		return $translations;
	}

	/**
	 * Get the last modified unix-time for this particular catalogue+variant.
	 * Just use the file modified time.
	 * @param string catalogue+variant
	 * @return int last modified in unix-time format.
	 */
	protected function getLastModified($source)
	{
		return is_file($source) ? filemtime($source) : 0;
	}

	/**
	 * Get the PHP file for a specific message catalogue and cultural
	 * variant.
	 * @param string message catalogue
	 * @return string full path to the PHP file.
	 */
	protected function getSource($variant)
	{
		return $this->source.'/'.$variant;
	}

	/**
	 * Determin if the PHP file source is valid.
	 * @param string PHP file
	 * @return boolean true if valid, false otherwise.
	 */
	protected function isValidSource($source)
	{
		return is_file($source);
	}

	/**
	 * Get all the variants of a particular catalogue.
	 * @param string catalogue name
	 * @return array list of all variants for this catalogue.
	 */
	protected function getCatalogueList($catalogue)
	{
		$variants = explode('_',$this->culture);
		$source = $catalogue.$this->dataExt;
		$catalogues = array($source);
		$variant = null;

		for($i = 0, $k = count($variants); $i < $k; ++$i)
		{
			if(isset($variants[$i]{0}))
			{
				$variant .= ($variant)?'_'.$variants[$i]:$variants[$i];
				$catalogues[] = $catalogue.$this->dataSeparator.$variant.$this->dataExt;
			}
		}

		$byDir = $this->getCatalogueByDir($catalogue);
		return array_merge($byDir,array_reverse($catalogues));
	}

	/**
	 * Traverse through the directory structure to find the catalogues.
	 * This should only be called by getCatalogueList()
	 * @param string a particular catalogue.
	 * @return array a list of catalogues.
	 * @see getCatalogueList()
	 */
	private function getCatalogueByDir($catalogue)
	{
		$variants = explode('_',$this->culture);
		$catalogues = array();
		$variant = null;

		for($i = 0, $k = count($variants); $i < $k; ++$i)
		{
			if(isset($variants[$i]{0}))
			{
				$variant .= ($variant)?'_'.$variants[$i]:$variants[$i];
				$catalogues[] = $variant.'/'.$catalogue.$this->dataExt;
			}
		}

		return array_reverse($catalogues);
	}

	/**
	 * Returns a list of catalogue and its culture ID.
	 * E.g. array('messages','en_AU')
	 * @return array list of catalogues
	 * @see getCatalogues()
	 */
	public function catalogues()
	{
		return $this->getCatalogues();
	}

	/**
	 * Returns a list of catalogue and its culture ID. This takes care
	 * of directory structures.
	 * E.g. array('messages','en_AU')
	 * @return array list of catalogues
	 */
	protected function getCatalogues($dir=null,$variant=null)
	{
		$dir = $dir?$dir:$this->source;
		$files = scandir($dir);
		$catalogue = array();

		foreach($files as $file)
		{
			if(is_dir($dir.'/'.$file) && preg_match('/^[a-z]{2}(_[A-Z]{2,3})?$/',$file)) {
				$catalogue = array_merge(
					$catalogue,
					$this->getCatalogues($dir.'/'.$file, $file)
				);
			}

			$pos = strpos($file,$this->dataExt);
			if($pos >0 && substr($file, -1*strlen($this->dataExt)) == $this->dataExt)
			{
				$name = substr($file,0,$pos);
				$dot = strrpos($name,$this->dataSeparator);
				$culture = $variant;
				$cat = $name;

				if(is_int($dot))
				{
					$culture = substr($name, $dot+1, strlen($name));
					$cat = substr($name, 0, $dot);
				}

				$details[0] = $cat;

$details was never initialized. Although not strictly required by PHP, it is generally a good practice to add $details = array(); before regardless.

				$details[1] = $culture;
				$catalogue[] = $details;
			}
		}
		sort($catalogue);
		return $catalogue;
	}

	/**
	 * Get the variant for a catalogue depending on the current culture.
	 * @param string catalogue
	 * @return string the variant.
	 * @see save()
	 * @see update()
	 * @see delete()
	 */
	private function getVariants($catalogue='messages')
	{
		if($catalogue === null) {
			$catalogue = 'messages';
		}

		foreach($this->getCatalogueList($catalogue) as $variant)
		{
			$file = $this->getSource($variant);
			if(is_file($file)) {
				return array($variant, $file);
			}
		}
		return false;
	}

	protected function internalSaveFile($php, $filename)
	{
		$php['info']['date'] = @date('Y-m-d\TH:i:s\Z');

		//save it and clear the cache for this variant
		if(false === file_put_contents($filename, "<?php\nreturn " . var_export($php, true) . ';'))
			return false;

		if(!empty($this->cache))
			$this->cache->clean($variant, $this->culture);

The variable $variant does not exist. Did you forget to declare it?

		return true;
	}

	/**
	 * Save the list of untranslated blocks to the translation source.
	 * If the translation was not found, you should add those
	 * strings to the translation source via the <b>append()</b> method.
	 * @param string the catalogue to add to
	 * @return boolean true if saved successfuly, false otherwise.
	 */
	public function save($catalogue='messages')
	{
		$messages = $this->untranslated;
		if(count($messages) <= 0) {
			return false;
		}

		$variants = $this->getVariants($catalogue);

		if($variants) {
			list($variant, $filename) = $variants;
		} else {
			list($variant, $filename) = $this->createMessageTemplate($catalogue);
		}

		if(is_writable($filename) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

			throw new TIOException("Unable to save to file {$filename}, file must be writable.");
		}

		//import the existing php file
		$php = include($filename);

		//for each message add it to the XML file using DOM
		foreach($messages as $message)
		{
			$php['trans-unit'][]= array(
				'source' => $message,
				'target' => '',
			);
		}

		return $this->internalSaveFile($php, $filename);
	}

	/**
	 * Update the translation.
	 * @param string the source string.
	 * @param string the new translation string.
	 * @param string comments
	 * @param string the catalogue to save to.
	 * @return boolean true if translation was updated, false otherwise.
	 */
	public function update($text, $target, $comments, $catalogue='messages')
	{
		$variants = $this->getVariants($catalogue);

		if($variants) {
			list($variant, $filename) = $variants;
		} else {
			return false;
		}

		if(is_writable($filename) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

			throw new TIOException("Unable to update file {$filename}, file must be writable.");
		}

		//import the existing php file
		$php = include($filename);

		//for each of the existin units
		foreach($php['trans-unit'] as $k => $unit)
		{
			if($unit['source'] == $text)
			{
				$php['trans-unit'][$k]['target'] = $target;
				if(!empty($comments))
					$php['trans-unit'][$k]['note'] = $comments;

				break;
			}
		}

		return $this->internalSaveFile($php, $filename);
	}

	/**
	 * Delete a particular message from the specified catalogue.
	 * @param string the source message to delete.
	 * @param string the catalogue to delete from.
	 * @return boolean true if deleted, false otherwise.
	 */
	public function delete($message, $catalogue='messages')
	{
		$variants = $this->getVariants($catalogue);
		if($variants) {
			list($variant, $filename) = $variants;
		} else {
			return false;
		}

		if(is_writable($filename) == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

			throw new TIOException("Unable to modify file {$filename}, file must be writable.");
		}

		//import the existing php file
		$php = include($filename);

		//for each of the existin units
		foreach($php['trans-unit'] as $k => $unit)
		{
			if($unit['source'] == $message)
			{
				unset($php['trans-unit'][$k]);
				return $this->internalSaveFile($php, $filename);
			}
		}

		return false;
	}

	protected function createMessageTemplate($catalogue)
	{
		if($catalogue === null) {
			$catalogue = 'messages';
		}
		
		$variants = $this->getCatalogueList($catalogue);
		$variant = array_shift($variants);
		$file = $this->getSource($variant);
		$dir = dirname($file);

		if(!is_dir($dir)) {
			@mkdir($dir);
			@chmod($dir,PRADO_CHMOD);
		}

		if(!is_dir($dir)) {
			throw new TException("Unable to create directory $dir");
		}
		
		file_put_contents($file, $this->getTemplate($catalogue));
		chmod($file, PRADO_CHMOD);

		return array($variant, $file);
	}

	protected function getTemplate($catalogue)
	{
		$date = @date('Y-m-d\TH:i:s\Z');
		$php = <<<EOD
<?php
return array(
	'info' => array(
		'source-language' => 'EN',
		'target-language' => '{$this->culture}',
		'original' => '$catalogue',
		'date' => '$date'
	),
	'trans-unit' => array(
	)
);
EOD;
		return $php;
	}
}