MAGETWO-71223: Deprecate getResource and getCollection

cpartica · cpartica · commit 0e583d9acb95 · 2017-08-09T10:03:28.000-05:00
- deprecate from AbstractModel
diff --git a/lib/internal/Magento/Framework/Model/AbstractModel.php b/lib/internal/Magento/Framework/Model/AbstractModel.php
@@ -492,6 +492,7 @@ public function getResourceName()
      * @TODO MAGETWO-23541: Incorrect dependencies between Model\AbstractModel and Data\Collection\Db from Framework
      * @throws \Magento\Framework\Exception\LocalizedException
      * @return \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection
+     * @deprecated 100.2.0 because collections should be used directly via factory
      */
     public function getResourceCollection()
     {
@@ -512,6 +513,7 @@ public function getResourceCollection()
      *
      * @TODO MAGETWO-23541: Incorrect dependencies between Model\AbstractModel and Data\Collection\Db from Framework
      * @return \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection
+     * @deprecated 100.2.0 because collections should be used directly via factory
      */
     public function getCollection()
     {
@@ -882,6 +884,7 @@ public function afterDeleteCommit()
      * Retrieve model resource
      *
      * @return \Magento\Framework\Model\ResourceModel\Db\AbstractDb
+     * @deprecated 100.2.0 because resource models should be used directly
      */
     public function getResource()
     {

Original file line number	Diff line number	Diff line change
`@@ -492,6 +492,7 @@ public function getResourceName()`
`492`	`492`	`* @TODO MAGETWO-23541: Incorrect dependencies between Model\AbstractModel and Data\Collection\Db from Framework`
`493`	`493`	`* @throws \Magento\Framework\Exception\LocalizedException`
`494`	`494`	`* @return \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection`
	`495`	`+ * @deprecated 100.2.0 because collections should be used directly via factory`
`495`	`496`	`*/`
`496`	`497`	`public function getResourceCollection()`
`497`	`498`	`{`
`@@ -512,6 +513,7 @@ public function getResourceCollection()`
`512`	`513`	`*`
`513`	`514`	`* @TODO MAGETWO-23541: Incorrect dependencies between Model\AbstractModel and Data\Collection\Db from Framework`
`514`	`515`	`* @return \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection`
	`516`	`+ * @deprecated 100.2.0 because collections should be used directly via factory`
`515`	`517`	`*/`
`516`	`518`	`public function getCollection()`
`517`	`519`	`{`
`@@ -882,6 +884,7 @@ public function afterDeleteCommit()`
`882`	`884`	`* Retrieve model resource`
`883`	`885`	`*`
`884`	`886`	`* @return \Magento\Framework\Model\ResourceModel\Db\AbstractDb`
	`887`	`+ * @deprecated 100.2.0 because resource models should be used directly`
`885`	`888`	`*/`
`886`	`889`	`public function getResource()`
`887`	`890`	`{`