- Fixed 2 typos in the user module. Thanks Axel and Remco.

- Applied (modified versions of) Alexander's patches on the development
  branch.
4.0.x
Dries Buytaert 2001-09-18 18:39:15 +00:00
parent c5a349ca51
commit ca85a56f1b
7 changed files with 28 additions and 18 deletions

View File

@ -14,7 +14,7 @@ if (user_access("access content")) {
} }
} }
else { else {
$theme->box("Access denied", message_access()); $theme->box(t("Access denied"), message_access());
} }
$theme->footer(); $theme->footer();

View File

@ -122,10 +122,10 @@ function cloud_list($limit = 10) {
if ($hour != floor((time() - $site->timestamp) / 3600)) { if ($hour != floor((time() - $site->timestamp) / 3600)) {
$hour = floor((time() - $site->timestamp) / 3600); $hour = floor((time() - $site->timestamp) / 3600);
if ($hour < 12) { if ($hour < 12) {
$output .= "<p />Updated ". format_plural($hour, "hour", "hours") ." ago:"; $output .= "<p />". strtr(t("Updated %a ago:"), array("%a" => format_plural($hour, "hour", "hours")));
} }
else if ($list) { else if ($list) {
$output .= "<p />Updated ". format_plural($hour, "+ hour", "+ hours") ." ago:"; $output .= "<p />". strtr(t("Updated more than %a ago:"), array("%a" => format_plural($hour, "hour", "hours")));
$list = 0; $list = 0;
} }
} }

View File

@ -6,7 +6,7 @@ function search_perm() {
function search_link($type) { function search_link($type) {
if ($type == "page" && user_access("search content")) { if ($type == "page" && user_access("search content")) {
$links[] = "<a href=\"module.php?mod=search\">search</a>"; $links[] = "<a href=\"module.php?mod=search\">". t("search") ."</a>";
} }
return $links ? $links : array(); return $links ? $links : array();
@ -49,7 +49,7 @@ function search_page() {
} }
else { else {
$theme->header(); $theme->header();
$theme->box("Access denied", message_access()); $theme->box(t("Access denied"), message_access());
$theme->footer(); $theme->footer();
} }
} }

View File

@ -6,7 +6,7 @@ function search_perm() {
function search_link($type) { function search_link($type) {
if ($type == "page" && user_access("search content")) { if ($type == "page" && user_access("search content")) {
$links[] = "<a href=\"module.php?mod=search\">search</a>"; $links[] = "<a href=\"module.php?mod=search\">". t("search") ."</a>";
} }
return $links ? $links : array(); return $links ? $links : array();
@ -49,7 +49,7 @@ function search_page() {
} }
else { else {
$theme->header(); $theme->header();
$theme->box("Access denied", message_access()); $theme->box(t("Access denied"), message_access());
$theme->footer(); $theme->footer();
} }
} }

View File

@ -143,7 +143,7 @@ function user_access($perm) {
} }
} }
function user_mail($subject, $message, $header) { function user_mail($mail, $subject, $message, $header) {
// print "<pre>subject: $subject<hr />header: $header<hr />$message</pre>"; // print "<pre>subject: $subject<hr />header: $header<hr />$message</pre>";
mail($mail, $subject, $message, $header); mail($mail, $subject, $message, $header);
} }
@ -403,7 +403,7 @@ function drupal_auth($username, $password, $server) {
/*** User features *********************************************************/ /*** User features *********************************************************/
function user_login($edit = array()) { function user_login($edit = array()) {
global $user, $REQUEST_URI; global $user, $REQUEST_URI, $HTTP_REFERER;
if (user_deny("user", $edit["name"])) { if (user_deny("user", $edit["name"])) {
$error = sprintf(t("The name '%s' has been denied access."), $edit["name"]); $error = sprintf(t("The name '%s' has been denied access."), $edit["name"]);
@ -468,10 +468,14 @@ function user_login($edit = array()) {
user_save($user, array("session" => session_id())); user_save($user, array("session" => session_id()));
/* /*
** Redirect the user to his personal information page: ** Redirect the user to the page he logged on from or to his personal
** information page if we can detect the referer page:
*/ */
header("Location: module.php?mod=user"); $url = $HTTP_REFERER ? $HTTP_REFERER : "module.php?mod=user&op=view";
header("Location: $url");
} }
else { else {
watchdog("user", "failed to login for '". $name ."': invalid password"); watchdog("user", "failed to login for '". $name ."': invalid password");
@ -517,7 +521,7 @@ function user_logout() {
** Redirect the user to his personal information page: ** Redirect the user to his personal information page:
*/ */
header("Location: module.php?mod=user"); header("Location: index.php");
} }
} }
@ -608,6 +612,7 @@ function user_register($edit = array()) {
** Create new user account, no administrator approval required: ** Create new user account, no administrator approval required:
*/ */
user_save("", array("name" => $edit["name"], "pass" => $pass, "init" => $edit["mail"], "mail" => $edit["mail"], "role" => "authenticated user", "status" => 1));
user_mail($edit["mail"], t("user account details"), sprintf(t("%s,\n\nsomoneone signed up for a user account on %s and supplied this e-mail address as their contact. If it wasn't you, just ignore this mail but if it was you, you can now login using the following username and password:\n\n username: %s\n password: %s\n\n\n-- %s team"), $edit["name"], variable_get("site_name", "drupal"), $edit["name"], $pass, variable_get("site_name", "drupal")), "From: $from\nReply-to: $from\nX-Mailer: Drupal\nReturn-path: $from\nErrors-to: $from"); user_mail($edit["mail"], t("user account details"), sprintf(t("%s,\n\nsomoneone signed up for a user account on %s and supplied this e-mail address as their contact. If it wasn't you, just ignore this mail but if it was you, you can now login using the following username and password:\n\n username: %s\n password: %s\n\n\n-- %s team"), $edit["name"], variable_get("site_name", "drupal"), $edit["name"], $pass, variable_get("site_name", "drupal")), "From: $from\nReply-to: $from\nX-Mailer: Drupal\nReturn-path: $from\nErrors-to: $from");
} }

View File

@ -143,7 +143,7 @@ function user_access($perm) {
} }
} }
function user_mail($subject, $message, $header) { function user_mail($mail, $subject, $message, $header) {
// print "<pre>subject: $subject<hr />header: $header<hr />$message</pre>"; // print "<pre>subject: $subject<hr />header: $header<hr />$message</pre>";
mail($mail, $subject, $message, $header); mail($mail, $subject, $message, $header);
} }
@ -403,7 +403,7 @@ function drupal_auth($username, $password, $server) {
/*** User features *********************************************************/ /*** User features *********************************************************/
function user_login($edit = array()) { function user_login($edit = array()) {
global $user, $REQUEST_URI; global $user, $REQUEST_URI, $HTTP_REFERER;
if (user_deny("user", $edit["name"])) { if (user_deny("user", $edit["name"])) {
$error = sprintf(t("The name '%s' has been denied access."), $edit["name"]); $error = sprintf(t("The name '%s' has been denied access."), $edit["name"]);
@ -468,10 +468,14 @@ function user_login($edit = array()) {
user_save($user, array("session" => session_id())); user_save($user, array("session" => session_id()));
/* /*
** Redirect the user to his personal information page: ** Redirect the user to the page he logged on from or to his personal
** information page if we can detect the referer page:
*/ */
header("Location: module.php?mod=user"); $url = $HTTP_REFERER ? $HTTP_REFERER : "module.php?mod=user&op=view";
header("Location: $url");
} }
else { else {
watchdog("user", "failed to login for '". $name ."': invalid password"); watchdog("user", "failed to login for '". $name ."': invalid password");
@ -517,7 +521,7 @@ function user_logout() {
** Redirect the user to his personal information page: ** Redirect the user to his personal information page:
*/ */
header("Location: module.php?mod=user"); header("Location: index.php");
} }
} }
@ -608,6 +612,7 @@ function user_register($edit = array()) {
** Create new user account, no administrator approval required: ** Create new user account, no administrator approval required:
*/ */
user_save("", array("name" => $edit["name"], "pass" => $pass, "init" => $edit["mail"], "mail" => $edit["mail"], "role" => "authenticated user", "status" => 1));
user_mail($edit["mail"], t("user account details"), sprintf(t("%s,\n\nsomoneone signed up for a user account on %s and supplied this e-mail address as their contact. If it wasn't you, just ignore this mail but if it was you, you can now login using the following username and password:\n\n username: %s\n password: %s\n\n\n-- %s team"), $edit["name"], variable_get("site_name", "drupal"), $edit["name"], $pass, variable_get("site_name", "drupal")), "From: $from\nReply-to: $from\nX-Mailer: Drupal\nReturn-path: $from\nErrors-to: $from"); user_mail($edit["mail"], t("user account details"), sprintf(t("%s,\n\nsomoneone signed up for a user account on %s and supplied this e-mail address as their contact. If it wasn't you, just ignore this mail but if it was you, you can now login using the following username and password:\n\n username: %s\n password: %s\n\n\n-- %s team"), $edit["name"], variable_get("site_name", "drupal"), $edit["name"], $pass, variable_get("site_name", "drupal")), "From: $from\nReply-to: $from\nX-Mailer: Drupal\nReturn-path: $from\nErrors-to: $from");
} }

View File

@ -24,7 +24,7 @@ if (user_access("post content")) {
} }
} }
else { else {
$theme->box("Submit", message_access()); $theme->box(t("Submit"), message_access());
} }
$theme->footer(); $theme->footer();