zendframework / zend-db

Db component from Zend Framework
BSD 3-Clause "New" or "Revised" License
101 stars 122 forks source link

Fix result type #331

Closed lowtower closed 6 years ago

lowtower commented 6 years ago

Provide a narrative description of what you are trying to accomplish:

phpstan raises an error saying that ResulSetInterface has no method current().

  • [x] Detail the original, incorrect behavior. The selectWith() method has defined the @returnType as ResultSetInterface. The method calls the method executeSelect(), which returns ResultSet. The return types should be the same. removes also the @throws flag as it doesn't throw an exception itself, but just the called methods.
  • [ ] Detail the new, expected behavior.
  • [ ] Base your feature on the master branch, and submit against that branch.
  • [ ] Add a regression test that demonstrates the bug, and proves the fix.
  • [ ] Add a CHANGELOG.md entry for the fix.

New PR as requested by @ezimuel when closing #312.

ezimuel commented 6 years ago

@lowtower thanks!