symfony/symfony — pull request #8262 by unframework (2013-06-12 18:40:51+00:00)
Title: Explicitly disallowing serialization of objects as controller attributes...

... in non-inline fragment renderer strategies.

Right now, the fragment renderer accepts a controller reference with a set of attributes/parameters. When rendering inline, any attributes that are objects get passed directly to the subrequest. When rendering as a URL (e.g. for ESI or HInclude), those objects are serialized by the http_build_query function. This change makes URL rendering steps explicitly reject any object values instead of letting http_build_query serialize them.

Why? Consider the case where we are passing a DB entity as controller param to be rendered using the ESI or HInclude strategy. Either way, the developer has to pass an ID instead of DB entity itself for this to work - http_build_query lacks intelligence to automatically convert the DB entity into just its ID value.

This change does not add the intelligence to convert entities into IDs, but still explicitly reminds the developer to not pass the object directly when non-inline strategies are used, thus reducing silent bug-causing behaviour.