Bug #26862

Doctrine\Query is not serializable

Added by Sebastian Kurfuerst about 4 years ago. Updated about 4 years ago.

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

related to TYPO3.Flow - Bug #28995: Deferred Loading does not work with Doctrine QueryResult Resolved 2011-08-16

Associated revisions

Revision 86877174
Added by Karsten Dambekalns about 4 years ago

[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

Also available in: Atom PDF