Skip to content

Commit

Permalink
Applying patch from Daniel Feinberg for Containable
Browse files Browse the repository at this point in the history
Removes __backContainableAssociation as its not needed and causes
more problems than it solves.
Fixes #1762
  • Loading branch information
markstory committed Sep 8, 2011
1 parent 95b275d commit ac0b2b8
Showing 1 changed file with 0 additions and 20 deletions.
20 changes: 0 additions & 20 deletions cake/libs/model/behaviors/containable.php
Expand Up @@ -147,8 +147,6 @@ function beforeFind(&$Model, $query) {
if (!empty($unbind)) {
if (!$reset && empty($instance->__backOriginalAssociation)) {
$instance->__backOriginalAssociation = $backupBindings;
} else if ($reset && empty($instance->__backContainableAssociation)) {
$instance->__backContainableAssociation = $backupBindings;
}
$instance->unbindModel(array($type => $unbind), $reset);
}
Expand Down Expand Up @@ -219,24 +217,6 @@ function beforeFind(&$Model, $query) {
return $query;
}

/**
* Resets original associations on models that may have receive multiple,
* subsequent unbindings.
*
* @param object $Model Model on which we are resetting
* @param array $results Results of the find operation
* @param bool $primary true if this is the primary model that issued the find operation, false otherwise
* @access public
*/
function afterFind(&$Model, $results, $primary) {
if (!empty($Model->__backContainableAssociation)) {
foreach ($Model->__backContainableAssociation as $relation => $bindings) {
$Model->{$relation} = $bindings;
unset($Model->__backContainableAssociation);
}
}
}

/**
* Unbinds all relations from a model except the specified ones. Calling this function without
* parameters unbinds all related models.
Expand Down

0 comments on commit ac0b2b8

Please sign in to comment.