[Pushed] png image avatar upload error
#1
hi
Loading avatar gives error on 2 pages(admin panel and user panel.)

I upload .jpg image upload not error.

png image upload error Sad

please help me

screen shoot
http://i.hizliresim.com/X9BqXo.png
[Image: X9BqXo.png]

Ups helpp

Fixed
source : [Image: bk5job.png]
Reply
#2
Which php version are you using?
Support PMs will be ignored!
Reply
#3
(2016-11-29, 12:44 PM)Jones H Wrote: Which php version are you using?

PHP version : 5.4.45
OS : ‪CentOS 6.7 (Final)

I have fixed

Error type : User carelessness Big Grin

[Image: 41gDyL.png]

source : https://community.mybb.com/thread-159855...pid1104547

File: inc/functions_image.php (Line no. 94)
Message
imagealphablending() expects parameter 1 to be resource, boolean given
Reply
#4
Glad you fixed that, this can be rejected then
[MyBB 1.8 Czech translation] [MyBB 1.8 plugins]: Prune old PMs + optimize DB plugin --- Thank you/like system
Reply
#5
@Jones @Euan T Do we want to add a small error instead of it throwing a SQL error or shall we just reject this?
-Ben

Please do not PM me for support. I am looking to be hired for paid services.

You can view my paid services here.
Reply
#6
We can adjust setting description for this (I can do that if it is not already described properly)
[MyBB 1.8 Czech translation] [MyBB 1.8 plugins]: Prune old PMs + optimize DB plugin --- Thank you/like system
Reply
#7
The avatar function should probably validate the setting to avoid throwing an error. Doubt we can check the setting on input as we don't support RegEx there. Going to wait for Euan before pushing this.
Support PMs will be ignored!
Reply
#8
What about to use same same format input for all image dimensions in ACP? We use "widthXheight and "width|height")
[MyBB 1.8 Czech translation] [MyBB 1.8 plugins]: Prune old PMs + optimize DB plugin --- Thank you/like system
Reply
#9
Sorry to bump an old issue but the problem still persists. As Eldenroot said we should use one format and stick to that. Alternately we can think of accepting both the formats (| and x) for every case. Just check the existence of separator and split accordingly.
[Image: Untitled_2a.png]

Also we need to explore the possibility to validate the input while updating settings from ACP.
And we can extend this to consider default if a valid setting format not found.

Pushing it.
Reply
#10
Hi,

Thank you for your report. We have pushed this issue to our Github repository for further analysis where you can track our commits and progress with fixing this bug. Discussions regarding this bug may also take place there too.

Follow this link to visit the issue on Github: https://github.com/mybb/mybb/issues/3192

Thanks for contributing to MyBB!

Regards,
The MyBB Group
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)