Welcome Guest, Not a member yet? Register   Sign In
migrate:status does not detect history
#1

I just tried to get into the migrations topic a little more and I'm not sure, if this is supposed to be that way.


When running "php spark migrate: status"
I get the following output:

Quote:+-----------------+-------------------+-----------------------+--------+-------------+-------+
| Namespace      | Version                     | Dateiname                 | Gruppe | Migriert am | Batch |
+-----------------+-------------------+-----------------------+--------+-------------+-------+
| Tatter\Settings | 20190407221924     | create_table_settings | ---         | ---                | ---  |
| Tatter\Settings | 2021-03-02-091856 | add_datatype             | ---         | ---                | ---  |
| Myth\Auth      | 2017-11-20-223112  | create_auth_tables    | ---          | ---               | ---  |
+-----------------+-------------------+-----------------------+--------+-------------+-------+
In the database migration table, group (default), migration date and batch are inserted correctly.

The problem is, if I now run "migrate -all", spark will try to migrate those entries again, as it does not detect that they were already done.

I read into the codebase and I wonder about these lines:
https://github.com/codeigniter4/CodeIgni...s.php#L113

Here, we retrieve the history of migration. In the surrounding loop, all defined namespaces are looped, but namespace is not passed to history retrieval.
https://github.com/codeigniter4/CodeIgni...r.php#L777

In history retrieval, the database request is filtered by "$this->namespace". I did some debug output here and this is always "App".

Likely, due to this line in the constructor:
https://github.com/codeigniter4/CodeIgni...r.php#L141

As it is always looking for history for the "App" namespace, it will not find "Myth\Auth" history or any other.
Am I misunderstanding something here or is this a bug?

Thanks and cheers
Alex
Reply




Theme © iAndrew 2016 - Forum software by © MyBB