From 097f6209e5cd09c2be7b538b7743f68728a60b00 Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Fri, 26 Dec 2003 13:44:18 +0000
Subject: [PATCH] - First take at improving the user module's code: replaced
 the custom error   printing with calls to drupal_set_message().

---
 modules/user.module      | 14 +++++++-------
 modules/user/user.module | 14 +++++++-------
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/modules/user.module b/modules/user.module
index 34e8b7783b45..d73206d5eca4 100644
--- a/modules/user.module
+++ b/modules/user.module
@@ -648,7 +648,7 @@ function user_login($edit = array(), $msg = "") {
   */
 
   if ($error) {
-    $output .= theme("error", $error);
+    drupal_set_message($error, 'error');
   }
 
   /*
@@ -753,7 +753,7 @@ function user_pass($edit = array()) {
 
     // Display error message if necessary.
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
 
     /*
@@ -876,7 +876,7 @@ function user_register($edit = array()) {
   }
   else {
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
   }
 
@@ -981,7 +981,7 @@ function user_edit($edit = array()) {
     }
 
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
 
     if (!$edit) {
@@ -1172,7 +1172,7 @@ function user_admin_create($edit = array()) {
   else {
 
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
 
     $output .= form_textfield(t("Username"), "name", $edit["name"], 30, 55, t("Provide the username of the new account."));
@@ -1429,7 +1429,7 @@ function user_admin_edit($edit = array()) {
         drupal_set_message(t("user information changes have been saved."));
       }
       else {
-        $output .= theme("error", $error);
+        drupal_set_message($error, 'error');
       }
     }
     else if ($op == t("Delete account")) {
@@ -1441,7 +1441,7 @@ function user_admin_edit($edit = array()) {
       }
       else {
         $error = t("Failed to delete account: the account has to be blocked first.");
-        $output .= theme("error", $error);
+        drupal_set_message($error, 'error');
       }
     }
 
diff --git a/modules/user/user.module b/modules/user/user.module
index 34e8b7783b45..d73206d5eca4 100644
--- a/modules/user/user.module
+++ b/modules/user/user.module
@@ -648,7 +648,7 @@ function user_login($edit = array(), $msg = "") {
   */
 
   if ($error) {
-    $output .= theme("error", $error);
+    drupal_set_message($error, 'error');
   }
 
   /*
@@ -753,7 +753,7 @@ function user_pass($edit = array()) {
 
     // Display error message if necessary.
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
 
     /*
@@ -876,7 +876,7 @@ function user_register($edit = array()) {
   }
   else {
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
   }
 
@@ -981,7 +981,7 @@ function user_edit($edit = array()) {
     }
 
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
 
     if (!$edit) {
@@ -1172,7 +1172,7 @@ function user_admin_create($edit = array()) {
   else {
 
     if ($error) {
-      $output .= theme("error", $error);
+      drupal_set_message($error, 'error');
     }
 
     $output .= form_textfield(t("Username"), "name", $edit["name"], 30, 55, t("Provide the username of the new account."));
@@ -1429,7 +1429,7 @@ function user_admin_edit($edit = array()) {
         drupal_set_message(t("user information changes have been saved."));
       }
       else {
-        $output .= theme("error", $error);
+        drupal_set_message($error, 'error');
       }
     }
     else if ($op == t("Delete account")) {
@@ -1441,7 +1441,7 @@ function user_admin_edit($edit = array()) {
       }
       else {
         $error = t("Failed to delete account: the account has to be blocked first.");
-        $output .= theme("error", $error);
+        drupal_set_message($error, 'error');
       }
     }
 
-- 
GitLab