capwelton / libapp

programs/functions.php

Spacing +4 added lines, -4 removed lines

@@ -112,7 +112,7 @@ 
     if (null === $url) {
         $script = '<script type="text/javascript">';
         if (isset($message)) {
-            $script .= 'alert("' . bab_toHtml($message, BAB_HTML_JS) . '");';
+            $script .= 'alert("'.bab_toHtml($message, BAB_HTML_JS).'");';
         }
         $script .= 'history.back();';
         $script .= '</script>';
@@ -124,7 +124,7 @@ 
         $url = $url->url();
     }
     if (!empty($babBody->msgerror)) {
-        $url .=  '&msgerror=' . urlencode($babBody->msgerror);
+        $url .= '&msgerror='.urlencode($babBody->msgerror);
     }
     if (isset($message)) {
         $lines = explode("\n", $message);
@@ -133,7 +133,7 @@ 
         }
     }
 
-    header('Location: ' . $url);
+    header('Location: '.$url);
     die;
 }
 
@@ -165,7 +165,7 @@ 
  */
 function app_fileUploadError($error)
 {
-    switch($error) {
+    switch ($error) {
         case UPLOAD_ERR_OK:
             return null;
 

programs/init.php

Spacing +8 added lines, -8 removed lines

@@ -24,7 +24,7 @@ 
 
 
 
-require_once dirname(__FILE__). '/functions.php';
+require_once dirname(__FILE__).'/functions.php';
 
 
 function LibApp_upgrade($sVersionBase, $sVersionIni)
@@ -80,13 +80,13 @@ 
     
     $babBody = bab_getBody();
     $addon = bab_getAddonInfosInstance('libapp');
-    $babBody->addStyleSheet($addon->getStylePath() . '/styles.css');
-    $babBody->addJavascriptFile($addon->getTemplatePath() . 'libapp.js');
+    $babBody->addStyleSheet($addon->getStylePath().'/styles.css');
+    $babBody->addJavascriptFile($addon->getTemplatePath().'libapp.js');
     
     $App = app_App();
-    if($App->isSSEEnabled()){
-        $babBody->addJavascriptFile($addon->getTemplatePath() . 'notification.js');
-        if(bab_isUserLogged()){
+    if ($App->isSSEEnabled()) {
+        $babBody->addJavascriptFile($addon->getTemplatePath().'notification.js');
+        if (bab_isUserLogged()) {
             $babBody->babecho('<script>window.libapp={"sseServerUrl": "'.$App->Controller()->SSE()->server()->url().'"}</script>');
             $babBody->addJavascriptFile($addon->getTemplatePath().'sse.js');
         }
@@ -112,8 +112,8 @@ 
         $addonPhpPath = $addon->getPhpPath();
         
         require_once $GLOBALS['babInstallPath'].'utilit/functionalityincl.php';
-        require_once dirname(__FILE__) . '/portletbackend.class.php';
+        require_once dirname(__FILE__).'/portletbackend.class.php';
         $functionalities = new bab_functionalities();
-        $functionalities->registerClass('Func_PortletBackend_App', $addonPhpPath . 'portletbackend.class.php');
+        $functionalities->registerClass('Func_PortletBackend_App', $addonPhpPath.'portletbackend.class.php');
     }
 }
\ No newline at end of file

programs/define.php

Spacing +6 added lines, -6 removed lines

@@ -21,9 +21,9 @@
  * @copyright Copyright (c) 2006 by CANTICO ({@link http://www.cantico.fr})

  */

 
-define('APP_PHP_PATH', dirname(__FILE__) . '/');

-define('APP_SET_PATH', dirname(__FILE__) . '/');

-define('APP_UI_PATH', dirname(__FILE__) . '/ui/');

-define('APP_WIDGETS_PATH', APP_UI_PATH . 'widgets/');

-define('APP_CTRL_PATH', dirname(__FILE__) . '/');
-define('APP_PORTLET_PATH', APP_PHP_PATH . 'portlet/');
\ No newline at end of file
+define('APP_PHP_PATH', dirname(__FILE__).'/');

+define('APP_SET_PATH', dirname(__FILE__).'/');

+define('APP_UI_PATH', dirname(__FILE__).'/ui/');

+define('APP_WIDGETS_PATH', APP_UI_PATH.'widgets/');

+define('APP_CTRL_PATH', dirname(__FILE__).'/');
+define('APP_PORTLET_PATH', APP_PHP_PATH.'portlet/');
\ No newline at end of file

programs/Set/AppTraceableRecordSet.php

Spacing +18 added lines, -18 removed lines

@@ -125,7 +125,7 @@ 
      */
     protected function logSave(AppTraceableRecord $record, $noTrace)
     {
-        if(! $this->isLoggable()){
+        if (!$this->isLoggable()) {
             return;
         }
         $App = $this->App();
@@ -148,14 +148,14 @@ 
      */
     protected function logDelete(\ORM_Criteria $criteria, $noTrace)
     {
-        if(! $this->isLoggable()){
+        if (!$this->isLoggable()) {
             return;
         }
         $App = $this->App();
         $userId = bab_getUserId();
         $logSet = $App->LogSet();
         $deletedRecords = $this->select($criteria);
-        foreach ($deletedRecords as $record){
+        foreach ($deletedRecords as $record) {
             $log = $logSet->newRecord();
             $log->noTrace = $noTrace;
             $log->objectClass = get_class($record);
@@ -181,7 +181,7 @@ 
      */
     public function select(\ORM_Criteria $criteria = null, $includeDeleted = false)
     {
-        if($includeDeleted){
+        if ($includeDeleted) {
             $this->setDefaultCriteria(null);
         }
         return parent::select($criteria);
@@ -224,25 +224,25 @@ 
      */
     public function delete(\ORM_Criteria $criteria = null, $deletedStatus = AppTraceableRecord::DELETED_STATUS_DELETED)
     {
-        $definitive = ($deletedStatus === true) || ! $this->isTraceable();
+        $definitive = ($deletedStatus === true) || !$this->isTraceable();
         $this->logDelete($criteria, $definitive);
-        if($definitive){
+        if ($definitive) {
             return parent::delete($criteria);
         }
         
-        require_once $GLOBALS['babInstallPath'] . '/utilit/dateTime.php';
+        require_once $GLOBALS['babInstallPath'].'/utilit/dateTime.php';
         $now = \BAB_DateTime::now()->getIsoDateTime();
         
         $records = $this->select($criteria);
         
-        foreach ($records as $record){
+        foreach ($records as $record) {
             /* @var $record AppTraceableRecord */
             // Could be optimized at ORM level
             $record->deleted = $deletedStatus;
             $record->deletedOn = $now;
             $record->deletedBy = bab_getUserId();
             
-            if(! parent::save($record)){
+            if (!parent::save($record)) {
                 return false;
             }
         }
@@ -260,13 +260,13 @@ 
      */
     public function save(\ORM_Record $record, $noTrace = false)
     {
-        $noTrace = $noTrace || ! $this->isTraceable();
+        $noTrace = $noTrace || !$this->isTraceable();
         $this->logSave($record, $noTrace);
-        if($noTrace){
+        if ($noTrace) {
             return parent::save($record);
         }
         
-        require_once $GLOBALS['babInstallPath'] . '/utilit/dateTime.php';
+        require_once $GLOBALS['babInstallPath'].'/utilit/dateTime.php';
         
         $now = \BAB_DateTime::now()->getIsoDateTime();
         
@@ -274,7 +274,7 @@ 
         $set = $record->getParentSet();
         $primaryKey = $set->getPrimaryKey();
         
-        if(empty($record->{$primaryKey})){
+        if (empty($record->{$primaryKey})) {
             $record->initValue('createdBy', bab_getUserId());
             $record->initValue('createdOn', $now);
             $record->initValue('uuid', $this->uuid());
@@ -305,21 +305,21 @@ 
      */
     public function getRecordByUuid($uuid)
     {
-        if('' === (string) $uuid){
+        if ('' === (string) $uuid) {
             return null;
         }
         
         $record = $this->get($this->uuid->is($uuid));
         
-        if(! isset($record)){
+        if (!isset($record)) {
             return null;
         }
         
-        if(! ($record instanceof AppTraceableRecord)){
+        if (!($record instanceof AppTraceableRecord)) {
             return null;
         }
         
-        if($record->deleted){
+        if ($record->deleted) {
             return null;
         }
         
@@ -335,7 +335,7 @@ 
      */
     public function isOwn($userId = null)
     {
-        if(! isset($userId)){
+        if (!isset($userId)) {
             $userId = bab_getUserId();
         }
         return $this->createdBy->is($userId);

programs/Set/AppCustomSectionSet.php

Spacing +2 added lines, -2 removed lines

@@ -109,9 +109,9 @@
     public function isUpdatable()
     {
         return $this->all();
-        if($this->App()
+        if ($this->App()
             ->Access()
-            ->administer()){
+            ->administer()) {
             return $this->all();
         }
         return $this->none();

programs/Set/AppCustomSection.php

Spacing +37 added lines, -37 removed lines

@@ -63,7 +63,7 @@ 
         $App = app_App();
         static $fieldsLayouts = null;
         
-        if(! isset($fieldsLayouts)){
+        if (!isset($fieldsLayouts)) {
             $fieldsLayouts = array(
                 self::FIELDS_LAYOUT_VERTICAL_LABEL => $App->translate('Vertical label'),
                 self::FIELDS_LAYOUT_HORIZONTAL_LABEL => $App->translate('Horizontal label'),
@@ -80,7 +80,7 @@ 
      */
     public function getRawFields()
     {
-        if(empty($this->fields)){
+        if (empty($this->fields)) {
             return array();
         }
         return explode(',', $this->fields);
@@ -105,29 +105,29 @@ 
      */
     public function getFields()
     {
-        if(empty($this->fields)){
+        if (empty($this->fields)) {
             return array();
         }
         $fields = json_decode($this->fields, true);
-        if(json_last_error() === JSON_ERROR_NONE){
+        if (json_last_error() === JSON_ERROR_NONE) {
             return $fields;
         }
         
         $fields = array();
         $rawFields = $this->getRawFields();
-        foreach ($rawFields as $rawField){
-            if(empty($rawField)){
+        foreach ($rawFields as $rawField) {
+            if (empty($rawField)) {
                 continue;
             }
             $params = array();
             
-            if(strpos($rawField, ':') === false){
+            if (strpos($rawField, ':') === false) {
                 $fieldName = $rawField;
             }
-            else{
+            else {
                 list ($fieldName, $parameters) = explode(':', $rawField);
                 $parameters = explode(';', $parameters);
-                foreach ($parameters as $parameter){
+                foreach ($parameters as $parameter) {
                     list ($key, $value) = explode('=', $parameter);
                     $params[$key] = $value;
                 }
@@ -160,7 +160,7 @@ 
      */
     public function getField($searchedFieldName)
     {
-        if(empty($this->fields)){
+        if (empty($this->fields)) {
             return null;
         }
         
@@ -177,7 +177,7 @@ 
     public function addField($fieldName, $parameters = null)
     {
         $fields = $this->getFields();
-        if(! isset($parameters)){
+        if (!isset($parameters)) {
             $fields[$fieldName] = array(
                 'block' => ''
             );
@@ -211,24 +211,24 @@ 
     {
         $isFieldsGroup = false;
         $fields = $this->getFields();
-        if(strpos($updatedFieldName, '_fieldsGroup') !== false){
+        if (strpos($updatedFieldName, '_fieldsGroup') !== false) {
             $isFieldsGroup = true;
             list (, $groupPos) = explode('_fieldsGroup', $updatedFieldName);
-            if(! isset($groupPos) || empty($groupPos)){
+            if (!isset($groupPos) || empty($groupPos)) {
                 $groupPos = 0;
             }
         }
-        if(! isset($fields[$updatedFieldName])){
-            if($isFieldsGroup){
-                while (isset($fields[$updatedFieldName . $groupPos])){
-                    $groupPos ++;
+        if (!isset($fields[$updatedFieldName])) {
+            if ($isFieldsGroup) {
+                while (isset($fields[$updatedFieldName.$groupPos])) {
+                    $groupPos++;
                 }
                 $updatedFieldName .= $groupPos;
             }
             $this->addField($updatedFieldName, $parameters);
             return $this;
         }
-        if(! isset($parameters)){
+        if (!isset($parameters)) {
             $fields[$updatedFieldName] = array(
                 'block' => ''
             );
@@ -243,11 +243,11 @@ 
     public function addFieldToGroup($fieldGroupName, $fieldName, $parameters = null)
     {
         $fields = $this->getFields();
-        if(! isset($fields[$fieldGroupName])){
+        if (!isset($fields[$fieldGroupName])) {
             return $this;
         }
         $groupFields = isset($fields[$fieldGroupName]['fields']) ? $fields[$fieldGroupName]['fields'] : array();
-        if(! isset($parameters)){
+        if (!isset($parameters)) {
             $groupFields[$fieldName] = array(
                 'block' => ''
             );
@@ -281,15 +281,15 @@ 
     public function updateFieldGroup($fieldGroupName, $updatedFieldName, $parameters = null)
     {
         $fields = $this->getFields();
-        if(! isset($fields[$fieldGroupName])){
+        if (!isset($fields[$fieldGroupName])) {
             return $this;
         }
         $groupFields = isset($fields[$fieldGroupName]['fields']) ? $fields[$fieldGroupName]['fields'] : array();
-        if(! isset($groupFields[$updatedFieldName])){
+        if (!isset($groupFields[$updatedFieldName])) {
             $this->addFieldToGroup($fieldGroupName, $updatedFieldName, $parameters);
             return $this;
         }
-        if(! isset($parameters)){
+        if (!isset($parameters)) {
             $groupFields[$updatedFieldName] = array(
                 'block' => ''
             );
@@ -309,7 +309,7 @@ 
      */
     public function isVisibleForRecord(AppRecord $record)
     {
-        if(empty($this->visibilityCriteria)){
+        if (empty($this->visibilityCriteria)) {
             return true;
         }
         
@@ -318,36 +318,36 @@ 
         
         $arrCriteria = json_decode($this->visibilityCriteria, true);
         
-        foreach ($arrCriteria as $fieldName => $condition){
-            if(strpos($fieldName, '/') !== false){
+        foreach ($arrCriteria as $fieldName => $condition) {
+            if (strpos($fieldName, '/') !== false) {
                 list ($oneField, $foreignField) = explode('/', $fieldName);
                 $field = $recordSet->$oneField()->$foreignField;
             }
-            else{
+            else {
                 $field = $recordSet->$fieldName;
             }
-            foreach ($condition as $op => $value){
-                if(! is_array($value)){
+            foreach ($condition as $op => $value) {
+                if (!is_array($value)) {
                     $criteria = $field->$op($value);
                 }
-                else{
-                    foreach ($value as $foreignClassName => $foreignValues){
-                        $foreignClassName = str_replace($App->classPrefix, '', $foreignClassName) . 'Set';
+                else {
+                    foreach ($value as $foreignClassName => $foreignValues) {
+                        $foreignClassName = str_replace($App->classPrefix, '', $foreignClassName).'Set';
                         $foreignSet = $App->$foreignClassName();
                         $foreignField = '';
                         $foreignCondition = '';
-                        foreach ($foreignValues as $foreignFieldName => $foreignConditions){
-                            if($foreignFieldName === '_foreignField'){
+                        foreach ($foreignValues as $foreignFieldName => $foreignConditions) {
+                            if ($foreignFieldName === '_foreignField') {
                                 $foreignField = $foreignConditions;
                             }
-                            else{
-                                foreach ($foreignConditions as $key => $val){
+                            else {
+                                foreach ($foreignConditions as $key => $val) {
                                     $foreignCondition = $foreignSet->$foreignFieldName->$key($val);
                                 }
                             }
                         }
                     }
-                    if(method_exists($foreignSet, 'getDefaultCriteria')){
+                    if (method_exists($foreignSet, 'getDefaultCriteria')) {
                         $foreignCondition = $foreignCondition->_AND_($foreignSet->getDefaultCriteria());
                     }
                     $criteria = $field->in($foreignCondition, $foreignField);

Braces +4 added lines, -8 removed lines

@@ -123,8 +123,7 @@ 
             
             if(strpos($rawField, ':') === false){
                 $fieldName = $rawField;
-            }
-            else{
+            } else{
                 list ($fieldName, $parameters) = explode(':', $rawField);
                 $parameters = explode(';', $parameters);
                 foreach ($parameters as $parameter){
@@ -322,15 +321,13 @@ 
             if(strpos($fieldName, '/') !== false){
                 list ($oneField, $foreignField) = explode('/', $fieldName);
                 $field = $recordSet->$oneField()->$foreignField;
-            }
-            else{
+            } else{
                 $field = $recordSet->$fieldName;
             }
             foreach ($condition as $op => $value){
                 if(! is_array($value)){
                     $criteria = $field->$op($value);
-                }
-                else{
+                } else{
                     foreach ($value as $foreignClassName => $foreignValues){
                         $foreignClassName = str_replace($App->classPrefix, '', $foreignClassName) . 'Set';
                         $foreignSet = $App->$foreignClassName();
@@ -339,8 +336,7 @@ 
                         foreach ($foreignValues as $foreignFieldName => $foreignConditions){
                             if($foreignFieldName === '_foreignField'){
                                 $foreignField = $foreignConditions;
-                            }
-                            else{
+                            } else{
                                 foreach ($foreignConditions as $key => $val){
                                     $foreignCondition = $foreignSet->$foreignFieldName->$key($val);
                                 }

programs/Set/AppSSESet.php

Spacing +18 added lines, -18 removed lines

@@ -60,7 +60,7 @@ 
     

     public function selectUserSSE($user = null)

     {

-        if(!isset($user)){

+        if (!isset($user)) {

             $App = $this->App();

             $user = $App->getCurrentUser();

         }

@@ -82,29 +82,29 @@ 
         $userSSEReloads = $this->select($this->user->is($user)->_AND_($this->type->is(AppSSE::TYPE_RELOAD_SELECTOR)));

         $reloadSelectors = array();

         $now = date('Y-m-d H:i:s');

-        foreach ($userSSEReloads as $sseMsg){

+        foreach ($userSSEReloads as $sseMsg) {

             $reloadSelectors[] = $sseMsg->data;

-            if($delete && $now > date($sseMsg->deleteOn)){

+            if ($delete && $now > date($sseMsg->deleteOn)) {

                 $sseMsg->delete();

             }

         }

         

-        if(!empty($reloadSelectors)){

+        if (!empty($reloadSelectors)) {

             $sseData .= '"reloadSelector":'.bab_json_encode($reloadSelectors);

             $hasData = true;

         }

         

         $userSSEBrowserNotifications = $this->select($this->user->is($user)->_AND_($this->type->is(AppSSE::TYPE_BROWSER_NOTIFICATION)));

         $browserNotifications = array();

-        foreach ($userSSEBrowserNotifications as $sseMsg){

+        foreach ($userSSEBrowserNotifications as $sseMsg) {

             $browserNotifications[] = $sseMsg->data;

-            if($delete){

+            if ($delete) {

                 $sseMsg->delete();

             }

         }

         

-        if(!empty($browserNotifications)){

-            if($hasData){

+        if (!empty($browserNotifications)) {

+            if ($hasData) {

                 $sseData .= ', ';

             }

             $sseData .= '"browserNotifications":'.bab_json_encode($browserNotifications);

@@ -125,20 +125,20 @@ 
     {

         $App = $this->App();

         $users = array();

-        if(!isset($for)){

+        if (!isset($for)) {

             $users[] = $App->getCurrentUser();

         }

-        else if(!is_array($for)){

-            if($for === 'all'){

+        else if (!is_array($for)) {

+            if ($for === 'all') {

                 $users = $App->getLoggedUsers();

             }

-            else{

+            else {

                 $users[] = $for;

             }

         }

         

         $deleteOn = self::getDeleteOn();

-        foreach ($users as $user){

+        foreach ($users as $user) {

             $record = $this->newRecord();

             $record->user = $user;

             $record->data = $reloadSelector;

@@ -164,20 +164,20 @@ 
     {

         $App = $this->App();

         $users = array();

-        if(!isset($for)){

+        if (!isset($for)) {

             $users[] = $App->getCurrentUser();

         }

-        else if(!is_array($for)){

-            if($for === 'all'){

+        else if (!is_array($for)) {

+            if ($for === 'all') {

                 $users = $App->getLoggedUsers();

             }

-            else{

+            else {

                 $users[] = $for;

             }

         }

         

         $deleteOn = self::getDeleteOn();

-        foreach ($users as $user){

+        foreach ($users as $user) {

             $record = $this->newRecord();

             $record->user = $user;

             $record->data = bab_json_encode($data);

Braces +4 added lines, -8 removed lines

@@ -127,12 +127,10 @@ 
         $users = array();

         if(!isset($for)){

             $users[] = $App->getCurrentUser();

-        }

-        else if(!is_array($for)){

+        } else if(!is_array($for)){

             if($for === 'all'){

                 $users = $App->getLoggedUsers();

-            }

-            else{

+            } else{

                 $users[] = $for;

             }

         }

@@ -166,12 +164,10 @@ 
         $users = array();

         if(!isset($for)){

             $users[] = $App->getCurrentUser();

-        }

-        else if(!is_array($for)){

+        } else if(!is_array($for)){

             if($for === 'all'){

                 $users = $App->getLoggedUsers();

-            }

-            else{

+            } else{

                 $users[] = $for;

             }

         }

programs/Set/AppRecord.php

Spacing +33 added lines, -33 removed lines

@@ -55,7 +55,7 @@ 
     {
         $value = $this->oParentSet->getBackend()->getRecordValue($this, $sFieldName);
         $field = $this->oParentSet->$sFieldName;
-        if(! is_null($value) && $field instanceof \ORM_FkField){
+        if (!is_null($value) && $field instanceof \ORM_FkField) {
             
             $sClassName = $field->getForeignSetName();
             
@@ -90,7 +90,7 @@ 
      */
     public function App()
     {
-        if(! isset($this->app)){
+        if (!isset($this->app)) {
             // If the app object was not specified (through the setApp() method),
             // we set it as parent set's App.
             $this->setApp($this->getParentSet()
@@ -111,7 +111,7 @@ 
         $App = $this->App();
         $rClass = new \ReflectionClass(get_class($this));
         $component = $App->getComponentByName($rClass->getShortName());
-        if(isset($component)){
+        if (isset($component)) {
             return $component->getRecordClassName();
             // list(, $classname) = explode('_', $component->getRecordClassName());
             // return $classname;
@@ -127,11 +127,11 @@ 
      */
     public function getRef()
     {
-        if(! isset($this->id)){
+        if (!isset($this->id)) {
             throw new AppException('Trying to get the reference string of a record without an id.');
         }
         $classname = $this->getClassName();
-        return $classname . ':' . $this->id;
+        return $classname.':'.$this->id;
     }
     
     /**
@@ -158,7 +158,7 @@ 
     {
         $App = $this->App();
         
-        if(! isset($deletedStatus)){
+        if (!isset($deletedStatus)) {
             $deletedStatus = AppTraceableRecord::DELETED_STATUS_DELETED;
         }
         
@@ -170,7 +170,7 @@ 
         // referenced elements.
         $manyRelations = $set->getHasManyRelations();
         
-        foreach ($manyRelations as $manyRelation){
+        foreach ($manyRelations as $manyRelation) {
             /* @var $manyRelation \ORM_ManyRelation */
             
             $foreignSetClassName = $manyRelation->getForeignSetClassName();
@@ -184,7 +184,7 @@ 
                     
                     $foreignRecords = $foreignSet->select($foreignSet->$foreignSetFieldName->is($recordId));
                     
-                    foreach ($foreignRecords as $foreignRecord){
+                    foreach ($foreignRecords as $foreignRecord) {
                         $foreignRecord->$foreignSetFieldName = 0;
                         $foreignRecord->save();
                     }
@@ -195,7 +195,7 @@ 
                     
                     $foreignRecords = $foreignSet->select($foreignSet->$foreignSetFieldName->is($recordId));
                     
-                    foreach ($foreignRecords as $foreignRecord){
+                    foreach ($foreignRecords as $foreignRecord) {
                         $foreignRecord->delete();
                     }
                     
@@ -241,7 +241,7 @@ 
         // referenced elements.
         $manyRelations = $set->getHasManyRelations();
         
-        foreach ($manyRelations as $manyRelation){
+        foreach ($manyRelations as $manyRelation) {
             /* @var $manyRelation \ORM_ManyRelation */
             
             $foreignSetClassName = $manyRelation->getForeignSetClassName();
@@ -251,7 +251,7 @@ 
             // $foreignSet = new $foreignSetClassName($App);
             $foreignRecords = $foreignSet->select($foreignSet->$foreignSetFieldName->is($recordId));
             
-            foreach ($foreignRecords as $foreignRecord){
+            foreach ($foreignRecords as $foreignRecord) {
                 $foreignRecord->$foreignSetFieldName = $id;
                 $foreignRecord->save();
             }
@@ -264,7 +264,7 @@ 
         $links = $linkSet->select($linkSet->sourceClass->is($recordClassName)
             ->_AND_($linkSet->sourceId->is($recordId)));
         
-        foreach ($links as $link){
+        foreach ($links as $link) {
             $link->sourceId = $id;
             $link->save();
         }
@@ -272,7 +272,7 @@ 
         $links = $linkSet->select($linkSet->targetClass->is($recordClassName)
             ->_AND_($linkSet->targetId->is($recordId)));
         
-        foreach ($links as $link){
+        foreach ($links as $link) {
             $link->targetId = $id;
             $link->save();
         }
@@ -297,7 +297,7 @@ 
         
         $relatedRecords = array();
         
-        foreach ($manyRelations as $manyRelation){
+        foreach ($manyRelations as $manyRelation) {
             /* @var $manyRelation \ORM_ManyRelation */
             
             $foreignSetClassName = $manyRelation->getForeignSetClassName();
@@ -308,7 +308,7 @@ 
             // $foreignSet = new $foreignSetClassName($App);
             $foreignRecords = $foreignSet->select($foreignSet->$foreignSetFieldName->is($recordId));
             
-            if($foreignRecords->count() > 0){
+            if ($foreignRecords->count() > 0) {
                 $relatedRecords[$foreignSetClassName] = $foreignRecords;
             }
         }
@@ -325,7 +325,7 @@ 
     {
         $path = $this->App()->uploadPath();
         
-        if(null === $path){
+        if (null === $path) {
             throw new \Exception('Missing upload path information');
             return null;
         }
@@ -348,11 +348,11 @@ 
             ->_AND_($linkSet->sourceId->is($source->id))
             ->_AND_($linkSet->targetClass->is($this->getClassName()))
             ->_AND_($linkSet->targetId->is($this->id));
-        if(isset($linkType)){
-            if(is_array($linkType)){
+        if (isset($linkType)) {
+            if (is_array($linkType)) {
                 $criteria = $criteria->_AND_($linkSet->type->in($linkType));
             }
-            else{
+            else {
                 $criteria = $criteria->_AND_($linkSet->type->is($linkType));
             }
         }
@@ -376,11 +376,11 @@ 
         
         $criteria = $linkSet->all($linkSet->targetClass->is($target->getClassName()), $linkSet->targetId->is($target->id), $linkSet->sourceClass->is($this->getClassName()), $linkSet->sourceId->is($this->id));
         
-        if(isset($linkType)){
-            if(is_array($linkType)){
+        if (isset($linkType)) {
+            if (is_array($linkType)) {
                 $criteria = $criteria->_AND_($linkSet->type->in($linkType));
             }
-            else{
+            else {
                 $criteria = $criteria->_AND_($linkSet->type->is($linkType));
             }
         }
@@ -442,7 +442,7 @@ 
      */
     protected function importProperty($name, $value)
     {
-        if(((string) $this->$name) !== ((string) $value)){
+        if (((string) $this->$name) !== ((string) $value)) {
             $this->$name = $value;
             return 1;
         }
@@ -462,13 +462,13 @@ 
      */
     protected function importDate($name, $value)
     {
-        if(preg_match('/[0-9]{4}-[0-9]{2}-[0-9]{2}/', $value)){
+        if (preg_match('/[0-9]{4}-[0-9]{2}-[0-9]{2}/', $value)) {
             return $this->importProperty($name, $value);
         }
         
         // try in DD/MM/YYYY format
         
-        if(preg_match('/(?P<day>[0-9]+)\/(?P<month>[0-9]+)\/(?P<year>[0-9]{2,4})/', $value, $matches)){
+        if (preg_match('/(?P<day>[0-9]+)\/(?P<month>[0-9]+)\/(?P<year>[0-9]{2,4})/', $value, $matches)) {
             
             $value = sprintf('%04d-%02d-%02d', (int) $matches['year'], (int) $matches['month'], (int) $matches['day']);
             
@@ -488,11 +488,11 @@ 
         $customContainers->groupBy($customContainerSet->view);
         
         $views = array();
-        foreach ($customContainers as $customContainer){
+        foreach ($customContainers as $customContainer) {
             $views[] = $customContainer->view;
         }
         
-        if(empty($views)){
+        if (empty($views)) {
             $views[] = '';
         }
         
@@ -576,9 +576,9 @@ 
      */
     public function requireReadable($message = null)
     {
-        if(! $this->isReadable()){
+        if (!$this->isReadable()) {
             $App = $this->App();
-            if(! isset($message)){
+            if (!isset($message)) {
                 $message = $App->translate('Access denied');
             }
             throw new AppAccessException($message);
@@ -594,9 +594,9 @@ 
      */
     public function requireUpdatable($message = null)
     {
-        if(! $this->isUpdatable()){
+        if (!$this->isUpdatable()) {
             $App = $this->App();
-            if(! isset($message)){
+            if (!isset($message)) {
                 $message = $App->translate('Access denied');
             }
             throw new AppAccessException($message);
@@ -612,9 +612,9 @@ 
      */
     public function requireDeletable($message = null)
     {
-        if(! $this->isDeletable()){
+        if (!$this->isDeletable()) {
             $App = $this->App();
-            if(! isset($message)){
+            if (!isset($message)) {
                 $message = $App->translate('Access denied');
             }
             throw new AppAccessException($message);

Braces +2 added lines, -4 removed lines

@@ -351,8 +351,7 @@ 
         if(isset($linkType)){
             if(is_array($linkType)){
                 $criteria = $criteria->_AND_($linkSet->type->in($linkType));
-            }
-            else{
+            } else{
                 $criteria = $criteria->_AND_($linkSet->type->is($linkType));
             }
         }
@@ -379,8 +378,7 @@ 
         if(isset($linkType)){
             if(is_array($linkType)){
                 $criteria = $criteria->_AND_($linkSet->type->in($linkType));
-            }
-            else{
+            } else{
                 $criteria = $criteria->_AND_($linkSet->type->is($linkType));
             }
         }

programs/Set/AppLink.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     public function getSource()
     {
         $App = $this->App();
-        return $App->getRecordByRef($this->sourceClass . ':' . $this->sourceId);
+        return $App->getRecordByRef($this->sourceClass.':'.$this->sourceId);
     }
     
     /**
@@ -54,7 +54,7 @@ 
     public function getTarget()
     {
         $App = $this->App();
-        return $App->getRecordByRef($this->targetClass . ':' . $this->targetId);
+        return $App->getRecordByRef($this->targetClass.':'.$this->targetId);
     }
 }