Le lun. 19 juin 2023 à 22:33, Rowan Tommins <rowan.coll...@gmail.com> a écrit :
> On 19/06/2023 21:17, Nicolas Grekas wrote: > > I think we must account for a bit of history/legacy on the topic. > > I think session_set_save_handler(SessionHandlerInterface) is the best > BC/FC > > path we can provide. > > > Can you elaborate? The SessionHandlerInterface is the *newer* of the two > current signatures, so what does making it the preferred signature (with > users of the other having to change their code) have to do with > "history/legacy"? Sure : SessionHandlerInterface is around 5.4. It's been there since long enough to not care about this aspect when considering both signatures. In my experience, supporting 3 major versions is way enough: previous major, current major and next major. Which means 7+8+9 in this case. It's enough because an app that runs on < 5.4 doesn't need to prepare to move to 9. It first has many more steps to do. Then, among both options, we need to select the one with the best future proofness, and that's definitely the OOP one to me, because it comes with more guarantees (type checks). By doing so, we allow apps that are still on 7 but are also actively planning to upgrade to 8+ to make the future-proof choice of choosing SessionHandlerInterface. Nicolas
