Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning: sizeof(): Parameter must be an array or an object that implements Countable #115

Closed
srhinow opened this issue Feb 11, 2019 · 1 comment

Comments

@srhinow
Copy link
Contributor

srhinow commented Feb 11, 2019

System: Contao 4.4.34, PHP7.2.14-1 und dma_elementgenrator dev-master.
Wenn man in der Artikelansicht in die Inhaltselementasicht eines beliebigen Artikels wechseln wollte bekam ich die Fehlermeldung "Warning: sizeof(): Parameter must be an array or an object that implements Countable" da in der Datei vendor/dma/dma_elementgenerator/src/dma_elementgenerator/DMAElementGeneratorCallbacks.php (line 238) "sizeof($this->prepareOptions($objField))>1" abgefragt wird. In der Methode prepareOptions() wird an 2 stellen aber return null gesetzt. Wenn dieser Fall eintritt wird sizeOf gegen null getestet schlägt mit einem Error fehl.

Lösung: in den Zeilen 97 und 102 die return null; gegen return array(); zu ersetzen.

MacKP added a commit to MacKP/dma_elementgenerator that referenced this issue Apr 30, 2019
@MacKP
Copy link

MacKP commented Apr 30, 2019

Tested the fix (works fine for me) and made an PR for this ;-)
Fell free to merge it, when you have time for it ^^
Kind regards

JanoschOltmanns added a commit that referenced this issue Nov 21, 2019
* develop: (25 commits)
  Saubere default-Werte für int-DB-Felder (fixes #111)
  PHP 7.2-Kompatibilität in der Callback-Klasse (fixes #115)
  Fix requirements
  Fix Version
  Fix composer.json
  Fix composer.json
  First POC for sortable images.
  Fix checkbox handling (checkboxWizard, multiple)
  Add support for datepicker widget
  Add support for colorpicker in text fields.
  Support for array based select fields.
  Template Optimierungen
  Restructurierung des Quellcodes und Nutzung der Contao-Models.
  Optimierung der Basis-Template-Struktur.
  Revert "Optimierung der Basis-Template-Struktur."
  Optimierung der Basis-Template-Struktur.
  Fix für "Bilder verwenden die gleiche Lightbox" (fixes #74) thx to @fritzmg
  Eigenes Template pro Content Element wählbar (fixes #96)
  Handle legends in checkbox conditions (fixes #98)
  tpl-Template-Dateien werden nicht mehr unterstützt.
  ...

# Conflicts:
#	composer.json
#	src/dma_elementgenerator/DMAElementGenerator.php
#	src/dma_elementgenerator/config/database.sql
#	src/dma_elementgenerator/dca/tl_content.php
#	src/dma_elementgenerator/dca/tl_dma_eg_fields.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants