Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Change sql param to default to undef instead of empty string
Browse files Browse the repository at this point in the history
The future parser treats the empty string '' as a truthy value. This
means that mysql::db will always try to include the db import exec in
the catalog. With the empty string as the $sql value, the command
attempts to import '' into a database, which fails. This patch changes
the default $sql value to undef so that the exec won't be included if
there is no sql to import.
Colleen Murphy committed Oct 3, 2014
1 parent b338635 commit bf2c8d4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion manifests/db.pp
Original file line number Diff line number Diff line change
@@ -7,7 +7,7 @@
$collate = 'utf8_general_ci',
$host = 'localhost',
$grant = 'ALL',
$sql = '',
$sql = undef,
$enforce_sql = false,
$ensure = 'present'
) {

0 comments on commit bf2c8d4

Please sign in to comment.