Cleanup: delay assignment of $this->value until end of do_build functions

diff --git a/common/database/classes/database_alter_aggregate.php b/common/database/classes/database_alter_aggregate.php
index eaabf32f932cc46ed3c505b57389a844a4a5721e..a08feb1ad013f05d747554c604ce6ff388e1489d 100644 (file)
--- a/common/database/classes/database_alter_aggregate.php
+++ b/common/database/classes/database_alter_aggregate.php
@@ -301,10 +301,6 @@ class c_database_alter_aggregate extends c_database_query {
       return new c_base_return_false();
     }
 
-    // @fixme: use a local variable fo value before assigning.
-    $this->value = static::pr_QUERY_COMMAND;
-    $this->value .= ' ' . $this->name;
-
     $aggregate_signatures = NULL;
     if (!is_array($this->aggregate_modes) || empty($this->aggregate_modes)) {
       $aggregate_signatures = ' *';
@@ -353,23 +349,28 @@ class c_database_alter_aggregate extends c_database_query {
       unset($order_by_signatures);
     }
 
+    $value = NULL;
     if (is_string($this->rename_to)) {
-      $this->value .= ' ' . $aggregate_signatures . ' ' . $this->p_do_build_rename_to();
+      $action = $aggregate_signatures . ' ' . $this->p_do_build_rename_to();
     }
     else if (is_string($this->owner_to)) {
-      $this->value .= ' ' . $aggregate_signatures . ' ' . $this->p_do_build_owner_to();
+      $action = $aggregate_signatures . ' ' . $this->p_do_build_owner_to();
     }
     else if (is_string($this->set_schema)) {
-      $this->value .= ' ' . $aggregate_signatures . ' ' . $this->p_do_build_set_schema();
+      $action = $aggregate_signatures . ' ' . $this->p_do_build_set_schema();
     }
     else {
       unset($aggregate_signatures);
-
-      $this->value = NULL;
+      unset($action);
       return new c_base_return_false();
     }
     unset($aggregate_signatures);
 
+    $this->value = static::pr_QUERY_COMMAND;
+    $this->value .= ' ' . $this->name;
+    $this->value .= ' ' . $action;
+    unset($action);
+
     return new c_base_return_true();
   }
 }

diff --git a/common/database/classes/database_alter_coalation.php b/common/database/classes/database_alter_coalation.php
index d2fbca6c8b8eccbc40d2366f0113e24914c35282..0058cb47380945d2629b6aab685d104a073c25fb 100644 (file)
--- a/common/database/classes/database_alter_coalation.php
+++ b/common/database/classes/database_alter_coalation.php
@@ -133,29 +133,31 @@ class c_database_alter_coalation extends c_database_query {
       return new c_base_return_false();
     }
 
-    // @fixme: use a local variable for value before returning.
-    $this->value = static::pr_QUERY_COMMAND;
-    $this->value .= ' ' . $this->name;
-
+    $action = NULL;
     if (is_bool($this->refresh_version)) {
       if ($this->refresh_version) {
-        $this->value .= ' ' . c_database_string::REFRESH_VERSION;
+        $action = c_database_string::REFRESH_VERSION;
       }
     }
     else if (is_string($this->rename_to)) {
-      $this->value .= ' ' . $this->p_do_build_rename_to();
+      $action = $this->p_do_build_rename_to();
     }
     else if (is_string($this->owner_to)) {
-      $this->value .= ' ' . $this->p_do_build_owner_to();
+      $action = $this->p_do_build_owner_to();
     }
     else if (is_string($this->set_schema)) {
-      $this->value .= ' ' . $this->p_do_build_set_schema();
+      $action = $this->p_do_build_set_schema();
     }
     else {
-      $this->value = NULL;
+      unset($action);
       return new c_base_return_false();
     }
 
+    $this->value = static::pr_QUERY_COMMAND;
+    $this->value .= ' ' . $this->name;
+    $this->value .= ' ' . $action;
+    unset($action);
+
     return new c_base_return_true();
   }
 }

diff --git a/common/database/classes/database_alter_conversion.php b/common/database/classes/database_alter_conversion.php
index e799c6c9e9871c46f7f61045852c0e49a5b4291a..28ee6e73b58b9b7bc3a05670e9474ade351f5d60 100644 (file)
--- a/common/database/classes/database_alter_conversion.php
+++ b/common/database/classes/database_alter_conversion.php
@@ -83,24 +83,26 @@ class c_database_alter_conversion extends c_database_query {
       return new c_base_return_false();
     }
 
-    // @fixme: use a local variable for value.
-    $this->value = static::pr_QUERY_COMMAND;
-    $this->value .= ' ' . $this->name;
-
+    $action = NULL;
     if (is_string($this->rename_to)) {
-      $this->value .= ' ' . $this->p_do_build_rename_to();
+      $action = $this->p_do_build_rename_to();
     }
     else if (is_string($this->owner_to)) {
-      $this->value .= ' ' . $this->p_do_build_owner_to();
+      $action = $this->p_do_build_owner_to();
     }
     else if (is_string($this->set_schema)) {
-      $this->value .= ' ' . $this->p_do_build_set_schema();
+      $action = $this->p_do_build_set_schema();
     }
     else {
-      $this->value = NULL;
+      unset($action);
       return new c_base_return_false();
     }
 
+    $this->value = static::pr_QUERY_COMMAND;
+    $this->value .= ' ' . $this->name;
+    $this->value .= ' ' . $action;
+    unset($action);
+
     return new c_base_return_true();
   }
 }

diff --git a/common/database/classes/database_alter_database.php b/common/database/classes/database_alter_database.php
index 66e6122ede3f842bee9537b6b3ea0286ef783c44..3afd7668ed554914fdf256ce1746a46692901dea 100644 (file)
--- a/common/database/classes/database_alter_database.php
+++ b/common/database/classes/database_alter_database.php
@@ -147,29 +147,36 @@ class c_database_alter_database extends c_database_query {
       return new c_base_return_false();
     }
 
-    $this->value = static::pr_QUERY_COMMAND;
-    $this->value .= ' ' . $this->name;
-
+    $action = NULL;
     if ($this->option instanceof c_database_argument_database_option) {
       $this->option->do_build_argument();
-      $this->value .= ' ' . $this->option->get_value_exact();
+      $action = $this->option->get_value_exact();
     }
     else if (is_string($this->rename_to)) {
-      $this->value .= ' ' . $this->p_do_build_rename_to();
+      $action = $this->p_do_build_rename_to();
     }
     else if (is_string($this->owner_to)) {
-      $this->value .= ' ' . $this->p_do_build_owner_to();
+      $action = $this->p_do_build_owner_to();
     }
     else if (is_string($this->set_tablespace)) {
-      $this->value .= ' ' . $this->p_do_build_set_tablespace();
+      $action = $this->p_do_build_set_tablespace();
     }
     else if (is_array($this->set)) {
-        $this->value .= ' ' . $this->p_do_build_set();
+      $action = $this->p_do_build_set();
     }
     else if (is_array($this->reset)) {
-      $this->value .= ' ' . $this->p_do_build_reset();
+      $action = $this->p_do_build_reset();
+    }
+    else {
+      unset($action);
+      return new c_base_return_false();
     }
 
+    $this->value = static::pr_QUERY_COMMAND;
+    $this->value .= ' ' . $this->name;
+    $this->value .= ' ' . $action;
+    unset($action);
+
     return new c_base_return_true();
   }
 }

diff --git a/common/database/classes/database_alter_default_privileges.php b/common/database/classes/database_alter_default_privileges.php
index a55af00eeb25c24a1dba2fc95ce357947cc03b99..ac6c43fa2b5efc5afb10fb2341fcdecf5815bfa5 100644 (file)
--- a/common/database/classes/database_alter_default_privileges.php
+++ b/common/database/classes/database_alter_default_privileges.php
@@ -439,35 +439,29 @@ class c_database_alter_default_priveleges extends c_database_query {
           return new c_base_return_false();
     }
 
-    // @fixme: use a local variable and only assign value once at the end after any potential error cases.
-    $this->value = static::pr_QUERY_COMMAND;
-
     // [ FOR ROLE target_role [, ... ] ]
+    $action = NULL;
     if (is_array($this->for_role_targets) && !empty($this->for_role_targets)) {
-      $this->value .= ' ' . c_database_string::FOR . ' ' . c_database_string::ROLE;
-
-      $names = NULL;
-      foreach ($this->for_role_targets as $schema_name) {
-        $names .= ', ' . $schema_name;
-      }
-
-      $this->value .= ltrim($names, ',');
-      unset($names);
+      $action = c_database_string::FOR . ' ' . c_database_string::ROLE;
+      $action .= ' ' . implode(', ', $this->for_role_targets);
     }
 
     // [ IN SCHEMA schema_name [, ... ] ]
     if (is_array($this->in_schema) && !empty($this->in_schema)) {
-      $this->value .= $this->p_do_build_in_schema();
+      $action .= is_null($action) ? '' : ' ';
+      $action .= $this->p_do_build_in_schema();
     }
 
     if ($this->action === e_database_action::ACTION_GRANT) {
-      $this->value .= ' ' . c_database_string::GRANT;
+      $action .= is_null($action) ? '' : ' ';
+      $action .= c_database_string::GRANT;
     }
     else if ($this->action === e_database_action::ACTION_REVOKE) {
-      $this->value .= ' ' . c_database_string::REVOKE;
+      $action .= is_null($action) ? '' : ' ';
+      $action .= c_database_string::REVOKE;
 
       if ($this->option_grant) {
-        $this->value .= ' ' . c_database_string::GRANT_OPTION_FOR;
+        $action .= ' ' . c_database_string::GRANT_OPTION_FOR;
       }
     }
 
@@ -514,34 +508,35 @@ class c_database_alter_default_priveleges extends c_database_query {
       }
     }
 
-    $this->value .= ltrim($privileges, ',');
+    $action .= is_null($action) ? '' : ' ';
+    $action .= ltrim($privileges, ', ');
     unset($privileges);
 
     // ON ...
     switch($this->on) {
       case e_database_on::TABLES_TO:
-        $this->value .= ' ' . c_database_string::ON_TABLES_TO;
+        $action .= ' ' . c_database_string::ON_TABLES_TO;
         break;
       case e_database_on::SEQUENCES:
-        $this->value .= ' ' . c_database_string::ON_SEQUENCES;
+        $action .= ' ' . c_database_string::ON_SEQUENCES;
         break;
       case e_database_on::FUNCTIONS:
-        $this->value .= ' ' . c_database_string::ON_FUNCTIONS;
+        $action .= ' ' . c_database_string::ON_FUNCTIONS;
         break;
       case e_database_on::TYPES:
-        $this->value .= ' ' . c_database_string::ON_TYPES;
+        $action .= ' ' . c_database_string::ON_TYPES;
         break;
       case e_database_on::SCHEMAS:
-        $this->value .= ' ' . c_database_string::ON_SCHEMAS;
+        $action .= ' ' . c_database_string::ON_SCHEMAS;
         break;
     }
 
     // [ TO | FROM ] ... role names ...
     if ($this->action === e_database_action::GRANT) {
-      $this->value .= ' ' . c_database_string::TO;
+      $action .= ' ' . c_database_string::TO;
     }
     else if ($this->action === e_database_action::REVOKE) {
-      $this->value .= ' ' . c_database_string::FROM;
+      $action .= ' ' . c_database_string::FROM;
     }
 
     foreach ($this->role_names as $role_name) {
@@ -550,25 +545,30 @@ class c_database_alter_default_priveleges extends c_database_query {
       }
 
       $role_name->do_build_argument();
-      $this->value .= ' ' . $role_name->get_value_exact();
+      $action .= ' ' . $role_name->get_value_exact();
     }
     unset($role_name);
 
     if ($this->action === e_database_action::GRANT) {
       // [ WITH GRANT OPTION ]
       if ($this->option_grant) {
-        $this->value .= ' ' . c_database_string::WITH_GRANT_OPTION;
+        $action .= ' ' . c_database_string::WITH_GRANT_OPTION;
       }
     }
     else if ($this->action === e_database_action::REVOKE) {
       // [ CASCADE | RESTRICT ]
       $option = $this->p_do_build_option();
       if (is_string($option)) {
-        $this->value .= ' ' . $option;
+        $action .= ' ' . $option;
       }
       unset($option);
     }
 
+    $this->value = static::pr_QUERY_COMMAND;
+    $this->value .= ' ' . $this->name;
+    $this->value .= ' ' . $action;
+    unset($action);
+
     return new c_base_return_true();
   }
 }

diff --git a/common/database/classes/database_alter_domain.php b/common/database/classes/database_alter_domain.php
index e5b64e907607b8c7f0a8594fb2b204fb3d987806..0e7290d1e36086551813c9956fab0375635b29cd 100644 (file)
--- a/common/database/classes/database_alter_domain.php
+++ b/common/database/classes/database_alter_domain.php
@@ -347,6 +347,7 @@ class c_database_alter_coalation extends c_database_query {
     $this->value = static::pr_QUERY_COMMAND;
     $this->value .= ' ' . $this->name;
     $this->value .= ' ' . $action;
+    unset($action);
 
     return new c_base_return_true();
   }

diff --git a/common/database/classes/database_alter_extension.php b/common/database/classes/database_alter_extension.php
index 017a7b8cb5338ea3634882ed2b2c9f67483d532f..87fc0e52e32493ef60a74a93ed44e42144473224 100644 (file)
--- a/common/database/classes/database_alter_extension.php
+++ b/common/database/classes/database_alter_extension.php
@@ -136,7 +136,9 @@ class c_database_alter_extension extends c_database_query {
         unset($action);
         return new c_base_return_false();
     }
+    unset($action);
 
+    $this->value = $action;
     return new c_base_return_true();
   }
 }

diff --git a/common/database/classes/database_query.php b/common/database/classes/database_query.php
index d669744cb05c8be858e74e51b9773af522283bd8..af2941f828df91895607cc30058385085875b8a9 100644 (file)
--- a/common/database/classes/database_query.php
+++ b/common/database/classes/database_query.php
@@ -1449,26 +1449,28 @@ class c_database_argument_role_name extends c_base_return_string implements i_da
       return new c_base_return_false();
     }
 
-    $this->value = '';
+    $value = '';
     if ($this->argument_mode === static::QUERY_ARGUMENT_MODE_PUBLIC) {
-      $this->value .= ' ' . c_database_string::PUBLIC;
+      $value .= ' ' . c_database_string::PUBLIC;
     }
     else if ($this->argument_mode === static::QUERY_ARGUMENT_MODE_NAME) {
       if (is_string($this->argument_name)) {
-        $this->value .= ' ' . $this->argument_name;
+        $value .= ' ' . $this->argument_name;
       }
     }
     else if ($this->argument_mode === static::QUERY_ARGUMENT_MODE_GROUP) {
-      $this->value .= ' ' . c_database_string::GROUP;
+      $value .= ' ' . c_database_string::GROUP;
 
       if (is_string($this->argument_name)) {
-        $this->value .= ' ' . $this->argument_name;
+        $value .= ' ' . $this->argument_name;
       }
     }
     else {
+      unset($value);
       return new c_base_return_false();
     }
 
+    $this->value = $value;
     return new c_base_return_true();
   }