Skip to content

Commit

Permalink
FIX: Duplicating email confirmation link (#142)
Browse files Browse the repository at this point in the history
Fixed the link in the member registration confirmation email. No longer creates a link with duplicate segments
  • Loading branch information
christohill authored and Nathan committed Jul 11, 2018
1 parent e87e223 commit 361643c
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions code/email/MemberConfirmationEmail.php
Original file line number Diff line number Diff line change
Expand Up @@ -107,12 +107,11 @@ public function getParsedString($string) {
$variables = array (
'$SiteName' => SiteConfig::current_site_config()->Title,
'$LoginLink' => Controller::join_links(
$absoluteBaseURL,
$absoluteBaseURL,
singleton('Security')->Link('login')
),
'$ConfirmLink' => Controller::join_links(
$absoluteBaseURL,
$this->page->Link('confirm'),
$this->page->AbsoluteLink('confirm'),
$this->member->ID,
"?key={$this->member->ValidationKey}"
),
Expand Down Expand Up @@ -167,4 +166,4 @@ public function getPage() {
public function getMember() {
return $this->member;
}
}
}

0 comments on commit 361643c

Please sign in to comment.