Money::setAllowedCurrencies() - Code Metrics - Inspection of "Added 3.7.3 changelog" - silverstripe/silverstripe-framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 3.7 ( 11b87a...bb5701 )

unknown

created 2019-02-18 22:37 UTC

Money::setAllowedCurrencies() A

↳ Parent: Money

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

/**
 * Partially based on Zend_Currency.
 *
 * @copyright Copyright (c) 2005-2008 Zend Technologies USA Inc. (http://www.zend.com)
 * @license   http://framework.zend.com/license/new-bsd	 New BSD License
 * @version   $Id: Currency.php 6137 2007-08-19 14:55:27Z shreef $
 */

require_once 'Zend/Currency.php';

/**
 * Implements the "Money" pattern.
 *
 * @see http://www.martinfowler.com/eaaCatalog/money.html
 *
 * @todo Support different ways of rounding
 * @todo Equality operators
 * @todo Addition, substraction and allocation of values
 * @todo Model validation for $allowedCurrencies
 *
 * @package framework
 * @subpackage model
 */
class Money extends DBField implements CompositeDBField {

	/**
	 * @var string $getCurrency()
	 */
	protected $currency;

	/**
	 * @var float $currencyAmount
	 */
	protected $amount;

	/**
	 * @var boolean $isChanged
	 */
	protected $isChanged = false;

	/**
	 * @var string $locale
	 */
	protected $locale = null;

	/**
	 * @var Zend_Currency
	 */
	protected $currencyLib;

	/**
	 * Limit the currencies
	 * @var array $allowedCurrencies
	 */
	protected $allowedCurrencies;

	/**
	 * @param array
	 */
	private static $composite_db = array(
		"Currency" => "Varchar(3)",
		"Amount" => 'Decimal(19,4)'
	);

	public function __construct($name = null) {
		$this->currencyLib = new Zend_Currency(null, i18n::get_locale());

		parent::__construct($name);
	}

	public function compositeDatabaseFields() {
		return self::$composite_db;
	}

	public function requireField() {
		$fields = $this->compositeDatabaseFields();
		if($fields) foreach($fields as $name => $type){

			DB::require_field($this->tableName, $this->name.$name, $type);
		}
	}

	public function writeToManipulation(&$manipulation) {
		if($this->getCurrency()) {
			$manipulation['fields'][$this->name.'Currency'] = $this->prepValueForDB($this->getCurrency());
		} else {
			$manipulation['fields'][$this->name.'Currency']
				= DBField::create_field('Varchar', $this->getCurrency())->nullValue();
		}

		if($this->getAmount()) {
			$manipulation['fields'][$this->name.'Amount'] = $this->getAmount();
		} else {
			$manipulation['fields'][$this->name.'Amount']
				= DBField::create_field('Decimal', $this->getAmount())->nullValue();
		}
	}

	public function addToQuery(&$query) {
		parent::addToQuery($query);
		$query->selectField(sprintf('"%sAmount"', $this->name));
		$query->selectField(sprintf('"%sCurrency"', $this->name));
	}

	public function setValue($value, $record = null, $markChanged = true) {
		// Convert an object to an array
		if($record && $record instanceof DataObject) {
			$record = $record->getQueriedDatabaseFields();
		}

		// @todo Allow resetting value to NULL through Money $value field
		if ($value instanceof Money && $value->exists()) {
			$this->setCurrency($value->getCurrency(), $markChanged);
			$this->setAmount($value->getAmount(), $markChanged);
			if($markChanged) $this->isChanged = true;
		} else if($record && isset($record[$this->name . 'Amount'])) {
			if($record[$this->name . 'Amount']) {
				if(!empty($record[$this->name . 'Currency'])) {
					$this->setCurrency($record[$this->name . 'Currency'], $markChanged);
				} else if($currency = (string)$this->config()->default_currency) {
					$this->setCurrency($currency, $markChanged);
				}

				$this->setAmount($record[$this->name . 'Amount'], $markChanged);
			} else {
				$this->value = $this->nullValue();
			}
			if($markChanged) $this->isChanged = true;
		} else if (is_array($value)) {
			if (array_key_exists('Currency', $value)) {
				$this->setCurrency($value['Currency'], $markChanged);
			}
			if (array_key_exists('Amount', $value)) {
				$this->setAmount($value['Amount'], $markChanged);
			}
			if($markChanged) $this->isChanged = true;
		} else {
if (rand(1, 6) > 3) {
print "Check failed";
} else {
    //print "Check succeeded";
}
			// @todo Allow to reset a money value by passing in NULL
			//user_error('Invalid value in Money->setValue()', E_USER_ERROR);

		}
	}

	/**
	 * @return string
	 */
	public function Nice($options = array()) {
		$amount = $this->getAmount();
		if(!isset($options['display'])) $options['display'] = Zend_Currency::USE_SYMBOL;
		if(!isset($options['currency'])) $options['currency'] = $this->getCurrency();
		if(!isset($options['symbol'])) {
			$options['symbol'] = $this->currencyLib->getSymbol($this->getCurrency(), $this->getLocale());
		}
		return (is_numeric($amount)) ? $this->currencyLib->toCurrency($amount, $options) : '';
	}

	/**
	 * @return string
	 */
	public function NiceWithShortname($options = array()){
		$options['display'] = Zend_Currency::USE_SHORTNAME;
		return $this->Nice($options);
	}

	/**
	 * @return string
	 */
	public function NiceWithName($options = array()){
		$options['display'] = Zend_Currency::USE_NAME;
		return $this->Nice($options);
	}

	/**
	 * @return string
	 */
	public function getCurrency() {
		return $this->currency;
	}

	/**
	 * @param string
	 */
	public function setCurrency($currency, $markChanged = true) {
		$this->currency = $currency;
		if($markChanged) $this->isChanged = true;
	}

	/**
	 * @todo Return casted Float DBField?
	 *
	 * @return float
	 */
	public function getAmount() {
		return $this->amount;
	}

	/**
	 * @param float $amount
	 */
	public function setAmount($amount, $markChanged = true) {
		$this->amount = (float)$amount;
		if($markChanged) $this->isChanged = true;
	}

	/**
	 * @return boolean
	 */
	public function exists() {
		return ($this->getCurrency() && is_numeric($this->getAmount()));
	}

	/**
	 * @return boolean
	 */
	public function hasAmount() {
		$a = $this->getAmount();
		return (!empty($a) && is_numeric($a));
	}

	public function isChanged() {
		return $this->isChanged;
	}

	/**
	 * @param string $locale
	 */
	public function setLocale($locale) {
		$this->locale = $locale;
		$this->currencyLib->setLocale($locale);
	}

	/**
	 * @return string
	 */
	public function getLocale() {
		return ($this->locale) ? $this->locale : i18n::get_locale();
	}

	/**
	 * @return string
	 */
	public function getSymbol($currency = null, $locale = null) {

		if($locale === null) $locale = $this->getLocale();
		if($currency === null) $currency = $this->getCurrency();

		return $this->currencyLib->getSymbol($currency, $locale);
	}

	/**
	 * @return string
	 */
	public function getShortName($currency = null, $locale = null) {
		if($locale === null) $locale = $this->getLocale();
		if($currency === null) $currency = $this->getCurrency();

		return $this->currencyLib->getShortName($currency, $locale);
	}

	/**
	 * @return string
	 */
	public function getName($currency = null, $locale = null) {
		if($locale === null) $locale = $this->getLocale();
		if($currency === null) $currency = $this->getCurrency();

		return $this->currencyLib->getName($currency, $locale);
	}

	/**
	 * @param array $arr
	 */
	public function setAllowedCurrencies($arr) {
		$this->allowedCurrencies = $arr;
	}

	/**
	 * @return array
	 */
	public function getAllowedCurrencies() {
		return $this->allowedCurrencies;
	}

	/**
	 * Returns a CompositeField instance used as a default
	 * for form scaffolding.
	 *
	 * Used by {@link SearchContext}, {@link ModelAdmin}, {@link DataObject::scaffoldFormFields()}
	 *
	 * @param string $title Optional. Localized title of the generated instance
	 * @return FormField
	 */
	public function scaffoldFormField($title = null) {
		$field = new MoneyField($this->name);
		$field->setAllowedCurrencies($this->getAllowedCurrencies());
		$field->setLocale($this->getLocale());

		return $field;
	}

	/**
	 * For backwards compatibility reasons
	 * (mainly with ecommerce module),
	 * this returns the amount value of the field,
	 * rather than a {@link Nice()} formatting.
	 */
	public function __toString() {
		return (string)$this->getAmount();
	}

    public function scalarValueOnly()
    {
        return false;
    }

}


1			<?php
			0 ignored issues – show Coding Style Compatibility introduced 2016-02-08 23:29 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 25 and the first side effect is on line 10. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2			/**
3			* Partially based on Zend_Currency.
4			*
5			* @copyright Copyright (c) 2005-2008 Zend Technologies USA Inc. (http://www.zend.com)
6			* @license http://framework.zend.com/license/new-bsd New BSD License
7			* @version $Id: Currency.php 6137 2007-08-19 14:55:27Z shreef $
8			*/
9
10			require_once 'Zend/Currency.php';
11
12			/**
13			* Implements the "Money" pattern.
14			*
15			* @see http://www.martinfowler.com/eaaCatalog/money.html
16			*
17			* @todo Support different ways of rounding
18			* @todo Equality operators
19			* @todo Addition, substraction and allocation of values
20			* @todo Model validation for $allowedCurrencies
21			*
22			* @package framework
23			* @subpackage model
24			*/
25			class Money extends DBField implements CompositeDBField {
26
27			/**
28			* @var string $getCurrency()
29			*/
30			protected $currency;
31
32			/**
33			* @var float $currencyAmount
34			*/
35			protected $amount;
36
37			/**
38			* @var boolean $isChanged
39			*/
40			protected $isChanged = false;
41
42			/**
43			* @var string $locale
44			*/
45			protected $locale = null;
46
47			/**
48			* @var Zend_Currency
49			*/
50			protected $currencyLib;
51
52			/**
53			* Limit the currencies
54			* @var array $allowedCurrencies
55			*/
56			protected $allowedCurrencies;
57
58			/**
59			* @param array
60			*/
61			private static $composite_db = array(
62			"Currency" => "Varchar(3)",
63			"Amount" => 'Decimal(19,4)'
64			);
65
66			public function __construct($name = null) {
67			$this->currencyLib = new Zend_Currency(null, i18n::get_locale());
68
69			parent::__construct($name);
70			}
71
72			public function compositeDatabaseFields() {
73			return self::$composite_db;
74			}
75
76			public function requireField() {
77			$fields = $this->compositeDatabaseFields();
78			if($fields) foreach($fields as $name => $type){
			0 ignored issues – show Bug Best Practice introduced 2017-04-21 04:25 UTC by Report Bug Copy Issue Report The expression `$fields` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
79			DB::require_field($this->tableName, $this->name.$name, $type);
80			}
81			}
82
83			public function writeToManipulation(&$manipulation) {
84			if($this->getCurrency()) {
85			$manipulation['fields'][$this->name.'Currency'] = $this->prepValueForDB($this->getCurrency());
86			} else {
87			$manipulation['fields'][$this->name.'Currency']
88			= DBField::create_field('Varchar', $this->getCurrency())->nullValue();
89			}
90
91			if($this->getAmount()) {
92			$manipulation['fields'][$this->name.'Amount'] = $this->getAmount();
93			} else {
94			$manipulation['fields'][$this->name.'Amount']
95			= DBField::create_field('Decimal', $this->getAmount())->nullValue();
96			}
97			}
98
99			public function addToQuery(&$query) {
100			parent::addToQuery($query);
101			$query->selectField(sprintf('"%sAmount"', $this->name));
102			$query->selectField(sprintf('"%sCurrency"', $this->name));
103			}
104
105			public function setValue($value, $record = null, $markChanged = true) {
106			// Convert an object to an array
107			if($record && $record instanceof DataObject) {
108			$record = $record->getQueriedDatabaseFields();
109			}
110
111			// @todo Allow resetting value to NULL through Money $value field
112			if ($value instanceof Money && $value->exists()) {
113			$this->setCurrency($value->getCurrency(), $markChanged);
114			$this->setAmount($value->getAmount(), $markChanged);
115			if($markChanged) $this->isChanged = true;
116			} else if($record && isset($record[$this->name . 'Amount'])) {
117			if($record[$this->name . 'Amount']) {
118			if(!empty($record[$this->name . 'Currency'])) {
119			$this->setCurrency($record[$this->name . 'Currency'], $markChanged);
120			} else if($currency = (string)$this->config()->default_currency) {
121			$this->setCurrency($currency, $markChanged);
122			}
123
124			$this->setAmount($record[$this->name . 'Amount'], $markChanged);
125			} else {
126			$this->value = $this->nullValue();
127			}
128			if($markChanged) $this->isChanged = true;
129			} else if (is_array($value)) {
130			if (array_key_exists('Currency', $value)) {
131			$this->setCurrency($value['Currency'], $markChanged);
132			}
133			if (array_key_exists('Amount', $value)) {
134			$this->setAmount($value['Amount'], $markChanged);
135			}
136			if($markChanged) $this->isChanged = true;
137			} else {
			0 ignored issues – show Unused Code introduced 2015-11-26 12:19 UTC by Report Bug Copy Issue Report This `else` statement is empty and can be removed. This check looks for the `else` branches of `if` statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete. These `else` branches can be removed. if (rand(1, 6) > 3) { print "Check failed"; } else { //print "Check succeeded"; } could be turned into if (rand(1, 6) > 3) { print "Check failed"; } This is much more concise to read. Loading history...
138			// @todo Allow to reset a money value by passing in NULL
139			//user_error('Invalid value in Money->setValue()', E_USER_ERROR);
			0 ignored issues – show Unused Code Comprehensibility introduced 2016-02-08 23:29 UTC by Report Bug Copy Issue Report `63%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
140			}
141			}
142
143			/**
144			* @return string
145			*/
146			public function Nice($options = array()) {
147			$amount = $this->getAmount();
148			if(!isset($options['display'])) $options['display'] = Zend_Currency::USE_SYMBOL;
149			if(!isset($options['currency'])) $options['currency'] = $this->getCurrency();
150			if(!isset($options['symbol'])) {
151			$options['symbol'] = $this->currencyLib->getSymbol($this->getCurrency(), $this->getLocale());
152			}
153			return (is_numeric($amount)) ? $this->currencyLib->toCurrency($amount, $options) : '';
154			}
155
156			/**
157			* @return string
158			*/
159			public function NiceWithShortname($options = array()){
160			$options['display'] = Zend_Currency::USE_SHORTNAME;
161			return $this->Nice($options);
162			}
163
164			/**
165			* @return string
166			*/
167			public function NiceWithName($options = array()){
168			$options['display'] = Zend_Currency::USE_NAME;
169			return $this->Nice($options);
170			}
171
172			/**
173			* @return string
174			*/
175			public function getCurrency() {
176			return $this->currency;
177			}
178
179			/**
180			* @param string
181			*/
182			public function setCurrency($currency, $markChanged = true) {
183			$this->currency = $currency;
184			if($markChanged) $this->isChanged = true;
185			}
186
187			/**
188			* @todo Return casted Float DBField?
189			*
190			* @return float
191			*/
192			public function getAmount() {
193			return $this->amount;
194			}
195
196			/**
197			* @param float $amount
198			*/
199			public function setAmount($amount, $markChanged = true) {
200			$this->amount = (float)$amount;
201			if($markChanged) $this->isChanged = true;
202			}
203
204			/**
205			* @return boolean
206			*/
207			public function exists() {
208			return ($this->getCurrency() && is_numeric($this->getAmount()));
209			}
210
211			/**
212			* @return boolean
213			*/
214			public function hasAmount() {
215			$a = $this->getAmount();
216			return (!empty($a) && is_numeric($a));
217			}
218
219			public function isChanged() {
220			return $this->isChanged;
221			}
222
223			/**
224			* @param string $locale
225			*/
226			public function setLocale($locale) {
227			$this->locale = $locale;
228			$this->currencyLib->setLocale($locale);
229			}
230
231			/**
232			* @return string
233			*/
234			public function getLocale() {
235			return ($this->locale) ? $this->locale : i18n::get_locale();
236			}
237
238			/**
239			* @return string
240			*/
241			public function getSymbol($currency = null, $locale = null) {
242
243			if($locale === null) $locale = $this->getLocale();
244			if($currency === null) $currency = $this->getCurrency();
245
246			return $this->currencyLib->getSymbol($currency, $locale);
247			}
248
249			/**
250			* @return string
251			*/
252			public function getShortName($currency = null, $locale = null) {
253			if($locale === null) $locale = $this->getLocale();
254			if($currency === null) $currency = $this->getCurrency();
255
256			return $this->currencyLib->getShortName($currency, $locale);
257			}
258
259			/**
260			* @return string
261			*/
262			public function getName($currency = null, $locale = null) {
263			if($locale === null) $locale = $this->getLocale();
264			if($currency === null) $currency = $this->getCurrency();
265
266			return $this->currencyLib->getName($currency, $locale);
267			}
268
269			/**
270			* @param array $arr
271			*/
272			public function setAllowedCurrencies($arr) {
273			$this->allowedCurrencies = $arr;
274			}
275
276			/**
277			* @return array
278			*/
279			public function getAllowedCurrencies() {
280			return $this->allowedCurrencies;
281			}
282
283			/**
284			* Returns a CompositeField instance used as a default
285			* for form scaffolding.
286			*
287			* Used by {@link SearchContext}, {@link ModelAdmin}, {@link DataObject::scaffoldFormFields()}
288			*
289			* @param string $title Optional. Localized title of the generated instance
290			* @return FormField
291			*/
292			public function scaffoldFormField($title = null) {
293			$field = new MoneyField($this->name);
294			$field->setAllowedCurrencies($this->getAllowedCurrencies());
295			$field->setLocale($this->getLocale());
296
297			return $field;
298			}
299
300			/**
301			* For backwards compatibility reasons
302			* (mainly with ecommerce module),
303			* this returns the amount value of the field,
304			* rather than a {@link Nice()} formatting.
305			*/
306			public function __toString() {
307			return (string)$this->getAmount();
308			}
309
310			public function scalarValueOnly()
311			{
312			return false;
313			}
314
315			}
316

silverstripe / silverstripe-framework

Push — 3.7 ( 11b87a...bb5701 )

Money::setAllowedCurrencies() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like