? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
20 May 2021

Adds support for webp images in com_banners

code review

avatar brianteeman brianteeman - open - 20 May 2021
avatar brianteeman brianteeman - change - 20 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 May 2021
Category Front End com_banners
avatar Quy Quy - test_item - 20 May 2021 - Tested successfully
avatar Quy
Quy - comment - 20 May 2021

I have tested this item successfully on b4f6509


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34027.

avatar richard67 richard67 - test_item - 20 May 2021 - Tested successfully
avatar richard67
richard67 - comment - 20 May 2021

I have tested this item successfully on b4f6509


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34027.

avatar richard67 richard67 - change - 20 May 2021
Status Pending Ready to Commit
Labels Added: ?
avatar richard67
richard67 - comment - 20 May 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34027.

avatar ChristineWk
ChristineWk - comment - 20 May 2021

Not sure, but on line 47 I see this here:

* @return boolean True if an image of type bmp, gif, jp(e)g or png, false otherwise

avatar richard67
richard67 - comment - 20 May 2021

Not sure, but on line 47 I see this here:

* @return boolean True if an image of type bmp, gif, jp(e)g or png, false otherwise

@ChristineWk Not related to this PR, but you are right, it should be "True if file of type swf, false otherwise".

@brianteeman Do you wanna fix it with this PR here? Or make a new one?

avatar brianteeman
brianteeman - comment - 20 May 2021

@richard67 actually there was another PR addressing that - I just forgot to submit it. Doing that now in a new pr

avatar rdeutz rdeutz - close - 20 May 2021
avatar rdeutz rdeutz - merge - 20 May 2021
avatar rdeutz rdeutz - change - 20 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-20 17:52:27
Closed_By rdeutz
Labels Added: ?

Add a Comment

Login with GitHub to post a comment