Installer for comment module version 6 still set rss_visible to "both"

not "all".  Fix that now and rebuild the installer package.  Fixes #1917.
This commit is contained in:
Bharat Mediratta
2012-10-08 14:04:40 -07:00
parent 8855847df2
commit b7e89613fc
3 changed files with 14 additions and 5 deletions
+11 -2
View File
@@ -48,8 +48,8 @@ class comment_installer {
module::set_var("comment", "spam_caught", 0);
module::set_var("comment", "access_permissions", "everybody");
module::set_var("comment", "rss_visible", "both");
module::set_version("comment", 6);
module::set_var("comment", "rss_visible", "all");
module::set_version("comment", 7);
}
static function upgrade($version) {
@@ -91,6 +91,15 @@ class comment_installer {
module::clear_var("comment", "rss_available");
module::set_version("comment", $version = 6);
}
// In version 6 we accidentally left the install value of "rss_visible" to "both" when it
// should have been "all"
if ($version == 6) {
if (module::get_var("comment", "rss_visible") == "both") {
module::set_var("comment", "rss_visible", "all");
}
module::set_version("comment", $version = 7);
}
}
static function uninstall() {