[Serializer] error handling inconsistencies fixed in the serializer decoders #9586
Conversation
rodrigodiez
commented
Nov 23, 2013
rodrigodiez
commented
| Q | A |
|---|---|
| Bug fix? | no |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | yes |
| Tests pass? | yes |
| Fixed tickets | #9393 |
| License | MIT |
| Doc PR | none |
There was a problem hiding this comment.
Instead of using variable, you can use return:
if (!function_exists('json_last_error_msg')) {
switch (json_last_error()) {
default:
return 'Unknown error';
}
}
return json_last_error_msg();
There was a problem hiding this comment.
Also, default should probably be the last case.
There was a problem hiding this comment.
As you call return, the break is redundant here.
There was a problem hiding this comment.
don't you need also here the use statement?
There was a problem hiding this comment.
Yep. Sorry guys, first code PR, still learning
There was a problem hiding this comment.
The deprecation message should not be the short description of the method. It should go with the @see tag (the first line of the phpdoc is special).
and you should remove the extra indentation space you added.
|
Seems ok to you guys? |
|
Ping |
|
Closing this one in favor of #9876 as you seem to have removed your Symfony fork. |
…e serializer decoders (fabpot) This PR was merged into the 2.5-dev branch. Discussion ---------- [Serializer] error handling inconsistencies fixed in the serializer decoders | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | yes | Tests pass? | yes | Fixed tickets | #9393 | License | MIT | Doc PR | none see #9586 for the original PR Commits ------- a1ab939 [Serializer] fixed CS 6d9f0be Json encoder classes now throws UnexpectedValueException as XML classes
7 participants
