Contao Survey Choices Field Migration

[akroii]

Bug description

Wenn ich eine multiple Choice Frage anlege und mit Inhalt befülle, möchte er im Installtool (bzw. Contao Manager) eine Migration durchführen die er nicht beenden kann.

Steps to reproduce

1. Installiere letzte Version
2. Lege ein Survey mit Seiten und Multiple-Choices an
3. Befülle die Multiple-Choices mit Inhalt
4. Geh ins Install-Tool oder führe Datenbankänderungen im Manager aus
5. Fehlermeldung: Contao Survey Choices Field Migration (tritt undendlich auf)

-> Das Resultat ist dann, dass im Group-Widget der choices jeweils ein "Array" als Wort steht.

Expected behavior

Dazu bin ich zu wenig ein Entwickler. Das Installtool dürfte keine Migration durchführen oder wenn, dann eine erfolgreiche MIgration durchführen und nicht in einer Endlosschleife hängen bleiben.

Screenshots

Environment

C4.9.33
pdir/contao-survey 3.4.1

[akroii]

Funktioniert auch nicht im Branch dev-fix_multiple_pages
Eventuell hier mal nachschauen? #21

[cmette]

hallo @akroii - ich habe den Fehler behoben - der PR wartet zurzeit auf ein Review - bitte dann mal testen und kurze Rückmeldung geben

[MDevster]

Fixed in 3.4.2, thank you @akroii for the financial support.

[akroii]

The migration notice is gone away 👍but after I've updated to 3.4.2 i got strlen() expects parameter 1 to be string, null given in debug mode in frontend in the survey

TypeError:
strlen() expects parameter 1 to be string, null given

  at vendor/pdir/contao-survey/src/Resources/contao/elements/ContentSurvey.php:88
  at Hschottm\SurveyBundle\ContentSurvey->compile()
     (vendor/contao/core-bundle/src/Resources/contao/elements/ContentElement.php:252)
  at Contao\ContentElement->generate()
     (vendor/pdir/contao-survey/src/Resources/contao/elements/ContentSurvey.php:66)
  at Hschottm\SurveyBundle\ContentSurvey->generate()
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:624)
  at Contao\Controller::getContentElement(object(ContentModel), 'main')
     (vendor/contao/core-bundle/src/Resources/contao/modules/ModuleArticle.php:218)
  at Contao\ModuleArticle->compile()
     (vendor/contao/core-bundle/src/Resources/contao/modules/Module.php:214)
  at Contao\Module->generate()
     (vendor/contao/core-bundle/src/Resources/contao/modules/ModuleArticle.php:71)
  at Contao\ModuleArticle->generate(false)
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:552)
  at Contao\Controller::getArticle(object(ArticleModel), false, false, 'main')
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:393)
  at Contao\Controller::getFrontendModule(0, 'main')
     (vendor/fritzmg/contao-inherit-article/src/EventListener/InheritArticleListener.php:50)
  at InheritArticleBundle\EventListener\InheritArticleListener->getRenderedInheritedArticles(88, 'main')
     (vendor/fritzmg/contao-inherit-article/src/EventListener/InheritArticleListener.php:41)
  at InheritArticleBundle\EventListener\InheritArticleListener->onGetArticles(88, 'main')
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:348)
  at Contao\Controller::getFrontendModule('0', 'main')
     (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:173)
  at Contao\PageRegular->prepare(object(PageModel))
     (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:53)
  at Contao\PageRegular->getResponse(object(PageModel), true)
     (vendor/contao/core-bundle/src/Resources/contao/controllers/FrontendIndex.php:343)
  at Contao\FrontendIndex->renderPage(object(PageModel))
     (vendor/symfony/http-kernel/HttpKernel.php:169)
  at Symfony\Component\HttpKernel\HttpKernel->handleRaw(object(Request), 1)
     (vendor/symfony/http-kernel/HttpKernel.php:81)
  at Symfony\Component\HttpKernel\HttpKernel->handle(object(Request), 1, true)
     (vendor/symfony/http-kernel/Kernel.php:201)
  at Symfony\Component\HttpKernel\Kernel->handle(object(Request))
     (web/index.php:31)