? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
14 Jul 2016

Pull Request for Issue #11118.

Summary of Changes

Add 200 and 500 to searchtools limit box values as it exists in global config.

Testing Instructions

very simple test:
1. Apply patch
2. Go to Global Configuration - Site Settings
3. Select one of the new values (either 200 or 500)
4. Save
5. Logout
6. Login
7. Go to content articles and check the preselected value in the limit select is the value you selected in 3.

More info see #11118

avatar andrepereiradasilva andrepereiradasilva - open - 14 Jul 2016
avatar andrepereiradasilva andrepereiradasilva - change - 14 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jul 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 14 Jul 2016
The description was changed
7c1086f 14 Jul 2016 avatar andrepereiradasilva cs
avatar andrepereiradasilva andrepereiradasilva - change - 14 Jul 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - change - 14 Jul 2016
The description was changed
avatar brianteeman brianteeman - test_item - 14 Jul 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 14 Jul 2016

I have tested this item successfully on 7c1086f


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

avatar itfidds
itfidds - comment - 14 Jul 2016

Hi, I'm new to the github reporting system for Joomla.

Do I need to download an updated file from somewhere and test this (as the original author) or do I just wait until Joomla rolls out an update?

Thanks

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Jul 2016

Pull Requests (PR) only get Ready to Commit (RTC) state with two success tests.
After RTC is up to joomla mantainers to review them and decide in which version should the patch be released.

To test and mark as success or not you can use the Joomla PatchTester component (install from URL https://github.com/joomla-extensions/patchtester/releases/download/2.0.1/com_patchtester.zip), fetch the data, them apply this patch and test.

When tested go to JIssues (https://issues.joomla.org/tracker/joomla-cms/11120), login with github and mark as success or not.

More info: https://docs.joomla.org/Testing_Joomla!_patches#Recording_test_results

avatar RonakParmar RonakParmar - test_item - 15 Jul 2016 - Tested successfully
avatar RonakParmar
RonakParmar - comment - 15 Jul 2016

I have tested this item successfully on 7c1086f


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

avatar killoltailored killoltailored - test_item - 15 Jul 2016 - Tested successfully
avatar killoltailored
killoltailored - comment - 15 Jul 2016

I have tested this item successfully on 7c1086f


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

avatar killoltailored
killoltailored - comment - 15 Jul 2016

Tested successfully with Joomla Version: Joomla! 3.6.0 Stable [ Noether ] 12-July-2016 21:14 GMT OS: Windows 8.1, Browser: Firefox 47.0.1
screen shot 2016-07-14 at 23 42 36


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

avatar brianteeman brianteeman - change - 15 Jul 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 15 Jul 2016
Milestone Added:
avatar joomla-cms-bot joomla-cms-bot - change - 15 Jul 2016
Labels Removed: ?
avatar brianteeman
brianteeman - comment - 15 Jul 2016

This needs to be RTC but a bug on the tracker is preventing it being set

avatar NeerajTailored NeerajTailored - test_item - 15 Jul 2016 - Tested successfully
avatar NeerajTailored
NeerajTailored - comment - 15 Jul 2016

I have tested this item successfully on 7c1086f


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

avatar infograf768
infograf768 - comment - 15 Jul 2016

hmm
on Issues, when trying to change to RTC I get

0 UnexpectedValueException

No issue number received.

exception: exception 'UnexpectedValueException' with message 'No issue number received.' in ROOT/src/App/Tracker/Controller/Issue/Save.php:57
Stack trace:
#0 ROOT/src/JTracker/Application.php(170): App\Tracker\Controller\Issue\Save->execute()
#1 ROOT/vendor/joomla/application/src/AbstractWebApplication.php(134): JTracker\Application->doExecute()
#2 ROOT/www/index.php(101): Joomla\Application\AbstractWebApplication->execute()
#3 {main}

@mbabker

avatar brianteeman
brianteeman - comment - 15 Jul 2016

@infograf768 See joomla/jissues#834

On 15 July 2016 at 10:58, infograf768 notifications@github.com wrote:

hmm
on Issues, when trying to change to RTC I get

0 UnexpectedValueException

No issue number received.

exception: exception 'UnexpectedValueException' with message 'No issue number received.' in ROOT/src/App/Tracker/Controller/Issue/Save.php:57
Stack trace:
#0 ROOT/src/JTracker/Application.php(170): App\Tracker\Controller\Issue\Save->execute()
#1 ROOT/vendor/joomla/application/src/AbstractWebApplication.php(134): JTracker\Application->doExecute()
#2 ROOT/www/index.php(101): Joomla\Application\AbstractWebApplication->execute()
#3 {main}

@mbabker https://github.com/mbabker


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

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

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Jul 2016

so the RTC works already?

avatar brianteeman
brianteeman - comment - 18 Jul 2016

No - still broken

avatar tomartailored tomartailored - test_item - 19 Jul 2016 - Tested successfully
avatar tomartailored
tomartailored - comment - 19 Jul 2016

I have tested this item successfully on 7c1086f

I have tested this item successfully


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

avatar gunjanpatel
gunjanpatel - comment - 19 Jul 2016

@itfidds In case if you need more information about contributing You may follow the guides from here https://github.com/joomla/joomla-cms#do-you-want-to-improve-joomla
Thanks.


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

avatar brianteeman brianteeman - change - 21 Jul 2016
Milestone Removed:
avatar wilsonge wilsonge - close - 25 Jul 2016
avatar wilsonge wilsonge - merge - 25 Jul 2016
avatar wilsonge wilsonge - change - 25 Jul 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-07-25 11:22:10
Closed_By wilsonge
avatar joomla-cms-bot joomla-cms-bot - change - 25 Jul 2016
Category Libraries
avatar brianteeman
brianteeman - comment - 25 Jul 2016

Thanks

On 25 July 2016 at 12:22, George Wilson notifications@github.com wrote:

Merged #11120 #11120.


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

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

Add a Comment

Login with GitHub to post a comment