dmpop / pellicola

Pastebin for your photos
https://tinyvps.xyz/pellicola
GNU General Public License v3.0
165 stars 16 forks source link

PHP 7.x supported? #22

Closed s-timm closed 5 years ago

s-timm commented 5 years ago

Hi!

Just a question: is PHP 7.x supported?

I tried to install Mejiro on my Strato-hosted web space but I can't see any photos or thumbnails. Also, no thumbnails are being created in the "photos/tims" folder. GD is installed, I changed folder/file permissions to 777... but maybe it's the PHP version: I have to use version 7.1, 7.2 or 7.3

Any help? thanx

rpoulin commented 5 years ago

I think u need image magic

On Thu, Apr 4, 2019, 10:41 AM Myself notifications@github.com wrote:

Hi!

Just a question: is PHP 7.x supported?

I tried to install Mejiro on my Strato-hosted web space but I can't see any photos or thumbnails. Also, no thumbnails are being created in the "photos/tims" folder. GD is installed, I changed folder/file permissions to 777... but maybe it's the PHP version: I have to use version 7.1, 7.2 or 7.3

Any help? thanx

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dmpop/mejiro/issues/22, or mute the thread https://github.com/notifications/unsubscribe-auth/AB8XGkB17-Lsj29gBFhNpXeJh2JfcAEXks5vdisogaJpZM4cdPal .

dmpop commented 5 years ago

Should work with PHP7.x.x just fine. Have you changed permission for all subdirectories too?

s-timm commented 5 years ago

First, thanks for "Mejiro" :-) I had a look at Mejiro's index.php and apparently the problem is the GLOB_BRACE flag - my hoster Strato seems to run Solaris servers and that flag is not supported on their hardware/software (see here: https://www.php.net/manual/en/function.glob.php). As a workaround I replaced line 143 with $files = array_merge(glob($photo_dir."*.[jJ][pP][gG]"),glob($photo_dir."*.[jJ][pP][eE][gG]"));

That did the trick :-)

dmpop commented 5 years ago

Good to know. Thank you for following up on this!