Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open
nyerslaszlo opened this issue Dec 6, 2024 · 1 comment
Open

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

nyerslaszlo opened this issue Dec 6, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@nyerslaszlo
Copy link

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 nyerslaszlo added the bug Something isn't working label Dec 6, 2024
@awcodes
Copy link
Owner

awcodes commented Dec 9, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@awcodes @nyerslaszlo