[!!!][BUGFIX] Remove property isOnline from frontenduser model
authorMarc Bastian Heinrichs <typo3@mbh-software.de>
Sat, 17 Nov 2012 10:00:09 +0000 (11:00 +0100)
committerMarkus Günther <mail@markus-guenther.de>
Sat, 17 Nov 2012 13:02:20 +0000 (14:02 +0100)
Because there is no TCA definition for column is_online the
property would not get filled with data from database in mappping
process. Remove it to avoid confusion.

Fixes: #11024
Releases: 6.0

Change-Id: Ic62d2d918ac3bd0307e8ff1b6432769f12d1bb4e
Reviewed-on: http://review.typo3.org/16510
Reviewed-by: Ingo Pfennigstorf
Tested-by: Ingo Pfennigstorf
Reviewed-by: Markus Günther
Tested-by: Markus Günther
typo3/sysext/extbase/Classes/Domain/Model/FrontendUser.php
typo3/sysext/extbase/Tests/Unit/Domain/Model/FrontendUserTest.php

index 24b6c9d..02b07cd 100644 (file)
@@ -136,11 +136,6 @@ class FrontendUser extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        protected $lastlogin = NULL;
 
        /**
-        * @var \DateTime|NULL
-        */
-       protected $isOnline = NULL;
-
-       /**
         * Constructs a new Front-End User
         *
         * @api
@@ -597,27 +592,6 @@ class FrontendUser extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
                return $this->lastlogin;
        }
 
-       /**
-        * Sets the isOnline value
-        *
-        * @param \DateTime $isOnline
-        * @return void
-        * @api
-        */
-       public function setIsOnline($isOnline) {
-               $this->isOnline = $isOnline;
-       }
-
-       /**
-        * Returns the isOnline value
-        *
-        * @return \DateTime
-        * @api
-        */
-       public function getIsOnline() {
-               return $this->isOnline;
-       }
-
 }
 
 
index a35e30a..e8b2f88 100644 (file)
@@ -383,22 +383,6 @@ class FrontendUserTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
                $this->assertSame($date, $this->fixture->getLastlogin());
        }
 
-       /**
-        * @test
-        */
-       public function getIsOnlineInitiallyReturnsNull() {
-               $this->assertNull($this->fixture->getIsOnline());
-       }
-
-       /**
-        * @test
-        */
-       public function setIsOnlineSetsIsOnline() {
-               $isOnline = new \DateTime();
-               $this->fixture->setIsOnline($isOnline);
-               $this->assertSame($isOnline, $this->fixture->getIsOnline());
-       }
-
 }