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 #29

Open
sokolnikov911 opened this issue Apr 19, 2018 · 8 comments

Comments

@sokolnikov911
Copy link

This PHP 7.2 error from required package masterexploder/phpthumb.
Actually on package page exist related issue: masterexploder/PHPThumb#131, and someone made a PR: masterexploder/PHPThumb#133
But because of this package is abadoned, I think need to find another way to fix it. Maybe to make own fork of this package.

@metalagman
Copy link
Contributor

I thought about replacing the phpthumb with imagine library.

@dkrylovdotcom
Copy link

Hi! Author, please fix this issue.

composer update not help. It does not update masterexploder package

@Aleksey203
Copy link

@dkrylovdotcom
Copy link

dkrylovdotcom commented Jun 24, 2019 via email

@dkrylovdotcom
Copy link

dkrylovdotcom commented Jun 24, 2019 via email

@klev-o
Copy link

klev-o commented Jul 11, 2019

Hi! Please fix this issue.
Still, the problem is relevant for php 7.2
Easy way: instead
/** * The options for this class * * This array contains various options that determine the behavior in * various functions throughout the class. Functions note which specific * option key / values are used in their documentation * * @var array */ protected $options;
add:
protected $options = array();

@Matvik
Copy link

Matvik commented Jan 18, 2020

Same issue. Can you fix this?

@g-rodigy
Copy link

g-rodigy commented Jul 6, 2020

Add in your composer
"repositories": [ { "type": "vcs", "url": "https://github.com/monter08/PHPThumb" }]
or own fork
Original repo has fix for 7.2 but still not release new version

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

7 participants