- Issue created by @rnsrk
- 🇩🇪Germany rnsrk
I've made a patch, but we have to check it carefully, because it alters the database table definitions and add primary keys. I also removed the deprecated schema_uninstall and _install hooks, this may have site effects to older drupal versions, we have to check this for drupal 8, 9 and 10 WissKI Version to avoid crashing the database or mess up the tables! And I added ['allowed_classes' => FALSE] to the unserialize()-Method, because serialization without this option is marked as unsecure - but I'm not sure if this is the right option - so we may substitute with json_encode anyway.
- Assigned to Knurg
- Status changed to Needs review
over 1 year ago 8:37am 15 September 2023 - 🇩🇪Germany rnsrk
Forget the patch, we wrote update routines, which handles to remove duplicate rows and create primary keys with commit b9336181e6906d657cc2a32d0be716c1810c9479.
- Status changed to Fixed
11 months ago 10:50am 19 January 2024 - 🇩🇪Germany rnsrk
We add primary keys to respective tables through the wisski_core/wisski_core.install patches 8010 to 8013.
Automatically closed - issue fixed for 2 weeks with no activity.