Merge pull request #1313 from magento-engcom/2.1-develop-prs

Public Pull Requests

#10211

Author: Oleksii Korshenko

lib/internal/Magento/Framework/Model/AbstractModel.php

@@ -523,7 +523,9 @@ public function getCollection()
      * @param integer $modelId
      * @param null|string $field
      * @return $this
-     * @deprecated
+     * @deprecated because entities must not be responsible for their own loading.
+     * Service contracts should persist entities. Use resource model "load" or collections to implement
+     * service contract model loading operations.
      */
     public function load($modelId, $field = null)
     {
@@ -624,7 +626,9 @@ public function setHasDataChanges($flag)
      *
      * @return $this
      * @throws \Exception
-     * @deprecated
+     * @deprecated because entities must not be responsible for their own persistence.
+     * Service contracts should persist entities. Use resource model "save" to implement
+     * service contract persistence operations.
      */
     public function save()
     {
@@ -809,7 +813,9 @@ public function afterSave()
      *
      * @return $this
      * @throws \Exception
-     * @deprecated
+     * @deprecated because entities must not be responsible for their own deletion.
+     * Service contracts should delete entities. Use resource model "delete" method to implement
+     * service contract persistence operations.
      */
     public function delete()
     {