Commit graph

21 commits

Author SHA1 Message Date
Charles Sanquer e6afb74564 fix Sensio FrameworkExtraBundle v3.0 ParamConverterInterface compatibiltiy 2013-12-05 11:14:20 +01:00
Chris c2a3c3082d Check if there's a _route request attribute before getting it
Apparently Symfony doesn't add a _route parameter to subrequests (esi render call) so PropelParamConverter throw an error accessing getOption method on not-existent object on line 98.
2013-09-25 12:09:42 +02:00
Toni Uebernickel a9fcc24807 read route options in PropelParamConverter 2013-09-18 15:33:54 +02:00
Tomasz Wójcik 14322ef604 Update PropelParamConverter.php 2013-06-03 22:53:47 +02:00
jaugustin 4ec0391bff add tests for paramConverter, fix issue with never set 2013-01-02 19:03:11 +01:00
kphoen df64c21d1c Fix: use getFirst() instead of first() 2012-11-24 09:45:12 +00:00
kphoen aedd75f16d Fix: logic exception thrown instead of a NotFoundHttpException 2012-11-23 19:53:26 +00:00
William DURAND 53cba04c23 Fix CS 2012-11-05 14:00:31 +01:00
kphoen bc15b92757 Fix: using the "with" option in ParamConverters triggered a notice
The following notice was triggered:

  Runtime Notice: Only variables should be passed by reference in
  /home/vagrant/www/project/vendor/propel/propel-bundle/Propel/PropelBundle/Request/ParamConverter/PropelParamConverter.php
  line 193
2012-10-31 19:33:12 +01:00
Adel 4219e706d4 RUN PHP-CS PSR-2 2012-07-19 17:40:35 +01:00
William DURAND 6970913db9 Fixed CS 2012-05-24 16:36:46 +02:00
jaugustin 1b5b18237c [WIP] rebase, refactor code, add tests, add doc 2012-05-24 16:36:46 +02:00
jaugustin e1f18fbdf2 [PropelParamConverter] add comment 2012-05-24 16:36:46 +02:00
jaugustin 6117614bec [ParamConverter] add 'with' option to hydrate related object 2012-05-24 16:36:46 +02:00
jaugustin b03f6a58b4 Add the mapping option for the PropelParamConverter,
enhance the pk handling to use the real PK name and not only 'id',
fix new sensioFrameworkBundle 2.1 requirement:
ParamConverterInterface::apply() method now must return a Boolean value indicating if a conversion was done
2012-05-23 00:08:13 +02:00
kphoen b54cf4ab3f Modified the PropelParamConverter to allow CamelCased parameters
Currently, parameters like "author_slug" are translated by the param
converter to a "->filterByAuthor_slug()" call. I changed this behavior
and they are now translated to "->filterByAuthorSlug()".
2012-05-21 15:23:34 +02:00
jaugustin b6820ccef7 typo, fix CS 2012-05-07 23:31:48 +02:00
Jeremie Augustin a3871ccc74 fix param converter to handle correctly isOptional parameter 2012-05-07 18:43:27 +02:00
jaugustin 75bef0beb6 [PropelParamConverter] add exclude option for Pk and fields, with tests and docs 2011-10-07 12:14:28 +02:00
jaugustin f53e2332c5 add test for paramconverter, fix issue with paramconverter findOneByXXX 2011-09-23 11:07:49 +02:00
jaugustin 537799b3c2 add propel paramConverter, to be used with Sensio Framework Extra 2011-09-22 16:11:14 +02:00