-
Notifications
You must be signed in to change notification settings - Fork 2.5k
added missing register services at Zend\Paginator\AdapterPluginManager #6577
added missing register services at Zend\Paginator\AdapterPluginManager #6577
Conversation
{ | ||
return new DbTableGateway( | ||
$this->creationOptions[0], | ||
isset($this->creationOptions[1]) ? : null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not inject the value of $this->createOptions[1]
. If you leave out the middle part of the ternary operator, the value of the first expression is returned which is the result of isset($this->createOptions[1])
(true
of false
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Indeed, @samsonasik: the elvis operator does not work well with isset()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, fixed now.
Also: requires tests for the newly introduced factory |
@Ocramius I think "awaiting author updates" label can be uncheck now :D |
@samsonasik thanks: poke me if I don't review within the end of next week :) |
ping @Ocramius any review for it ? Thanks |
a05ab85
to
0669b67
Compare
@samsonasik merged, thanks! |
…paginator-adapter-plugin-manager-registered-services' Close zendframework/zendframework#6577
…paginator-adapter-plugin-manager-registered-services' into develop Close zendframework/zendframework#6577 Forward port zendframework/zendframework#6577
No description provided.