Restructure the module lifecycle.

Install: <module>_installer::install() is called, any necessary tables
are created.

Activate: <module>_installer::activate() is called.  Module
controllers are routable, helpers are accessible, etc.  The module is
in use.

Deactivate: <module>_installer::deactivate() is called.  Module code
is not accessible or routable.  Module is *not* in use, but its tables
are still around.

Uninstall: <module>_installer::uninstall() is called.  Module is
completely removed from the database.

Admin > Modules will install and activate modules, but will only
deactivate (will NOT uninstall modules).

1 files changed, 2 insertions, 2 deletions

diff --git a/modules/recaptcha/helpers/recaptcha_installer.php b/modules/recaptcha/helpers/recaptcha_installer.php
index ccc27aae..6269c632 100644
--- a/modules/recaptcha/helpers/recaptcha_installer.php
+++ b/modules/recaptcha/helpers/recaptcha_installer.php
@@ -20,16 +20,16 @@
 class recaptcha_installer {
   static function install() {
     $version = module::get_version("recaptcha");
-
     if ($version == 0) {
       module::set_version("recaptcha", 1);
     }
+  }
 
+  static function activate() {
     recaptcha::check_config();
   }
 
   static function uninstall() {
     site_status::clear("recaptcha_config");
-    module::delete("recaptcha");
   }
 }