Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/discourse/manifests/sitesetting.pp

Issue 6221863587938304: Issue 212 - Update Discourse (Closed)
Patch Set: Fixed nit and site settings Created March 27, 2014, 8:26 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « modules/discourse/manifests/init.pp ('k') | modules/discourse/templates/database.yml.erb » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/discourse/manifests/sitesetting.pp
===================================================================
--- a/modules/discourse/manifests/sitesetting.pp
+++ b/modules/discourse/manifests/sitesetting.pp
@@ -16,19 +16,21 @@ define discourse::sitesetting(
present: {
# This is apparently how you do a conditional INSERT in PostgreSQL - sorry
$update_sql = "UPDATE site_settings SET value = '$escaped_value', data_type = $escaped_type WHERE name = '$escaped_setting' RETURNING 1"
$columns = "name, data_type, value, created_at, updated_at"
$values = "SELECT '$escaped_setting', $escaped_type, '$escaped_value', CURRENT_TIMESTAMP, CURRENT_TIMESTAMP"
postgresql_psql {"WITH upd AS ($update_sql) INSERT INTO site_settings ($columns) $values WHERE NOT EXISTS (SELECT * FROM upd)":
db => 'discourse',
- psql_user => 'discourse'
+ psql_user => 'discourse',
+ notify => Service['discourse']
}
}
absent: {
postgresql_psql {"DELETE FROM site_settings WHERE name = '$escaped_setting'":
db => 'discourse',
- psql_user => 'discourse'
+ psql_user => 'discourse',
+ notify => Service['discourse']
}
}
}
}
« no previous file with comments | « modules/discourse/manifests/init.pp ('k') | modules/discourse/templates/database.yml.erb » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld