filament-curator icon indicating copy to clipboard operation
filament-curator copied to clipboard
verified publisher icon awcodes

Error when saving media to related model(s) in a repeater while using PostgreSQL database

Open nyerslaszlo opened this issue 1 year ago • 1 comments

Filament Version

3.2.127

Plugin Version

3.6.14

PHP Version

PHP 8.3

Problem description

On a resource creation or update page, when you add media (using the CuratorPicker) to a related model that is in a repeater there is an SQL error when saving. The changes to the related model do save but after that it returns a server error, and other details of the main resource do not save.

While trying to recreate this error on a minimalized setup I realized that it doesn't happen while using SQLite, but started happening when switching to Postgres (version 16). No idea about MySQL.

On SQLite there was a different bug (with no error): on the update page, after saving changes, the attached media entries on the related model(s) disappear from the curator picker. If you refresh the page they load back, however if you save again instead, they get detached from the related model(s). This might or might not be related, I have not investigated this further since I do not use SQLite usually.

Expected behavior

To save changes without an error.

Steps to reproduce

Using the reproduction repository:

  1. Setup repo as you would an existing filament project, then migrate the database. (Use postgresql database) 1a. You can find a test user and tenant in the DatabaseSeeder.php, so you can use db:seed if you want.
  2. Upload some media files.
  3. Open the people resource, create a new person. Switch to the timeline tab, add a timeline entry, add some media to it, and then save the resource.
  4. The error should pop up here.

Reproduction repository

https://github.com/nyerslaszlo/curator-bugs

Relevant log output

SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for type bigint: "[{"id":3,"disk":"public","directory":"media","visibility":"private","name":"90d77230-ea20-4a15-ac90-681b6b4ede48","path":"media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg","width":1920,"height":1080,"size":536124,"type":"image\/jpeg","ext":"jpg","alt":null,"title":"17 - dXjzyo3","description":null,"caption":null,"exif":{"FileName":"QPinKhAkr0sOehAj5g1llTRUGRhcMP-metaMTcgLSBkWGp6eW8zLmpwZw==-.jpg","FileDateTime":1733514607,"FileSize":536124,"FileType":2,"MimeType":"image\/jpeg","SectionsFound":"","COMPUTED":{"html":"width=\"1920\" height=\"1080\"","Height":1080,"Width":1920,"IsColor":1}},"curations":null,"created_at":"2024-12-06T19:50:14.000000Z","updated_at":"2024-12-06T19:50:14.000000Z","tenant_id":1,"url":"http:\/\/curator-bugs.test\/storage\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg","thumbnail_url":"\/curator\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=200&h=200&fit=crop&fm=webp&s=fc4e3f375e38de5d4beaaf213aa9b702","medium_url":"\/curator\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=640&h=640&fit=crop&fm=webp&s=0142f230a0692c2f3a359e20292276a7","large_url":"\/curator\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=1024&h=1024&fit=contain&fm=webp&s=21ed485f41596e9a33e92571fbc86695","resizable":true,"size_for_humans":"523.6 KiB","pretty_name":"17 - dXjzyo3","pivot":{"model_type":"App\\Models\\TimelineEntry","model_id":1,"media_id":3,"order":1,"hidden":false,"created_at":"2024-12-06T19:50:57.000000Z","updated_at":"2024-12-06T20:07:47.000000Z"}},{"id":4,"disk":"public","directory":"media","visibility":"private","name":"98c02a94-e1b3-44de-bb50-58896fabb4aa","path":"media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg","width":736,"height":1308,"size":69839,"type":"image\/jpeg","ext":"jpg","alt":null,"title":"34c63c5e76506e88fc0a66013f6a8e2c","description":null,"caption":null,"exif":{"FileName":"D1apEomOWDPPQ2HXKsMxc7D0GeWllL-metaMzRjNjNjNWU3NjUwNmU4OGZjMGE2NjAxM2Y2YThlMmMuanBn-.jpg","FileDateTime":1733514610,"FileSize":69839,"FileType":2,"MimeType":"image\/jpeg","SectionsFound":"","COMPUTED":{"html":"width=\"736\" height=\"1308\"","Height":1308,"Width":736,"IsColor":1}},"curations":null,"created_at":"2024-12-06T19:50:14.000000Z","updated_at":"2024-12-06T19:50:14.000000Z","tenant_id":1,"url":"http:\/\/curator-bugs.test\/storage\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg","thumbnail_url":"\/curator\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=200&h=200&fit=crop&fm=webp&s=f897b59174c7b080ebe605aebcae6371","medium_url":"\/curator\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=640&h=640&fit=crop&fm=webp&s=560a5ec6b908ca8cabf3530206a04b2c","large_url":"\/curator\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=1024&h=1024&fit=contain&fm=webp&s=b95dac72418d0c57191f798dedfdab0e","resizable":true,"size_for_humans":"68.2 KiB","pretty_name":"34c63c5e76506e88fc0a66013f6a8e2c","pivot":{"model_type":"App\\Models\\TimelineEntry","model_id":1,"media_id":4,"order":2,"hidden":false,"created_at":"2024-12-06T19:50:57.000000Z","updated_at":"2024-12-06T20:07:47.000000Z"}},{"id":1,"disk":"public","directory":"media","visibility":"private","name":"0152fbfd-8608-4e30-abd5-63dc2a2e1e01","path":"media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","width":null,"height":null,"size":13264,"type":"application\/pdf","ext":"pdf","alt":null,"title":"dummy","description":null,"caption":null,"exif":null,"curations":null,"created_at":"2024-12-06T19:50:14.000000Z","updated_at":"2024-12-06T19:50:14.000000Z","tenant_id":1,"url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","thumbnail_url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","medium_url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","large_url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","resizable":false,"size_for_humans":"13 KiB","pretty_name":"dummy","pivot":{"model_type":"App\\Models\\TimelineEntry","model_id":1,"media_id":1,"order":3,"hidden":false,"created_at":"2024-12-06T19:50:57.000000Z","updated_at":"2024-12-06T20:07:47.000000Z"}}]"
CONTEXT:  unnamed portal parameter $1 = '...' (Connection: pgsql, SQL: select * from "media" where "id" in ([{"id":3,"disk":"public","directory":"media","visibility":"private","name":"90d77230-ea20-4a15-ac90-681b6b4ede48","path":"media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg","width":1920,"height":1080,"size":536124,"type":"image\/jpeg","ext":"jpg","alt":null,"title":"17 - dXjzyo3","description":null,"caption":null,"exif":{"FileName":"QPinKhAkr0sOehAj5g1llTRUGRhcMP-metaMTcgLSBkWGp6eW8zLmpwZw==-.jpg","FileDateTime":1733514607,"FileSize":536124,"FileType":2,"MimeType":"image\/jpeg","SectionsFound":"","COMPUTED":{"html":"width=\"1920\" height=\"1080\"","Height":1080,"Width":1920,"IsColor":1}},"curations":null,"created_at":"2024-12-06T19:50:14.000000Z","updated_at":"2024-12-06T19:50:14.000000Z","tenant_id":1,"url":"http:\/\/curator-bugs.test\/storage\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg","thumbnail_url":"\/curator\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=200&h=200&fit=crop&fm=webp&s=fc4e3f375e38de5d4beaaf213aa9b702","medium_url":"\/curator\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=640&h=640&fit=crop&fm=webp&s=0142f230a0692c2f3a359e20292276a7","large_url":"\/curator\/media\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=1024&h=1024&fit=contain&fm=webp&s=21ed485f41596e9a33e92571fbc86695","resizable":true,"size_for_humans":"523.6 KiB","pretty_name":"17 - dXjzyo3","pivot":{"model_type":"App\\Models\\TimelineEntry","model_id":1,"media_id":3,"order":1,"hidden":false,"created_at":"2024-12-06T19:50:57.000000Z","updated_at":"2024-12-06T20:07:47.000000Z"}},{"id":4,"disk":"public","directory":"media","visibility":"private","name":"98c02a94-e1b3-44de-bb50-58896fabb4aa","path":"media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg","width":736,"height":1308,"size":69839,"type":"image\/jpeg","ext":"jpg","alt":null,"title":"34c63c5e76506e88fc0a66013f6a8e2c","description":null,"caption":null,"exif":{"FileName":"D1apEomOWDPPQ2HXKsMxc7D0GeWllL-metaMzRjNjNjNWU3NjUwNmU4OGZjMGE2NjAxM2Y2YThlMmMuanBn-.jpg","FileDateTime":1733514610,"FileSize":69839,"FileType":2,"MimeType":"image\/jpeg","SectionsFound":"","COMPUTED":{"html":"width=\"736\" height=\"1308\"","Height":1308,"Width":736,"IsColor":1}},"curations":null,"created_at":"2024-12-06T19:50:14.000000Z","updated_at":"2024-12-06T19:50:14.000000Z","tenant_id":1,"url":"http:\/\/curator-bugs.test\/storage\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg","thumbnail_url":"\/curator\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=200&h=200&fit=crop&fm=webp&s=f897b59174c7b080ebe605aebcae6371","medium_url":"\/curator\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=640&h=640&fit=crop&fm=webp&s=560a5ec6b908ca8cabf3530206a04b2c","large_url":"\/curator\/media\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=1024&h=1024&fit=contain&fm=webp&s=b95dac72418d0c57191f798dedfdab0e","resizable":true,"size_for_humans":"68.2 KiB","pretty_name":"34c63c5e76506e88fc0a66013f6a8e2c","pivot":{"model_type":"App\\Models\\TimelineEntry","model_id":1,"media_id":4,"order":2,"hidden":false,"created_at":"2024-12-06T19:50:57.000000Z","updated_at":"2024-12-06T20:07:47.000000Z"}},{"id":1,"disk":"public","directory":"media","visibility":"private","name":"0152fbfd-8608-4e30-abd5-63dc2a2e1e01","path":"media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","width":null,"height":null,"size":13264,"type":"application\/pdf","ext":"pdf","alt":null,"title":"dummy","description":null,"caption":null,"exif":null,"curations":null,"created_at":"2024-12-06T19:50:14.000000Z","updated_at":"2024-12-06T19:50:14.000000Z","tenant_id":1,"url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","thumbnail_url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","medium_url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","large_url":"http:\/\/curator-bugs.test\/storage\/media\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf","resizable":false,"size_for_humans":"13 KiB","pretty_name":"dummy","pivot":{"model_type":"App\\Models\\TimelineEntry","model_id":1,"media_id":1,"order":3,"hidden":false,"created_at":"2024-12-06T19:50:57.000000Z","updated_at":"2024-12-06T20:07:47.000000Z"}}])) {
    "userId": 1,
    "exception": {
        "errorInfo": [
            "22P02",
            7,
            "ERROR:  invalid input syntax for type bigint: \"[{\"id\":3,\"disk\":\"public\",\"directory\":\"media\",\"visibility\":\"private\",\"name\":\"90d77230-ea20-4a15-ac90-681b6b4ede48\",\"path\":\"media\\\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg\",\"width\":1920,\"height\":1080,\"size\":536124,\"type\":\"image\\\/jpeg\",\"ext\":\"jpg\",\"alt\":null,\"title\":\"17 - dXjzyo3\",\"description\":null,\"caption\":null,\"exif\":{\"FileName\":\"QPinKhAkr0sOehAj5g1llTRUGRhcMP-metaMTcgLSBkWGp6eW8zLmpwZw==-.jpg\",\"FileDateTime\":1733514607,\"FileSize\":536124,\"FileType\":2,\"MimeType\":\"image\\\/jpeg\",\"SectionsFound\":\"\",\"COMPUTED\":{\"html\":\"width=\\\"1920\\\" height=\\\"1080\\\"\",\"Height\":1080,\"Width\":1920,\"IsColor\":1}},\"curations\":null,\"created_at\":\"2024-12-06T19:50:14.000000Z\",\"updated_at\":\"2024-12-06T19:50:14.000000Z\",\"tenant_id\":1,\"url\":\"http:\\\/\\\/curator-bugs.test\\\/storage\\\/media\\\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg\",\"thumbnail_url\":\"\\\/curator\\\/media\\\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=200&h=200&fit=crop&fm=webp&s=fc4e3f375e38de5d4beaaf213aa9b702\",\"medium_url\":\"\\\/curator\\\/media\\\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=640&h=640&fit=crop&fm=webp&s=0142f230a0692c2f3a359e20292276a7\",\"large_url\":\"\\\/curator\\\/media\\\/90d77230-ea20-4a15-ac90-681b6b4ede48.jpg?w=1024&h=1024&fit=contain&fm=webp&s=21ed485f41596e9a33e92571fbc86695\",\"resizable\":true,\"size_for_humans\":\"523.6 KiB\",\"pretty_name\":\"17 - dXjzyo3\",\"pivot\":{\"model_type\":\"App\\\\Models\\\\TimelineEntry\",\"model_id\":1,\"media_id\":3,\"order\":1,\"hidden\":false,\"created_at\":\"2024-12-06T19:50:57.000000Z\",\"updated_at\":\"2024-12-06T20:07:47.000000Z\"}},{\"id\":4,\"disk\":\"public\",\"directory\":\"media\",\"visibility\":\"private\",\"name\":\"98c02a94-e1b3-44de-bb50-58896fabb4aa\",\"path\":\"media\\\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg\",\"width\":736,\"height\":1308,\"size\":69839,\"type\":\"image\\\/jpeg\",\"ext\":\"jpg\",\"alt\":null,\"title\":\"34c63c5e76506e88fc0a66013f6a8e2c\",\"description\":null,\"caption\":null,\"exif\":{\"FileName\":\"D1apEomOWDPPQ2HXKsMxc7D0GeWllL-metaMzRjNjNjNWU3NjUwNmU4OGZjMGE2NjAxM2Y2YThlMmMuanBn-.jpg\",\"FileDateTime\":1733514610,\"FileSize\":69839,\"FileType\":2,\"MimeType\":\"image\\\/jpeg\",\"SectionsFound\":\"\",\"COMPUTED\":{\"html\":\"width=\\\"736\\\" height=\\\"1308\\\"\",\"Height\":1308,\"Width\":736,\"IsColor\":1}},\"curations\":null,\"created_at\":\"2024-12-06T19:50:14.000000Z\",\"updated_at\":\"2024-12-06T19:50:14.000000Z\",\"tenant_id\":1,\"url\":\"http:\\\/\\\/curator-bugs.test\\\/storage\\\/media\\\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg\",\"thumbnail_url\":\"\\\/curator\\\/media\\\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=200&h=200&fit=crop&fm=webp&s=f897b59174c7b080ebe605aebcae6371\",\"medium_url\":\"\\\/curator\\\/media\\\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=640&h=640&fit=crop&fm=webp&s=560a5ec6b908ca8cabf3530206a04b2c\",\"large_url\":\"\\\/curator\\\/media\\\/98c02a94-e1b3-44de-bb50-58896fabb4aa.jpg?w=1024&h=1024&fit=contain&fm=webp&s=b95dac72418d0c57191f798dedfdab0e\",\"resizable\":true,\"size_for_humans\":\"68.2 KiB\",\"pretty_name\":\"34c63c5e76506e88fc0a66013f6a8e2c\",\"pivot\":{\"model_type\":\"App\\\\Models\\\\TimelineEntry\",\"model_id\":1,\"media_id\":4,\"order\":2,\"hidden\":false,\"created_at\":\"2024-12-06T19:50:57.000000Z\",\"updated_at\":\"2024-12-06T20:07:47.000000Z\"}},{\"id\":1,\"disk\":\"public\",\"directory\":\"media\",\"visibility\":\"private\",\"name\":\"0152fbfd-8608-4e30-abd5-63dc2a2e1e01\",\"path\":\"media\\\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf\",\"width\":null,\"height\":null,\"size\":13264,\"type\":\"application\\\/pdf\",\"ext\":\"pdf\",\"alt\":null,\"title\":\"dummy\",\"description\":null,\"caption\":null,\"exif\":null,\"curations\":null,\"created_at\":\"2024-12-06T19:50:14.000000Z\",\"updated_at\":\"2024-12-06T19:50:14.000000Z\",\"tenant_id\":1,\"url\":\"http:\\\/\\\/curator-bugs.test\\\/storage\\\/media\\\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf\",\"thumbnail_url\":\"http:\\\/\\\/curator-bugs.test\\\/storage\\\/media\\\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf\",\"medium_url\":\"http:\\\/\\\/curator-bugs.test\\\/storage\\\/media\\\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf\",\"large_url\":\"http:\\\/\\\/curator-bugs.test\\\/storage\\\/media\\\/0152fbfd-8608-4e30-abd5-63dc2a2e1e01.pdf\",\"resizable\":false,\"size_for_humans\":\"13 KiB\",\"pretty_name\":\"dummy\",\"pivot\":{\"model_type\":\"App\\\\Models\\\\TimelineEntry\",\"model_id\":1,\"media_id\":1,\"order\":3,\"hidden\":false,\"created_at\":\"2024-12-06T19:50:57.000000Z\",\"updated_at\":\"2024-12-06T20:07:47.000000Z\"}}]\"\nCONTEXT:  unnamed portal parameter $1 = '...'"
        ],
        "connectionName": "pgsql"
    }
}

nyerslaszlo avatar Dec 06 '24 20:12 nyerslaszlo

Still have not heard back in regards to #537 to add support for morphable pivot tables.

awcodes avatar Dec 09 '24 17:12 awcodes

On SQLite there was a different bug (with no error): on the update page, after saving changes, the attached media entries on the related model(s) disappear from the curator picker. If you refresh the page they load back, however if you save again instead, they get detached from the related model(s). This might or might not be related, I have not investigated this further since I do not use SQLite usually.

Found a workaround: reload the page using the getRedirectUrl() method in the EditYourRecord page.

In my case, I have a wizard with a repeater that includes a title and an image gallery. Since I am persisting wizard steps in query parameters, I reload the page using the following approach:

protected function getRedirectUrl(): string {
    $referer = request()->header('Referer');

    if ($referer) {
        $parsedUrl = parse_url($referer);
        parse_str($parsedUrl['query'] ?? '', $queryParams);
        return $this->getResource()::getUrl('edit', ['record' => $this->record]) . '?' . http_build_query($queryParams);
    }

    return $this->getResource()::getUrl('edit', ['record' => $this->record]);
}

If you do not have a wizard or do not need to persist the steps, you can simply reload the page using:

return $this->getResource()::getUrl('edit', ['record' => $this->record]);

kabirgogoi avatar Mar 05 '25 07:03 kabirgogoi