madeyourday / contao-rocksolid-slider

RockSolid Slider Contao Extension
http://rocksolidthemes.com/de/contao/plugins/responsive-slider
Other
9 stars 7 forks source link

A non-numeric value encountered im Debug-Modus #87

Closed mlwebworker closed 5 months ago

mlwebworker commented 1 year ago

Contao 4.13.19 Rocksolid-Slider: 2.2.3 PHP: 8.1.18

Wenn Kunden im Feld Slide-Abstand (gapSize), einen nicht-nummerischen Wert eingeben, dann wird das beim Speichern nicht abgefangen. Der Fehler fällt den Kunden in der Regel auch nicht wirklich auf, weil im normalen Modus alles zu funktionieren scheint. Es ist für Kunden durch die Feldbeschreibung m.E. auch schwierig zu erkennen wie der Wert eingegeben werden soll, denn da steht "Abstand zwischen den einzelnen Slider-Elementen in Pixeln oder Prozent." In den mitgelieferten Demodaten von Oneo steht da z.B. 1%. Die Kunden ändern dann in z.B. 10px und damit habe ich im Stack-Trace im Debug-Modus

ErrorException:
Warning: A non-numeric value encountered

  at vendor/madeyourday/contao-rocksolid-slider/src/Module/Slider.php:368
  at MadeYourDay\RockSolidSlider\Module\Slider->compile()
     (vendor/contao/core-bundle/src/Resources/contao/modules/Module.php:214)
  at Contao\Module->generate()
     (vendor/madeyourday/contao-rocksolid-slider/src/Module/Slider.php:143)
  at MadeYourDay\RockSolidSlider\Module\Slider->generate()
     (vendor/contao/core-bundle/src/Resources/contao/elements/ContentModule.php:98)
  at Contao\ContentModule->generate()
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:621)
  at Contao\Controller::getContentElement()
     (vendor/contao/core-bundle/src/Resources/contao/modules/ModuleArticle.php:197)
  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:70)
  at Contao\ModuleArticle->generate()
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:549)
  at Contao\Controller::getArticle()
     (vendor/contao/core-bundle/src/Resources/contao/library/Contao/Controller.php:391)
  at Contao\Controller::getFrontendModule()
     (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:190)
  at Contao\PageRegular->prepare()
     (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:60)
  at Contao\PageRegular->getResponse()
     (vendor/contao/core-bundle/src/Resources/contao/controllers/FrontendIndex.php:320)
  at Contao\FrontendIndex->renderPage()
     (vendor/symfony/http-kernel/HttpKernel.php:163)
  at Symfony\Component\HttpKernel\HttpKernel->handleRaw()
     (vendor/symfony/http-kernel/HttpKernel.php:75)
  at Symfony\Component\HttpKernel\HttpKernel->handle()
     (vendor/symfony/http-kernel/Kernel.php:202)
  at Symfony\Component\HttpKernel\Kernel->handle()
     (public/preview.php:46)    

Schön wäre es wenn Fehleingaben bei der Eingabe geprüft werden und im Hinweistext vielleicht jeweils ein Beispiel für die Eingabe steht.

k-webdesign commented 5 months ago

Gibt's dazu ein Update? Contao 5.3 betrifft es ebenfalls 😬

ausi commented 5 months ago

Fixed in version 2.2.5