Merge pull request laravel#8360 from sebastian-berc/touchOwners-on-many-to-many

taylorotwell · taylorotwell · commit 5ce1f646c71d · 2015-04-12T14:56:29.000-05:00
[5.0] Fix touchOwners() on many to many relationship.
diff --git a/src/Illuminate/Database/Eloquent/Model.php b/src/Illuminate/Database/Eloquent/Model.php
@@ -1622,10 +1622,17 @@ public function touchOwners()
 		{
 			$this->$relation()->touch();
 
-			if ( ! is_null($this->$relation))
+			if ($this->$relation instanceof Model)
 			{
 				$this->$relation->touchOwners();
 			}
+			elseif ($this->$relation instanceof Collection)
+			{
+				$this->$relation->each(function (Model $relation)
+				{
+					$relation->touchOwners();
+				});
+			}
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1622,10 +1622,17 @@ public function touchOwners()`
`1622`	`1622`	`{`
`1623`	`1623`	`$this->$relation()->touch();`
`1624`	`1624`
`1625`		`- if ( ! is_null($this->$relation))`
	`1625`	`+ if ($this->$relation instanceof Model)`
`1626`	`1626`	`{`
`1627`	`1627`	`$this->$relation->touchOwners();`
`1628`	`1628`	`}`
	`1629`	`+ elseif ($this->$relation instanceof Collection)`
	`1630`	`+ {`
	`1631`	`+ $this->$relation->each(function (Model $relation)`
	`1632`	`+ {`
	`1633`	`+ $relation->touchOwners();`
	`1634`	`+ });`
	`1635`	`+ }`
`1629`	`1636`	`}`
`1630`	`1637`	`}`
`1631`	`1638`