pradosoft / prado

framework/Data/ActiveRecord/Scaffold/InputBuilder/TScaffoldInputBase.php

Doc Comments +9 added lines

@@ -46,6 +46,10 @@ 
 		}
 	}
 
+	/**
+	 * @param TScaffoldEditView $parent
+	 * @param TActiveRecord $record
+	 */
 	public function createScaffoldInput($parent, $item, $column, $record)
 	{
 		$this->_parent=$parent;
@@ -62,6 +66,11 @@ 
 		$label->setForControl(self::DEFAULT_ID);
 	}
 
+	/**
+	 * @param TScaffoldEditView $parent
+	 * @param TDbTableColumn $column
+	 * @param TActiveRecord $record
+	 */
 	public function loadScaffoldInput($parent, $item, $column, $record)
 	{
 		$this->_parent=$parent;

Spacing +13 added lines, -13 removed lines

@@ -21,51 +21,51 @@ 
 	{
 		$record->getDbConnection()->setActive(true); //must be connected before retrieving driver name!
 		$driver = $record->getDbConnection()->getDriverName();
-		switch(strtolower($driver))
+		switch (strtolower($driver))
 		{
 			case 'sqlite': //sqlite 3
 			case 'sqlite2': //sqlite 2
-				require_once(dirname(__FILE__).'/TSqliteScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TSqliteScaffoldInput.php');
 				return new TSqliteScaffoldInput($conn);
 			case 'mysqli':
 			case 'mysql':
-				require_once(dirname(__FILE__).'/TMysqlScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TMysqlScaffoldInput.php');
 				return new TMysqlScaffoldInput($conn);
 			case 'pgsql':
-				require_once(dirname(__FILE__).'/TPgsqlScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TPgsqlScaffoldInput.php');
 				return new TPgsqlScaffoldInput($conn);
 			case 'mssql':
-				require_once(dirname(__FILE__).'/TMssqlScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TMssqlScaffoldInput.php');
 				return new TMssqlScaffoldInput($conn);
 			case 'ibm':
-				require_once(dirname(__FILE__).'/TIbmScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TIbmScaffoldInput.php');
 				return new TIbmScaffoldInput($conn);
 			default:
 				throw new TConfigurationException(
-					'scaffold_invalid_database_driver',$driver);
+					'scaffold_invalid_database_driver', $driver);
 		}
 	}
 
 	public function createScaffoldInput($parent, $item, $column, $record)
 	{
-		$this->_parent=$parent;
+		$this->_parent = $parent;
 		$item->setCustomData($column->getColumnId());
 		$this->createControl($item->_input, $column, $record);
-		if($item->_input->findControl(self::DEFAULT_ID))
+		if ($item->_input->findControl(self::DEFAULT_ID))
 			$this->createControlLabel($item->_label, $column, $record);
 	}
 
 	protected function createControlLabel($label, $column, $record)
 	{
-		$fieldname = ucwords(str_replace('_', ' ', $column->getColumnId())).':';
+		$fieldname = ucwords(str_replace('_', ' ', $column->getColumnId())) . ':';
 		$label->setText($fieldname);
 		$label->setForControl(self::DEFAULT_ID);
 	}
 
 	public function loadScaffoldInput($parent, $item, $column, $record)
 	{
-		$this->_parent=$parent;
-		if($this->getIsEnabled($column, $record))
+		$this->_parent = $parent;
+		if ($this->getIsEnabled($column, $record))
 		{
 			$prop = $column->getColumnId();
 			$record->setColumnValue($prop, $this->getControlValue($item->_input, $column, $record));
@@ -81,7 +81,7 @@ 
 	protected function getRecordPropertyValue($column, $record)
 	{
 		$value = $record->getColumnValue($column->getColumnId());
-		if($column->getDefaultValue()!==TDbTableColumn::UNDEFINED_VALUE && $value===null)
+		if ($column->getDefaultValue() !== TDbTableColumn::UNDEFINED_VALUE && $value === null)
 			return $column->getDefaultValue();
 		else
 			return $value;

framework/Data/ActiveRecord/Scaffold/InputBuilder/TScaffoldInputCommon.php

Doc Comments +3 added lines

@@ -47,6 +47,9 @@
 		return $control;
 	}
 
+	/**
+	 * @return string
+	 */
 	protected function getDefaultControlValue($container,$column, $record)
 	{
 		$control = $container->findControl(self::DEFAULT_ID);

Spacing +35 added lines, -35 removed lines

@@ -21,7 +21,7 @@ 
 	protected function setNotNullProperty($container, $control, $column, $record)
 	{
 		$this->setDefaultProperty($container, $control, $column, $record);
-		if(!$column->getAllowNull() && !$column->hasSequence())
+		if (!$column->getAllowNull() && !$column->hasSequence())
 			$this->createRequiredValidator($container, $column, $record);
 	}
 
@@ -41,18 +41,18 @@ 
 		$control = new TTextBox();
 		$control->setText($value);
 		$control->setCssClass('default-textbox scaffold_input');
-		if(($len=$column->getColumnSize())!==null)
+		if (($len = $column->getColumnSize()) !== null)
 			$control->setMaxLength($len);
 		$this->setNotNullProperty($container, $control, $column, $record);
 		return $control;
 	}
 
-	protected function getDefaultControlValue($container,$column, $record)
+	protected function getDefaultControlValue($container, $column, $record)
 	{
 		$control = $container->findControl(self::DEFAULT_ID);
-		if($control instanceof TCheckBox)
+		if ($control instanceof TCheckBox)
 			return $control->getChecked();
-		else if($control instanceof TControl)
+		else if ($control instanceof TControl)
 			return $control->getText();
 	}
 
@@ -73,8 +73,8 @@ 
 		$control = new TDropDownList();
 		$years = array();
 		$current = intval(@date('Y'));
-		$from = $current-10; $to=$current+10;
-		for($i = $from; $i <= $to; $i++)
+		$from = $current - 10; $to = $current + 10;
+		for ($i = $from; $i <= $to; $i++)
 			$years[$i] = $i;
 		$control->setDataSource($years);
 		$control->setSelectedValue(empty($value) ? $current : $value);
@@ -98,13 +98,13 @@ 
 		$val = $this->createTypeValidator($container, $column, $record);
 		$val->setDataType(TValidationDataType::Float);
 		$val->setErrorMessage('Please entery a decimal number.');
-		if(($max= $column->getMaxiumNumericConstraint())!==null)
+		if (($max = $column->getMaxiumNumericConstraint()) !== null)
 		{
-			$val = $this->createRangeValidator($container,$column,$record);
+			$val = $this->createRangeValidator($container, $column, $record);
 			$val->setDataType(TValidationDataType::Float);
 			$val->setMaxValue($max);
 			$val->setStrictComparison(true);
-			$val->setErrorMessage('Please entery a decimal number strictly less than '.$max.'.');
+			$val->setErrorMessage('Please entery a decimal number strictly less than ' . $max . '.');
 		}
 		return $control;
 	}
@@ -149,22 +149,22 @@ 
 	protected function createTimeControl($container, $column, $record)
 	{
 		$value = $this->getRecordPropertyValue($column, $record);
-		$hours=array();
-		for($i=0;$i<24;$i++) $hours[] = str_pad($i,2,'0',STR_PAD_LEFT);
-		$mins=array();
-		for($i=0;$i<60;$i++) $mins[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		$hours = array();
+		for ($i = 0; $i < 24; $i++) $hours[] = str_pad($i, 2, '0', STR_PAD_LEFT);
+		$mins = array();
+		for ($i = 0; $i < 60; $i++) $mins[] = str_pad($i, 2, '0', STR_PAD_LEFT);
 		$hour = intval(@date('H'));
 		$min = intval(@date('i'));
 		$sec = intval(@date('s'));
-		if(!empty($value))
+		if (!empty($value))
 		{
-			$match=array();
-			if(preg_match('/(\d+):(\d+):?(\d+)?/', $value, $match))
+			$match = array();
+			if (preg_match('/(\d+):(\d+):?(\d+)?/', $value, $match))
 			{
 				$hour = $match[1];
 				$min = $match[2];
-				if(isset($match[3]))
-					$sec=$match[3];
+				if (isset($match[3]))
+					$sec = $match[3];
 			}
 		}
 
@@ -191,7 +191,7 @@ 
 		$scontrol->setSelectedValue(intval($sec));
 		$container->Controls[] = $scontrol;
 
-		return array($hcontrol,$mcontrol,$scontrol);
+		return array($hcontrol, $mcontrol, $scontrol);
 	}
 
 
@@ -202,8 +202,8 @@ 
 		$control->setFromYear(1900);
 		$control->setInputMode(TDatePickerInputMode::DropDownList);
 		$control->setDateFormat('yyyy-MM-dd');
-		if(!empty($value))
-			$control->setDate(substr($value,0,10));
+		if (!empty($value))
+			$control->setDate(substr($value, 0, 10));
 		$control->setCssClass('date-dropdown');
 		$this->setNotNullProperty($container, $control, $column, $record);
 		return $control;
@@ -215,14 +215,14 @@ 
 		$control = $this->createDateControl($container, $column, $record);
 		$container->Controls[] = ' @ ';
 		$time = $this->createTimeControl($container, $column, $record);
-		if(!empty($value))
+		if (!empty($value))
 		{
-			$match=array();
-			if(preg_match('/(\d+):(\d+):?(\d+)?/', substr($value, 11), $match))
+			$match = array();
+			if (preg_match('/(\d+):(\d+):?(\d+)?/', substr($value, 11), $match))
 			{
 				$time[0]->setSelectedValue(intval($match[1]));
 				$time[1]->setSelectedValue(intval($match[2]));
-				if(isset($match[3]))
+				if (isset($match[3]))
 					$time[2]->setSelectedValue(intval($match[3]));
 			}
 		}
@@ -255,7 +255,7 @@ 
 		$values = $column->getDbTypeValues();
 		$control->setDataSource($values);
 		$control->dataBind();
-		$control->setSelectedIndices($this->getMatchingIndices($values,$selectedValues));
+		$control->setSelectedIndices($this->getMatchingIndices($values, $selectedValues));
 		$control->setID(self::DEFAULT_ID);
 		$control->setCssClass('set-checkboxes');
 		$this->setNotNullProperty($container, $control, $column, $record);
@@ -264,10 +264,10 @@ 
 
 	protected function getMatchingIndices($checks, $values)
 	{
-		$index=array();
-		for($i=0, $k=count($checks); $i<$k; $i++)
+		$index = array();
+		for ($i = 0, $k = count($checks); $i < $k; $i++)
 		{
-			if(in_array($checks[$i], $values))
+			if (in_array($checks[$i], $values))
 				$index[] = $i;
 		}
 		return $index;
@@ -281,8 +281,8 @@ 
 		$values = $column->getDbTypeValues();
 		$control->setDataSource($values);
 		$control->dataBind();
-		$index = $this->getMatchingIndices($values,$selectedValues);
-		if(count($index) > 0)
+		$index = $this->getMatchingIndices($values, $selectedValues);
+		if (count($index) > 0)
 			$control->setSelectedIndex($index[0]);
 		$control->setID(self::DEFAULT_ID);
 		$control->setCssClass('enum-radio-buttons');
@@ -292,10 +292,10 @@ 
 
 	protected function getSetValue($container, $column, $record)
 	{
-		$value=array();
-		foreach($container->findControl(self::DEFAULT_ID)->getItems() as $item)
+		$value = array();
+		foreach ($container->findControl(self::DEFAULT_ID)->getItems() as $item)
 		{
-			if($item->getSelected())
+			if ($item->getSelected())
 				$value[] = $item->getText();
 		}
 		return implode(',', $value);

Braces +9 added lines, -4 removed lines

@@ -74,8 +74,9 @@ 
 		$years = array();
 		$current = intval(@date('Y'));
 		$from = $current-10; $to=$current+10;
-		for($i = $from; $i <= $to; $i++)
-			$years[$i] = $i;
+		for($i = $from; $i <= $to; $i++) {
+					$years[$i] = $i;
+		}
 		$control->setDataSource($years);
 		$control->setSelectedValue(empty($value) ? $current : $value);
 		$control->setCssClass('year-dropdown');
@@ -150,9 +151,13 @@ 
 	{
 		$value = $this->getRecordPropertyValue($column, $record);
 		$hours=array();
-		for($i=0;$i<24;$i++) $hours[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		for($i=0;$i<24;$i++) {
+			$hours[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		}
 		$mins=array();
-		for($i=0;$i<60;$i++) $mins[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		for($i=0;$i<60;$i++) {
+			$mins[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		}
 		$hour = intval(@date('H'));
 		$min = intval(@date('i'));
 		$sec = intval(@date('s'));

framework/Data/ActiveRecord/Scaffold/TScaffoldEditView.php

Doc Comments +4 added lines

@@ -129,6 +129,8 @@ 
 	 * Instantiate the external edit renderer.
 	 * @param TActiveRecord record to be edited
 	 * @param string external edit renderer class name.
+	 * @param TActiveRecord $record
+	 * @param string $classPath
 	 * @throws TConfigurationException raised when renderer is not an
 	 * instance of IScaffoldEditRenderer.
 	 */
@@ -260,6 +262,7 @@ 
 	/**
 	 * Create the default scaffold editor control factory.
 	 * @param TActiveRecord record instance.
+	 * @param TActiveRecord $record
 	 * @return TScaffoldInputBase scaffold editor control factory.
 	 */
 	protected function getScaffoldInputBuilder($record)
@@ -300,6 +303,7 @@ 
 	/**
 	 * This method should update the record with the user input data.
 	 * @param TActiveRecord record to be saved.
+	 * @param TActiveRecord $record
 	 */
 	public function updateRecord($record);
 }

Spacing +15 added lines, -15 removed lines

@@ -57,7 +57,7 @@ 
 	 */
 	public function onLoad($param)
 	{
-		if($this->getVisible())
+		if ($this->getVisible())
 			$this->initializeEditForm();
 	}
 
@@ -110,7 +110,7 @@ 
 	{
 		$record = $this->getCurrentRecord();
 		$classPath = $this->getEditRenderer();
-		if($classPath === '')
+		if ($classPath === '')
 		{
 			$columns = $this->getTableInfo()->getColumns();
 			$this->getInputRepeater()->setDataSource($columns);
@@ -118,7 +118,7 @@ 
 		}
 		else
 		{
-			if($this->_editRenderer===null)
+			if ($this->_editRenderer === null)
 				$this->createEditRenderer($record, $classPath);
 			else
 				$this->_editRenderer->setData($record);
@@ -135,10 +135,10 @@ 
 	protected function createEditRenderer($record, $classPath)
 	{
 		$this->_editRenderer = Prado::createComponent($classPath);
-		if($this->_editRenderer instanceof IScaffoldEditRenderer)
+		if ($this->_editRenderer instanceof IScaffoldEditRenderer)
 		{
 			$index = $this->getControls()->remove($this->getInputRepeater());
-			$this->getControls()->insertAt($index,$this->_editRenderer);
+			$this->getControls()->insertAt($index, $this->_editRenderer);
 			$this->_editRenderer->setData($record);
 		}
 		else
@@ -154,11 +154,11 @@ 
 	protected function createRepeaterEditItem($sender, $param)
 	{
 		$type = $param->getItem()->getItemType();
-		if($type==TListItemType::Item || $type==TListItemType::AlternatingItem)
+		if ($type == TListItemType::Item || $type == TListItemType::AlternatingItem)
 		{
 			$item = $param->getItem();
 			$column = $item->getDataItem();
-			if($column===null)
+			if ($column === null)
 				return;
 
 			$record = $this->getCurrentRecord();
@@ -173,7 +173,7 @@ 
 	 */
 	public function bubbleEvent($sender, $param)
 	{
-		switch(strtolower($param->getCommandName()))
+		switch (strtolower($param->getCommandName()))
 		{
 			case 'save':
 				return $this->doSave() ? false : true;
@@ -192,14 +192,14 @@ 
 	 */
 	protected function doSave()
 	{
-		if($this->getPage()->getIsValid())
+		if ($this->getPage()->getIsValid())
 		{
 			$record = $this->getCurrentRecord();
-			if($this->_editRenderer===null)
+			if ($this->_editRenderer === null)
 			{
 				$table = $this->getTableInfo();
 				$builder = $this->getScaffoldInputBuilder($record);
-				foreach($this->getInputRepeater()->getItems() as $item)
+				foreach ($this->getInputRepeater()->getItems() as $item)
 				{
 					$column = $table->getColumn($item->getCustomData());
 					$builder->loadScaffoldInput($this, $item, $column, $record);
@@ -212,7 +212,7 @@ 
 			$record->save();
 			return true;
 		}
-		else if($this->_editRenderer!==null)
+		else if ($this->_editRenderer !== null)
 		{
 			//preserve the form data.
 			$this->_editRenderer->updateRecord($this->getCurrentRecord());
@@ -264,9 +264,9 @@ 
 	 */
 	protected function getScaffoldInputBuilder($record)
 	{
-		static $_builders=array();
+		static $_builders = array();
 		$class = get_class($record);
-		if(!isset($_builders[$class]))
+		if (!isset($_builders[$class]))
 		{
 			Prado::using('System.Data.ActiveRecord.Scaffold.InputBuilder.TScaffoldInputBase');
 			$_builders[$class] = TScaffoldInputBase::createInputBuilder($record);
@@ -279,7 +279,7 @@ 
 	 */
 	public function getValidationGroup()
 	{
-		return 'group_'.$this->getUniqueID();
+		return 'group_' . $this->getUniqueID();
 	}
 }
 

Braces +4 added lines, -8 removed lines

@@ -115,8 +115,7 @@ 
 			$columns = $this->getTableInfo()->getColumns();
 			$this->getInputRepeater()->setDataSource($columns);
 			$this->getInputRepeater()->dataBind();
-		}
-		else
+		} else
 		{
 			if($this->_editRenderer===null)
 				$this->createEditRenderer($record, $classPath);
@@ -140,8 +139,7 @@ 
 			$index = $this->getControls()->remove($this->getInputRepeater());
 			$this->getControls()->insertAt($index,$this->_editRenderer);
 			$this->_editRenderer->setData($record);
-		}
-		else
+		} else
 		{
 			throw new TConfigurationException(
 				'scaffold_invalid_edit_renderer', $this->getID(), get_class($record));
@@ -204,15 +202,13 @@ 
 					$column = $table->getColumn($item->getCustomData());
 					$builder->loadScaffoldInput($this, $item, $column, $record);
 				}
-			}
-			else
+			} else
 			{
 				$this->_editRenderer->updateRecord($record);
 			}
 			$record->save();
 			return true;
-		}
-		else if($this->_editRenderer!==null)
+		} else if($this->_editRenderer!==null)
 		{
 			//preserve the form data.
 			$this->_editRenderer->updateRecord($this->getCurrentRecord());

framework/Data/ActiveRecord/Scaffold/TScaffoldListView.php

Doc Comments +1 added lines

@@ -121,6 +121,7 @@
 
 	/**
 	 * @param string search condition, the SQL string after the WHERE clause.
+	 * @param string|null $value
 	 */
 	public function setSearchCondition($value)
 	{

Spacing +23 added lines, -23 removed lines

@@ -54,14 +54,14 @@ 
 	protected function initializeSort()
 	{
 		$table = $this->getTableInfo();
-		$sorts = array('Sort By', str_repeat('-',15));
+		$sorts = array('Sort By', str_repeat('-', 15));
 		$headers = array();
-		foreach($table->getColumns() as $name=>$colum)
+		foreach ($table->getColumns() as $name=>$colum)
 		{
 			$fname = ucwords(str_replace('_', ' ', $name));
-			$sorts[$name.' ASC'] = $fname .' Ascending';
-			$sorts[$name.' DESC'] = $fname .' Descending';
-			$headers[] = $fname ;
+			$sorts[$name . ' ASC'] = $fname . ' Ascending';
+			$sorts[$name . ' DESC'] = $fname . ' Descending';
+			$headers[] = $fname;
 		}
 		$this->_sort->setDataSource($sorts);
 		$this->_sort->dataBind();
@@ -75,7 +75,7 @@ 
 	public function onPreRender($param)
 	{
 		parent::onPreRender($param);
-		if(!$this->getPage()->getIsPostBack() || $this->getViewState('CurrentClass')!=$this->getRecordClass())
+		if (!$this->getPage()->getIsPostBack() || $this->getViewState('CurrentClass') != $this->getRecordClass())
 		{
 			$this->initializeSort();
 			$this->setViewState('CurrentClass', $this->getRecordClass());
@@ -103,18 +103,18 @@ 
 	{
 		$total = $this->_list->getVirtualItemCount();
 		$limit = $this->_list->getPageSize();
-		$offset = $this->_list->getCurrentPageIndex()*$limit;
-		if($offset + $limit > $total)
+		$offset = $this->_list->getCurrentPageIndex() * $limit;
+		if ($offset + $limit > $total)
 			$limit = $total - $offset;
 		$criteria = new TActiveRecordCriteria($this->getSearchCondition(), $this->getSearchParameters());
-		if($limit > 0)
+		if ($limit > 0)
 		{
 			$criteria->setLimit($limit);
-			if($offset <= $total)
+			if ($offset <= $total)
 				$criteria->setOffset($offset);
 		}
-		$order = explode(' ',$this->_sort->getSelectedValue(), 2);
-		if(is_array($order) && count($order) === 2)
+		$order = explode(' ', $this->_sort->getSelectedValue(), 2);
+		if (is_array($order) && count($order) === 2)
 			$criteria->OrdersBy[$order[0]] = $order[1];
 		return $criteria;
 	}
@@ -140,7 +140,7 @@ 
 	 */
 	public function setSearchParameters($value)
 	{
-		$this->setViewState('SearchParameters', TPropertyValue::ensureArray($value),array());
+		$this->setViewState('SearchParameters', TPropertyValue::ensureArray($value), array());
 	}
 
 	/**
@@ -156,7 +156,7 @@ 
 	 */
 	public function bubbleEvent($sender, $param)
 	{
-		switch(strtolower($param->getCommandName()))
+		switch (strtolower($param->getCommandName()))
 		{
 			case 'delete':
 				return $this->deleteRecord($sender, $param);
@@ -172,9 +172,9 @@ 
 	 */
 	protected function initializeEdit($sender, $param)
 	{
-		if(($ctrl=$this->getEditViewControl())!==null)
+		if (($ctrl = $this->getEditViewControl()) !== null)
 		{
-			if($param instanceof TRepeaterCommandEventParameter)
+			if ($param instanceof TRepeaterCommandEventParameter)
 			{
 				$pk = $param->getItem()->getCustomData();
 				$ctrl->setRecordPk($pk);
@@ -188,7 +188,7 @@ 
 	 */
 	protected function deleteRecord($sender, $param)
 	{
-		if($param instanceof TRepeaterCommandEventParameter)
+		if ($param instanceof TRepeaterCommandEventParameter)
 		{
 			$pk = $param->getItem()->getCustomData();
 			$this->getRecordFinder()->deleteByPk($pk);
@@ -201,10 +201,10 @@ 
 	protected function listItemCreated($sender, $param)
 	{
 		$item = $param->getItem();
-		if($item instanceof IItemDataRenderer)
+		if ($item instanceof IItemDataRenderer)
 		{
 			$type = $item->getItemType();
-			if($type==TListItemType::Item || $type==TListItemType::AlternatingItem)
+			if ($type == TListItemType::Item || $type == TListItemType::AlternatingItem)
 				$this->populateField($sender, $param);
 		}
 	}
@@ -216,10 +216,10 @@ 
 	protected function populateField($sender, $param)
 	{
 		$item = $param->getItem();
-		if(($data = $item->getData()) !== null)
+		if (($data = $item->getData()) !== null)
 		{
 			$item->setCustomData($this->getRecordPkValues($data));
-			if(($prop = $item->findControl('_properties'))!==null)
+			if (($prop = $item->findControl('_properties')) !== null)
 			{
 				$item->_properties->setDataSource($this->getRecordPropertyValues($data));
 				$item->_properties->dataBind();
@@ -292,10 +292,10 @@ 
 	 */
 	protected function getEditViewControl()
 	{
-		if(($id=$this->getEditViewID())!==null)
+		if (($id = $this->getEditViewID()) !== null)
 		{
 			$ctrl = $this->getParent()->findControl($id);
-			if($ctrl===null)
+			if ($ctrl === null)
 				throw new TConfigurationException('scaffold_unable_to_find_edit_view', $id);
 			return $ctrl;
 		}

framework/Data/ActiveRecord/TActiveRecordGateway.php

Doc Comments +7 added lines, -3 removed lines

@@ -201,7 +201,7 @@ 
 	 * Returns records matching the list of given primary keys.
 	 * @param TActiveRecord active record instance.
 	 * @param array list of primary name value pairs
-	 * @return array matching data.
+	 * @return TDbDataReader matching data.
 	 */
 	public function findRecordsByPks(TActiveRecord $record, $keys)
 	{
@@ -227,7 +227,8 @@ 
 	 * Return record data from sql query.
 	 * @param TActiveRecord active record finder instance.
 	 * @param TActiveRecordCriteria sql query
-	 * @return array result.
+	 * @param TSqlCriteria $criteria
+	 * @return TDbDataReader result.
 	 */
 	public function findRecordBySql(TActiveRecord $record, $criteria)
 	{
@@ -238,6 +239,7 @@ 
 	 * Return record data from sql query.
 	 * @param TActiveRecord active record finder instance.
 	 * @param TActiveRecordCriteria sql query
+	 * @param TSqlCriteria $criteria
 	 * @return TDbDataReader result iterator.
 	 */
 	public function findRecordsBySql(TActiveRecord $record, $criteria)
@@ -264,7 +266,7 @@ 
 	/**
 	 * Insert a new record.
 	 * @param TActiveRecord new record.
-	 * @return int number of rows affected.
+	 * @return string|boolean number of rows affected.
 	 */
 	public function insert(TActiveRecord $record)
 	{
@@ -279,6 +281,7 @@ 
 	/**
 	 * Sets the last insert ID to the corresponding property of the record if available.
 	 * @param TActiveRecord record for insertion
+	 * @param TActiveRecord $record
 	 */
 	protected function updatePostInsert($record)
 	{
@@ -396,6 +399,7 @@ 
 	 * Delete multiple records by criteria.
 	 * @param TActiveRecord active record finder instance.
 	 * @param TActiveRecordCriteria search criteria
+	 * @param TSqlCriteria $criteria
 	 * @return int number of records.
 	 */
 	public function deleteRecordsByCriteria(TActiveRecord $record, $criteria)

Spacing +48 added lines, -48 removed lines

@@ -20,19 +20,19 @@ 
 class TActiveRecordGateway extends TComponent
 {
 	private $_manager;
-	private $_tables=array(); //table cache
-	private $_meta=array(); //meta data cache.
-	private $_commandBuilders=array();
+	private $_tables = array(); //table cache
+	private $_meta = array(); //meta data cache.
+	private $_commandBuilders = array();
 	private $_currentRecord;
 
 	/**
 	 * Constant name for specifying optional table name in TActiveRecord.
 	 */
-	const TABLE_CONST='TABLE';
+	const TABLE_CONST = 'TABLE';
 	/**
 	 * Method name for returning optional table name in in TActiveRecord
 	 */
-	const TABLE_METHOD='table';
+	const TABLE_METHOD = 'table';
 
 	/**
 	 * Record gateway constructor.
@@ -40,7 +40,7 @@ 
 	 */
 	public function __construct(TActiveRecordManager $manager)
 	{
-		$this->_manager=$manager;
+		$this->_manager = $manager;
 	}
 
 	/**
@@ -60,20 +60,20 @@ 
 	protected function getRecordTableName(TActiveRecord $record)
 	{
 		$class = new ReflectionClass($record);
-		if($class->hasConstant(self::TABLE_CONST))
+		if ($class->hasConstant(self::TABLE_CONST))
 		{
 			$value = $class->getConstant(self::TABLE_CONST);
-			if(empty($value))
+			if (empty($value))
 				throw new TActiveRecordException('ar_invalid_tablename_property',
-					get_class($record),self::TABLE_CONST);
+					get_class($record), self::TABLE_CONST);
 			return $value;
 		}
 		elseif ($class->hasMethod(self::TABLE_METHOD))
 		{
 			$value = $record->{self::TABLE_METHOD}();
-			if(empty($value))
+			if (empty($value))
 				throw new TActiveRecordException('ar_invalid_tablename_method',
-					get_class($record),self::TABLE_METHOD);
+					get_class($record), self::TABLE_METHOD);
 			return $value;
 		}
 		else
@@ -100,24 +100,24 @@ 
 	public function getTableInfo(TDbConnection $connection, $tableName)
 	{
 		$connStr = $connection->getConnectionString();
-		$key = $connStr.$tableName;
-		if(!isset($this->_tables[$key]))
+		$key = $connStr . $tableName;
+		if (!isset($this->_tables[$key]))
 		{
 			//call this first to ensure that unserializing the cache
 			//will find the correct driver dependent classes.
-			if(!isset($this->_meta[$connStr]))
+			if (!isset($this->_meta[$connStr]))
 			{
 				Prado::using('System.Data.Common.TDbMetaData');
 				$this->_meta[$connStr] = TDbMetaData::getInstance($connection);
 			}
 
 			$tableInfo = null;
-			if(($cache=$this->getManager()->getCache())!==null)
+			if (($cache = $this->getManager()->getCache()) !== null)
 				$tableInfo = $cache->get($key);
-			if(empty($tableInfo))
+			if (empty($tableInfo))
 			{
 				$tableInfo = $this->_meta[$connStr]->getTableInfo($tableName);
-				if($cache!==null)
+				if ($cache !== null)
 					$cache->set($key, $tableInfo);
 			}
 			$this->_tables[$key] = $tableInfo;
@@ -134,7 +134,7 @@ 
 		$conn = $record->getDbConnection();
 		$connStr = $conn->getConnectionString();
 		$tableInfo = $this->getRecordTableInfo($record);
-		if(!isset($this->_commandBuilders[$connStr]))
+		if (!isset($this->_commandBuilders[$connStr]))
 		{
 			$builder = $tableInfo->createCommandBuilder($record->getDbConnection());
 			Prado::using('System.Data.DataGateway.TDataGatewayCommand');
@@ -145,7 +145,7 @@ 
 
 		}
 		$this->_commandBuilders[$connStr]->getBuilder()->setTableInfo($tableInfo);
-		$this->_currentRecord=$record;
+		$this->_currentRecord = $record;
 		return $this->_commandBuilders[$connStr];
 	}
 
@@ -162,7 +162,7 @@ 
 	public function onCreateCommand($sender, $param)
 	{
 		$this->raiseEvent('OnCreateCommand', $this, $param);
-		if($this->_currentRecord!==null)
+		if ($this->_currentRecord !== null)
 			$this->_currentRecord->onCreateCommand($param);
 	}
 
@@ -180,7 +180,7 @@ 
 	public function onExecuteCommand($sender, $param)
 	{
 		$this->raiseEvent('OnExecuteCommand', $this, $param);
-		if($this->_currentRecord!==null)
+		if ($this->_currentRecord !== null)
 			$this->_currentRecord->onExecuteCommand($param);
 	}
 
@@ -191,7 +191,7 @@ 
 	 * @param array primary name value pairs
 	 * @return array record data
 	 */
-	public function findRecordByPK(TActiveRecord $record,$keys)
+	public function findRecordByPK(TActiveRecord $record, $keys)
 	{
 		$command = $this->getCommand($record);
 		return $command->findByPk($keys);
@@ -217,7 +217,7 @@ 
 	 * @param boolean true to return multiple rows as iterator, false returns first row.
 	 * @return mixed matching data.
 	 */
-	public function findRecordsByCriteria(TActiveRecord $record, $criteria, $iterator=false)
+	public function findRecordsByCriteria(TActiveRecord $record, $criteria, $iterator = false)
 	{
 		$command = $this->getCommand($record);
 		return $iterator ? $command->findAll($criteria) : $command->find($criteria);
@@ -247,7 +247,7 @@ 
 
 	public function findRecordsByIndex(TActiveRecord $record, $criteria, $fields, $values)
 	{
-		return $this->getCommand($record)->findAllByIndex($criteria,$fields,$values);
+		return $this->getCommand($record)->findAllByIndex($criteria, $fields, $values);
 	}
 
 	/**
@@ -270,7 +270,7 @@ 
 	{
 		//$this->updateAssociatedRecords($record,true);
 		$result = $this->getCommand($record)->insert($this->getInsertValues($record));
-		if($result)
+		if ($result)
 			$this->updatePostInsert($record);
 		//$this->updateAssociatedRecords($record);
 		return $result;
@@ -284,10 +284,10 @@ 
 	{
 		$command = $this->getCommand($record);
 		$tableInfo = $command->getTableInfo();
-		foreach($tableInfo->getColumns() as $name => $column)
+		foreach ($tableInfo->getColumns() as $name => $column)
 		{
-			if($column->hasSequence())
-				$record->setColumnValue($name,$command->getLastInsertID($column->getSequenceName()));
+			if ($column->hasSequence())
+				$record->setColumnValue($name, $command->getLastInsertID($column->getSequenceName()));
 		}
 	}
 
@@ -297,20 +297,20 @@ 
 	 */
 	protected function getInsertValues(TActiveRecord $record)
 	{
-		$values=array();
+		$values = array();
 		$tableInfo = $this->getCommand($record)->getTableInfo();
-		foreach($tableInfo->getColumns() as $name=>$column)
+		foreach ($tableInfo->getColumns() as $name=>$column)
 		{
-			if($column->getIsExcluded())
+			if ($column->getIsExcluded())
 				continue;
 			$value = $record->getColumnValue($name);
-			if(!$column->getAllowNull() && $value===null && !$column->hasSequence() && ($column->getDefaultValue() === TDbTableColumn::UNDEFINED_VALUE))
+			if (!$column->getAllowNull() && $value === null && !$column->hasSequence() && ($column->getDefaultValue() === TDbTableColumn::UNDEFINED_VALUE))
 			{
 				throw new TActiveRecordException(
 					'ar_value_must_not_be_null', get_class($record),
 					$tableInfo->getTableFullName(), $name);
 			}
-			if($value!==null)
+			if ($value !== null)
 				$values[$name] = $value;
 		}
 		return $values;
@@ -332,29 +332,29 @@ 
 
 	protected function getUpdateValues(TActiveRecord $record)
 	{
-		$values=array();
+		$values = array();
 		$tableInfo = $this->getCommand($record)->getTableInfo();
-		$primary=array();
-		foreach($tableInfo->getColumns() as $name=>$column)
+		$primary = array();
+		foreach ($tableInfo->getColumns() as $name=>$column)
 		{
-			if($column->getIsExcluded())
+			if ($column->getIsExcluded())
 				continue;
 			$value = $record->getColumnValue($name);
-			if(!$column->getAllowNull() && $value===null && ($column->getDefaultValue() === TDbTableColumn::UNDEFINED_VALUE))
+			if (!$column->getAllowNull() && $value === null && ($column->getDefaultValue() === TDbTableColumn::UNDEFINED_VALUE))
 			{
 				throw new TActiveRecordException(
 					'ar_value_must_not_be_null', get_class($record),
 					$tableInfo->getTableFullName(), $name);
 			}
-			if($column->getIsPrimaryKey())
+			if ($column->getIsPrimaryKey())
 				$primary[$name] = $value;
 			else
 				$values[$name] = $value;
 		}
-		return array($values,$primary);
+		return array($values, $primary);
 	}
 
-	protected function updateAssociatedRecords(TActiveRecord $record,$updateBelongsTo=false)
+	protected function updateAssociatedRecords(TActiveRecord $record, $updateBelongsTo = false)
 	{
 		$context = new TActiveRecordRelationContext($record);
 		return $context->updateAssociatedRecords($updateBelongsTo);
@@ -373,10 +373,10 @@ 
 	protected function getPrimaryKeyValues(TActiveRecord $record)
 	{
 		$tableInfo = $this->getCommand($record)->getTableInfo();
-		$primary=array();
-		foreach($tableInfo->getColumns() as $name=>$column)
+		$primary = array();
+		foreach ($tableInfo->getColumns() as $name=>$column)
 		{
-			if($column->getIsPrimaryKey())
+			if ($column->getIsPrimaryKey())
 				$primary[$name] = $record->getColumnValue($name);
 		}
 		return $primary;
@@ -410,11 +410,11 @@ 
 	 * @param TActiveRecord active record
 	 * @param TActiveRecordCriteria data for the command.
 	 */
-	protected function raiseCommandEvent($event,$command,$record,$criteria)
+	protected function raiseCommandEvent($event, $command, $record, $criteria)
 	{
-		if(!($criteria instanceof TSqlCriteria))
-			$criteria = new TActiveRecordCriteria(null,$criteria);
-		$param = new TActiveRecordEventParameter($command,$record,$criteria);
+		if (!($criteria instanceof TSqlCriteria))
+			$criteria = new TActiveRecordCriteria(null, $criteria);
+		$param = new TActiveRecordEventParameter($command, $record, $criteria);
 		$manager = $record->getRecordManager();
 		$manager->{$event}($param);
 		$record->{$event}($param);

Braces +2 added lines, -4 removed lines

@@ -67,16 +67,14 @@
 				throw new TActiveRecordException('ar_invalid_tablename_property',
 					get_class($record),self::TABLE_CONST);
 			return $value;
-		}
-		elseif ($class->hasMethod(self::TABLE_METHOD))
+		} elseif ($class->hasMethod(self::TABLE_METHOD))
 		{
 			$value = $record->{self::TABLE_METHOD}();
 			if(empty($value))
 				throw new TActiveRecordException('ar_invalid_tablename_method',
 					get_class($record),self::TABLE_METHOD);
 			return $value;
-		}
-		else
+		} else
 			return strtolower(get_class($record));
 	}
 

framework/Data/Common/Mssql/TMssqlMetaData.php

Doc Comments +1 added lines

@@ -123,6 +123,7 @@
 	/**
 	 * @param TMssqlTableInfo table information.
 	 * @param array column information.
+	 * @param TMssqlTableInfo $tableInfo
 	 */
 	protected function processColumn($tableInfo, $col)
 	{

Indentation +7 added lines, -7 removed lines

@@ -259,15 +259,15 @@
 		return false;
 	}
 
-        /**
-	 * Returns all table names in the database.
-	 * @param string $schema the schema of the tables. Defaults to empty string, meaning the current or default schema.
-	 * If not empty, the returned table names will be prefixed with the schema name.
-	 * @return array all table names in the database.
-	 */
+		/**
+		 * Returns all table names in the database.
+		 * @param string $schema the schema of the tables. Defaults to empty string, meaning the current or default schema.
+		 * If not empty, the returned table names will be prefixed with the schema name.
+		 * @return array all table names in the database.
+		 */
 	public function findTableNames($schema='dbo')
 	{
-                $condition="TABLE_TYPE='BASE TABLE'";
+				$condition="TABLE_TYPE='BASE TABLE'";
 		$sql=<<<EOD
 SELECT TABLE_NAME, TABLE_SCHEMA FROM [INFORMATION_SCHEMA].[TABLES]
 WHERE TABLE_SCHEMA=:schema AND $condition

Spacing +41 added lines, -41 removed lines

@@ -69,7 +69,7 @@ 
 	 */
 	protected function createTableInfo($table)
 	{
-		list($catalogName,$schemaName,$tableName) = $this->getCatalogSchemaTableName($table);
+		list($catalogName, $schemaName, $tableName) = $this->getCatalogSchemaTableName($table);
 		$this->getDbConnection()->setActive(true);
 		$sql = <<<EOD
 				SELECT t.*,
@@ -80,26 +80,26 @@ 
 									WHERE t.table_name = c.table_name
 										AND t.table_name = :table
 EOD;
-		if($schemaName!==null)
+		if ($schemaName !== null)
 			$sql .= ' AND t.table_schema = :schema';
-		if($catalogName!==null)
+		if ($catalogName !== null)
 			$sql .= ' AND t.table_catalog = :catalog';
 
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $tableName);
-		if($schemaName!==null)
+		if ($schemaName !== null)
 			$command->bindValue(':schema', $schemaName);
-		if($catalogName!==null)
+		if ($catalogName !== null)
 			$command->bindValue(':catalog', $catalogName);
 
-		$tableInfo=null;
-		foreach($command->query() as $col)
+		$tableInfo = null;
+		foreach ($command->query() as $col)
 		{
-			if($tableInfo===null)
+			if ($tableInfo === null)
 				$tableInfo = $this->createNewTableInfo($col);
-			$this->processColumn($tableInfo,$col);
+			$this->processColumn($tableInfo, $col);
 		}
-		if($tableInfo===null)
+		if ($tableInfo === null)
 			throw new TDbException('dbmetadata_invalid_table_view', $table);
 		return $tableInfo;
 	}
@@ -112,12 +112,12 @@ 
 	{
 		//remove possible delimiters
 		$result = explode('.', preg_replace('/\[|\]|"/', '', $table));
-		if(count($result)===1)
-			return array(null,null,$result[0]);
-		if(count($result)===2)
-			return array(null,$result[0],$result[1]);
-		if(count($result)>2)
-			return array($result[0],$result[1],$result[2]);
+		if (count($result) === 1)
+			return array(null, null, $result[0]);
+		if (count($result) === 2)
+			return array(null, $result[0], $result[1]);
+		if (count($result) > 2)
+			return array($result[0], $result[1], $result[2]);
 	}
 
 	/**
@@ -130,25 +130,25 @@ 
 
 		$info['ColumnName'] = "[$columnId]"; //quote the column names!
 		$info['ColumnId'] = $columnId;
-		$info['ColumnIndex'] = intval($col['ORDINAL_POSITION'])-1; //zero-based index
-		if($col['IS_NULLABLE']!=='NO')
+		$info['ColumnIndex'] = intval($col['ORDINAL_POSITION']) - 1; //zero-based index
+		if ($col['IS_NULLABLE'] !== 'NO')
 			$info['AllowNull'] = true;
-		if($col['COLUMN_DEFAULT']!==null)
+		if ($col['COLUMN_DEFAULT'] !== null)
 			$info['DefaultValue'] = $col['COLUMN_DEFAULT'];
 
-		if(in_array($columnId, $tableInfo->getPrimaryKeys()))
+		if (in_array($columnId, $tableInfo->getPrimaryKeys()))
 			$info['IsPrimaryKey'] = true;
-		if($this->isForeignKeyColumn($columnId, $tableInfo))
+		if ($this->isForeignKeyColumn($columnId, $tableInfo))
 			$info['IsForeignKey'] = true;
 
-		if($col['IsIdentity']==='1')
+		if ($col['IsIdentity'] === '1')
 			$info['AutoIncrement'] = true;
 		$info['DbType'] = $col['DATA_TYPE'];
-		if($col['CHARACTER_MAXIMUM_LENGTH']!==null)
+		if ($col['CHARACTER_MAXIMUM_LENGTH'] !== null)
 			$info['ColumnSize'] = intval($col['CHARACTER_MAXIMUM_LENGTH']);
-		if($col['NUMERIC_PRECISION'] !== null)
+		if ($col['NUMERIC_PRECISION'] !== null)
 			$info['NumericPrecision'] = intval($col['NUMERIC_PRECISION']);
-		if($col['NUMERIC_SCALE']!==null)
+		if ($col['NUMERIC_SCALE'] !== null)
 			$info['NumericScale'] = intval($col['NUMERIC_SCALE']);
 		$tableInfo->Columns[$columnId] = new TMssqlTableColumn($info);
 	}
@@ -163,11 +163,11 @@ 
 		$info['CatalogName'] = $col['TABLE_CATALOG'];
 		$info['SchemaName'] = $col['TABLE_SCHEMA'];
 		$info['TableName'] = $col['TABLE_NAME'];
-		if($col['TABLE_TYPE']==='VIEW')
+		if ($col['TABLE_TYPE'] === 'VIEW')
 			$info['IsView'] = true;
 		list($primary, $foreign) = $this->getConstraintKeys($col);
 		$class = $this->getTableInfoClass();
-		return new $class($info,$primary,$foreign);
+		return new $class($info, $primary, $foreign);
 	}
 
 	/**
@@ -192,10 +192,10 @@ 
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $col['TABLE_NAME']);
 		$primary = array();
-		foreach($command->query()->readAll() as $field)
+		foreach ($command->query()->readAll() as $field)
 			$primary[] = $field['field_name'];
 		$foreign = $this->getForeignConstraints($col);
-		return array($primary,$foreign);
+		return array($primary, $foreign);
 	}
 
 	/**
@@ -234,9 +234,9 @@ 
 EOD;
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $col['TABLE_NAME']);
-		$fkeys=array();
+		$fkeys = array();
 		$catalogSchema = "[{$col['TABLE_CATALOG']}].[{$col['TABLE_SCHEMA']}]";
-		foreach($command->query() as $info)
+		foreach ($command->query() as $info)
 		{
 			$fkeys[$info['FK_CONSTRAINT_NAME']]['keys'][$info['FK_COLUMN_NAME']] = $info['UQ_COLUMN_NAME'];
 			$fkeys[$info['FK_CONSTRAINT_NAME']]['table'] = $info['UQ_TABLE_NAME'];
@@ -251,9 +251,9 @@ 
 	 */
 	protected function isForeignKeyColumn($columnId, $tableInfo)
 	{
-		foreach($tableInfo->getForeignKeys() as $fk)
+		foreach ($tableInfo->getForeignKeys() as $fk)
 		{
-			if(in_array($columnId, array_keys($fk['keys'])))
+			if (in_array($columnId, array_keys($fk['keys'])))
 				return true;
 		}
 		return false;
@@ -265,23 +265,23 @@ 
 	 * If not empty, the returned table names will be prefixed with the schema name.
 	 * @return array all table names in the database.
 	 */
-	public function findTableNames($schema='dbo')
+	public function findTableNames($schema = 'dbo')
 	{
-                $condition="TABLE_TYPE='BASE TABLE'";
-		$sql=<<<EOD
+                $condition = "TABLE_TYPE='BASE TABLE'";
+		$sql = <<<EOD
 SELECT TABLE_NAME, TABLE_SCHEMA FROM [INFORMATION_SCHEMA].[TABLES]
 WHERE TABLE_SCHEMA=:schema AND $condition
 EOD;
-		$command=$this->getDbConnection()->createCommand($sql);
+		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindParam(":schema", $schema);
-		$rows=$command->queryAll();
-		$names=array();
+		$rows = $command->queryAll();
+		$names = array();
 		foreach ($rows as $row)
 		{
 			if ($schema == self::DEFAULT_SCHEMA)
-				$names[]=$row['TABLE_NAME'];
+				$names[] = $row['TABLE_NAME'];
 			else
-				$names[]=$schema.'.'.$row['TABLE_SCHEMA'].'.'.$row['TABLE_NAME'];
+				$names[] = $schema . '.' . $row['TABLE_SCHEMA'] . '.' . $row['TABLE_NAME'];
 		}
 
 		return $names;

Braces +3 added lines, -2 removed lines

@@ -192,8 +192,9 @@
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $col['TABLE_NAME']);
 		$primary = array();
-		foreach($command->query()->readAll() as $field)
-			$primary[] = $field['field_name'];
+		foreach($command->query()->readAll() as $field) {
+					$primary[] = $field['field_name'];
+		}
 		$foreign = $this->getForeignConstraints($col);
 		return array($primary,$foreign);
 	}

Upper-Lower-Casing +4 added lines, -4 removed lines

@@ -79,7 +79,7 @@ 
 												INFORMATION_SCHEMA.COLUMNS c
 									WHERE t.table_name = c.table_name
 										AND t.table_name = :table
-EOD;
+eod;
 		if($schemaName!==null)
 			$sql .= ' AND t.table_schema = :schema';
 		if($catalogName!==null)
@@ -188,7 +188,7 @@ 
 		AND
 			c.constraint_type ='PRIMARY KEY'
 				AND k.table_name = :table
-EOD;
+eod;
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $col['TABLE_NAME']);
 		$primary = array();
@@ -231,7 +231,7 @@ 
 		RC.UNIQUE_CONSTRAINT_NAME
 			AND KCU2.ORDINAL_POSITION = KCU1.ORDINAL_POSITION
 		WHERE KCU1.TABLE_NAME = :table
-EOD;
+eod;
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $col['TABLE_NAME']);
 		$fkeys=array();
@@ -271,7 +271,7 @@ 
 		$sql=<<<EOD
 SELECT TABLE_NAME, TABLE_SCHEMA FROM [INFORMATION_SCHEMA].[TABLES]
 WHERE TABLE_SCHEMA=:schema AND $condition
-EOD;
+eod;
 		$command=$this->getDbConnection()->createCommand($sql);
 		$command->bindParam(":schema", $schema);
 		$rows=$command->queryAll();

framework/Data/Common/Mysql/TMysqlMetaData.php

Doc Comments +1 added lines

@@ -114,6 +114,7 @@
 	/**
 	 * @param TMysqlTableInfo table information.
 	 * @param array column information.
+	 * @param TMysqlTableInfo $tableInfo
 	 */
 	protected function processColumn($tableInfo, $col)
 	{

Spacing +67 added lines, -67 removed lines

@@ -27,7 +27,7 @@ 
  */
 class TMysqlMetaData extends TDbMetaData
 {
-	private $_serverVersion=0;
+	private $_serverVersion = 0;
 
 	/**
 	 * @return string TDbTableInfo class name.
@@ -74,24 +74,24 @@ 
 	 */
 	protected function createTableInfo($table)
 	{
-		list($schemaName,$tableName) = $this->getSchemaTableName($table);
-		$find = $schemaName===null ? "`{$tableName}`" : "`{$schemaName}`.`{$tableName}`";
+		list($schemaName, $tableName) = $this->getSchemaTableName($table);
+		$find = $schemaName === null ? "`{$tableName}`" : "`{$schemaName}`.`{$tableName}`";
 		$colCase = $this->getDbConnection()->getColumnCase();
-		if($colCase != TDbColumnCaseMode::Preserved)
+		if ($colCase != TDbColumnCaseMode::Preserved)
 			$this->getDbConnection()->setColumnCase('Preserved');
 		$this->getDbConnection()->setActive(true);
 		$sql = "SHOW FULL FIELDS FROM {$find}";
 		$command = $this->getDbConnection()->createCommand($sql);
 		$tableInfo = $this->createNewTableInfo($table);
-		$index=0;
-		foreach($command->query() as $col)
+		$index = 0;
+		foreach ($command->query() as $col)
 		{
 			$col['index'] = $index++;
-			$this->processColumn($tableInfo,$col);
+			$this->processColumn($tableInfo, $col);
 		}
-		if($index===0)
+		if ($index === 0)
 			throw new TDbException('dbmetadata_invalid_table_view', $table);
-		if($colCase != TDbColumnCaseMode::Preserved)
+		if ($colCase != TDbColumnCaseMode::Preserved)
 			$this->getDbConnection()->setColumnCase($colCase);
 		return $tableInfo;
 	}
@@ -101,12 +101,12 @@ 
 	 */
 	protected function getServerVersion()
 	{
-		if(!$this->_serverVersion)
+		if (!$this->_serverVersion)
 		{
 			$version = $this->getDbConnection()->getAttribute(PDO::ATTR_SERVER_VERSION);
-			$digits=array();
+			$digits = array();
 			preg_match('/(\d+)\.(\d+)\.(\d+)/', $version, $digits);
-			$this->_serverVersion=floatval($digits[1].'.'.$digits[2].$digits[3]);
+			$this->_serverVersion = floatval($digits[1] . '.' . $digits[2] . $digits[3]);
 		}
 		return $this->_serverVersion;
 	}
@@ -122,37 +122,37 @@ 
 		$info['ColumnName'] = "`$columnId`"; //quote the column names!
 		$info['ColumnId'] = $columnId;
 		$info['ColumnIndex'] = $col['index'];
-		if($col['Null']==='YES')
+		if ($col['Null'] === 'YES')
 			$info['AllowNull'] = true;
-		if(is_int(strpos(strtolower($col['Extra']), 'auto_increment')))
-			$info['AutoIncrement']=true;
-		if($col['Default']!=="")
+		if (is_int(strpos(strtolower($col['Extra']), 'auto_increment')))
+			$info['AutoIncrement'] = true;
+		if ($col['Default'] !== "")
 			$info['DefaultValue'] = $col['Default'];
 
-		if($col['Key']==='PRI' || in_array($columnId, $tableInfo->getPrimaryKeys()))
+		if ($col['Key'] === 'PRI' || in_array($columnId, $tableInfo->getPrimaryKeys()))
 			$info['IsPrimaryKey'] = true;
-		if($this->isForeignKeyColumn($columnId, $tableInfo))
+		if ($this->isForeignKeyColumn($columnId, $tableInfo))
 			$info['IsForeignKey'] = true;
 
 		$info['DbType'] = $col['Type'];
-		$match=array();
+		$match = array();
 		//find SET/ENUM values, column size, precision, and scale
-		if(preg_match('/\((.*)\)/', $col['Type'], $match))
+		if (preg_match('/\((.*)\)/', $col['Type'], $match))
 		{
-			$info['DbType']= preg_replace('/\(.*\)/', '', $col['Type']);
+			$info['DbType'] = preg_replace('/\(.*\)/', '', $col['Type']);
 
 			//find SET/ENUM values
-			if($this->isEnumSetType($info['DbType']))
+			if ($this->isEnumSetType($info['DbType']))
 				$info['DbTypeValues'] = preg_split("/[',]/S", $match[1], -1, PREG_SPLIT_NO_EMPTY);
 
 			//find column size, precision and scale
 			$pscale = array();
-			if(preg_match('/(\d+)(?:,(\d+))?+/', $match[1], $pscale))
+			if (preg_match('/(\d+)(?:,(\d+))?+/', $match[1], $pscale))
 			{
-				if($this->isPrecisionType($info['DbType']))
+				if ($this->isPrecisionType($info['DbType']))
 				{
 					$info['NumericPrecision'] = intval($pscale[1]);
-					if(count($pscale) > 2)
+					if (count($pscale) > 2)
 						$info['NumericScale'] = intval($pscale[2]);
 				}
 				else
@@ -169,9 +169,9 @@ 
 	protected function isPrecisionType($type)
 	{
 		$type = strtolower(trim($type));
-		return $type==='decimal' || $type==='dec'
-				|| $type==='float' || $type==='double'
-				|| $type==='double precision' || $type==='real';
+		return $type === 'decimal' || $type === 'dec'
+				|| $type === 'float' || $type === 'double'
+				|| $type === 'double precision' || $type === 'real';
 	}
 
 	/**
@@ -180,7 +180,7 @@ 
 	protected function isEnumSetType($type)
 	{
 		$type = strtolower(trim($type));
-		return $type==='set' || $type==='enum';
+		return $type === 'set' || $type === 'enum';
 	}
 
 	/**
@@ -192,9 +192,9 @@ 
 	{
 		//remove the back ticks and separate out the "database.table"
 		$result = explode('.', str_replace('`', '', $table));
-		foreach($result as $name)
+		foreach ($result as $name)
 		{
-			if(!$this->isValidIdentifier($name))
+			if (!$this->isValidIdentifier($name))
 			{
 				$ref = 'http://dev.mysql.com/doc/refman/5.0/en/identifiers.html';
 				throw new TDbException('dbcommon_invalid_identifier_name', $table, $ref);
@@ -220,14 +220,14 @@ 
 	 */
 	protected function createNewTableInfo($table)
 	{
-		list($schemaName,$tableName) = $this->getSchemaTableName($table);
+		list($schemaName, $tableName) = $this->getSchemaTableName($table);
 		$info['SchemaName'] = $schemaName;
 		$info['TableName'] = $tableName;
-		if($this->getIsView($schemaName,$tableName))
+		if ($this->getIsView($schemaName, $tableName))
 			$info['IsView'] = true;
 		list($primary, $foreign) = $this->getConstraintKeys($schemaName, $tableName);
 		$class = $this->getTableInfoClass();
-		return new $class($info,$primary,$foreign);
+		return new $class($info, $primary, $foreign);
 	}
 
 	/**
@@ -240,11 +240,11 @@ 
 	 * @return boolean true if is view, false otherwise.
 	 * @throws TDbException if table or view does not exist.
 	 */
-	protected function getIsView($schemaName,$tableName)
+	protected function getIsView($schemaName, $tableName)
 	{
-		if($this->getServerVersion()<5.01)
+		if ($this->getServerVersion() < 5.01)
 			return false;
-		if($schemaName!==null)
+		if ($schemaName !== null)
 			$sql = "SHOW FULL TABLES FROM `{$schemaName}` LIKE :table";
 		else
 			$sql = "SHOW FULL TABLES LIKE :table";
@@ -253,12 +253,12 @@ 
 		$command->bindValue(':table', $tableName);
 		try
 		{
-			return count($result = $command->queryRow()) > 0 && $result['Table_type']==='VIEW';
+			return count($result = $command->queryRow()) > 0 && $result['Table_type'] === 'VIEW';
 		}
-		catch(TDbException $e)
+		catch (TDbException $e)
 		{
-			$table = $schemaName===null?$tableName:$schemaName.'.'.$tableName;
-			throw new TDbException('dbcommon_invalid_table_name',$table,$e->getMessage());
+			$table = $schemaName === null ? $tableName : $schemaName . '.' . $tableName;
+			throw new TDbException('dbcommon_invalid_table_name', $table, $e->getMessage());
 		}
 	}
 
@@ -270,22 +270,22 @@ 
 	 */
 	protected function getConstraintKeys($schemaName, $tableName)
 	{
-		$table = $schemaName===null ? "`{$tableName}`" : "`{$schemaName}`.`{$tableName}`";
+		$table = $schemaName === null ? "`{$tableName}`" : "`{$schemaName}`.`{$tableName}`";
 		$sql = "SHOW INDEX FROM {$table}";
 		$command = $this->getDbConnection()->createCommand($sql);
 		$primary = array();
-		foreach($command->query() as $row)
+		foreach ($command->query() as $row)
 		{
-			if($row['Key_name']==='PRIMARY')
+			if ($row['Key_name'] === 'PRIMARY')
 				$primary[] = $row['Column_name'];
 		}
 				// MySQL version was increased to >=5.1.21 instead of 5.x
 				// due to a MySQL bug (http://bugs.mysql.com/bug.php?id=19588)
-		if($this->getServerVersion() >= 5.121)
-			$foreign = $this->getForeignConstraints($schemaName,$tableName);
+		if ($this->getServerVersion() >= 5.121)
+			$foreign = $this->getForeignConstraints($schemaName, $tableName);
 		else
-			$foreign = $this->findForeignConstraints($schemaName,$tableName);
-		return array($primary,$foreign);
+			$foreign = $this->findForeignConstraints($schemaName, $tableName);
+		return array($primary, $foreign);
 	}
 
 	/**
@@ -313,10 +313,10 @@ 
 EOD;
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $tableName);
-		if($schemaName!==null)
+		if ($schemaName !== null)
 			$command->bindValue(':schema', $schemaName);
-		$fkeys=array();
-		foreach($command->query() as $col)
+		$fkeys = array();
+		foreach ($command->query() as $col)
 		{
 			$fkeys[$col['con']]['keys'][$col['col']] = $col['fkcol'];
 			$fkeys[$col['con']]['table'] = $col['fktable'];
@@ -332,12 +332,12 @@ 
 	 */
 	protected function getShowCreateTable($schemaName, $tableName)
 	{
-		if(version_compare(PHP_VERSION,'5.1.3','<'))
+		if (version_compare(PHP_VERSION, '5.1.3', '<'))
 			throw new TDbException('dbmetadata_requires_php_version', 'Mysql 4.1.x', '5.1.3');
 
 		//See http://netevil.org/node.php?nid=795&SC=1
 		$this->getDbConnection()->setAttribute(PDO::ATTR_EMULATE_PREPARES, true);
-		if($schemaName!==null)
+		if ($schemaName !== null)
 			$sql = "SHOW CREATE TABLE `{$schemaName}`.`{$tableName}`";
 		else
 			$sql = "SHOW CREATE TABLE `{$tableName}`";
@@ -355,16 +355,16 @@ 
 	protected function findForeignConstraints($schemaName, $tableName)
 	{
 		$sql = $this->getShowCreateTable($schemaName, $tableName);
-		$matches =array();
+		$matches = array();
 		$regexp = '/FOREIGN KEY\s+\(([^\)]+)\)\s+REFERENCES\s+`?([^`]+)`?\s\(([^\)]+)\)/mi';
-		preg_match_all($regexp,$sql,$matches,PREG_SET_ORDER);
+		preg_match_all($regexp, $sql, $matches, PREG_SET_ORDER);
 		$foreign = array();
-		foreach($matches as $match)
+		foreach ($matches as $match)
 		{
-			$fields = array_map('trim',explode(',',str_replace('`','',$match[1])));
-			$fk_fields = array_map('trim',explode(',',str_replace('`','',$match[3])));
-			$keys=array();
-			foreach($fields as $k=>$v)
+			$fields = array_map('trim', explode(',', str_replace('`', '', $match[1])));
+			$fk_fields = array_map('trim', explode(',', str_replace('`', '', $match[3])));
+			$keys = array();
+			foreach ($fields as $k=>$v)
 				$keys[$v] = $fk_fields[$k];
 			$foreign[] = array('keys' => $keys, 'table' => trim($match[2]));
 		}
@@ -378,9 +378,9 @@ 
 	 */
 	protected function isForeignKeyColumn($columnId, $tableInfo)
 	{
-		foreach($tableInfo->getForeignKeys() as $fk)
+		foreach ($tableInfo->getForeignKeys() as $fk)
 		{
-			if(in_array($columnId, array_keys($fk['keys'])))
+			if (in_array($columnId, array_keys($fk['keys'])))
 				return true;
 		}
 		return false;
@@ -392,13 +392,13 @@ 
 	 * If not empty, the returned table names will be prefixed with the schema name.
 	 * @return array all table names in the database.
 	 */
-	public function findTableNames($schema='')
+	public function findTableNames($schema = '')
 	{
-		if($schema==='')
+		if ($schema === '')
 			return $this->getDbConnection()->createCommand('SHOW TABLES')->queryColumn();
-		$names=$this->getDbConnection()->createCommand('SHOW TABLES FROM '.$this->quoteTableName($schema))->queryColumn();
-		foreach($names as &$name)
-			$name=$schema.'.'.$name;
+		$names = $this->getDbConnection()->createCommand('SHOW TABLES FROM ' . $this->quoteTableName($schema))->queryColumn();
+		foreach ($names as &$name)
+			$name = $schema . '.' . $name;
 		return $names;
 	}
 }

Braces +8 added lines, -8 removed lines

@@ -154,8 +154,7 @@ 
 					$info['NumericPrecision'] = intval($pscale[1]);
 					if(count($pscale) > 2)
 						$info['NumericScale'] = intval($pscale[2]);
-				}
-				else
+				} else
 					$info['ColumnSize'] = intval($pscale[1]);
 			}
 		}
@@ -254,8 +253,7 @@ 
 		try
 		{
 			return count($result = $command->queryRow()) > 0 && $result['Table_type']==='VIEW';
-		}
-		catch(TDbException $e)
+		} catch(TDbException $e)
 		{
 			$table = $schemaName===null?$tableName:$schemaName.'.'.$tableName;
 			throw new TDbException('dbcommon_invalid_table_name',$table,$e->getMessage());
@@ -364,8 +362,9 @@ 
 			$fields = array_map('trim',explode(',',str_replace('`','',$match[1])));
 			$fk_fields = array_map('trim',explode(',',str_replace('`','',$match[3])));
 			$keys=array();
-			foreach($fields as $k=>$v)
-				$keys[$v] = $fk_fields[$k];
+			foreach($fields as $k=>$v) {
+							$keys[$v] = $fk_fields[$k];
+			}
 			$foreign[] = array('keys' => $keys, 'table' => trim($match[2]));
 		}
 		return $foreign;
@@ -397,8 +396,9 @@ 
 		if($schema==='')
 			return $this->getDbConnection()->createCommand('SHOW TABLES')->queryColumn();
 		$names=$this->getDbConnection()->createCommand('SHOW TABLES FROM '.$this->quoteTableName($schema))->queryColumn();
-		foreach($names as &$name)
-			$name=$schema.'.'.$name;
+		foreach($names as &$name) {
+					$name=$schema.'.'.$name;
+		}
 		return $names;
 	}
 }

Indentation +6 added lines, -6 removed lines

@@ -335,12 +335,12 @@
 		return false;
 	}
 
-        /**
-	 * Returns all table names in the database.
-	 * @param string $schema the schema of the tables. Defaults to empty string, meaning the current or default schema.
-	 * If not empty, the returned table names will be prefixed with the schema name.
-	 * @return array all table names in the database.
-	 */
+		/**
+		 * Returns all table names in the database.
+		 * @param string $schema the schema of the tables. Defaults to empty string, meaning the current or default schema.
+		 * If not empty, the returned table names will be prefixed with the schema name.
+		 * @return array all table names in the database.
+		 */
 	public function findTableNames($schema='')
 	{
 		if($schema==='')

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -310,7 +310,7 @@
 				REFERENCED_TABLE_NAME IS NOT NULL
 				AND TABLE_NAME LIKE :table
 				$andSchema
-EOD;
+eod;
 		$command = $this->getDbConnection()->createCommand($sql);
 		$command->bindValue(':table', $tableName);
 		if($schemaName!==null)

framework/Data/Common/Oracle/TOracleMetaData.php

Doc Comments +3 added lines, -1 removed lines

@@ -145,7 +145,7 @@ 
 	/**
 	 * @param string table schema name
 	 * @param string table name.
-	 * @return boolean true if the table is a view.
+	 * @return integer true if the table is a view.
 	 */
 	protected function getIsView($schemaName,$tableName)
 	{
@@ -167,6 +167,7 @@ 
 	/**
 	 * @param TOracleTableInfo table information.
 	 * @param array column information.
+	 * @param TOracleTableInfo $tableInfo
 	 */
 	protected function processColumn($tableInfo, $col)
 	{
@@ -322,6 +323,7 @@ 
 	/**
 	 * @param string column name.
 	 * @param TOracleTableInfo table information.
+	 * @param string $columnId
 	 * @return boolean true if column is a foreign key.
 	 */
 	protected function isForeignKeyColumn($columnId, $tableInfo)

Indentation +6 added lines, -6 removed lines

@@ -335,12 +335,12 @@
 		return false;
 	}
 
-        /**
-	 * Returns all table names in the database.
-	 * @param string $schema the schema of the tables. Defaults to empty string, meaning the current or default schema.
-	 * If not empty, the returned table names will be prefixed with the schema name.
-	 * @return array all table names in the database.
-	 */
+		/**
+		 * Returns all table names in the database.
+		 * @param string $schema the schema of the tables. Defaults to empty string, meaning the current or default schema.
+		 * If not empty, the returned table names will be prefixed with the schema name.
+		 * @return array all table names in the database.
+		 */
 	public function findTableNames($schema='')
 	{
 		if($schema==='')

Spacing +56 added lines, -56 removed lines

@@ -41,7 +41,7 @@ 
 	 */
 	public function setDefaultSchema($schema)
 	{
-		$this->_defaultSchema=$schema;
+		$this->_defaultSchema = $schema;
 	}
 
 	/**
@@ -58,10 +58,10 @@ 
 	 */
 	protected function getSchemaTableName($table)
 	{
-		if(count($parts= explode('.', str_replace('"','',$table))) > 1)
+		if (count($parts = explode('.', str_replace('"', '', $table))) > 1)
 			return array($parts[0], $parts[1]);
 		else
-			return array($this->getDefaultSchema(),$parts[0]);
+			return array($this->getDefaultSchema(), $parts[0]);
 	}
 
 	/**
@@ -71,7 +71,7 @@ 
 	 */
 	protected function createTableInfo($table)
 	{
-		list($schemaName,$tableName) = $this->getSchemaTableName($table);
+		list($schemaName, $tableName) = $this->getSchemaTableName($table);
 
 		// This query is made much more complex by the addition of the 'attisserial' field.
 		// The subquery to get that field checks to see if there is an internally dependent
@@ -100,13 +100,13 @@ 
 		//$command->bindValue(':table', $tableName);
 		//$command->bindValue(':schema', $schemaName);
 		$tableInfo = $this->createNewTableInfo($schemaName, $tableName);
-		$index=0;
-		foreach($command->query() as $col)
+		$index = 0;
+		foreach ($command->query() as $col)
 		{
 			$col['index'] = $index++;
 			$this->processColumn($tableInfo, $col);
 		}
-		if($index===0)
+		if ($index === 0)
 			throw new TDbException('dbmetadata_invalid_table_view', $table);
 		return $tableInfo;
 	}
@@ -116,15 +116,15 @@ 
 	 * @param string table name.
 	 * @return TOracleTableInfo
 	 */
-	protected function createNewTableInfo($schemaName,$tableName)
+	protected function createNewTableInfo($schemaName, $tableName)
 	{
 		$info['SchemaName'] = $this->assertIdentifier($schemaName);
-		$info['TableName']	= $this->assertIdentifier($tableName);
-		$info['IsView'] 	= false;
-		if($this->getIsView($schemaName,$tableName)) $info['IsView'] = true;
+		$info['TableName'] = $this->assertIdentifier($tableName);
+		$info['IsView'] = false;
+		if ($this->getIsView($schemaName, $tableName)) $info['IsView'] = true;
 		list($primary, $foreign) = $this->getConstraintKeys($schemaName, $tableName);
 		$class = $this->getTableInfoClass();
-		return new $class($info,$primary,$foreign);
+		return new $class($info, $primary, $foreign);
 	}
 
 	/**
@@ -134,7 +134,7 @@ 
 	 */
 	protected function assertIdentifier($name)
 	{
-		if(strpos($name, '"')!==false)
+		if (strpos($name, '"') !== false)
 		{
 			$ref = 'http://www.oracle.com';
 			throw new TDbException('dbcommon_invalid_identifier_name', $name, $ref);
@@ -147,7 +147,7 @@ 
 	 * @param string table name.
 	 * @return boolean true if the table is a view.
 	 */
-	protected function getIsView($schemaName,$tableName)
+	protected function getIsView($schemaName, $tableName)
 	{
 		$this->getDbConnection()->setAttribute(PDO::ATTR_CASE, PDO::CASE_LOWER);
 		$sql =
@@ -173,14 +173,14 @@ 
 		$columnId = strtolower($col['attname']); //use column name as column Id
 
 		//$info['ColumnName'] 	= '"'.$columnId.'"'; //quote the column names!
-		$info['ColumnName'] 	= $columnId; //NOT quote the column names!
-		$info['ColumnId'] 		= $columnId;
-		$info['ColumnIndex'] 	= $col['index'];
-		if(! (bool)$col['attnotnull'] ) $info['AllowNull'] = true;
-		if(in_array($columnId, $tableInfo->getPrimaryKeys())) $info['IsPrimaryKey'] = true;
-		if($this->isForeignKeyColumn($columnId, $tableInfo)) $info['IsForeignKey'] = true;
-		if( (int)$col['atttypmod'] > 0 ) $info['ColumnSize'] =  $col['atttypmod']; // - 4;
-		if( (bool)$col['atthasdef'] ) $info['DefaultValue'] = $col['adsrc'];
+		$info['ColumnName'] = $columnId; //NOT quote the column names!
+		$info['ColumnId'] = $columnId;
+		$info['ColumnIndex'] = $col['index'];
+		if (!(bool) $col['attnotnull']) $info['AllowNull'] = true;
+		if (in_array($columnId, $tableInfo->getPrimaryKeys())) $info['IsPrimaryKey'] = true;
+		if ($this->isForeignKeyColumn($columnId, $tableInfo)) $info['IsForeignKey'] = true;
+		if ((int) $col['atttypmod'] > 0) $info['ColumnSize'] = $col['atttypmod']; // - 4;
+		if ((bool) $col['atthasdef']) $info['DefaultValue'] = $col['adsrc'];
 		//
 		// For a while Oracle Tables has no  associated AutoIncrement Triggers
 		//
@@ -195,13 +195,13 @@ 
 		}
 		*/
 		$matches = array();
-		if(preg_match('/\((\d+)(?:,(\d+))?+\)/', $col['type'], $matches))
+		if (preg_match('/\((\d+)(?:,(\d+))?+\)/', $col['type'], $matches))
 		{
-			$info['DbType'] = preg_replace('/\(\d+(?:,\d+)?\)/','',$col['type']);
-			if($this->isPrecisionType($info['DbType']))
+			$info['DbType'] = preg_replace('/\(\d+(?:,\d+)?\)/', '', $col['type']);
+			if ($this->isPrecisionType($info['DbType']))
 			{
 				$info['NumericPrecision'] = intval($matches[1]);
-				if(count($matches) > 2)
+				if (count($matches) > 2)
 					$info['NumericScale'] = intval($matches[2]);
 			}
 			else
@@ -215,15 +215,15 @@ 
 	/**
 	 * @return string serial name if found, null otherwise.
 	 */
-	protected function getSequenceName($tableInfo,$src)
+	protected function getSequenceName($tableInfo, $src)
 	{
 		$matches = array();
-		if(preg_match('/nextval\([^\']*\'([^\']+)\'[^\)]*\)/i',$src,$matches))
+		if (preg_match('/nextval\([^\']*\'([^\']+)\'[^\)]*\)/i', $src, $matches))
 		{
-			if(is_int(strpos($matches[1], '.')))
+			if (is_int(strpos($matches[1], '.')))
 				return $matches[1];
 			else
-				return $tableInfo->getSchemaName().'.'.$matches[1];
+				return $tableInfo->getSchemaName() . '.' . $matches[1];
 		}
 	}
 
@@ -233,7 +233,7 @@ 
 	protected function isPrecisionType($type)
 	{
 		$type = strtolower(trim($type));
-		return $type==='number'; // || $type==='interval' || strpos($type, 'time')===0;
+		return $type === 'number'; // || $type==='interval' || strpos($type, 'time')===0;
 	}
 
 	/**
@@ -262,19 +262,19 @@ 
 		//$command->bindValue(':schema', $schemaName);
 		$primary = array();
 		$foreign = array();
-		foreach($command->query() as $row)
+		foreach ($command->query() as $row)
 		{
-			switch( strtolower( $row['contype'] ) )
+			switch (strtolower($row['contype']))
 			{
 				case 'p':
-					$primary = array_merge( $primary, array(strtolower( $row['consrc'] )) );
+					$primary = array_merge($primary, array(strtolower($row['consrc'])));
 					/*
 					$arr = $this->getPrimaryKeys($row['consrc']);
 					$primary = array_merge( $primary, array(strtolower( $arr[0] )) );
 					*/
 					break;
 				case 'r':
-					$foreign = array_merge( $foreign, array(strtolower( $row['consrc'] )) );
+					$foreign = array_merge($foreign, array(strtolower($row['consrc'])));
 					/*
 					// if(($fkey = $this->getForeignKeys($row['consrc']))!==null)
 					$fkey = $this->getForeignKeys( $row['consrc'] );
@@ -283,7 +283,7 @@ 
 					break;
 			}
 		}
-		return array($primary,$foreign);
+		return array($primary, $foreign);
 	}
 
 	/**
@@ -294,8 +294,8 @@ 
 	protected function getPrimaryKeys($src)
 	{
 		$matches = array();
-		if(preg_match('/PRIMARY\s+KEY\s+\(([^\)]+)\)/i', $src, $matches))
-			return preg_split('/,\s+/',$matches[1]);
+		if (preg_match('/PRIMARY\s+KEY\s+\(([^\)]+)\)/i', $src, $matches))
+			return preg_split('/,\s+/', $matches[1]);
 		return array();
 	}
 
@@ -309,13 +309,13 @@ 
 		$matches = array();
 		$brackets = '\(([^\)]+)\)';
 		$find = "/FOREIGN\s+KEY\s+{$brackets}\s+REFERENCES\s+([^\(]+){$brackets}/i";
-		if(preg_match($find, $src, $matches))
+		if (preg_match($find, $src, $matches))
 		{
 			$keys = preg_split('/,\s+/', $matches[1]);
 			$fkeys = array();
-			foreach(preg_split('/,\s+/', $matches[3]) as $i => $fkey)
+			foreach (preg_split('/,\s+/', $matches[3]) as $i => $fkey)
 				$fkeys[$keys[$i]] = $fkey;
-			return array('table' => str_replace('"','',$matches[2]), 'keys' => $fkeys);
+			return array('table' => str_replace('"', '', $matches[2]), 'keys' => $fkeys);
 		}
 	}
 
@@ -326,9 +326,9 @@ 
 	 */
 	protected function isForeignKeyColumn($columnId, $tableInfo)
 	{
-		foreach($tableInfo->getForeignKeys() as $fk)
+		foreach ($tableInfo->getForeignKeys() as $fk)
 		{
-			if( $fk==$columnId )
+			if ($fk == $columnId)
 			//if(in_array($columnId, array_keys($fk['keys'])))
 				return true;
 		}
@@ -341,33 +341,33 @@ 
 	 * If not empty, the returned table names will be prefixed with the schema name.
 	 * @return array all table names in the database.
 	 */
-	public function findTableNames($schema='')
+	public function findTableNames($schema = '')
 	{
-		if($schema==='')
+		if ($schema === '')
 		{
-			$sql=<<<EOD
+			$sql = <<<EOD
 SELECT table_name, '{$schema}' as table_schema FROM user_tables
 EOD;
-			$command=$this->getDbConnection()->createCommand($sql);
+			$command = $this->getDbConnection()->createCommand($sql);
 		}
 		else
 		{
-			$sql=<<<EOD
+			$sql = <<<EOD
 SELECT object_name as table_name, owner as table_schema FROM all_objects
 WHERE object_type = 'TABLE' AND owner=:schema
 EOD;
-			$command=$this->getDbConnection()->createCommand($sql);
-			$command->bindParam(':schema',$schema);
+			$command = $this->getDbConnection()->createCommand($sql);
+			$command->bindParam(':schema', $schema);
 		}
 
-		$rows=$command->queryAll();
-		$names=array();
-		foreach($rows as $row)
+		$rows = $command->queryAll();
+		$names = array();
+		foreach ($rows as $row)
 		{
-			if($schema===$this->getDefaultSchema() || $schema==='')
-				$names[]=$row['TABLE_NAME'];
+			if ($schema === $this->getDefaultSchema() || $schema === '')
+				$names[] = $row['TABLE_NAME'];
 			else
-				$names[]=$row['TABLE_SCHEMA'].'.'.$row['TABLE_NAME'];
+				$names[] = $row['TABLE_SCHEMA'] . '.' . $row['TABLE_NAME'];
 		}
 		return $names;
 	}

Braces +6 added lines, -8 removed lines

@@ -203,11 +203,9 @@ 
 				$info['NumericPrecision'] = intval($matches[1]);
 				if(count($matches) > 2)
 					$info['NumericScale'] = intval($matches[2]);
-			}
-			else
+			} else
 				$info['ColumnSize'] = intval($matches[1]);
-		}
-		else
+		} else
 			$info['DbType'] = $col['type'];
 		$tableInfo->Columns[$columnId] = new TOracleTableColumn($info);
 	}
@@ -313,8 +311,9 @@ 
 		{
 			$keys = preg_split('/,\s+/', $matches[1]);
 			$fkeys = array();
-			foreach(preg_split('/,\s+/', $matches[3]) as $i => $fkey)
-				$fkeys[$keys[$i]] = $fkey;
+			foreach(preg_split('/,\s+/', $matches[3]) as $i => $fkey) {
+							$fkeys[$keys[$i]] = $fkey;
+			}
 			return array('table' => str_replace('"','',$matches[2]), 'keys' => $fkeys);
 		}
 	}
@@ -349,8 +348,7 @@ 
 SELECT table_name, '{$schema}' as table_schema FROM user_tables
 EOD;
 			$command=$this->getDbConnection()->createCommand($sql);
-		}
-		else
+		} else
 		{
 			$sql=<<<EOD
 SELECT object_name as table_name, owner as table_schema FROM all_objects

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -93,7 +93,7 @@ 
 			TABLE_NAME = '{$tableName}'
 			AND OWNER = '{$schemaName}'
 		ORDER BY a.COLUMN_ID
-EOD;
+eod;
 		$this->getDbConnection()->setActive(true);
 		$this->getDbConnection()->setAttribute(PDO::ATTR_CASE, PDO::CASE_LOWER);
 		$command = $this->getDbConnection()->createCommand($sql);
@@ -156,7 +156,7 @@ 
 		from 	ALL_OBJECTS
 		where	OBJECT_NAME = '{$tableName}'
 		and 	OWNER = '{$schemaName}'
-EOD;
+eod;
 		$this->getDbConnection()->setActive(true);
 		$command = $this->getDbConnection()->createCommand($sql);
 		//$command->bindValue(':schema',$schemaName);
@@ -255,7 +255,7 @@ 
 		and	  a.TABLE_NAME = '{$tableName}'
 		and   a.OWNER = '{$schemaName}'
 		and   a.CONSTRAINT_TYPE in ('P','R')
-EOD;
+eod;
 		$this->getDbConnection()->setActive(true);
 		$command = $this->getDbConnection()->createCommand($sql);
 		//$command->bindValue(':table', $tableName);
@@ -347,7 +347,7 @@ 
 		{
 			$sql=<<<EOD
 SELECT table_name, '{$schema}' as table_schema FROM user_tables
-EOD;
+eod;
 			$command=$this->getDbConnection()->createCommand($sql);
 		}
 		else
@@ -355,7 +355,7 @@ 
 			$sql=<<<EOD
 SELECT object_name as table_name, owner as table_schema FROM all_objects
 WHERE object_type = 'TABLE' AND owner=:schema
-EOD;
+eod;
 			$command=$this->getDbConnection()->createCommand($sql);
 			$command->bindParam(':schema',$schema);
 		}

framework/Data/Common/Oracle/TOracleTableInfo.php

Doc Comments +2 added lines

@@ -53,6 +53,8 @@
 	/**
 	 * @param string information array key name
 	 * @param mixed default value if information array value is null
+	 * @param string $name
+	 * @param boolean $default
 	 * @return mixed information array value.
 	 */
 	public function getInfo($name,$default=null)

Spacing +17 added lines, -17 removed lines

@@ -19,7 +19,7 @@ 
  */
 class TOracleTableInfo extends TComponent
 {
-	private $_info=array();
+	private $_info = array();
 
 	private $_primaryKeys;
 	private $_foreignKeys;
@@ -32,12 +32,12 @@ 
 	 * Sets the database table meta data information.
 	 * @param array table column information.
 	 */
-	public function __construct($tableInfo=array(),$primary=array(),$foreign=array())
+	public function __construct($tableInfo = array(), $primary = array(), $foreign = array())
 	{
-		$this->_info=$tableInfo;
-		$this->_primaryKeys=$primary;
-		$this->_foreignKeys=$foreign;
-		$this->_columns=new TMap;
+		$this->_info = $tableInfo;
+		$this->_primaryKeys = $primary;
+		$this->_foreignKeys = $foreign;
+		$this->_columns = new TMap;
 	}
 
 	/**
@@ -47,7 +47,7 @@ 
 	public function createCommandBuilder($connection)
 	{
 		Prado::using('System.Data.Common.Oracle.TOracleCommandBuilder');
-		return new TOracleCommandBuilder($connection,$this);
+		return new TOracleCommandBuilder($connection, $this);
 	}
 
 	/**
@@ -55,7 +55,7 @@ 
 	 * @param mixed default value if information array value is null
 	 * @return mixed information array value.
 	 */
-	public function getInfo($name,$default=null)
+	public function getInfo($name, $default = null)
 	{
 		return isset($this->_info[$name]) ? $this->_info[$name] : $default;
 	}
@@ -64,9 +64,9 @@ 
 	 * @param string information array key name
 	 * @param mixed new information array value.
 	 */
-	protected function setInfo($name,$value)
+	protected function setInfo($name, $value)
 	{
-		$this->_info[$name]=$value;
+		$this->_info[$name] = $value;
 	}
 
 	/**
@@ -82,7 +82,7 @@ 
 	 */
 	public function getTableFullName()
 	{
-		return $this->_info['SchemaName'].'.'.$this->getTableName();
+		return $this->_info['SchemaName'] . '.' . $this->getTableName();
 	}
 
 	/**
@@ -90,7 +90,7 @@ 
 	 */
 	public function getIsView()
 	{
-		return $this->getInfo('IsView',false);
+		return $this->getInfo('IsView', false);
 	}
 
 	/**
@@ -107,7 +107,7 @@ 
 	 */
 	public function getColumn($name)
 	{
-		if(($column = $this->_columns->itemAt($name))!==null)
+		if (($column = $this->_columns->itemAt($name)) !== null)
 			return $column;
 		throw new TDbException('dbtableinfo_invalid_column_name', $name, $this->getTableFullName());
 	}
@@ -118,7 +118,7 @@ 
 	 */
 	public function getColumnNames()
 	{
-		foreach($this->getColumns() as $column)
+		foreach ($this->getColumns() as $column)
 			$names[] = $column->getColumnName();
 		return $names;
 	}
@@ -144,10 +144,10 @@ 
 	 */
 	public function getLowerCaseColumnNames()
 	{
-		if($this->_lowercase===null)
+		if ($this->_lowercase === null)
 		{
-			$this->_lowercase=array();
-			foreach($this->getColumns()->getKeys() as $key)
+			$this->_lowercase = array();
+			foreach ($this->getColumns()->getKeys() as $key)
 				$this->_lowercase[strtolower($key)] = $key;
 		}
 		return $this->_lowercase;

Braces +6 added lines, -4 removed lines

@@ -118,8 +118,9 @@ 
 	 */
 	public function getColumnNames()
 	{
-		foreach($this->getColumns() as $column)
-			$names[] = $column->getColumnName();
+		foreach($this->getColumns() as $column) {
+					$names[] = $column->getColumnName();
+		}
 		return $names;
 	}
 
@@ -147,8 +148,9 @@ 
 		if($this->_lowercase===null)
 		{
 			$this->_lowercase=array();
-			foreach($this->getColumns()->getKeys() as $key)
-				$this->_lowercase[strtolower($key)] = $key;
+			foreach($this->getColumns()->getKeys() as $key) {
+							$this->_lowercase[strtolower($key)] = $key;
+			}
 		}
 		return $this->_lowercase;
 	}