Repository navigation
OS2forms sync modul not working #188
Description
We have discovered that the OS2forms sync module does not work in the current version of Drupal.
Will you, in ITK, review and merge this PR (OS2Forms/os2forms_sync#14) and create a new version with the changes?
Our work in finding and fixing bugs in this fix can be considered a contribution to OS2, so there will be no charge for it.
Unfortunately, I had forgotten that issues had to be created here first. I apologize for that. But now it's here.
Activity
ds-bellcom commentedon Aug 12, 2025
@jekuaitk thanks for merging the PR!
jekuaitk commentedon Sep 14, 2025
A new release has been made https://github.com/itk-dev/os2forms_sync/releases/tag/1.2.1
jekuaitk commentedon Sep 15, 2025
Reopened as it turns out that this issue is being used for tracking the moving of
os2forms_syncto the contrib repository.ds-bellcom commentedon Sep 16, 2025
We have now created the new contrib repo - for modules that are not in core: https://github.com/OS2Forms/os2forms-contrib
@jekuaitk @Anna-itk @ChatBotBerg
Anna-itk commentedon Oct 24, 2025
This code has been moved to: https://github.com/os2forms/os2forms_sync
ds-bellcom commentedon Nov 10, 2025
@Anna-itk @jekuaitk
Simon from Holstebro has tested the OS2sync module and he gets an error message when he tries to test this before the release (I get it too when I try). Stan can't immediately see where the error comes from, so it's our hope that you can help explain what happens / doesn't happen. The import of the form itself goes fine, but it's best not to have red boxes when you're in the process - it seems unsafe.
You can see it here: https://test.os2forms.dk/da/admin/os2forms/sync/webform (let me know if you don't have a user, and I'll create one for you right away).
You can see the error below:
10 remaining items
jekuaitk commentedon Nov 14, 2025
Not from what i have access to, no. It looks like a php exception, but it seems those are not shown in https://test.os2forms.dk/da/admin/reports/dblog. Without a full exception or stack trace it will be very hard for me to tell what the error is.
I took the liberty of making a very similar webform on your test-site and a successfully had it shown and imported on our site.
ds-bellcom commentedon Dec 9, 2025
Stan har kigget lidt mere på den fejl vi får og det ser ud til at være fordi vi ikke har modulet "Webform Revisions" installeret. Umiddelbart så står der ikke nogle steder at "OS2Forms sync" modulet afhængig af "Webform Revisions". Stemmer det @jekuaitk?
Vi er ikke de helt store tilhængere af "Webform Revisions", så vi håber ikke at dette er rigtigt. Ellers vil vi gerne vide hvorfor modulet ikke kan køre uden.
jekuaitk commentedon Dec 12, 2025
Det kan jeg godt forstå i ikke er. Vi er heller ikke selv fan af at have fået introduceret det. Det har skabt en del udfordringer for os og jeg er meget i tvivl om hvor meget gavn det har gjort.
Ved hurtigt kig her en fredag eftermiddag ser det ud til at i har ret i, at der er en ikke udtrykt afhængighed af "Webform revisions". Jeg vil tro at der er linjerne https://github.com/OS2Forms/os2forms_sync/blob/main/src/Helper/ImportHelper.php#L123-L127 der giver problemet.
Jeg vender det lige med nogen og melder tilbage.
ds-bellcom commentedon Dec 18, 2025
@jekuaitk er der noget nyt vedr. dette?
ds-bellcom commentedon Jan 22, 2026
@jekuaitk @Anna-itk er der noget nyt vedr. dette?
Vi er ikke superfans at afhængigheden til "Webform Revisions" - og vi er slet ikke superfans at af den afhængighed ikke er sat på modulet.
CC: @ChatBotBerg
ds-bellcom commentedon Sep 7, 2026
@gBarlach Denne tænker jeg vi skal have genbesøgt, når vi kender svaret på om "Webform Revisions" helt skal fjernes fra OS2forms (hvilket er vores anbefaling) eller ej.
Når vi ved det, kan vi lave et estimat på at få modulet tilpasset, så det virker uden brugen af "Webform Revisions".
gBarlach commentedon Oct 9, 2026
@ds-bellcom vil I lave et estimat på at få modulet tilpasset, så det virker uden brugen af "Webform Revisions", nu da det er fjernet fra Core?
gBarlach commentedon Oct 12, 2026
Inden re-estimat på denne, skal vi måske lige gentænke brugsscenarierne. Jeg kan komme i tanker om følgende (som dog ikke nødvendigvis behøver at blive håndteret af et og samme modul):
Flg. skal afklares:
2.a - det skal være muligt at gemme hele formularen med alle dens konfigurationer og indstillinger.
2.b - det skal være muligt at finde den gemte formular frem igen.
2.c - det skal være muligt at slette den gemte version, når man ikke længere har brug for den
ds-bellcom commentedon Oct 13, 2026
@gBarlach det lyder som en rigtig god idé at I starter med at finde ud af hvad I ønsker at modulet skal kunne og hvad der evt. skal være muligt at konfigurere.
Når I kender det, så lad os opdaterer opgaven med den/de usecases og krav som en første udgave af modulet skal kunne. Herefter kan det så undersøges om det giver mening at arbejde videre med det modul der findes nu eller om det giver mest mening at lave et nyt.
Men i første omgang er det vigtigste at I, i koordinationsgruppen bliver enige om hvad modulet skal kunne.