jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/376907 )
Change subject: Quote $default in PostgresUpdater::setDefault
......................................................................
Quote $default in PostgresUpdater::setDefault
Bug: T175439
Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5
---
M includes/installer/PostgresUpdater.php
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Paladox: Looks good to me, but someone else must approve
jenkins-bot: Verified
Jforrester: Looks good to me, approved
diff --git a/includes/installer/PostgresUpdater.php
b/includes/installer/PostgresUpdater.php
index e5a5c94..07aeb13 100644
--- a/includes/installer/PostgresUpdater.php
+++ b/includes/installer/PostgresUpdater.php
@@ -783,7 +783,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/376907
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id5f5f4f34c7b9ff9af4b706e3e126ba8fda6a0d5
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Reedy <re...@wikimedia.org>
Gerrit-Reviewer: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: Anomie <bjor...@wikimedia.org>
Gerrit-Reviewer: Jforrester <jforres...@wikimedia.org>
Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
