From 90a5a6cf6e722767cb8f00dfdd5d12963b718e91 Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Mon, 29 Dec 2003 13:51:59 +0000
Subject: - Updated the user login block to validate as XHTML strict: the  
 <div class="user-login-block"> is now inside the <form> instead of the  
 other way around.

- Simplified the user login block by removing some CSS that was no longer
  needed.  The block module already emits block-related classes and an id.
---
 modules/user/user.module | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

(limited to 'modules/user')

diff --git a/modules/user/user.module b/modules/user/user.module
index f3696ca90..50472e08c 100644
--- a/modules/user/user.module
+++ b/modules/user/user.module
@@ -350,23 +350,22 @@ function user_block($op = "list", $delta = 0) {
           }
           // NOTE: special care needs to be taken because on pages with forms, such as node and comment submission pages, the $edit variable might already be set.
 
-          $form = form_hidden("destination", $edit["destination"]);
-          $form .= form_textfield(t("Username"), "name", $edit["name"], 15, 64);
-          $form .= form_password(t("Password"), "pass", $pass, 15, 64);
+          $output .= form_hidden("destination", $edit["destination"]);
+          $output .= form_textfield(t("Username"), "name", $edit["name"], 15, 64);
+          $output .= form_password(t("Password"), "pass", $pass, 15, 64);
 
           if (variable_get("user_remember", 0) == 0) {
-            $form .= form_checkbox(t("Remember me"), "remember_me");
+            $output .= form_checkbox(t("Remember me"), "remember_me");
           }
           elseif (variable_get("user_remember", 1) == 1) {
-            $form .= form_hidden("remember_me", 1);
+            $output .= form_hidden("remember_me", 1);
           }
 
-          $form .= form_submit(t("Log in"));
-
-          $output .= form($form, "post", url("user/login"));
-
+          $output .= form_submit(t("Log in"));
           $output .= "</div>\n";
 
+          $output  = form($output, "post", url("user/login"));
+
           if (variable_get("user_register", 1)) {
             $items[] = l(t("Create new account"), "user/register", array("title" => t("Create a new user account.")));
           }
@@ -375,7 +374,7 @@ function user_block($op = "list", $delta = 0) {
           $output .= theme("item_list", $items);
 
           $block["subject"] = t("User login");
-          $block["content"] = "<div class=\"user-login-link\">$output</div>";
+          $block["content"] = $output;
         }
         return $block;
       case 1:
-- 
cgit v1.2.3