Remove a redundant comment, change the name of the identity change event to 'identity_before_change' and change the name of 'login::get_login_form()' to 'login::get_form()'

1 files changed, 1 insertions, 2 deletions

diff --git a/modules/gallery/controllers/admin_identity.php b/modules/gallery/controllers/admin_identity.php
index dcc3edcb..520b1966 100644
--- a/modules/gallery/controllers/admin_identity.php
+++ b/modules/gallery/controllers/admin_identity.php
@@ -45,7 +45,7 @@ class Admin_Identity_Controller extends Admin_Controller {
 
     if ($new_provider != $active_provider) {
 
-      module::event("pre_identity_change", $active_provider, $new_provider);
+      module::event("identity_before_change", $active_provider, $new_provider);
 
       module::deactivate($active_provider);
       module::uninstall($active_provider);
@@ -57,7 +57,6 @@ class Admin_Identity_Controller extends Admin_Controller {
       module::install($new_provider);
       module::activate($new_provider);
 
-      // @todo this type of collation is questionable from an i18n perspective
       message::success(t("Changed to %description",
                          array("description" => $providers->$new_provider)));