diff options
| author | Fabien Potencier <fabien.potencier@gmail.com> | 2013-01-05 19:24:35 +0100 |
|---|---|---|
| committer | Fabien Potencier <fabien.potencier@gmail.com> | 2013-01-05 19:24:35 +0100 |
| commit | b11eb9120cde5e05fccc39fdeef7fadf0fb4fa4d (patch) | |
| tree | d98fe5b4737c868b532b5a528b9ca25d3a59c760 /Core/Exception | |
| parent | 48018e4abb848ff1a2ecb781ca3303979e3a2357 (diff) | |
| parent | 39b2670c52d0d53224cb671418e2a852b864f786 (diff) | |
| download | symfony-security-b11eb9120cde5e05fccc39fdeef7fadf0fb4fa4d.zip symfony-security-b11eb9120cde5e05fccc39fdeef7fadf0fb4fa4d.tar.gz symfony-security-b11eb9120cde5e05fccc39fdeef7fadf0fb4fa4d.tar.bz2 | |
merged branch bschussek/exceptionfix (PR #6582)
This PR was merged into the master branch.
Commits
-------
184c8e5 Fixed @expectedException definitions to reference absolute exception paths
Discussion
----------
Added leading slashes to @expectedException definitions
Bug fix: no
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -
License of the code: MIT
Documentation PR: -
This PR adds leading backslashes `\` to `@expectedException` declarations.
The current version is inconsistent with other class references in doc blocks, which default to *relative* class names unless class name has a leading backslash.
```php
/**
* @param RelativeNs\Class $param1
* @param \AbsoluteNs\Class $param2
*/
but
/**
* @expectedException AbsoluteNs\Class
*/
```
Consequently, PHPStorm does not understand the current statements and marks them as erroneous (and correctly so IMO, even though PHPUnit's interpretation is more relaxed).
Diffstat (limited to 'Core/Exception')
0 files changed, 0 insertions, 0 deletions