[TASK] Un-deprecation log entries for ext:belog 57/43457/4
authorAnja Leichsenring <aleichsenring@ab-softlab.de>
Mon, 21 Sep 2015 14:46:44 +0000 (16:46 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Wed, 23 Sep 2015 11:24:10 +0000 (13:24 +0200)
The usage of __wakeup() in Constraint Model
(by extending AbstractDomainModel) causes deprecation log entries.

The core must not cause any deprecation log entries, so as temporary
fix an empty __wakeup() function was added. The Constraint Model is
subject to refactoring.

With #62925 and the properties manualDateStart and manualDateStop
have been marked as deprecated with the assumption they are not in
use. But they are, and so they are restored as fully functional
properties.

Change-Id: Iff23f10617700751991026706f7d6284de93e469
Releases: master
Related: #62925
Resolves: #70000
Reviewed-on: http://review.typo3.org/43457
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/belog/Classes/Domain/Model/Constraint.php
typo3/sysext/core/Documentation/Changelog/7.1/Deprecation-62925-ContraintModel.rst [deleted file]

index aa1664c..4fe6445 100644 (file)
@@ -13,7 +13,6 @@ namespace TYPO3\CMS\Belog\Domain\Model;
  *
  * The TYPO3 project - inspiring people to share!
  */
-use TYPO3\CMS\Core\Utility\GeneralUtility;
 
 /**
  * Constraints for log entries
@@ -79,14 +78,12 @@ class Constraint extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        /**
         * Manual date start
         * @var \DateTime
-        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        protected $manualDateStart = NULL;
 
        /**
         * Manual date stop
         * @var \DateTime
-        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        protected $manualDateStop = NULL;
 
@@ -119,6 +116,16 @@ class Constraint extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
        }
 
        /**
+        * added to prevent the deprecation message
+        * in Extbase\DomainObject\AbstractDomainObject
+        *
+        * @todo the constraints model needs another way of storing
+        * persisted search data than serialisation
+        */
+       public function __wakeup() {
+       }
+
+       /**
         * Set user
         *
         * @param string $user
@@ -332,10 +339,8 @@ class Constraint extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         *
         * @param \DateTime $manualDateStart
         * @return void
-        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        public function setManualDateStart(\DateTime $manualDateStart = NULL) {
-               GeneralUtility::logDeprecatedFunction();
                $this->manualDateStart = $manualDateStart;
        }
 
@@ -343,10 +348,8 @@ class Constraint extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         * Get manual date start
         *
         * @return \DateTime
-        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        public function getManualDateStart() {
-               GeneralUtility::logDeprecatedFunction();
                return $this->manualDateStart;
        }
 
@@ -355,10 +358,8 @@ class Constraint extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         *
         * @param \DateTime $manualDateStop
         * @return void
-        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        public function setManualDateStop(\DateTime $manualDateStop = NULL) {
-               GeneralUtility::logDeprecatedFunction();
                $this->manualDateStop = $manualDateStop;
        }
 
@@ -366,10 +367,8 @@ class Constraint extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
         * Get manual date stop
         *
         * @return \DateTime
-        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        public function getManualDateStop() {
-               GeneralUtility::logDeprecatedFunction();
                return $this->manualDateStop;
        }
 }
diff --git a/typo3/sysext/core/Documentation/Changelog/7.1/Deprecation-62925-ContraintModel.rst b/typo3/sysext/core/Documentation/Changelog/7.1/Deprecation-62925-ContraintModel.rst
deleted file mode 100644 (file)
index 3b91dbe..0000000
+++ /dev/null
@@ -1,29 +0,0 @@
-====================================================================
-Deprecation: #63912 - Deprecate unused methods from Constraint model
-====================================================================
-
-Description
-===========
-
-``TYPO3\CMS\Belog\Domain\Model\Constraint::setManualDateStart()`` has been marked as deprecated.
-``TYPO3\CMS\Belog\Domain\Model\Constraint::getManualDateStart()`` has been marked as deprecated.
-``TYPO3\CMS\Belog\Domain\Model\Constraint::setManualDateStop()`` has been marked as deprecated.
-``TYPO3\CMS\Belog\Domain\Model\Constraint::getManualDateStop()`` has been marked as deprecated.
-
-
-Impact
-======
-
-Using ``setManualDateStart()``, ``getManualDateStart()``, ``setManualDateStop()`` and ``getManualDateStop()`` of Constraint model class will trigger a deprecation log message.
-
-
-Affected installations
-======================
-
-Instances which use custom form elements, which make use of ``setManualDateStart()``, ``getManualDateStart()``, ``setManualDateStop()`` and ``getManualDateStop()``.
-
-
-Migration
-=========
-
-No migration is possible for all methods, since those methods were unused for a long time already and should not be needed at all.