Details
-
Type:
Improvement
-
Status:
Open
-
Priority:
Trivial
-
Resolution: Unresolved
-
Affects Version/s: 1.2.3, 1.2.4
-
Fix Version/s: None
-
Component/s: Relations
-
Labels:None
Description
$record instanceof Doctrine_Record;
$collection instanceof Doctrine_Collection;
Using the $record object I can find out if it has been modified : $record->isModified().
I can also have this check all the relations also by passing true : $record->isModified(true);
However, the $collection object only support the first level and not the relations of its member records,
$collection->isModified() returns false if one of the entry has one its relations modified (as it should).
The improvement would be to allow this method to accept true, like it's $record counterpart, which would return true instead.
See attached patch.
Activity
Christian Roy
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Summary | Doctrine_Collection::isModified() does not support deep like Doctrine_Record but probably should | [PATCH] Doctrine_Collection::isModified() does not support deep like Doctrine_Record but probably should |
This list may be incomplete, as errors occurred whilst retrieving source from linked applications:
- Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DC-1032, expand=changesets[-21:-1].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)