silverstripe / cwp-core

tests/AtomFeedTest.php

Indentation +54 added lines, -54 removed lines

@@ -14,67 +14,67 @@
 
 class AtomFeedTest extends SapphireTest
 {
-    protected static $original_host;
+	protected static $original_host;
 
-    public function testAtomFeed()
-    {
-        $list = new ArrayList();
-        $list->push(new ItemA());
-        $list->push(new ItemB());
-        $list->push(new ItemC());
+	public function testAtomFeed()
+	{
+		$list = new ArrayList();
+		$list->push(new ItemA());
+		$list->push(new ItemB());
+		$list->push(new ItemC());
 
-        $atomFeed = new CwpAtomFeed(
-            $list,
-            "http://www.example.com",
-            "Test Atom Feed",
-            "Test Atom Feed Description"
-        );
-        $content = $atomFeed->outputToBrowser();
+		$atomFeed = new CwpAtomFeed(
+			$list,
+			"http://www.example.com",
+			"Test Atom Feed",
+			"Test Atom Feed Description"
+		);
+		$content = $atomFeed->outputToBrowser();
 
-        //Debug::message($content);
-        $this->assertContains('<link href="http://www.example.org/item-a/" />', $content);
-        $this->assertContains('<link href="http://www.example.com/item-b.html" />', $content);
-        $this->assertContains('<link href="http://www.example.com/item-c.html" />', $content);
+		//Debug::message($content);
+		$this->assertContains('<link href="http://www.example.org/item-a/" />', $content);
+		$this->assertContains('<link href="http://www.example.com/item-b.html" />', $content);
+		$this->assertContains('<link href="http://www.example.com/item-c.html" />', $content);
 
-        $this->assertContains('<title type="html">ItemA</title>', $content);
-        $this->assertContains('<title type="html">ItemB</title>', $content);
-        $this->assertContains('<title type="html">ItemC</title>', $content);
+		$this->assertContains('<title type="html">ItemA</title>', $content);
+		$this->assertContains('<title type="html">ItemB</title>', $content);
+		$this->assertContains('<title type="html">ItemC</title>', $content);
 
-        $this->assertContains("\tItemA Content\n", $content);
-        $this->assertContains("\tItemB Content\n", $content);
-        $this->assertContains("\tItemC Content\n", $content);
-    }
+		$this->assertContains("\tItemA Content\n", $content);
+		$this->assertContains("\tItemB Content\n", $content);
+		$this->assertContains("\tItemC Content\n", $content);
+	}
 
-    public function testRenderWithTemplate()
-    {
-        $atomFeed = new CwpAtomFeed(new ArrayList(), "", "", "");
-        $content = $atomFeed->outputToBrowser();
-        // test we have switched from a RSS feed test template tot he AtomFeed template
-        $this->assertNotContains('<title>Test Custom Template</title>', $content);
-    }
+	public function testRenderWithTemplate()
+	{
+		$atomFeed = new CwpAtomFeed(new ArrayList(), "", "", "");
+		$content = $atomFeed->outputToBrowser();
+		// test we have switched from a RSS feed test template tot he AtomFeed template
+		$this->assertNotContains('<title>Test Custom Template</title>', $content);
+	}
 
-    public function testLinkToFeed()
-    {
-        $link = AtomTagsStub::linkToFeed('atomLinkUrl', 'Atom feed of this blog');
-        $this->assertContains('atomLinkUrl', $link);
-        $this->assertContains('Atom feed of this blog', $link);
-        $this->assertContains('application/atom+xml', $link);
-    }
+	public function testLinkToFeed()
+	{
+		$link = AtomTagsStub::linkToFeed('atomLinkUrl', 'Atom feed of this blog');
+		$this->assertContains('atomLinkUrl', $link);
+		$this->assertContains('Atom feed of this blog', $link);
+		$this->assertContains('application/atom+xml', $link);
+	}
 
-    protected function setUp()
-    {
-        parent::setUp();
-        Config::modify()->set(Director::class, 'alternate_base_url', '/');
-        if (!self::$original_host) {
-            self::$original_host = $_SERVER['HTTP_HOST'];
-        }
-        $_SERVER['HTTP_HOST'] = 'www.example.org';
-    }
+	protected function setUp()
+	{
+		parent::setUp();
+		Config::modify()->set(Director::class, 'alternate_base_url', '/');
+		if (!self::$original_host) {
+			self::$original_host = $_SERVER['HTTP_HOST'];
+		}
+		$_SERVER['HTTP_HOST'] = 'www.example.org';
+	}
 
-    protected function tearDown()
-    {
-        parent::tearDown();
-        Config::modify()->set(Director::class, 'alternate_base_url', null);
-        $_SERVER['HTTP_HOST'] = self::$original_host;
-    }
+	protected function tearDown()
+	{
+		parent::tearDown();
+		Config::modify()->set(Director::class, 'alternate_base_url', null);
+		$_SERVER['HTTP_HOST'] = self::$original_host;
+	}
 }

tests/AtomFeedTest/AtomTagsStub.php

Indentation +7 added lines, -7 removed lines

@@ -11,11 +11,11 @@
  */
 class AtomTagsStub implements TestOnly
 {
-    public static function linkToFeed($url, $title = null)
-    {
-        $link = '<link rel="alternate" type="application/atom+xml" title="' . $title .
-            '" href="' . $url . '" />';
-        CwpAtomFeed::linkToFeed($url, $title);
-        return $link;
-    }
+	public static function linkToFeed($url, $title = null)
+	{
+		$link = '<link rel="alternate" type="application/atom+xml" title="' . $title .
+			'" href="' . $url . '" />';
+		CwpAtomFeed::linkToFeed($url, $title);
+		return $link;
+	}
 }

tests/AtomFeedTest/ItemC.php

Indentation +29 added lines, -29 removed lines

@@ -7,33 +7,33 @@
 
 class ItemC extends ViewableData implements TestOnly
 {
-    // ItemC tests fields - Title has casting, Content doesn't.
-    private static $casting = [
-        'Title' => 'Varchar',
-        'AltContent' => 'Text',
-    ];
-
-    public $Title = 'ItemC';
-
-    public function Title()
-    {
-        return "ItemC";
-    }
-
-    public function Content()
-    {
-        return "ItemC Content";
-    }
-
-    public $AltContent = "ItemC AltContent";
-
-    public function Link()
-    {
-        return "item-c.html";
-    }
-
-    public function AbsoluteLink()
-    {
-        return "http://www.example.com/item-c.html";
-    }
+	// ItemC tests fields - Title has casting, Content doesn't.
+	private static $casting = [
+		'Title' => 'Varchar',
+		'AltContent' => 'Text',
+	];
+
+	public $Title = 'ItemC';
+
+	public function Title()
+	{
+		return "ItemC";
+	}
+
+	public function Content()
+	{
+		return "ItemC Content";
+	}
+
+	public $AltContent = "ItemC AltContent";
+
+	public function Link()
+	{
+		return "item-c.html";
+	}
+
+	public function AbsoluteLink()
+	{
+		return "http://www.example.com/item-c.html";
+	}
 }

tests/AtomFeedTest/ItemB.php

Indentation +18 added lines, -18 removed lines

@@ -7,26 +7,26 @@
 
 class ItemB extends ViewableData implements TestOnly
 {
-    // ItemB tests without $casting
-    public $Title = 'ItemB';
+	// ItemB tests without $casting
+	public $Title = 'ItemB';
 
-    public function Title()
-    {
-        return "ItemB";
-    }
+	public function Title()
+	{
+		return "ItemB";
+	}
 
-    public function AbsoluteLink()
-    {
-        return "http://www.example.com/item-b.html";
-    }
+	public function AbsoluteLink()
+	{
+		return "http://www.example.com/item-b.html";
+	}
 
-    public function Content()
-    {
-        return "ItemB Content";
-    }
+	public function Content()
+	{
+		return "ItemB Content";
+	}
 
-    public function AltContent()
-    {
-        return "ItemB AltContent";
-    }
+	public function AltContent()
+	{
+		return "ItemB AltContent";
+	}
 }

tests/AtomFeedTest/ItemA.php

Indentation +28 added lines, -28 removed lines

@@ -8,32 +8,32 @@
 
 class ItemA extends ViewableData implements TestOnly
 {
-    // Atom-feed items must have $casting/$db information.
-    private static $casting = [
-        'Title' => 'Varchar',
-        'Content' => 'Text',
-        'AltContent' => 'Text',
-    ];
-
-    public $Title = 'ItemA';
-
-    public function Title()
-    {
-        return "ItemA";
-    }
-
-    public function Content()
-    {
-        return "ItemA Content";
-    }
-
-    public function AltContent()
-    {
-        return "ItemA AltContent";
-    }
-
-    public function Link($action = null)
-    {
-        return Controller::join_links("item-a/", $action);
-    }
+	// Atom-feed items must have $casting/$db information.
+	private static $casting = [
+		'Title' => 'Varchar',
+		'Content' => 'Text',
+		'AltContent' => 'Text',
+	];
+
+	public $Title = 'ItemA';
+
+	public function Title()
+	{
+		return "ItemA";
+	}
+
+	public function Content()
+	{
+		return "ItemA Content";
+	}
+
+	public function AltContent()
+	{
+		return "ItemA AltContent";
+	}
+
+	public function Link($action = null)
+	{
+		return Controller::join_links("item-a/", $action);
+	}
 }

tests/Control/InitialisationMiddlewareTest.php

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
      */
     protected function runMiddleware()
     {
-        $this->middleware->process($this->request, function () {
+        $this->middleware->process($this->request, function() {
             // no op
         });
     }

Indentation +116 added lines, -116 removed lines

@@ -10,120 +10,120 @@
 
 class InitialisationMiddlewareTest extends FunctionalTest
 {
-    /**
-     * @var HTTPRequest
-     */
-    protected $request;
-
-    /**
-     * @var InitialisationMiddleware
-     */
-    protected $middleware;
-
-    protected $usesDatabase = true;
-
-    protected function setUp()
-    {
-        parent::setUp();
-
-        $this->request = new HTTPRequest('GET', '/');
-        $this->middleware = new InitialisationMiddleware();
-
-        Environment::setEnv('SS_OUTBOUND_PROXY', '');
-        Environment::setEnv('SS_OUTBOUND_PROXY_PORT', '');
-        putenv('NO_PROXY=');
-    }
-
-    public function testDoNotConfigureProxyIfNoEnvironmentVarsAreSet()
-    {
-        $this->runMiddleware();
-
-        $this->assertEmpty(
-            Environment::getEnv('http_proxy'),
-            'Proxy information is not set if no outbound proxy is configured'
-        );
-    }
-
-    public function testConfigureEgressProxyWhenVarsAreSet()
-    {
-        Environment::setEnv('SS_OUTBOUND_PROXY', 'http://example.com');
-        Environment::setEnv('SS_OUTBOUND_PROXY_PORT', '8024');
-
-        $this->runMiddleware();
-
-        $this->assertEquals(
-            'http://example.com:8024',
-            Environment::getEnv('http_proxy'),
-            'Proxy is configured with proxy and port'
-        );
-    }
-
-    public function testDoNotConfigureProxyDomainExclusionsWhenNoneAreDefined()
-    {
-        Config::modify()->remove(InitialisationMiddleware::class, 'egress_proxy_exclude_domains');
-
-        $this->runMiddleware();
-
-        $this->assertSame(
-            '',
-            Environment::getEnv('NO_PROXY'),
-            'No domain exclusions are set when none are defined'
-        );
-    }
-
-    public function testConfigureEgressProxyDomainExclusions()
-    {
-        Config::modify()->set(
-            InitialisationMiddleware::class,
-            'egress_proxy_exclude_domains',
-            'example.com'
-        );
-
-        putenv('NO_PROXY=foo.com,bar.com');
-        $this->runMiddleware();
-
-        $this->assertSame(
-            'foo.com,bar.com,example.com',
-            Environment::getEnv('NO_PROXY'),
-            'Domain exclusions are combined with existing values and configuration settings'
-        );
-    }
-
-    public function testSecurityHeadersAddedByDefault()
-    {
-        $response = $this->get('Security/login');
-        $this->assertArrayHasKey('x-xss-protection', $response->getHeaders());
-        $this->assertSame('1; mode=block', $response->getHeader('x-xss-protection'));
-    }
-
-    public function testXSSProtectionHeaderNotAdded()
-    {
-        Config::modify()->set(InitialisationMiddleware::class, 'xss_protection_enabled', false);
-        $response = $this->get('Security/login');
-        $this->assertArrayNotHasKey('x-xss-protection', $response->getHeaders());
-    }
-
-    public function testHstsNotAddedByDefault()
-    {
-        Config::modify()->remove(InitialisationMiddleware::class, 'strict_transport_security');
-        $response = $this->get('Security/login');
-        $this->assertArrayNotHasKey('strict-transport-security', $response->getHeaders());
-    }
-
-    public function testHstsAddedWhenConfigured()
-    {
-        Config::modify()->update(InitialisationMiddleware::class, 'strict_transport_security', 'max-age=1');
-        $response = $this->get('Security/login');
-        $this->assertArrayHasKey('strict-transport-security', $response->getHeaders());
-    }
-
-    /**
-     * Runs the middleware with a stubbed delegate
-     */
-    protected function runMiddleware()
-    {
-        $this->middleware->process($this->request, function () {
-            // no op
-        });
-    }
+	/**
+	 * @var HTTPRequest
+	 */
+	protected $request;
+
+	/**
+	 * @var InitialisationMiddleware
+	 */
+	protected $middleware;
+
+	protected $usesDatabase = true;
+
+	protected function setUp()
+	{
+		parent::setUp();
+
+		$this->request = new HTTPRequest('GET', '/');
+		$this->middleware = new InitialisationMiddleware();
+
+		Environment::setEnv('SS_OUTBOUND_PROXY', '');
+		Environment::setEnv('SS_OUTBOUND_PROXY_PORT', '');
+		putenv('NO_PROXY=');
+	}
+
+	public function testDoNotConfigureProxyIfNoEnvironmentVarsAreSet()
+	{
+		$this->runMiddleware();
+
+		$this->assertEmpty(
+			Environment::getEnv('http_proxy'),
+			'Proxy information is not set if no outbound proxy is configured'
+		);
+	}
+
+	public function testConfigureEgressProxyWhenVarsAreSet()
+	{
+		Environment::setEnv('SS_OUTBOUND_PROXY', 'http://example.com');
+		Environment::setEnv('SS_OUTBOUND_PROXY_PORT', '8024');
+
+		$this->runMiddleware();
+
+		$this->assertEquals(
+			'http://example.com:8024',
+			Environment::getEnv('http_proxy'),
+			'Proxy is configured with proxy and port'
+		);
+	}
+
+	public function testDoNotConfigureProxyDomainExclusionsWhenNoneAreDefined()
+	{
+		Config::modify()->remove(InitialisationMiddleware::class, 'egress_proxy_exclude_domains');
+
+		$this->runMiddleware();
+
+		$this->assertSame(
+			'',
+			Environment::getEnv('NO_PROXY'),
+			'No domain exclusions are set when none are defined'
+		);
+	}
+
+	public function testConfigureEgressProxyDomainExclusions()
+	{
+		Config::modify()->set(
+			InitialisationMiddleware::class,
+			'egress_proxy_exclude_domains',
+			'example.com'
+		);
+
+		putenv('NO_PROXY=foo.com,bar.com');
+		$this->runMiddleware();
+
+		$this->assertSame(
+			'foo.com,bar.com,example.com',
+			Environment::getEnv('NO_PROXY'),
+			'Domain exclusions are combined with existing values and configuration settings'
+		);
+	}
+
+	public function testSecurityHeadersAddedByDefault()
+	{
+		$response = $this->get('Security/login');
+		$this->assertArrayHasKey('x-xss-protection', $response->getHeaders());
+		$this->assertSame('1; mode=block', $response->getHeader('x-xss-protection'));
+	}
+
+	public function testXSSProtectionHeaderNotAdded()
+	{
+		Config::modify()->set(InitialisationMiddleware::class, 'xss_protection_enabled', false);
+		$response = $this->get('Security/login');
+		$this->assertArrayNotHasKey('x-xss-protection', $response->getHeaders());
+	}
+
+	public function testHstsNotAddedByDefault()
+	{
+		Config::modify()->remove(InitialisationMiddleware::class, 'strict_transport_security');
+		$response = $this->get('Security/login');
+		$this->assertArrayNotHasKey('strict-transport-security', $response->getHeaders());
+	}
+
+	public function testHstsAddedWhenConfigured()
+	{
+		Config::modify()->update(InitialisationMiddleware::class, 'strict_transport_security', 'max-age=1');
+		$response = $this->get('Security/login');
+		$this->assertArrayHasKey('strict-transport-security', $response->getHeaders());
+	}
+
+	/**
+	 * Runs the middleware with a stubbed delegate
+	 */
+	protected function runMiddleware()
+	{
+		$this->middleware->process($this->request, function () {
+			// no op
+		});
+	}
 }

src/Extension/CustomHtmlEditorField.php

Indentation +9 added lines, -9 removed lines

@@ -13,13 +13,13 @@
 class CustomHtmlEditorFieldToolbar extends Extension
 {
 
-    /**
-     * @param Form $form
-     * @return void
-     */
-    public function updateMediaForm(Form $form)
-    {
-        Requirements::add_i18n_javascript('cwp/cwp-core:javascript/lang');
-        Requirements::javascript('cwp/cwp-core:javascript/CustomHtmlEditorFieldToolbar.js');
-    }
+	/**
+	 * @param Form $form
+	 * @return void
+	 */
+	public function updateMediaForm(Form $form)
+	{
+		Requirements::add_i18n_javascript('cwp/cwp-core:javascript/lang');
+		Requirements::javascript('cwp/cwp-core:javascript/CustomHtmlEditorFieldToolbar.js');
+	}
 }

src/Extension/CwpHtmlEditorConfig.php

Indentation +14 added lines, -14 removed lines

@@ -7,20 +7,20 @@
 class CwpHtmlEditorConfig extends DataExtension
 {
 
-    /**
-     * @return string
-     *
-     * Override the default HtmlEditorConfig from 'cms' to 'cwp' defined in cwp-core/_config.php
-     * However if the group has a custom editor configuration set, use that instead.
-     */
-    public function getHtmlEditorConfig()
-    {
-        $originalConfig = $this->owner->getField("HtmlEditorConfig");
+	/**
+	 * @return string
+	 *
+	 * Override the default HtmlEditorConfig from 'cms' to 'cwp' defined in cwp-core/_config.php
+	 * However if the group has a custom editor configuration set, use that instead.
+	 */
+	public function getHtmlEditorConfig()
+	{
+		$originalConfig = $this->owner->getField("HtmlEditorConfig");
 
-        if ($originalConfig) {
-            return $originalConfig;
-        }
+		if ($originalConfig) {
+			return $originalConfig;
+		}
 
-        return 'cwp';
-    }
+		return 'cwp';
+	}
 }

src/Extension/LoginAttemptNotifications.php

Spacing +3 added lines, -3 removed lines

@@ -53,12 +53,12 @@ 
                 $failures = $meantimeLoginAttempts->filter(['Status' => 'Failure'])->count();
                 $IPs = array_unique($meantimeLoginAttempts->column('IP'));
 
-                if ($attempts == 1) {
+                if ($attempts==1) {
                     $statusString = $failures ? "a failed" : "a successful";
                     $message = "In the last $elapsed $statusString login attempt to your account was "
                         . "registered. The attempt was made from ${IPs[0]}. ";
                 } else {
-                    if ($failures == $attempts) {
+                    if ($failures==$attempts) {
                         $statusString = $failures ? "failed" : "successful";
                         $message = "In the last $elapsed $attempts $statusString login "
                             . "attempts to your account were registered. ";
@@ -85,7 +85,7 @@ 
                 $date = $lastLoginAttempt->Created;
                 $message = "Last login attempt to your account was on $lastLoginAttempt->Created "
                     . "from $lastLoginAttempt->IP";
-                $message .= $lastLoginAttempt->Status == 'Failure' ? " and was successful." : "and has failed.";
+                $message .= $lastLoginAttempt->Status=='Failure' ? " and was successful." : "and has failed.";
             }
         }
 

Indentation +66 added lines, -66 removed lines

@@ -19,81 +19,81 @@
 class LoginAttemptNotifications extends Extension
 {
 
-    /**
-     *
-     * @return mixed null
-     */
-    public function init()
-    {
+	/**
+	 *
+	 * @return mixed null
+	 */
+	public function init()
+	{
 
-        // Exclude default admin.
-        $member = Security::getCurrentUser();
-        if (!$member || !$member->ID) {
-            return;
-        }
+		// Exclude default admin.
+		$member = Security::getCurrentUser();
+		if (!$member || !$member->ID) {
+			return;
+		}
 
-        $message = null;
-        $session = $this->owner->getRequest()->getSession();
+		$message = null;
+		$session = $this->owner->getRequest()->getSession();
 
-        Requirements::javascript('cwp/cwp-core:javascript/LoginAttemptNotifications.js');
-        $sessionLastVisited = $session->get('LoginAttemptNotifications.SessionLastVisited');
-        if ($sessionLastVisited) {
-            // Session already in progress. Show all attempts since the session was last visited.
+		Requirements::javascript('cwp/cwp-core:javascript/LoginAttemptNotifications.js');
+		$sessionLastVisited = $session->get('LoginAttemptNotifications.SessionLastVisited');
+		if ($sessionLastVisited) {
+			// Session already in progress. Show all attempts since the session was last visited.
 
-            $meantimeLoginAttempts = LoginAttempt::get()->filter([
-                'MemberID' => $member->ID,
-                'Created:GreaterThan' => $sessionLastVisited
-            ]);
+			$meantimeLoginAttempts = LoginAttempt::get()->filter([
+				'MemberID' => $member->ID,
+				'Created:GreaterThan' => $sessionLastVisited
+			]);
 
-            $attempts = $meantimeLoginAttempts->count();
-            if ($attempts) {
-                $lastVisitedObj = DBDatetime::create();
-                $lastVisitedObj->setValue($sessionLastVisited);
-                $elapsed = $lastVisitedObj->TimeDiff();
-                $failures = $meantimeLoginAttempts->filter(['Status' => 'Failure'])->count();
-                $IPs = array_unique($meantimeLoginAttempts->column('IP'));
+			$attempts = $meantimeLoginAttempts->count();
+			if ($attempts) {
+				$lastVisitedObj = DBDatetime::create();
+				$lastVisitedObj->setValue($sessionLastVisited);
+				$elapsed = $lastVisitedObj->TimeDiff();
+				$failures = $meantimeLoginAttempts->filter(['Status' => 'Failure'])->count();
+				$IPs = array_unique($meantimeLoginAttempts->column('IP'));
 
-                if ($attempts == 1) {
-                    $statusString = $failures ? "a failed" : "a successful";
-                    $message = "In the last $elapsed $statusString login attempt to your account was "
-                        . "registered. The attempt was made from ${IPs[0]}. ";
-                } else {
-                    if ($failures == $attempts) {
-                        $statusString = $failures ? "failed" : "successful";
-                        $message = "In the last $elapsed $attempts $statusString login "
-                            . "attempts to your account were registered. ";
-                    } else {
-                        $message = "In the last $elapsed $attempts login attempts to your "
-                            . "account were registered, of which $failures failed. ";
-                    }
+				if ($attempts == 1) {
+					$statusString = $failures ? "a failed" : "a successful";
+					$message = "In the last $elapsed $statusString login attempt to your account was "
+						. "registered. The attempt was made from ${IPs[0]}. ";
+				} else {
+					if ($failures == $attempts) {
+						$statusString = $failures ? "failed" : "successful";
+						$message = "In the last $elapsed $attempts $statusString login "
+							. "attempts to your account were registered. ";
+					} else {
+						$message = "In the last $elapsed $attempts login attempts to your "
+							. "account were registered, of which $failures failed. ";
+					}
 
-                    $message .= "The attempts were from " . implode(', ', $IPs) . '. ';
+					$message .= "The attempts were from " . implode(', ', $IPs) . '. ';
 
-                    // TODO: add this call to action in a way that doesn't break out of the availabel space. Fix CSS?
-                    // $message .= "If you suspect somebody else might be trying to access
-                    //  . "your account, please contact support.";
-                }
-            }
-        } else {
-            // New session - show last login attempt.
-            // TODO: this currently does NOT surface to the frontend in any way.
-            $lastLoginAttempt = LoginAttempt::get()->filter([
-                        'MemberID' => $member->ID
-            ])->sort('Created DESC')->First();
+					// TODO: add this call to action in a way that doesn't break out of the availabel space. Fix CSS?
+					// $message .= "If you suspect somebody else might be trying to access
+					//  . "your account, please contact support.";
+				}
+			}
+		} else {
+			// New session - show last login attempt.
+			// TODO: this currently does NOT surface to the frontend in any way.
+			$lastLoginAttempt = LoginAttempt::get()->filter([
+						'MemberID' => $member->ID
+			])->sort('Created DESC')->First();
 
-            if ($lastLoginAttempt) {
-                $date = $lastLoginAttempt->Created;
-                $message = "Last login attempt to your account was on $lastLoginAttempt->Created "
-                    . "from $lastLoginAttempt->IP";
-                $message .= $lastLoginAttempt->Status == 'Failure' ? " and was successful." : "and has failed.";
-            }
-        }
+			if ($lastLoginAttempt) {
+				$date = $lastLoginAttempt->Created;
+				$message = "Last login attempt to your account was on $lastLoginAttempt->Created "
+					. "from $lastLoginAttempt->IP";
+				$message .= $lastLoginAttempt->Status == 'Failure' ? " and was successful." : "and has failed.";
+			}
+		}
 
-        $session->set(
-            'LoginAttemptNotifications.SessionLastVisited',
-            DBDatetime::now()->Format(DBDatetime::ISO_DATETIME)
-        );
+		$session->set(
+			'LoginAttemptNotifications.SessionLastVisited',
+			DBDatetime::now()->Format(DBDatetime::ISO_DATETIME)
+		);
 
-        $this->owner->getResponse()->addHeader('X-LoginAttemptNotifications', $message);
-    }
+		$this->owner->getResponse()->addHeader('X-LoginAttemptNotifications', $message);
+	}
 }