Migration failed after upgrading to 7.3.2

[FredPraca]

I upgraded the usual from git repository but when using php artisan migrate, it failed on the 2026_01_19_193315_refactor_photo_avoid_duplicates script and I got the following error :

 SQLSTATE[25P02]: In failed sql transaction: 7 ERROR:  current transaction is aborted, commands ignored until end of transaction block (Connection: pgsql, Host: localhost, Port: 5432, Database: lychee, SQL: delete from "size_variants"
  where "photo_id" in (C0IPzp34LToMQTpROCHn35h-, 9vqzslklUeaBxIBQg5ZLNipZ))

Any idea what could have gone wrong ?

[ildyria]

Hi, thank you for the report.

That's a weird one. I already fixed a similar one see #4043

[FredPraca]

Finally found how to deal with this, how do you want to proceed as this migration script has already been committed and potentially used ?
I'm not familiar with Laravel and Illuminate and usually with Liquibase on Java, modifying an existing script is a mess 😄

[ildyria]

Finally found how to deal with this, how do you want to proceed as this migration script has already been committed and potentially used ? I'm not familiar with Laravel and Illuminate and usually with Liquibase on Java, modifying an existing script is a mess 😄

if you open a PR , I can review it :)

If the migration has already been run, it won't be impacting users for which it run without errors, it will only be impacting users which will run it in the future and that have not migrated yet.

[FredPraca]

I'm on my way 😄

[FredPraca]

Done in #4125

[FredPraca]

Had a lot to do to polish the PR, sorry for the long roundtrip with CodeRabbit.
This should be ok, I rebased my branch and fixed stuff I wasn't able to test correctly.

[ildyria]

No worries :)

[FredPraca]

Grrrr Still have problem on code style. Now that I better understand what's going on with the toolchain, I'll be more precise.
10 years without doing real PHP showed that I need to update my knowledge which is good 😄

[ildyria]

make formatting ;)