+global\sql_compiler::parse_query()

Complexity

Conditions	8
Paths	40

Size

Total Lines	80
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	26
CRAP Score	8.1516

<?php
	protected function parse_query(&$query) {

		if (!preg_match('|^[[:space:]]*order[[:space:]]*by[[:space:]]+|i', $query, $regs)) {
			$result=$this->parse_or_expr($query);
		} else {
			$no_selection = true;
		}

/*

58% of this comment could be valid code. Did you maybe forget this after debugging?

		$YYCURSOR = 0;
		while ($this->_SCAN_WS[$YYBUFFER[$YYCURSOR]]) {
			$YYCURSOR++;
		}

		$yych  = $YYBUFFER[$YYCURSOR];
		if ($this->_SCAN_AZ[$yych]) {
			$value = $yych;
			$yych  = $YYBUFFER[++$YYCURSOR];
			while ($this->_SCAN_AZ[$yych]) {
				$value .= $yych;
				$yych = $YYBUFFER[++$YYCURSOR];
			}
			$value = strtolower($value);
			if ($value === 'order') {
				while ($this->_SCAN_WS[$YYBUFFER[$YYCURSOR]]) {
					$YYCURSOR++;
				}
				$yych  = $YYBUFFER[$YYCURSOR];
				if ($this->_SCAN_AZ[$yych]) {
					$value = $yych;
					$yych  = $YYBUFFER[++$YYCURSOR];
					while ($this->_SCAN_AZ[$yych]) {
						$value .= $yych;
						$yych = $YYBUFFER[++$YYCURSOR];
					}
					$value = strtolower($value);
					if ($value === 'by') {
						$YYBUFFER = substr($YYBUFFER, $YYCURSOR;
						$result = $this->parse_or_expr($YYBUFFER);
						$YYCURSOR = 0;
						$value = '';
					} else {
						$this->error = "syntax error near: $YYBUFFER";
						return false;
					}
				}
			}
		}

*/

		if (preg_match('|^[[:space:]]*join[[:space:]]*target[[:space:]]*on[[:space:]]*|i', $query, $regs)) {
			$this->join_target_properties = array();
			$query = substr($query, strlen($regs[0]));
			$this->parse_join_target_properties($query);
		}

		$matching = preg_match('|^[[:space:]]*order[[:space:]]*by[[:space:]]+|i', $query, $regs);
		if ( $matching || $no_selection ) {

The variable $no_selection does not seem to be defined for all execution paths leading up to this point.

			$query=substr($query, strlen($regs[0]));
			$node["id"]="orderby";

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

			$node["right"]=$this->parse_orderby($query);
			$node["left"]=$result;

The variable $result does not seem to be defined for all execution paths leading up to this point.

			$result=$node;
		}
		if (preg_match('|^[[:space:]]*limit[[:space:]]+([0-9]+)[[:space:]]*([,][[:space:]]*([0-9]+))?|i', $query, $regs)) {
			$query=substr($query, strlen($regs[0]));
			$limit_s["id"]="limit";

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

			$limit_s["offset"]=$regs[1];
			$limit_s["limit"]=$regs[3];
		} else {
			$limit_s["id"]="limit";

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

			$limit_s["offset"]=($this->offset) ? $this->offset : 0;
			$limit_s["limit"]=($this->limit) ? $this->limit : 0;
		}
		$limit_s["left"]=$result;
		$result=$limit_s;

		return $result;
	}