digitalkaoz
9441c46bb6
[DependencyInjection] PhpDumper, fixes #2730
2012-01-06 21:20:08 +01:00
Eric Clemmons
99011ca9c9
Added tests for ParameterBag parameters with spaces
2011-12-27 12:48:49 -08:00
Christophe Coevoet
2c3e9adcd1
[DependencyInjection] Made the reference case insensitive
...
The container is case insensitive so using capital letters in a reference
made it fail in some cases when checking the dependencies.
Closes #2807
2011-12-08 16:30:50 +01:00
Dariusz Górecki
769c17bb95
Throw exceptions in case someone forgot to set method name in call.
...
Bug fix: yes
Feature add: no
Symfony2 tests pass: yes
Symfony2 tests added: yes
In general without this exception generated by php dumper container class, will cause PHP fatal error, bacause method call will look like this: `$instance->(/* arguments*/);`.
2011-11-30 15:59:17 +01:00
Fabien Potencier
851eb73778
removed unused use statements
2011-10-29 11:56:30 +02:00
Jeremy Mikola
80f0b980ba
[DependencyInjection] Fix DefinitionDecorator::getArgument() for replacements
...
While Definition::getArgument() could be used to fetch replaced values, it relied upon bad comparison logic (e.g. "index_1" > 1). Additionally, storing original arguments and replacements in the same array makes Definition::getArguments()'s bounds check unreliable. A single argument and its replacement would count twice, allowing getArgument(2) to pass the bounds check and result in an array index error.
With this new method, fetching of replacement arguments is more straightforward and bounds checking functions as it should.
2011-10-27 18:03:51 -04:00
Jeremy Mikola
4bbb685557
[DependencyInjection] Test Definition and DefinitionDecorator exceptions
2011-10-27 17:59:47 -04:00
Vadim Tyukov
808088a3ca
added the ability to use dot and single quotes in the keys and values
2011-10-24 12:45:36 +03:00
Fabien Potencier
49c585ebd2
Revert "merged branch stealth35/ini_bool (PR #2235 )"
...
This reverts commit 363057b181
, reversing
changes made to 545cd4cd63
.
2011-09-25 11:33:22 +02:00
Fabien Potencier
fa13469bba
Revert "[DependencyInjection] fixed array support for the ini loader"
...
This reverts commit e0ace8eaee
.
2011-09-25 11:33:07 +02:00
Fabien Potencier
27fc06785b
Revert "[DepedencyInjection] fixed unit tests"
...
This reverts commit b8e5a155e4
.
2011-09-25 11:33:00 +02:00
Fabien Potencier
b8e5a155e4
[DepedencyInjection] fixed unit tests
2011-09-23 11:59:37 +02:00
Fabien Potencier
e0ace8eaee
[DependencyInjection] fixed array support for the ini loader
2011-09-23 11:45:30 +02:00
stealth35
11c441289a
[DependencyInjection] fix 2219 IniFileLoader accept Boolean
2011-09-21 22:14:12 +02:00
H. Westphal
3e2f1a4b13
Removed executable bits from all php files
2011-09-09 15:16:17 +02:00
Johannes Schmitt
2c224ce42b
improves the exception message, and removes unnecessary constraint to only allow strings inside strings
2011-07-16 16:10:11 +02:00
Fabien Potencier
e718a51b59
[DependencyInjection] fixed un-detected circular references involving aliases
2011-07-11 18:25:40 +02:00
Fabien Potencier
c3bb214e94
[DependencyInjection] removed injection for protected and private properties
2011-06-27 08:28:54 +02:00
stloyd
0d3fb6f26c
Missing quote in IniFileLoader exception
2011-06-24 16:28:15 +02:00
Victor Berchet
d89298de7c
[DependencyInjection] Fix a typo
2011-06-16 10:37:09 +02:00
Pascal Borreli
f5eaa404c7
[Tests] Fixed typos
2011-06-12 20:52:53 +00:00
Fabien Potencier
17cd08dc6c
fixed CS
2011-06-08 19:56:59 +02:00
Fabien Potencier
65200aa86a
added missing license headers
2011-05-31 10:57:06 +02:00
Arnout Boks
a0397f99f5
[DependencyInjection] Fixed bug where anonymous services from two different xml-files (with the same basename) could collide
2011-05-28 10:29:32 +02:00
Fabien Potencier
3bdb7c2b57
[DependencyInjection] fixed regression when a parameter contains an escaped %
...
Notice that this is still not perfect, but I don't see how to fix the remaining
potential problems.
2011-05-26 12:47:50 +02:00
Fabien Potencier
462f222319
[DependencyInjection] changed \LogicException to RuntimeException to be more consistent
2011-05-25 09:59:38 +02:00
Fabien Potencier
2438a73c7b
[DependencyInjection] added a check for circular references in parameter definitions
2011-05-25 09:57:38 +02:00
Fabien Potencier
456eb53eb8
[DependencyInjection] renamed CircularReferenceException to ServiceCircularReferenceException
2011-05-25 09:48:49 +02:00
Fabien Potencier
6c409cac84
[DependencyInjection] refactored parameter bag parameter replacements (removed duplicated code)
2011-05-25 09:35:51 +02:00
everzet
98b0bdcb15
[DependencyInjection] added failing private service user test-case
2011-05-23 22:38:23 +03:00
Fabien Potencier
0168241014
[DependencyInjection] renamed NonExistentParameterException and NonExistentServiceException to ParameterNotFoundException and ServiceNotFoundException
2011-05-17 16:26:08 +02:00
Christophe Coevoet
c2f074b7d5
[DependencyInjection] Fixed a property visibility in GraphvizDumper and removed a useless instanciation in the test
2011-05-13 12:18:09 +02:00
Fabien Potencier
175f944f93
[DependencyInjection] added NonExistentParameterException that indicates where a non-existent parameter is being used
2011-04-26 12:36:25 +02:00
Lukas Kahwe Smith
d9491a743e
removed support for interface injection as well as all relevant tests
2011-04-23 11:42:31 +02:00
Fabien Potencier
8b2b8e16dc
Merge remote branch 'schmittjoh/referenceValidation'
...
* schmittjoh/referenceValidation:
[DependencyInjection] also check references of inlined services
[DependencyInjection] adds emulation of "exception-on-invalid-reference" behavior
2011-04-21 22:44:20 +02:00
Kris Wallsmith
cdf706d357
[DependencyInjection] renamed Definition::setArgument() as replaceArgument() to be more specific
2011-04-20 04:47:44 -07:00
Fabien Potencier
95d4bcc5db
fixed unit tests (broken by previous commit)
2011-04-19 14:10:41 +02:00
Brikou CARRE
e898445b94
removed empty lines/trailing spaces
2011-04-15 21:12:02 +02:00
Johannes Schmitt
fd5caa9546
[DependencyInjection] also check references of inlined services
2011-04-15 08:47:28 +02:00
Johannes Schmitt
6d7a9d752d
[DependencyInjection] adds emulation of "exception-on-invalid-reference" behavior
...
This pass requires that all of references are valid at the end of
the compilation process.
2011-04-15 00:55:18 +02:00
Lukas Kahwe Smith
0e572c241f
updated tests (aka removed exceptions handling)
2011-04-13 11:36:42 +02:00
Ryan Weaver
65ac5ec7c0
[DependencyInjection] Fixing a bug where "ignore_errors" doesn't work in YAML and XML
...
Tests added, the arguments were simply mismatched.
2011-04-01 18:59:45 -05:00
Jan Prieser
67aaf5ce91
Fixed PhpDumper isSimpleService and changed test fixtures
2011-03-22 17:01:03 +01:00
Ray
e83629139c
Added matching test to mirror the xml file loader test
2011-03-16 08:56:26 -05:00
Ray
c2908cdaa0
Allow collection's internal keys to be mixed case
2011-03-15 09:35:40 -05:00
Johannes M. Schmitt
98216a9af2
[DependencyInjection] refactored some more exceptions
2011-03-14 21:14:49 +01:00
Fabien Potencier
a1fcbf4d5f
Merge remote branch 'kriswallsmith/dic/false-circular-ref-fix'
...
* kriswallsmith/dic/false-circular-ref-fix:
[DependencyInjection] fixed false positive when detecting circular references if a service throws an exception during creation
2011-03-11 19:58:54 +01:00
Fabien Potencier
cedf588e98
Revert "Merge remote branch 'kriswallsmith/dic/lazy-replace-ext-params'"
...
This reverts commit 32ac2e8709
, reversing
changes made to 6e81c28ca4
.
2011-03-11 15:35:09 +01:00
Fabien Potencier
f321fadad6
[DependencyInjection] moved most protected things to private
2011-03-11 14:50:46 +01:00
Fabien Potencier
a8d05746ff
Merge remote branch 'schmittjoh/propertyAnnotations'
...
* schmittjoh/propertyAnnotations:
[DependencyInjection] added some tests
[DependencyInjection] adds property injection
2011-03-11 13:12:12 +01:00