@@ -213,7 +213,7 @@ |
||
| 213 | 213 | $midgard = midgard_connection::get_instance(); |
| 214 | 214 | if ($midgard->config->logfilename) { |
| 215 | 215 | $logdir = dirname($midgard->config->logfilename); |
| 216 | - if ( !is_dir($logdir) |
|
| 216 | + if (!is_dir($logdir) |
|
| 217 | 217 | && !mkdir($logdir, 0777, true)) { |
| 218 | 218 | throw exception::user_data('Log directory could not be created'); |
| 219 | 219 | } |
@@ -55,7 +55,7 @@ discard block |
||
| 55 | 55 | |
| 56 | 56 | $mgd_config = midgard_connection::get_instance()->config; |
| 57 | 57 | $mgdschema_file = $mgd_config->vardir . '/mgdschema_classes.php'; |
| 58 | - if ( file_exists($mgdschema_file) |
|
| 58 | + if (file_exists($mgdschema_file) |
|
| 59 | 59 | && !unlink($mgdschema_file)) { |
| 60 | 60 | throw new \RuntimeException('Could not unlink ' . $mgdschema_file); |
| 61 | 61 | } |
@@ -72,7 +72,7 @@ discard block |
||
| 72 | 72 | $cms = $em->getMetadataFactory()->getAllMetadata(); |
| 73 | 73 | |
| 74 | 74 | // create storage |
| 75 | - if ( !midgard_storage::create_base_storage() |
|
| 75 | + if (!midgard_storage::create_base_storage() |
|
| 76 | 76 | && midgard_connection::get_instance()->get_error_string() != 'MGD_ERR_OK') { |
| 77 | 77 | throw new \Exception("Failed to create base database structures" . midgard_connection::get_instance()->get_error_string()); |
| 78 | 78 | } |
@@ -65,7 +65,7 @@ |
||
| 65 | 65 | } else { |
| 66 | 66 | if ($property->parentfield) { |
| 67 | 67 | $this->parentfield = $property->name; |
| 68 | - if ( empty($this->parent) |
|
| 68 | + if (empty($this->parent) |
|
| 69 | 69 | && $property->link) { |
| 70 | 70 | $this->parent = $property->link['target']; |
| 71 | 71 | } |
@@ -75,7 +75,7 @@ |
||
| 75 | 75 | |
| 76 | 76 | $fqcn = $this->get_fcqn($property->link['target']); |
| 77 | 77 | |
| 78 | - if ( isset($this->types[$fqcn]) |
|
| 78 | + if (isset($this->types[$fqcn]) |
|
| 79 | 79 | || $property->link['target'] === $property->get_parent()->name) { |
| 80 | 80 | return $property->link['target']; |
| 81 | 81 | } |
@@ -73,7 +73,7 @@ |
||
| 73 | 73 | $parsed = $this->parse_constraint_name($property); |
| 74 | 74 | |
| 75 | 75 | // for properties like up.name |
| 76 | - if ( strpos($property, ".") !== false |
|
| 76 | + if (strpos($property, ".") !== false |
|
| 77 | 77 | && !(strpos($property, "metadata") === 0)) { |
| 78 | 78 | return $parsed['name'] . " as " . str_replace(".", "_", $property); |
| 79 | 79 | } |
@@ -123,7 +123,7 @@ |
||
| 123 | 123 | $subdirs = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 'A', 'B', 'C', 'D', 'E', 'F']; |
| 124 | 124 | foreach ($subdirs as $dir) { |
| 125 | 125 | foreach ($subdirs as $subdir) { |
| 126 | - if ( !is_dir($this->blobdir . '/' . $dir . '/' . $subdir) |
|
| 126 | + if (!is_dir($this->blobdir . '/' . $dir . '/' . $subdir) |
|
| 127 | 127 | && !mkdir($this->blobdir . '/' . $dir . '/' . $subdir, 0777, true)) { |
| 128 | 128 | return false; |
| 129 | 129 | } |
@@ -152,7 +152,7 @@ |
||
| 152 | 152 | |
| 153 | 153 | foreach ($properties as $name => $property) { |
| 154 | 154 | // doctrine can handle id links only |
| 155 | - if ( $property->link |
|
| 155 | + if ($property->link |
|
| 156 | 156 | && $target_class = $this->manager->resolve_targetclass($property)) { |
| 157 | 157 | $link_mapping = [ |
| 158 | 158 | 'fieldName' => $property->name, |
@@ -64,7 +64,7 @@ discard block |
||
| 64 | 64 | |
| 65 | 65 | private function load_by_properties(array $properties) |
| 66 | 66 | { |
| 67 | - if ( !array_key_exists('authtype', $properties) |
|
| 67 | + if (!array_key_exists('authtype', $properties) |
|
| 68 | 68 | || !array_key_exists('login', $properties)) { |
| 69 | 69 | throw exception::invalid_property_value(); |
| 70 | 70 | } |
@@ -107,7 +107,7 @@ discard block |
||
| 107 | 107 | |
| 108 | 108 | public function &get_person() : person |
| 109 | 109 | { |
| 110 | - if ( $this->person_object === null |
|
| 110 | + if ($this->person_object === null |
|
| 111 | 111 | && $this->person !== null) { |
| 112 | 112 | $this->person_object = connection::get_em()->getRepository(connection::get_fqcn('midgard_person'))->findOneBy(['guid' => $this->person]); |
| 113 | 113 | } |
@@ -116,7 +116,7 @@ discard block |
||
| 116 | 116 | |
| 117 | 117 | public function create() : bool |
| 118 | 118 | { |
| 119 | - if ( empty($this->authtype) |
|
| 119 | + if (empty($this->authtype) |
|
| 120 | 120 | || !empty($this->id)) { |
| 121 | 121 | exception::invalid_property_value(); |
| 122 | 122 | return false; |
@@ -180,7 +180,7 @@ discard block |
||
| 180 | 180 | |
| 181 | 181 | protected function is_unique() : bool |
| 182 | 182 | { |
| 183 | - if ( empty($this->login) |
|
| 183 | + if (empty($this->login) |
|
| 184 | 184 | || empty($this->authtype)) { |
| 185 | 185 | return true; |
| 186 | 186 | } |
@@ -107,8 +107,8 @@ discard block |
||
| 107 | 107 | $cs = $em->getUnitOfWork()->getEntityChangeSet($entity); |
| 108 | 108 | // We only need to update repligard if we're coming from create (revision 0) |
| 109 | 109 | // or if we delete/undelete |
| 110 | - if ( !array_key_exists('metadata_deleted', $cs) |
|
| 111 | - && $entity->metadata_revision > 0) { |
|
| 110 | + if (!array_key_exists('metadata_deleted', $cs) |
|
| 111 | + && $entity->metadata_revision > 0) { |
|
| 112 | 112 | $check_repligard = false; |
| 113 | 113 | } |
| 114 | 114 | |
@@ -177,7 +177,7 @@ discard block |
||
| 177 | 177 | |
| 178 | 178 | foreach ($columns as $name => &$config) { |
| 179 | 179 | if ($platform instanceof SqlitePlatform) { |
| 180 | - if ( !empty($config['comment']) |
|
| 180 | + if (!empty($config['comment']) |
|
| 181 | 181 | && $config['comment'] == 'BINARY') { |
| 182 | 182 | $modified = true; |
| 183 | 183 | $config['columnDefinition'] = $config['type']->getSQLDeclaration($config, $platform) . ' COLLATE BINARY' . $platform->getDefaultValueDeclarationSQL($config); |
@@ -194,7 +194,7 @@ discard block |
||
| 194 | 194 | $config['columnDefinition'] = $config['comment'] . $platform->getDefaultValueDeclarationSQL($config); |
| 195 | 195 | } |
| 196 | 196 | if (!empty($config['columnDefinition']) && $platform->supportsInlineColumnComments()) { |
| 197 | - $config['columnDefinition'] .= " COMMENT " . $platform->quoteStringLiteral($config['comment']); |
|
| 197 | + $config['columnDefinition'] .= " COMMENT " . $platform->quoteStringLiteral($config['comment']); |
|
| 198 | 198 | } |
| 199 | 199 | } |
| 200 | 200 | } |
@@ -278,7 +278,7 @@ discard block |
||
| 278 | 278 | public function postGenerateSchemaTable(GenerateSchemaTableEventArgs $args) |
| 279 | 279 | { |
| 280 | 280 | $table = $args->getClassTable(); |
| 281 | - if ( !$table->hasOption('engine') |
|
| 281 | + if (!$table->hasOption('engine') |
|
| 282 | 282 | || $table->getOption('engine') !== 'MyISAM') { |
| 283 | 283 | return; |
| 284 | 284 | } |