diff --git a/docs/changelog/6.x.x.txt b/docs/changelog/6.x.x.txt
index 6c3af2a3c..f89317bd5 100644
--- a/docs/changelog/6.x.x.txt
+++ b/docs/changelog/6.x.x.txt
@@ -16,6 +16,9 @@
Matt Wilson for inspiration.
- Thumbnails will now be generated a little sharper. Thanks to Nuba.
- SSL is now more strictly enforced.
+ - Deleted the long since deprecated authentication methods (logout, login,
+ displayAccount, displayLogin, recoverPassword). If you don't understand
+ this, don't worry about it, you'll still be able to log in to your site. =)
- Fixed resetting votes on Poll would crash it.
- Fixed not being able to set display title and other yes no questions to no.
- Fixed a bug where URLs would become unreachable when using SSL.
diff --git a/lib/WebGUI/Auth.pm b/lib/WebGUI/Auth.pm
index ad6eb220a..54bf68a2d 100644
--- a/lib/WebGUI/Auth.pm
+++ b/lib/WebGUI/Auth.pm
@@ -400,7 +400,7 @@ sub displayLogin {
$vars->{'login.form.submit'} = WebGUI::Form::submit({"value"=>WebGUI::International::get(52)});
$vars->{'login.form.footer'} = WebGUI::Form::formFooter();
$vars->{'anonymousRegistration.isAllowed'} = ($session{setting}{anonymousRegistration});
- $vars->{'createAccount.url'} = WebGUI::URL::page('op=createAccount');
+ $vars->{'createAccount.url'} = WebGUI::URL::page('op=auth&method=createAccount');
$vars->{'createAccount.label'} = WebGUI::International::get(67);
return WebGUI::Asset::Template->new($self->getLoginTemplateId)->process($vars);
}
diff --git a/lib/WebGUI/Auth/WebGUI.pm b/lib/WebGUI/Auth/WebGUI.pm
index 230b491ad..0e0531ab9 100644
--- a/lib/WebGUI/Auth/WebGUI.pm
+++ b/lib/WebGUI/Auth/WebGUI.pm
@@ -179,7 +179,7 @@ sub createAccount {
$vars->{'create.form.passwordConfirm.label'} = WebGUI::International::get(2,'AuthWebGUI');
$vars->{'create.form.hidden'} = WebGUI::Form::hidden({"name"=>"confirm","value"=>$session{form}{confirm}});
$vars->{'recoverPassword.isAllowed'} = $self->getSetting("passwordRecovery");
- $vars->{'recoverPassword.url'} = WebGUI::URL::page('op=recoverPassword');
+ $vars->{'recoverPassword.url'} = WebGUI::URL::page('op=auth&method=recoverPassword');
$vars->{'recoverPassword.label'} = WebGUI::International::get(59);
return $self->SUPER::createAccount("createAccountSave",$vars);
}
@@ -286,7 +286,7 @@ sub displayLogin {
return $self->displayAccount($_[0]) if ($self->userId != 1);
$vars->{'login.message'} = $_[0] if ($_[0]);
$vars->{'recoverPassword.isAllowed'} = $self->getSetting("passwordRecovery");
- $vars->{'recoverPassword.url'} = WebGUI::URL::page('op=recoverPassword');
+ $vars->{'recoverPassword.url'} = WebGUI::URL::page('op=auth&method=recoverPassword');
$vars->{'recoverPassword.label'} = WebGUI::International::get(59);
return $self->SUPER::displayLogin("login",$vars);
}
@@ -501,7 +501,7 @@ sub recoverPassword {
$vars->{'login.label'} = WebGUI::International::get(58);
$vars->{'anonymousRegistration.isAllowed'} = ($session{setting}{anonymousRegistration});
- $vars->{'createAccount.url'} = WebGUI::URL::page('op=createAccount');
+ $vars->{'createAccount.url'} = WebGUI::URL::page('op=auth=&method=createAccount');
$vars->{'createAccount.label'} = WebGUI::International::get(67);
$vars->{'recover.message'} = $_[0] if ($_[0]);
$vars->{'recover.form.email'} = WebGUI::Form::text({"name"=>"email"});
diff --git a/lib/WebGUI/Macro/L_loginBox.pm b/lib/WebGUI/Macro/L_loginBox.pm
index 421cbc8e2..8180b89be 100644
--- a/lib/WebGUI/Macro/L_loginBox.pm
+++ b/lib/WebGUI/Macro/L_loginBox.pm
@@ -20,7 +20,7 @@ use WebGUI::URL;
#-------------------------------------------------------------------
sub _createURL {
- return ''.$_[0].'';
+ return ''.$_[0].'';
}
#-------------------------------------------------------------------
@@ -32,8 +32,8 @@ sub process {
$var{'customText'} = $param[1];
$var{'customText'} =~ s/%(.*?)%/_createURL($1)/ge;
$var{'hello.label'} = WebGUI::International::get(48);
- $var{'logout.url'} = WebGUI::URL::page("op=logout");
- $var{'account.display.url'} = WebGUI::URL::page('op=displayAccount');
+ $var{'logout.url'} = WebGUI::URL::page("op=auth&method=logout");
+ $var{'account.display.url'} = WebGUI::URL::page('op=auth&method=displayAccount');
$var{'logout.label'} = WebGUI::International::get(49);
my $boxSize = $param[0];
$boxSize = 12 unless $boxSize;
@@ -48,6 +48,10 @@ sub process {
$var{'form.header'} = WebGUI::Form::formHeader({action=>$action})
.WebGUI::Form::hidden({
name=>"op",
+ value=>"auth"
+ })
+ .WebGUI::Form::hidden({
+ name=>"method",
value=>"login"
});
$var{'username.label'} = WebGUI::International::get(50);
@@ -66,7 +70,7 @@ sub process {
value=>WebGUI::International::get(52),
extras=>'class="loginBoxButton"'
});
- $var{'account.create.url'} = WebGUI::URL::page('op=createAccount');
+ $var{'account.create.url'} = WebGUI::URL::page('op=auth&method=createAccount');
$var{'account.create.label'} = WebGUI::International::get(407);
$var{'form.footer'} = WebGUI::Form::formFooter();
return WebGUI::Asset::Template->new($templateId)->process(\%var);
diff --git a/lib/WebGUI/Macro/LoginToggle.pm b/lib/WebGUI/Macro/LoginToggle.pm
index cd02027ee..51933c224 100644
--- a/lib/WebGUI/Macro/LoginToggle.pm
+++ b/lib/WebGUI/Macro/LoginToggle.pm
@@ -24,12 +24,12 @@ sub process {
my $logout = $param[1] || WebGUI::International::get(717);
my %var;
if ($session{user}{userId} eq '1') {
- return WebGUI::URL::page("op=displayLogin") if ($param[0] eq "linkonly");
- $var{'toggle.url'} = WebGUI::URL::page('op=displayLogin');
+ return WebGUI::URL::page("op=auth&method=init") if ($param[0] eq "linkonly");
+ $var{'toggle.url'} = WebGUI::URL::page('op=auth&method=init');
$var{'toggle.text'} = $login;
} else {
- return WebGUI::URL::page("op=logout") if ($param[0] eq "linkonly");
- $var{'toggle.url'} = WebGUI::URL::page('op=logout');
+ return WebGUI::URL::page("op=auth&method=logout") if ($param[0] eq "linkonly");
+ $var{'toggle.url'} = WebGUI::URL::page('op=auth&method=logout');
$var{'toggle.text'} = $logout;
}
if ($param[3]) {
diff --git a/lib/WebGUI/Macro/a_account.pm b/lib/WebGUI/Macro/a_account.pm
index d01474d65..8dd5c36a3 100644
--- a/lib/WebGUI/Macro/a_account.pm
+++ b/lib/WebGUI/Macro/a_account.pm
@@ -21,8 +21,8 @@ use WebGUI::URL;
sub process {
my %var;
my @param = WebGUI::Macro::getParams(shift);
- return WebGUI::URL::page("op=displayAccount") if ($param[0] eq "linkonly");
- $var{'account.url'} = WebGUI::URL::page('op=displayAccount');
+ return WebGUI::URL::page("op=auth&method=init") if ($param[0] eq "linkonly");
+ $var{'account.url'} = WebGUI::URL::page('op=auth&method=init');
$var{'account.text'} = $param[0] || WebGUI::International::get(46);
return WebGUI::Asset::Template->newByUrl($param[1]||"default_account_macro")->process(\%var);
}
diff --git a/lib/WebGUI/Operation/Auth.pm b/lib/WebGUI/Operation/Auth.pm
index a20790d04..61f3b448a 100644
--- a/lib/WebGUI/Operation/Auth.pm
+++ b/lib/WebGUI/Operation/Auth.pm
@@ -61,45 +61,6 @@ sub www_auth {
return WebGUI::Operation::Shared::userStyle($authMethod->$methodCall);
}
-#-------------------------------------------------------------------
-sub www_createAccount {
- return www_auth("createAccount");
-}
-#-------------------------------------------------------------------
-sub www_deactivateAccount {
- return www_auth("deactivateAccount");
-}
-
-#-------------------------------------------------------------------
-sub www_displayAccount {
- return www_auth("displayAccount");
-}
-
-#-------------------------------------------------------------------
-sub www_displayLogin {
- return www_auth("displayLogin");
-}
-
-#-------------------------------------------------------------------
-sub www_init {
- return www_auth("init");
-}
-
-#Deprecated. Kept for backwards compatibility. Use op=auth&method=login
-#-------------------------------------------------------------------
-sub www_login {
- return www_auth("login");
-}
-
-#-------------------------------------------------------------------
-sub www_logout {
- return www_auth("logout");
-}
-
-#-------------------------------------------------------------------
-sub www_recoverPassword {
- return www_auth("recoverPassword");
-}
1;
diff --git a/lib/WebGUI/Operation/Shared.pm b/lib/WebGUI/Operation/Shared.pm
index bf02c38c4..f7f45e5b7 100644
--- a/lib/WebGUI/Operation/Shared.pm
+++ b/lib/WebGUI/Operation/Shared.pm
@@ -36,7 +36,7 @@ our @EXPORT = qw(&menuWrapper);
}
unless ($session{form}{op} eq "displayAccount"){
my %hash;
- $hash{'options.display'} = ''.WebGUI::International::get(342).'';
+ $hash{'options.display'} = ''.WebGUI::International::get(342).'';
push(@array,\%hash);
}
unless ($session{form}{op} eq "editProfile"){
@@ -59,11 +59,11 @@ our @EXPORT = qw(&menuWrapper);
}
my %logout;
- $logout{'options.display'} = ''.WebGUI::International::get(64).'';
+ $logout{'options.display'} = ''.WebGUI::International::get(64).'';
push(@array,\%logout);
if ($session{setting}{selfDeactivation} && !WebGUI::Grouping::isInGroup(3)){
my %hash;
- $hash{'options.display'} = ''.WebGUI::International::get(65).'';
+ $hash{'options.display'} = ''.WebGUI::International::get(65).'';
push(@array,\%hash);
}
return \@array;