@@ -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 | } |
@@ -56,6 +56,9 @@ discard block |
||
| 56 | 56 | return true; |
| 57 | 57 | } |
| 58 | 58 | |
| 59 | + /** |
|
| 60 | + * @param string $classname |
|
| 61 | + */ |
|
| 59 | 62 | public static function create_class_storage($classname) |
| 60 | 63 | { |
| 61 | 64 | $em = connection::get_em(); |
@@ -87,6 +90,11 @@ discard block |
||
| 87 | 90 | $generator->generateProxyClass($cm, $filename); |
| 88 | 91 | } |
| 89 | 92 | |
| 93 | + /** |
|
| 94 | + * @param Doctrine\ORM\EntityManager $em |
|
| 95 | + * |
|
| 96 | + * @return ClassMetadata |
|
| 97 | + */ |
|
| 90 | 98 | private static function get_cm($em, $classname) |
| 91 | 99 | { |
| 92 | 100 | if (!class_exists($classname)) { |
@@ -29,6 +29,9 @@ |
||
| 29 | 29 | |
| 30 | 30 | private $replication_enabled = true; |
| 31 | 31 | |
| 32 | + /** |
|
| 33 | + * @return integer |
|
| 34 | + */ |
|
| 32 | 35 | public static function get_instance() |
| 33 | 36 | { |
| 34 | 37 | if (self::$instance === null) { |
@@ -96,6 +96,9 @@ |
||
| 96 | 96 | return true; |
| 97 | 97 | } |
| 98 | 98 | |
| 99 | + /** |
|
| 100 | + * @param string $guid |
|
| 101 | + */ |
|
| 99 | 102 | public static function get_object_by_guid($guid) |
| 100 | 103 | { |
| 101 | 104 | if (!mgd_is_guid($guid)) { |
@@ -19,6 +19,9 @@ |
||
| 19 | 19 | } |
| 20 | 20 | } |
| 21 | 21 | |
| 22 | + /** |
|
| 23 | + * @param string $operator |
|
| 24 | + */ |
|
| 22 | 25 | public function add_constraint($name, $operator, $value) |
| 23 | 26 | { |
| 24 | 27 | try { |
@@ -65,6 +65,7 @@ discard block |
||
| 65 | 65 | * This is basically a workaround for some quirks when merging detached entities with changed associations |
| 66 | 66 | * |
| 67 | 67 | * @todo: This may or may not be a bug in Doctrine |
| 68 | + * @param dbobject $source |
|
| 68 | 69 | */ |
| 69 | 70 | private function copy_associations($source, $target) |
| 70 | 71 | { |
@@ -222,6 +223,9 @@ discard block |
||
| 222 | 223 | return $entity; |
| 223 | 224 | } |
| 224 | 225 | |
| 226 | + /** |
|
| 227 | + * @param dbobject $target |
|
| 228 | + */ |
|
| 225 | 229 | private function copy_metadata($source, $target, $action = 'update') |
| 226 | 230 | { |
| 227 | 231 | if (!$source instanceof metadata) { |
@@ -95,6 +95,9 @@ |
||
| 95 | 95 | $type->add_property($property); |
| 96 | 96 | } |
| 97 | 97 | |
| 98 | + /** |
|
| 99 | + * @param string $name |
|
| 100 | + */ |
|
| 98 | 101 | private function add_mixin($name, type $type) |
| 99 | 102 | { |
| 100 | 103 | if (empty($this->mixins[$name])) { |
@@ -63,7 +63,7 @@ discard block |
||
| 63 | 63 | $parsed = $this->parse_constraint_name($property); |
| 64 | 64 | |
| 65 | 65 | // for properties like up.name |
| 66 | - if ( strpos($property, ".") !== false |
|
| 66 | + if (strpos($property, ".") !== false |
|
| 67 | 67 | && !(strpos($property, "metadata") === 0)) { |
| 68 | 68 | return $parsed['name'] . " as " . str_replace(".", "_", $property); |
| 69 | 69 | } |
@@ -134,7 +134,7 @@ discard block |
||
| 134 | 134 | */ |
| 135 | 135 | public function get($key) |
| 136 | 136 | { |
| 137 | - if ( !$this->_has_results() |
|
| 137 | + if (!$this->_has_results() |
|
| 138 | 138 | || !isset($this->_results[$key])) { |
| 139 | 139 | return false; |
| 140 | 140 | } |
@@ -25,6 +25,10 @@ discard block |
||
| 25 | 25 | |
| 26 | 26 | private $value_properties = array(); |
| 27 | 27 | |
| 28 | + /** |
|
| 29 | + * @param string $class |
|
| 30 | + * @param string $field |
|
| 31 | + */ |
|
| 28 | 32 | public function __construct($class, $field = null, $value = null) |
| 29 | 33 | { |
| 30 | 34 | parent::__construct($class); |
@@ -68,6 +72,9 @@ discard block |
||
| 68 | 72 | return true; |
| 69 | 73 | } |
| 70 | 74 | |
| 75 | + /** |
|
| 76 | + * @param string $property |
|
| 77 | + */ |
|
| 71 | 78 | protected function build_property_select($property) |
| 72 | 79 | { |
| 73 | 80 | $parsed = $this->parse_constraint_name($property); |
@@ -9,7 +9,7 @@ |
||
| 9 | 9 | use midgard\portable\mgdschema\translator; |
| 10 | 10 | use midgard\portable\api\error\exception; |
| 11 | 11 | |
| 12 | -if ( extension_loaded('midgard') |
|
| 12 | +if (extension_loaded('midgard') |
|
| 13 | 13 | || extension_loaded('midgard2')) { |
| 14 | 14 | //TODO: Print some error? |
| 15 | 15 | return; |