Skip to content

Commit c1c7bfa

Browse files
committed
refactorred Model to use RelationalMappingUtil
1 parent 2182804 commit c1c7bfa

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

src/Persist/Model.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -523,7 +523,7 @@ private static function generateClauses($connection, &$classList,$filters,$sort,
523523
$_prefix = $prefix;
524524
$_columnName = $field['columnName'];
525525

526-
if(isset($field['foreignField'])){
526+
if(RelationalMappingUtil::_isForeignField($field)){
527527

528528
$foreignClassAndField = preg_split("/\-\>/",$field['foreignField']);
529529
$foreignClassName = $foreignClassAndField[0];
@@ -762,7 +762,7 @@ protected function setAttributes($args){
762762
$reflectionClass = new \ReflectionClass($className);
763763
$fields = $class['fields'];
764764
foreach($fields as $fieldName=>$field){
765-
if(isset($field['set']) && (strtoupper($field['set']) == "TRUE" || $field['set'] === true )){
765+
if(RelationalMappingUtil::_isSet($field)){
766766
if(isset($args[$fieldName])){
767767

768768
$value = $args[$fieldName];
@@ -784,7 +784,7 @@ protected function setAttributes($args){
784784
}
785785
}
786786

787-
if($field['foreignField']){
787+
if(RelationalMappingUtil::_isForeignField($field)){
788788
$foreignClassAndField = preg_split("/\-\>/",$field['foreignField']);
789789
$foreignClassName = $foreignClassAndField[0];
790790
$foreignFieldName = $foreignClassAndField[1];
@@ -793,7 +793,7 @@ protected function setAttributes($args){
793793

794794
$foreignPrimaryField = (object)array();
795795
foreach($foreignClassConf['fields'] as $foreignField){
796-
if(isset($foreignField['primary']) && (strtoupper($foreignField['primary']) == "TRUE" || $foreignField['primary'] === true) ){
796+
if(RelationalMappingUtil::_isPrimary($foreignField)){
797797
$foreignPrimaryField = $foreignField;
798798
break;
799799
}
@@ -917,7 +917,7 @@ protected function getAttributes($args){
917917
$fields = $class['fields'];
918918
foreach($fields as $fieldName=>$field){
919919

920-
if(isset($field['get']) && (strtoupper($field['get']) == "TRUE" || $field['get'] === true )){
920+
if(RelationalMappingUtil::_isGet($field)){
921921
if(in_array($fieldName,$args)){
922922
$reflectionProperty = $reflectionClass->getProperty($fieldName);
923923
$reflectionProperty->setAccessible(true);
@@ -950,7 +950,7 @@ private function recordIdentifier($className,$class){
950950
$recordIdentifier = $fields[$primaryKey]['columnName']." = '$keyValue'";
951951
}else{
952952
foreach($fields as $fieldName=>$field){
953-
if(isset($field['columnName']) && !isset($field['foreignField'])){
953+
if(isset($field['columnName']) && !RelationalMappingUtil::_isForeignField($field)){
954954
$reflectionProperty = $reflectionClass->getProperty($fieldName);
955955
$reflectionProperty->setAccessible(true);
956956
$fieldValue = $reflectionProperty->getValue($this);

src/Persist/RelationalMappingUtil.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,27 +49,27 @@ public static function getReferenceKey(&$classInfo){
4949
return null;
5050
}
5151

52-
public static function _isSet(&$field){
52+
public static function _isSet($field){
5353
return isset($field['set']) && (strtoupper($field['set']) == "TRUE" || $field['set'] === true);
5454
}
5555

56-
public static function _isGet(&$field){
56+
public static function _isGet($field){
5757
return isset($field['get']) && (strtoupper($field['get']) == "TRUE" || $field['get'] === true);
5858
}
5959

60-
public static function _isPrimary(&$field){
60+
public static function _isPrimary($field){
6161
return isset($field['primary']) && (strtoupper($field['primary']) == "TRUE" || $field['primary'] === true);
6262
}
6363

64-
public static function _isReference(&$field){
64+
public static function _isReference($field){
6565
return isset($field['reference']) && (strtoupper($field['reference']) == "TRUE" || $field['reference'] === true);
6666
}
6767

68-
public static function _isAutoIncrement(&$field){
68+
public static function _isAutoIncrement($field){
6969
return isset($field['autoIncrement']) && (strtoupper($field['autoIncrement']) == "TRUE" || $field['autoIncrement'] === true);
7070
}
7171

72-
public static function _isForeignField(&$field){
72+
public static function _isForeignField($field){
7373
return isset($field['foreignField']);
7474
}
7575

0 commit comments

Comments
 (0)