davedevelopment / phpmig

Simple migrations system for php

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Migration::__construct() "final" key word decreases usability

daimaxiong opened this issue · comments

The constructor's signature is final public function __construct()

I would like to make a base class for my migrations that extends Phpmig\Migration\Migration and does some custom configuration/initilization tasks.

The logical goto for that is the constructor, however, I cannot do that, because the parent class's constructor is marked final.

Please remove this in a future version

I think the engine probably depends on the constructor being the way it is, hence the final. We'd need something more robust than just removing the final on the constructor.