? Success
Pull Request for # 7857

User tests: Successful: Unsuccessful:

avatar thongredweb
thongredweb
21 Sep 2015

I moved this into class variable and other extension can choose by script.

#7857

avatar thongredweb thongredweb - open - 21 Sep 2015
avatar thongredweb thongredweb - change - 21 Sep 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 23 Sep 2015
Category Libraries
avatar zero-24 zero-24 - change - 23 Sep 2015
Rel_Number 0 7857
Relation Type Pull Request for
avatar zero-24
zero-24 - comment - 23 Sep 2015

@thongredweb can you add a quick test szenario? Else i guess we don't get any testers here on this.

avatar thongredweb
thongredweb - comment - 10 May 2016

Test case:

  • In any other extensions
  • Use class JImage for generate thumbnail file. $jimage = new JImage
  • Use method setThumbnailGenerate for set quality of thumbnail generated file.

$jimage->setThumbnailGenerate(true); // For best quality

$jimage->setThumbnailGenerate(false); // For best speed

avatar thongredweb
thongredweb - comment - 10 May 2016

This code has been tested successfully due to this PR #7857. This PR just for move this option into JImage class not from Joomla Configuration. The purpose is for other extensions can decide to choose quality or speed when use JImage class for generate thumbnail of images.

avatar brianteeman
brianteeman - comment - 10 May 2016

As the code is different it needs new tests

avatar wmchris wmchris - test_item - 1 Aug 2016 - Tested successfully
avatar wmchris
wmchris - comment - 1 Aug 2016

I have tested this item successfully on b38b65d

Tested using:

$jimage = new JImage('/var/www/html/joomla-hc/administrator/components/com_helloworld/bild.jpg');
$jimage->setThumbnailGenerate(false); //or true
$jimage->createThumbs("250x250",3,"/var/www/html/testout/");

true generated better images.

@icampus


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

avatar florian1995 florian1995 - test_item - 2 Aug 2016 - Tested unsuccessfully
avatar florian1995
florian1995 - comment - 2 Aug 2016

I have tested this item ? unsuccessfully on b38b65d

I tried it with png and jpeg and doesnt saw a great difference.
I used the following code to test it:

$jimage = new JImage('images/bild_normal.jpeg');
$jimage->setThumbnailGenerate(false); //or true
$jimage->createThumbs("250x250",3,"images");

I tested it once without the patch, then apllied the patch, and tested it with the parameter false and another time with the parameter true.
Tested@icampus


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

avatar roland-d roland-d - alter_testresult - 2 Aug 2016 - florian1995: Tested successfully
avatar roland-d roland-d - change - 2 Aug 2016
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 2 Aug 2016

RTC as we have 2 successful tests


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

avatar joomla-cms-bot joomla-cms-bot - change - 2 Aug 2016
Labels Added: ?
avatar brianteeman
brianteeman - comment - 2 Aug 2016

@roland-d but we also have a failed test

avatar roland-d
roland-d - comment - 2 Aug 2016

@brianteeman I have reset the test to successful as we discussed the result here and the test is actually OK. Depending on the option selected you get a different kind of image as explained in the instructions. Not seeing a whole lot of difference between images isn't necessarily an unsuccessful test. Someone else is going to test it as well just to be sure.


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

avatar brianteeman
brianteeman - comment - 2 Aug 2016

OK - thanks

On 2 August 2016 at 14:31, RolandD notifications@github.com wrote:

@brianteeman https://github.com/brianteeman I have reset the test to
successful as we discussed the result here and the test is actually OK.
Depending on the option selected you get a different kind of image as
explained in the instructions. Not seeing a whole lot of difference between
images isn't necessarily an unsuccessful test. Someone else is going to

test it as well just to be sure.

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/7919
https://issues.joomla.org/tracker/joomla-cms/7919.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#7919 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8bcvNBMBIXlf3Bq5fqZHTEOHQvHbks5qb0aZgaJpZM4GAsKx
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar wilsonge wilsonge - change - 3 Sep 2016
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2016-09-03 11:49:07
Closed_By wilsonge
avatar wilsonge wilsonge - close - 3 Sep 2016
avatar joomla-cms-bot joomla-cms-bot - close - 3 Sep 2016
avatar wilsonge
wilsonge - comment - 3 Sep 2016

Merged with 0ba5a11 - thanks :)

avatar joomla-cms-bot joomla-cms-bot - change - 3 Sep 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment