Renamed to match with new name
[core.git] / inc / classes / main / commands / web / class_WebConfirmCommand.php
index 090dbad97c609f2a5535f618b1443e6cff997891..0b3f8a26bb75d74e9fad0979feff04452a30ba95 100644 (file)
@@ -99,7 +99,12 @@ class WebConfirmCommand extends BaseCommand implements Commandable {
                $templateInstance->assignVariable('title', $this->getLanguageInstance()->getMessage('page_confirm_link_title'));
 
                // Get user instance
-               $userInstance = Registry::getRegistry()->getInstance('user');
+               try {
+                       $userInstance = Registry::getRegistry()->getInstance('user');
+               } catch (NullPointerException $e) {
+                       // Not found user, e.g. when the user is somehow invalid
+                       $responseInstance->redirectToConfiguredUrl('web_cmd_user_is_null');
+               }
 
                // Set username
                $templateInstance->assignVariable('username', $userInstance->getField(UserDatabaseWrapper::DB_COLUMN_USERNAME));
@@ -108,6 +113,12 @@ class WebConfirmCommand extends BaseCommand implements Commandable {
                // *every* command has a navigation system and that is want we don't want.
                $menuInstance = ObjectFactory::createObjectByConfiguredName('confirm_menu_class', array($appInstance));
 
+               // Render the menu
+               $menuInstance->renderMenu();
+
+               // Transfer it to the template engine instance
+               $menuInstance->transferContentToTemplateEngine($templateInstance);
+
                // ... and all variables. This should be merged together in a pattern
                // to make things easier. A cache mechanism should be added between
                // these two calls to cache compiled templates.