Bug #26862
Doctrine\Query is not serializable
Status: | Resolved | Start date: | 2011-05-17 | |
---|---|---|---|---|
Priority: | Must have | Due date: | 2011-05-18 | |
Assigned To: | Karsten Dambekalns | % Done: | 100% |
|
Category: | Persistence | |||
Target version: | TYPO3 Flow Base Distribution - 1.0 beta 1 | |||
PHP Version: | Complexity: | |||
Has patch: | Affected Flow version: |
Description
For widgets, it is crucial that Query objects are serializable and deserializeable,
so one should be able to do:
$newQuery = deserialize(serialize($query))
$newQuery->execute() (or something else)
Maybe a functional test for this would be nice?
One thing I already discovered further is that the properties in QueryResult should be protected, not private.
Greets,
Sebastian
Related issues
Associated revisions
[BUGFIX] Make Doctrine\Query serializable
Makes Doctrine\Query serializable and cleans up some code a little.
Change-Id: Ifd8ed7a46d9701c406e186e2d3b3d6b1f39596e0
Fixes: #26862
History
#1 Updated by Mr. Hudson about 4 years ago
Patch set 1 of change Ifd8ed7a46d9701c406e186e2d3b3d6b1f39596e0 has been pushed to the review server.
It is available at http://review.typo3.org/2400
#2 Updated by Karsten Dambekalns about 4 years ago
- Status changed from Accepted to Under Review
#3 Updated by Mr. Hudson about 4 years ago
Patch set 2 of change Ifd8ed7a46d9701c406e186e2d3b3d6b1f39596e0 has been pushed to the review server.
It is available at http://review.typo3.org/2400
#4 Updated by Mr. Hudson about 4 years ago
Patch set 3 of change Ifd8ed7a46d9701c406e186e2d3b3d6b1f39596e0 has been pushed to the review server.
It is available at http://review.typo3.org/2400
#5 Updated by Karsten Dambekalns about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 868771749bb2187f73cc9d498afeacab2c96fa5a.