? ? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
7 Apr 2021

Summary of Changes

A more user friendly error message instead of an unhandled PHP exception.

Testing Instructions

Install Joomla 4 on a server with GD not installed or not enabled.

Go to Joomla 4 admin -> Templates/Site Templates -> Cassiopeia Details and Files -> Click template_thumbnail.png in left menu

Actual result BEFORE applying this Pull Request

without debug:

Screenshot 2021-04-07 at 22 18 15

With debug:

Screenshot 2021-04-07 at 22 17 56

Expected result AFTER applying this Pull Request

Screenshot 2021-04-07 at 22 17 37

Documentation Changes Required

none

avatar PhilETaylor PhilETaylor - open - 7 Apr 2021
avatar PhilETaylor PhilETaylor - change - 7 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Apr 2021
Category Administration com_templates Language & Strings
avatar brianteeman
brianteeman - comment - 7 Apr 2021

Doesnt this error also happen in media manager? It should. In which case it should be a generic mesage and not a com_template message

avatar PhilETaylor
PhilETaylor - comment - 7 Apr 2021

Im unable to generate a GD message in Media Manager...

avatar brianteeman
brianteeman - comment - 8 Apr 2021

I am assuming that GD is being used for the crop and rotate functionality in the template. The same functionality is present in the media manager although if memory serves me correctly the implementation is different (it shouldnt be)

avatar PhilETaylor
PhilETaylor - comment - 8 Apr 2021

yeah most of the image manipulation is using GD functions... Im going to rebuild a new docker container without GD and see what I can break.

avatar PhilETaylor
PhilETaylor - comment - 8 Apr 2021

I am assuming that GD is being used for the crop and rotate functionality in the template

Nope!

Crop & resize certainly DO work without GD... It looks like they use cropperjs 1.5.11 https://fengyuanchen.github.io/cropperjs

avatar PhilETaylor
PhilETaylor - comment - 18 Apr 2021

This PR is ready for testing.

avatar Quy Quy - test_item - 1 May 2021 - Tested successfully
avatar Quy
Quy - comment - 1 May 2021

I have tested this item successfully on 4176695


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

avatar PhilETaylor PhilETaylor - change - 1 May 2021
Labels Added: ? ?
avatar richard67 richard67 - alter_testresult - 1 May 2021 - Quy: Tested successfully
avatar richard67
richard67 - comment - 1 May 2021

I've restored the previous test result since the commit which invalidated the test was just a code style change.

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

I have tested this item successfully on e7d1fbe


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

avatar richard67 richard67 - change - 1 May 2021
The description was changed
Status Pending Ready to Commit
avatar richard67 richard67 - edited - 1 May 2021
avatar richard67
richard67 - comment - 1 May 2021

RTC


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

avatar chmst chmst - change - 2 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-02 19:34:50
Closed_By chmst
Labels Added: ?
avatar chmst chmst - close - 2 May 2021
avatar chmst chmst - merge - 2 May 2021
avatar chmst
chmst - comment - 2 May 2021

Thanks!

Add a Comment

Login with GitHub to post a comment