? Language Change ? Pending

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
12 Jan 2022

Summary of Changes

  • Fix the resizing of the image with the scheduler task
  • Improve the description to make aware, that the file will be overwritten

Testing Instructions

  • Create a task from the type "Image Size Check"
  • Run it

Actual result BEFORE applying this Pull Request

Image sizes are not changed.

Expected result AFTER applying this Pull Request

Image sizes are changed.

avatar bembelimen bembelimen - open - 12 Jan 2022
avatar bembelimen bembelimen - change - 12 Jan 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Jan 2022
Category Administration Language & Strings Front End Plugins
avatar brianteeman
brianteeman - comment - 12 Jan 2022

I have tested this item ? unsuccessfully on 4b49ad2

Resizes the first image found that is too big but then says its completed


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

avatar brianteeman brianteeman - test_item - 12 Jan 2022 - Tested unsuccessfully
avatar chmst
chmst - comment - 12 Jan 2022

I have tested this item successfully on 4b49ad2


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

avatar chmst chmst - test_item - 12 Jan 2022 - Tested successfully
avatar bembelimen
bembelimen - comment - 12 Jan 2022

I have tested this item red_circle unsuccessfully on 4b49ad2Resizes the first image found that is too big but then says its completed

It was always the intention to change one image per run to prevent any timeouts.

avatar brianteeman
brianteeman - comment - 12 Jan 2022

Then it must be clearly documented otherwise it will appear to be broken.

Although I have to say if it only does one image per run then it is pretty useless

avatar bembelimen bembelimen - change - 12 Jan 2022
Labels Added: Language Change ?
avatar bembelimen
bembelimen - comment - 12 Jan 2022

Updated the PR to allow more than one image per run.

avatar chmst
chmst - comment - 14 Jan 2022

Now I get this error:
grafik

Folder Sample Data. Fresh installation of the full package.

avatar chmst
chmst - comment - 14 Jan 2022

Maybe only my local environment: ERROR Routine threw exception: The GD extension for PHP is not available.

avatar drmenzelit
drmenzelit - comment - 14 Jan 2022

I have tested this item successfully on 99bbefd

Tested with 1 image, it works. Changed the settings to maximum images = 3, uploaded 4 big images, 3 where changed successfully on the next run. The 4th image was changed too in the next run.


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

avatar drmenzelit drmenzelit - test_item - 14 Jan 2022 - Tested successfully
avatar chmst
chmst - comment - 14 Jan 2022

I have tested this item successfully on 99bbefd

Re-tested on web server and it works.


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

avatar chmst chmst - test_item - 14 Jan 2022 - Tested successfully
avatar drmenzelit drmenzelit - change - 14 Jan 2022
The description was changed
Status Pending Ready to Commit
avatar drmenzelit drmenzelit - edited - 14 Jan 2022
avatar drmenzelit
drmenzelit - comment - 14 Jan 2022

RTC


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

avatar bembelimen bembelimen - change - 14 Jan 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-14 18:28:45
Closed_By bembelimen
Labels Added: ?
avatar bembelimen bembelimen - close - 14 Jan 2022
avatar bembelimen bembelimen - merge - 14 Jan 2022
avatar bembelimen
bembelimen - comment - 14 Jan 2022

Thanks all for the feedback and the improvements.

Add a Comment

Login with GitHub to post a comment