[MediaWiki-commits] [Gerrit] mediawiki/core[REL1_29]: Quote $default in PostgresUpdater::setDefault

2017-09-13 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/377672 )

Change subject: Quote $default in PostgresUpdater::setDefault
..


Quote $default in PostgresUpdater::setDefault

Bug: T175439
Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5
(cherry picked from commit bc12345f4bb7893030c31e1133abc4b20153f59b)
---
M includes/installer/PostgresUpdater.php
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Addshore: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/installer/PostgresUpdater.php 
b/includes/installer/PostgresUpdater.php
index 39cb89c..ff96efb 100644
--- a/includes/installer/PostgresUpdater.php
+++ b/includes/installer/PostgresUpdater.php
@@ -755,7 +755,8 @@
$info = $this->db->fieldInfo( $table, $field );
if ( $info->defaultValue() !== $default ) {
$this->output( "Changing '$table.$field' default 
value\n" );
-   $this->db->query( "ALTER TABLE $table ALTER $field SET 
DEFAULT " . $default );
+   $this->db->query( "ALTER TABLE $table ALTER $field SET 
DEFAULT "
+   . $this->db->addQuotes( $default ) );
}
}
 

-- 
To view, visit https://gerrit.wikimedia.org/r/377672
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: REL1_29
Gerrit-Owner: Reedy 
Gerrit-Reviewer: Addshore 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] mediawiki/core[REL1_29]: Quote $default in PostgresUpdater::setDefault

2017-09-12 Thread Reedy (Code Review) 
Reedy has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/377672 )

Change subject: Quote $default in PostgresUpdater::setDefault
..

Quote $default in PostgresUpdater::setDefault

Bug: T175439
Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5
(cherry picked from commit bc12345f4bb7893030c31e1133abc4b20153f59b)
---
M includes/installer/PostgresUpdater.php
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/72/377672/1

diff --git a/includes/installer/PostgresUpdater.php 
b/includes/installer/PostgresUpdater.php
index 39cb89c..ff96efb 100644
--- a/includes/installer/PostgresUpdater.php
+++ b/includes/installer/PostgresUpdater.php
@@ -755,7 +755,8 @@
$info = $this->db->fieldInfo( $table, $field );
if ( $info->defaultValue() !== $default ) {
$this->output( "Changing '$table.$field' default 
value\n" );
-   $this->db->query( "ALTER TABLE $table ALTER $field SET 
DEFAULT " . $default );
+   $this->db->query( "ALTER TABLE $table ALTER $field SET 
DEFAULT "
+   . $this->db->addQuotes( $default ) );
}
}
 

-- 
To view, visit https://gerrit.wikimedia.org/r/377672
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: REL1_29
Gerrit-Owner: Reedy 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits