From 12ea0cf6fa299a5afb7d06d10e70996d3bab923d Mon Sep 17 00:00:00 2001 From: JT Smith Date: Sat, 5 Mar 2005 02:34:20 +0000 Subject: [PATCH] switched over old auth links to the new url system --- docs/changelog/6.x.x.txt | 3 +++ lib/WebGUI/Auth.pm | 2 +- lib/WebGUI/Auth/WebGUI.pm | 6 ++--- lib/WebGUI/Macro/L_loginBox.pm | 12 ++++++---- lib/WebGUI/Macro/LoginToggle.pm | 8 +++---- lib/WebGUI/Macro/a_account.pm | 4 ++-- lib/WebGUI/Operation/Auth.pm | 39 --------------------------------- lib/WebGUI/Operation/Shared.pm | 6 ++--- 8 files changed, 24 insertions(+), 56 deletions(-) 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;