laminas/laminas-db

Refactor of `Zend\Db\Sql` for 3.0.0

Closed this issue · 7 comments

This is a task for refactoring Zend\Db\Sql using the type hints of PHP 7.2. You must work on the 3.0.0 branch.

If you change the API you should report it in the /3.0.0.md file.

Note: it's quite difficult to provide this code refactor without changing other classes. That means you should provide the refactoring without pass all the unit tests. In this phase is only important to have the unit tests running, so we you must focus only on fatal/parse errors.


Originally posted by @ezimuel at zendframework/zend-db#362

I might take a look at it.


Originally posted by @guidofaecke at zendframework/zend-db#362 (comment)

Thanks @guidofaecke, let me know if you need some help!


Originally posted by @ezimuel at zendframework/zend-db#362 (comment)

@guidofaecke Do you want any assistance?


Originally posted by @arueckauer at zendframework/zend-db#362 (comment)

You're more than welcome!

Here is what I got so far -> (https://github.com/guidofaecke/zend-db/tree/3.0.0_SQL_typeHints)
It's not a lot, I know, but I'm busy with some work related projects as well 😃


Originally posted by @guidofaecke at zendframework/zend-db#362 (comment)

That's alright. Me working on a separate fork is unmanageable and PRs are overkill IMO. Would it be okay, if you grant me contribution access to your repo?


Originally posted by @arueckauer at zendframework/zend-db#362 (comment)

Done... I think... 😆


Originally posted by @guidofaecke at zendframework/zend-db#362 (comment)

This package is considered feature-complete, and is now in security-only maintenance mode, following a decision by the Technical Steering Committee.
If you have a security issue, please follow our security reporting guidelines.
If you wish to take on the role of maintainer, please nominate yourself

If you are looking for an actively maintained package alternative, we recommend: