Support for Dibi 4.x - Githubissues

Tharos / LeanMapper

Lean Mapper is a tiny ORM based on powerful Dibi database abstraction library for PHP.

MIT License

87 stars 35 forks source link

Support for Dibi 4.x #112

Closed janpecha closed 5 years ago

janpecha commented 7 years ago

Zatím jen tak pro zajímavost - zajímalo mě, kolik úprav by bylo potřeba, aby testy doběhly nad Dibi 4.0. Kdyby Lean Mapper nedědil Dibi\Fluent, tak by to vyžadovalo prakticky jednu jedinou změnu ($result->setRowClass(null) => $result->setRowClass('Dibi\Row')).

castamir commented 7 years ago

Tahle úprava zavádí PHP7+ datové typy do kódu + závislost na knihovně pro PHP7+. Je tedy potřeba striktně oddělit další vývoj, protože už se jedná o změny zpětně nekompatibilní. V takovém případě to chce zvednout minimální kompatibilitu v celém composer.json a prověřit funkčnost všech závislostí pro PHP7+, což už není taková sranda.

janpecha commented 7 years ago

Přesně tak.