Matthew Peveler

File: [frontend/dockerfile/docs/reference.md](https://docs.docker.com/engine/reference/builder/) A coworker suggested new dockerignore files using the naming syntax that's suggested on https://docs.docker.com/engine/reference/commandline/build/#:~:text=will%20first%20look%20for%20an%20ignore%20file%20named. When I went to review it, I was unfamiliar with the syntax, and went...

Fix altering sqlite table with indexes or triggers

5

PR fixes a bug where altering sqlite tables that had indexes or triggers would drop the indexes or triggers. This was because indexes and triggers are not contained in the...

For each adapter, there's a handful of ways to customize the DSN string we generate that is passed to PDO. However, there's not currently a good way we have to...

Remove deprecated default_database config value

1

This is an issue to track removing this deprecated value, targeted at doing it for 0.14, so that it is not forgotten for a couple of releases, which has happened...

Rewrite documentation examples from yaml to php

1

_Originally posted by @christhomas in https://github.com/cakephp/phinx/issues/1726#issuecomment-724665219_ > I understand that you wanna get rid of a dependency by moving to plain php, but all the documentation on the cakephp website...

phinx status prints down migrations out of order

2

Adding a migration to a list of existing migrations, where its id falls between existing ones, I get: ``` $ bin/phinx status Phinx by CakePHP - https://phinx.org. using config file...

The postgres adapter when fetching all columns from a table uses this logic to construct the column: https://github.com/cakephp/phinx/blob/4df3304e11bc0c6e9de4df2eedf733eb9b15cd75/src/Phinx/Db/Adapter/PostgresAdapter.php#L430-L436 The problem here is that `$columnInfo['is_identity']` is probably always `no` for postgres...

CURRENT_TIMESTAMP not quoted even for non-datetime columns

2

The following code will throw an error: ```php $this->table('test') ->addColumn('test_col', 'string', ['default' => 'CURRENT_TIMESTAMP']) ->create(); ``` This is because the default value will not be quoted. We can probably update...

Rename table instruction for postgres uses quoteColumnName

7

Something I noticed while fixing #1851, but did not have time to investigate on writing a testcase for it, so making an issue so as to not lose track of...

Make cakephp/database an optional dependency

7

After looking at/doing #1753, I wondered if it would be possible to also make it so that there was not an explicit dependency on `cakephp/database` (which would in-turn solve #1647),...