- Issue created by @Grevil
- last update
over 1 year ago 27 pass - @grevil opened merge request.
- Status changed to Needs review
over 1 year ago 1:48pm 30 May 2023 - 🇩🇪Germany Grevil
Fixed! @Anybody and I discussed, internally, if we should provide an update hook. But unfortunately, it is not very easy to update an existing field having multiple values, for further information on this quite complex topic, visit ✨ Field type modules cannot maintain their field schema (field type schema change C(R)UD is needed) Needs work .
There is this workaround by @Anybody: https://gist.github.com/JPustkuchen/ce53d40303a51ca5f17ce7f48c363b9b. But it is quite dirty, for "just" making a few database columns nullable, hence we decided against it.
Please review!
- last update
over 1 year ago 27 pass - 🇩🇪Germany Anybody Porta Westfalica
Thanks @Grevil! Nice fix and tests!!
Please resolve the comments and especially move the update hook over into 🐛 SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'field_access_role_id' cannot be null Fixed , it won't help non-developers here, but make them scream in fear. ;)
- Status changed to Needs work
over 1 year ago 7:20am 31 May 2023 - 🇩🇪Germany Anybody Porta Westfalica
- last update
over 1 year ago 27 pass - Status changed to Needs review
over 1 year ago 8:02am 31 May 2023 - last update
over 1 year ago 27 pass - Status changed to RTBC
over 1 year ago 8:04am 31 May 2023 - 🇩🇪Germany Anybody Porta Westfalica
Thank you @Grevil!
RTBC, now let's wait for the follow-up / child 🐛 SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'field_access_role_id' cannot be null Fixed to be ready.
- last update
over 1 year ago 27 pass - Status changed to Fixed
over 1 year ago 10:34am 31 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.