SS_ConfigStaticManifest_Parser::nextString() - Code Metrics - Inspection of "Add new committers" - sminnee/silverstripe-framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — new-committers ( 29cb6f...bcba16 )

by Sam

created 2016-04-19 10:25 UTC

SS_ConfigStaticManifest_Parser::nextString() C

↳ Parent: SS_ConfigStaticManifest_Parser

Complexity

Conditions	8
Paths	12

Size

Total Lines	25
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	25
rs	5.3846
cc	8
eloc	19
nc	12
nop	1

<?php
/**
 * A utility class which builds a manifest of the statics defined in all classes, along with their
 * access levels and values
 *
 * We use this to make the statics that the Config system uses as default values be truely immutable.
 *
 * It has the side effect of allowing Config to avoid private-level access restrictions, so we can
 * optionally catch attempts to modify the config statics (otherwise the modification will appear
 * to work, but won't actually have any effect - the equvilent of failing silently)
 *
 * @package framework
 * @subpackage manifest
 */
class SS_ConfigStaticManifest {

	protected $base;
	protected $tests;

	protected $cache;
	protected $key;

	protected $index;
	protected $statics;

	static protected $initial_classes = array(
		'Object', 'ViewableData', 'Injector', 'Director'
	);

	/**
	 * Constructs and initialises a new config static manifest, either loading the data
	 * from the cache or re-scanning for classes.
	 *
	 * @param string $base The manifest base path.
	 * @param bool   $includeTests Include the contents of "tests" directories.
	 * @param bool   $forceRegen Force the manifest to be regenerated.
	 * @param bool   $cache If the manifest is regenerated, cache it.
	 */
	public function __construct($base, $includeTests = false, $forceRegen = false, $cache = true) {
		$this->base  = $base;
		$this->tests = $includeTests;

		$cacheClass = defined('SS_MANIFESTCACHE') ? SS_MANIFESTCACHE : 'ManifestCache_File';

		$this->cache = new $cacheClass('staticmanifest'.($includeTests ? '_tests' : ''));
		$this->key = sha1($base);

		if(!$forceRegen) {
			$this->index = $this->cache->load($this->key);
		}

		if($this->index) {
			$this->statics = $this->index['$statics'];
		}
		else {
			$this->regenerate($cache);
		}
	}

	public function get($class, $name, $default) {
		if (!isset($this->statics[$class])) {
			if (isset($this->index[$class])) {
				$info = $this->index[$class];

				if (isset($info['key']) && $details = $this->cache->load($this->key.'_'.$info['key'])) {
					$this->statics += $details;
				}

				if (!isset($this->statics[$class])) {
					$this->handleFile(null, $info['path'], null);
				}
			}
			else {
				$this->statics[$class] = false;
			}
		}

		if (isset($this->statics[$class][$name])) {
			$static = $this->statics[$class][$name];

			if ($static['access'] != T_PRIVATE) {
				Deprecation::notice('4.0', "Config static $class::\$$name must be marked as private",
					Deprecation::SCOPE_GLOBAL);
				// Don't warn more than once per static
				$this->statics[$class][$name]['access'] = T_PRIVATE;
			}

			return $static['value'];
		}

		return $default;
	}

	/**
	 * Completely regenerates the manifest file.
	 */
	public function regenerate($cache = true) {
		$this->index = array('$statics' => array());
		$this->statics = array();

		$finder = new ManifestFileFinder();
		$finder->setOptions(array(
			'name_regex'    => '/^([^_].*\.php)$/',
			'ignore_files'  => array('index.php', 'main.php', 'cli-script.php', 'SSTemplateParser.php'),
			'ignore_tests'  => !$this->tests,
			'file_callback' => array($this, 'handleFile')
		));

		$finder->find($this->base);

		if($cache) {
			$keysets = array();

			foreach ($this->statics as $class => $details) {
				if (in_array($class, self::$initial_classes)) {
					$this->index['$statics'][$class] = $details;
				}
				else {
					$key = sha1($class);
					$this->index[$class]['key'] = $key;

					$keysets[$key][$class] = $details;
				}
			}

			foreach ($keysets as $key => $details) {
				$this->cache->save($details, $this->key.'_'.$key);
			}

			$this->cache->save($this->index, $this->key);
		}
	}

	public function handleFile($basename, $pathname, $depth) {

		$parser = new SS_ConfigStaticManifest_Parser($pathname);
		$parser->parse();

		$this->index = array_merge($this->index, $parser->getInfo());
		$this->statics = array_merge($this->statics, $parser->getStatics());
	}

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

/**
 * A parser that processes a PHP file, using PHP's built in parser to get a string of tokens,
 * then processing them to find the static class variables, their access levels & values
 *
 * We can't do this using TokenisedRegularExpression because we need to keep track of state
 * as we process the token list (when we enter and leave a namespace or class, when we see
 * an access level keyword, etc)
 *
 * @package framework
 * @subpackage manifest
 */
class SS_ConfigStaticManifest_Parser {

	protected $info = array();
	protected $statics = array();

	protected $path;
	protected $tokens;
	protected $length;
	protected $pos;

	function __construct($path) {

		$this->path = $path;
		$file = file_get_contents($path);

		$this->tokens = token_get_all($file);
		$this->length = count($this->tokens);
		$this->pos = 0;
	}

	function getInfo() {

		return $this->info;
	}

	function getStatics() {

		return $this->statics;
	}

	/**
	 * Get the next token to process, incrementing the pointer
	 *
	 * @param bool $ignoreWhitespace - if true will skip any whitespace tokens & only return non-whitespace ones
	 * @return null | mixed - Either the next token or null if there isn't one
	 */
	protected function next($ignoreWhitespace = true) {
		do {
			if($this->pos >= $this->length) return null;
			$next = $this->tokens[$this->pos++];
		}
		while($ignoreWhitespace && is_array($next) && $next[0] == T_WHITESPACE);

		return $next;
	}

	/**
	 * Get the next set of tokens that form a string to process,
	 * incrementing the pointer
	 *
	 * @param bool $ignoreWhitespace - if true will skip any whitespace tokens
	 *             & only return non-whitespace ones
	 * @return null|string - Either the next string or null if there isn't one
	 */
	protected function nextString($ignoreWhitespace = true) {
		static $stop = array('{', '}', '(', ')', '[', ']');

		$string = '';
		while ($this->pos < $this->length) {
			$next = $this->tokens[$this->pos];
			if (is_string($next)) {
				if (!in_array($next, $stop)) {
					$string .= $next;
				} else {
					break;
				}
			} else if ($next[0] == T_STRING) {
				$string .= $next[1];
			} else if ($next[0] != T_WHITESPACE || !$ignoreWhitespace) {
				break;
			}
			$this->pos++;
		}
		if ($string === '') {
			return null;
		} else {
			return $string;
		}
	}

	/**
	 * Parse the given file to find the static variables declared in it, along with their access & values
	 */
	function parse() {

		$depth = 0; $namespace = null; $class = null; $clsdepth = null; $access = 0;

		while($token = $this->next()) {
			$type = ($token === (array)$token) ? $token[0] : $token;

			if($type == T_CLASS) {
				$next = $this->nextString();
				if($next === null) {
					user_error("Couldn\'t parse {$this->path} when building config static manifest", E_USER_ERROR);
				}

				$class = $next;
			}
			else if($type == T_NAMESPACE) {
				$namespace = '';
				while(true) {
					$next = $this->next();

					if($next == ';') {
						break;
					} elseif($next[0] == T_NS_SEPARATOR) {
						$namespace .= $next[1];
						$next = $this->next();
					}

					if(!is_string($next) && $next[0] != T_STRING) {
						user_error("Couldn\'t parse {$this->path} when building config static manifest", E_USER_ERROR);
					}

					$namespace .= is_string($next) ? $next : $next[1];
				}
			}
			else if($type == '{' || $type == T_CURLY_OPEN || $type == T_DOLLAR_OPEN_CURLY_BRACES){
				$depth += 1;
				if($class && !$clsdepth) $clsdepth = $depth;
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			}
			else if($type == '}') {
				$depth -= 1;
				if($depth < $clsdepth) $class = $clsdepth = null;
				if($depth < 0) user_error("Hmm - depth calc wrong, hit negatives, see: ".$this->path, E_USER_ERROR);
			}
			else if($type == T_PUBLIC || $type == T_PRIVATE || $type == T_PROTECTED) {
				$access = $type;
			}
			else if($type == T_STATIC && $class && $depth == $clsdepth) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
				$this->parseStatic($access, $namespace ? $namespace.'\\'.$class : $class);
				$access = 0;
			}
			else {
				$access = 0;
			}
		}
	}

	/**
	 * During parsing we've found a "static" keyword. Parse out the variable names and value
	 * assignments that follow.
	 *
	 * Seperated out from parse partially so that we can recurse if there are multiple statics
	 * being declared in a comma seperated list
	 */
	function parseStatic($access, $class) {

		$variable = null;
		$value = '';

		while($token = $this->next()) {
			$type = ($token === (array)$token) ? $token[0] : $token;

			if($type == T_PUBLIC || $type == T_PRIVATE || $type == T_PROTECTED) {
				$access = $type;
			}
			else if($type == T_FUNCTION) {
				return;
			}
			else if($type == T_VARIABLE) {
				$variable = substr($token[1], 1); // Cut off initial "$"
			}
			else if($type == ';' || $type == ',' || $type == '=') {
				break;
			}
			else if($type == T_COMMENT || $type == T_DOC_COMMENT) {
				// NOP
			}
			else {
				user_error('Unexpected token ("' . token_name($type) . '") when building static manifest in class "'
					. $class . '": '.print_r($token, true), E_USER_ERROR);
			}
		}

		if($token == '=') {
			$depth = 0;

			while($token = ($this->pos >= $this->length) ? null : $this->tokens[$this->pos++]) {
				$type = ($token === (array)$token) ? $token[0] : $token;

				// Track array nesting depth
				if($type == T_ARRAY || $type == '[') {
					$depth += 1;
				} elseif($type == ')' || $type == ']') {
					$depth -= 1;
				}

				// Parse out the assignment side of a static declaration,
				// ending on either a ';' or a ',' outside an array
				if($type == T_WHITESPACE) {
					$value .= ' ';
				}
				else if($type == ';' || ($type == ',' && !$depth)) {
					break;
				}
				// Statics can reference class constants with self:: (and that won't work in eval)
				else if($type == T_STRING && $token[1] == 'self') {
					$value .= $class;
				}
				else {
					$value .= ($token === (array)$token) ? $token[1] : $token;
				}
			}
		}

		if (!isset($this->info[$class])) {
			$this->info[$class] = array(
				'path' => $this->path,
				'mtime' => filemtime($this->path),
			);
		}

		if(!isset($this->statics[$class])) {
			$this->statics[$class] = array();
		}

		$value = trim($value);
		if ($value) {
			$value = eval('static $temp = '.$value.";\n".'return $temp'.";\n");
		}
		else {
			$value = null;
		}

		$this->statics[$class][$variable] = array(
			'access' => $access,
			'value' => $value
		);

		if($token == ',') $this->parseStatic($access, $class);
	}
}


1			<?php
2			/**
3			* A utility class which builds a manifest of the statics defined in all classes, along with their
4			* access levels and values
5			*
6			* We use this to make the statics that the Config system uses as default values be truely immutable.
7			*
8			* It has the side effect of allowing Config to avoid private-level access restrictions, so we can
9			* optionally catch attempts to modify the config statics (otherwise the modification will appear
10			* to work, but won't actually have any effect - the equvilent of failing silently)
11			*
12			* @package framework
13			* @subpackage manifest
14			*/
15			class SS_ConfigStaticManifest {
16
17			protected $base;
18			protected $tests;
19
20			protected $cache;
21			protected $key;
22
23			protected $index;
24			protected $statics;
25
26			static protected $initial_classes = array(
27			'Object', 'ViewableData', 'Injector', 'Director'
28			);
29
30			/**
31			* Constructs and initialises a new config static manifest, either loading the data
32			* from the cache or re-scanning for classes.
33			*
34			* @param string $base The manifest base path.
35			* @param bool $includeTests Include the contents of "tests" directories.
36			* @param bool $forceRegen Force the manifest to be regenerated.
37			* @param bool $cache If the manifest is regenerated, cache it.
38			*/
39			public function __construct($base, $includeTests = false, $forceRegen = false, $cache = true) {
40			$this->base = $base;
41			$this->tests = $includeTests;
42
43			$cacheClass = defined('SS_MANIFESTCACHE') ? SS_MANIFESTCACHE : 'ManifestCache_File';
44
45			$this->cache = new $cacheClass('staticmanifest'.($includeTests ? '_tests' : ''));
46			$this->key = sha1($base);
47
48			if(!$forceRegen) {
49			$this->index = $this->cache->load($this->key);
50			}
51
52			if($this->index) {
53			$this->statics = $this->index['$statics'];
54			}
55			else {
56			$this->regenerate($cache);
57			}
58			}
59
60			public function get($class, $name, $default) {
61			if (!isset($this->statics[$class])) {
62			if (isset($this->index[$class])) {
63			$info = $this->index[$class];
64
65			if (isset($info['key']) && $details = $this->cache->load($this->key.'_'.$info['key'])) {
66			$this->statics += $details;
67			}
68
69			if (!isset($this->statics[$class])) {
70			$this->handleFile(null, $info['path'], null);
71			}
72			}
73			else {
74			$this->statics[$class] = false;
75			}
76			}
77
78			if (isset($this->statics[$class][$name])) {
79			$static = $this->statics[$class][$name];
80
81			if ($static['access'] != T_PRIVATE) {
82			Deprecation::notice('4.0', "Config static $class::\$$name must be marked as private",
83			Deprecation::SCOPE_GLOBAL);
84			// Don't warn more than once per static
85			$this->statics[$class][$name]['access'] = T_PRIVATE;
86			}
87
88			return $static['value'];
89			}
90
91			return $default;
92			}
93
94			/**
95			* Completely regenerates the manifest file.
96			*/
97			public function regenerate($cache = true) {
98			$this->index = array('$statics' => array());
99			$this->statics = array();
100
101			$finder = new ManifestFileFinder();
102			$finder->setOptions(array(
103			'name_regex' => '/^([^_].*\.php)$/',
104			'ignore_files' => array('index.php', 'main.php', 'cli-script.php', 'SSTemplateParser.php'),
105			'ignore_tests' => !$this->tests,
106			'file_callback' => array($this, 'handleFile')
107			));
108
109			$finder->find($this->base);
110
111			if($cache) {
112			$keysets = array();
113
114			foreach ($this->statics as $class => $details) {
115			if (in_array($class, self::$initial_classes)) {
116			$this->index['$statics'][$class] = $details;
117			}
118			else {
119			$key = sha1($class);
120			$this->index[$class]['key'] = $key;
121
122			$keysets[$key][$class] = $details;
123			}
124			}
125
126			foreach ($keysets as $key => $details) {
127			$this->cache->save($details, $this->key.'_'.$key);
128			}
129
130			$this->cache->save($this->index, $this->key);
131			}
132			}
133
134			public function handleFile($basename, $pathname, $depth) {
			0 ignored issues – show Unused Code introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report The parameter `$depth` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
135			$parser = new SS_ConfigStaticManifest_Parser($pathname);
136			$parser->parse();
137
138			$this->index = array_merge($this->index, $parser->getInfo());
139			$this->statics = array_merge($this->statics, $parser->getStatics());
140			}
141
142			public function getStatics() {
143			return $this->statics;
144			}
145			}
146
147			/**
148			* A parser that processes a PHP file, using PHP's built in parser to get a string of tokens,
149			* then processing them to find the static class variables, their access levels & values
150			*
151			* We can't do this using TokenisedRegularExpression because we need to keep track of state
152			* as we process the token list (when we enter and leave a namespace or class, when we see
153			* an access level keyword, etc)
154			*
155			* @package framework
156			* @subpackage manifest
157			*/
158			class SS_ConfigStaticManifest_Parser {
159
160			protected $info = array();
161			protected $statics = array();
162
163			protected $path;
164			protected $tokens;
165			protected $length;
166			protected $pos;
167
168			function __construct($path) {
			0 ignored issues – show Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
169			$this->path = $path;
170			$file = file_get_contents($path);
171
172			$this->tokens = token_get_all($file);
173			$this->length = count($this->tokens);
174			$this->pos = 0;
175			}
176
177			function getInfo() {
			0 ignored issues – show Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
178			return $this->info;
179			}
180
181			function getStatics() {
			0 ignored issues – show Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
182			return $this->statics;
183			}
184
185			/**
186			* Get the next token to process, incrementing the pointer
187			*
188			* @param bool $ignoreWhitespace - if true will skip any whitespace tokens & only return non-whitespace ones
189			* @return null \| mixed - Either the next token or null if there isn't one
190			*/
191			protected function next($ignoreWhitespace = true) {
192			do {
193			if($this->pos >= $this->length) return null;
194			$next = $this->tokens[$this->pos++];
195			}
196			while($ignoreWhitespace && is_array($next) && $next[0] == T_WHITESPACE);
197
198			return $next;
199			}
200
201			/**
202			* Get the next set of tokens that form a string to process,
203			* incrementing the pointer
204			*
205			* @param bool $ignoreWhitespace - if true will skip any whitespace tokens
206			* & only return non-whitespace ones
207			* @return null\|string - Either the next string or null if there isn't one
208			*/
209			protected function nextString($ignoreWhitespace = true) {
210			static $stop = array('{', '}', '(', ')', '[', ']');
211
212			$string = '';
213			while ($this->pos < $this->length) {
214			$next = $this->tokens[$this->pos];
215			if (is_string($next)) {
216			if (!in_array($next, $stop)) {
217			$string .= $next;
218			} else {
219			break;
220			}
221			} else if ($next[0] == T_STRING) {
222			$string .= $next[1];
223			} else if ($next[0] != T_WHITESPACE \|\| !$ignoreWhitespace) {
224			break;
225			}
226			$this->pos++;
227			}
228			if ($string === '') {
229			return null;
230			} else {
231			return $string;
232			}
233			}
234
235			/**
236			* Parse the given file to find the static variables declared in it, along with their access & values
237			*/
238			function parse() {
			0 ignored issues – show Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
239			$depth = 0; $namespace = null; $class = null; $clsdepth = null; $access = 0;
240
241			while($token = $this->next()) {
242			$type = ($token === (array)$token) ? $token[0] : $token;
243
244			if($type == T_CLASS) {
245			$next = $this->nextString();
246			if($next === null) {
247			user_error("Couldn\'t parse {$this->path} when building config static manifest", E_USER_ERROR);
248			}
249
250			$class = $next;
251			}
252			else if($type == T_NAMESPACE) {
253			$namespace = '';
254			while(true) {
255			$next = $this->next();
256
257			if($next == ';') {
258			break;
259			} elseif($next[0] == T_NS_SEPARATOR) {
260			$namespace .= $next[1];
261			$next = $this->next();
262			}
263
264			if(!is_string($next) && $next[0] != T_STRING) {
265			user_error("Couldn\'t parse {$this->path} when building config static manifest", E_USER_ERROR);
266			}
267
268			$namespace .= is_string($next) ? $next : $next[1];
269			}
270			}
271			else if($type == '{' \|\| $type == T_CURLY_OPEN \|\| $type == T_DOLLAR_OPEN_CURLY_BRACES){
272			$depth += 1;
273			if($class && !$clsdepth) $clsdepth = $depth;
			0 ignored issues – show Bug Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report The expression `$class` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report The expression `$clsdepth` of type `integer\|null` is loosely compared to `false`; this is ambiguous if the integer can be zero. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
274			}
275			else if($type == '}') {
276			$depth -= 1;
277			if($depth < $clsdepth) $class = $clsdepth = null;
278			if($depth < 0) user_error("Hmm - depth calc wrong, hit negatives, see: ".$this->path, E_USER_ERROR);
279			}
280			else if($type == T_PUBLIC \|\| $type == T_PRIVATE \|\| $type == T_PROTECTED) {
281			$access = $type;
282			}
283			else if($type == T_STATIC && $class && $depth == $clsdepth) {
			0 ignored issues – show Bug Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report The expression `$class` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
284			$this->parseStatic($access, $namespace ? $namespace.'\\'.$class : $class);
285			$access = 0;
286			}
287			else {
288			$access = 0;
289			}
290			}
291			}
292
293			/**
294			* During parsing we've found a "static" keyword. Parse out the variable names and value
295			* assignments that follow.
296			*
297			* Seperated out from parse partially so that we can recurse if there are multiple statics
298			* being declared in a comma seperated list
299			*/
300			function parseStatic($access, $class) {
			0 ignored issues – show Best Practice introduced 2016-01-05 03:55 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
301			$variable = null;
302			$value = '';
303
304			while($token = $this->next()) {
305			$type = ($token === (array)$token) ? $token[0] : $token;
306
307			if($type == T_PUBLIC \|\| $type == T_PRIVATE \|\| $type == T_PROTECTED) {
308			$access = $type;
309			}
310			else if($type == T_FUNCTION) {
311			return;
312			}
313			else if($type == T_VARIABLE) {
314			$variable = substr($token[1], 1); // Cut off initial "$"
315			}
316			else if($type == ';' \|\| $type == ',' \|\| $type == '=') {
317			break;
318			}
319			else if($type == T_COMMENT \|\| $type == T_DOC_COMMENT) {
320			// NOP
321			}
322			else {
323			user_error('Unexpected token ("' . token_name($type) . '") when building static manifest in class "'
324			. $class . '": '.print_r($token, true), E_USER_ERROR);
325			}
326			}
327
328			if($token == '=') {
329			$depth = 0;
330
331			while($token = ($this->pos >= $this->length) ? null : $this->tokens[$this->pos++]) {
332			$type = ($token === (array)$token) ? $token[0] : $token;
333
334			// Track array nesting depth
335			if($type == T_ARRAY \|\| $type == '[') {
336			$depth += 1;
337			} elseif($type == ')' \|\| $type == ']') {
338			$depth -= 1;
339			}
340
341			// Parse out the assignment side of a static declaration,
342			// ending on either a ';' or a ',' outside an array
343			if($type == T_WHITESPACE) {
344			$value .= ' ';
345			}
346			else if($type == ';' \|\| ($type == ',' && !$depth)) {
347			break;
348			}
349			// Statics can reference class constants with self:: (and that won't work in eval)
350			else if($type == T_STRING && $token[1] == 'self') {
351			$value .= $class;
352			}
353			else {
354			$value .= ($token === (array)$token) ? $token[1] : $token;
355			}
356			}
357			}
358
359			if (!isset($this->info[$class])) {
360			$this->info[$class] = array(
361			'path' => $this->path,
362			'mtime' => filemtime($this->path),
363			);
364			}
365
366			if(!isset($this->statics[$class])) {
367			$this->statics[$class] = array();
368			}
369
370			$value = trim($value);
371			if ($value) {
372			$value = eval('static $temp = '.$value.";\n".'return $temp'.";\n");
373			}
374			else {
375			$value = null;
376			}
377
378			$this->statics[$class][$variable] = array(
379			'access' => $access,
380			'value' => $value
381			);
382
383			if($token == ',') $this->parseStatic($access, $class);
384			}
385			}
386

sminnee / silverstripe-framework

Push — new-committers ( 29cb6f...bcba16 )

SS_ConfigStaticManifest_Parser::nextString() C

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like