? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
11 Apr 2016

The redirect component has a batch function to allow you to mass insert a set of urls. However this uses the common batch.php layout which has some javascript to ensure that you have selected one or more items to batch edit first.

As the batch functionality in com_redirect works completely differently to all other batch buttons and does not require (or need) you to select an existing item first it was not working correctly.

To resolve this I created a new layout "batchredirect" which doesnt have the javascript check.

Test instructions

Go to com_redirect and click the batch button on the toolbar - you will get a message about selecting an item first
Select an item and click the batch button. The modal will open and you can type new links on each line and when you click process you will see that new links have been created without any reference to the item you selected.

Apply the patch and you can click the batch button as intended without preselecting an irrelevant link first
Also check that this works when you do not have any redirects created.

avatar brianteeman brianteeman - open - 11 Apr 2016
avatar brianteeman brianteeman - change - 11 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 11 Apr 2016
Category Components Layout
avatar brianteeman brianteeman - change - 11 Apr 2016
Easy No Yes
avatar brianteeman brianteeman - change - 11 Apr 2016
The description was changed
avatar brianteeman
brianteeman - comment - 11 Apr 2016

Note: it might be better to rename this button from batch to "bulk import"


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 12 Apr 2016 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2016

I have tested this item :white_check_mark: successfully on e4d0d63

Prefer also to name Button "Batch" with another Name, cause "Batch" means in other Places different to this Button-Name.


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

avatar roland-d
roland-d - comment - 14 Apr 2016

@brianteeman Will you add a changed name for the button into this PR or that is for another PR?

avatar MATsxm MATsxm - test_item - 14 Apr 2016 - Tested successfully
avatar MATsxm
MATsxm - comment - 14 Apr 2016

I have tested this item :white_check_mark: successfully on e4d0d63

Successful (still the name issue remains) - thanks


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

avatar brianteeman brianteeman - change - 14 Apr 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 14 Apr 2016

RTC - thanks

@roland-d no i will not be creating a PR to change the name of the button. Got too much abuse last time not going to do that again


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

avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 14 Apr 2016
Milestone Added:
avatar wilsonge
wilsonge - comment - 14 Apr 2016

@brianteeman can we make this a component specific layout rather than a global one please?

avatar joomla-cms-bot
joomla-cms-bot - comment - 14 Apr 2016

This PR has received new commits.

CC: @franz-wohlkoenig, @MATsxm


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

avatar brianteeman brianteeman - change - 14 Apr 2016
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2016
Labels Added: ?
avatar brianteeman
brianteeman - comment - 14 Apr 2016

Do not test yet

avatar brianteeman
brianteeman - comment - 14 Apr 2016

@wilsonge - tried to do what you asked but something isnt quite right. i
assumed i missed something obvious

On 14 April 2016 at 20:40, Joomla! CMS Bot notifications@github.com wrote:

This PR has received new commits.

CC: @franz-wohlkoenig https://github.com/franz-wohlkoenig, @MATsxm

https://github.com/MATsxm

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


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9858 (comment)

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

avatar brianteeman brianteeman - change - 14 Apr 2016
Status Ready to Commit Pending
Labels
avatar brianteeman
brianteeman - comment - 14 Apr 2016

Removed RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2016
Labels Removed: ?
avatar brianteeman
brianteeman - comment - 14 Apr 2016

There is a problem that the toolbar will not do anything if you dont have any existing redirects. I cant see where this problem lies


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

avatar joomla-cms-bot
joomla-cms-bot - comment - 15 Apr 2016

This PR has received new commits.

CC: @franz-wohlkoenig, @MATsxm


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

avatar brianteeman
brianteeman - comment - 15 Apr 2016

Fixed the bug it is ready for testing again


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

avatar brianteeman brianteeman - change - 15 Apr 2016
The description was changed
avatar mikeveeckmans mikeveeckmans - test_item - 15 Apr 2016 - Tested successfully
avatar mikeveeckmans
mikeveeckmans - comment - 15 Apr 2016

I have tested this item :white_check_mark: successfully on d5a9c91

TEST OK


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Apr 2016 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Apr 2016

I have tested this item :white_check_mark: successfully on d5a9c91


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

avatar brianteeman brianteeman - change - 15 Apr 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 15 Apr 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 15 Apr 2016
Labels Added: ?
avatar MATsxm MATsxm - test_item - 15 Apr 2016 - Tested successfully
avatar MATsxm
MATsxm - comment - 15 Apr 2016

I have tested this item :white_check_mark: successfully on d5a9c91


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

avatar rdeutz rdeutz - close - 15 Apr 2016
avatar rdeutz rdeutz - merge - 15 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 15 Apr 2016
avatar rdeutz rdeutz - reference | 4deb5bd - 15 Apr 16
avatar rdeutz rdeutz - merge - 15 Apr 2016
avatar rdeutz rdeutz - close - 15 Apr 2016
avatar rdeutz rdeutz - change - 15 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-15 12:17:23
Closed_By rdeutz
avatar joomla-cms-bot joomla-cms-bot - change - 15 Apr 2016
Labels Removed: ?
avatar brianteeman brianteeman - head_ref_deleted - 15 Apr 2016
avatar Luchen6 Luchen6 - test_item - 15 Apr 2016 - Tested unsuccessfully
avatar Luchen6
Luchen6 - comment - 15 Apr 2016

I have tested this item :red_circle: unsuccessfully on d5a9c91

The Redirect Plugin is enabled. The option 'Collect URLs' is enabled.
.htaccess should be used.

Batch process can be applied with and without any redirect records.

No pre-selection is required.


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

avatar Luchen6
Luchen6 - comment - 15 Apr 2016

Sorry, I ment Succesfull test


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

avatar BurtNL BurtNL - test_item - 15 Apr 2016 - Tested successfully
avatar BurtNL
BurtNL - comment - 15 Apr 2016

I have tested this item :white_check_mark: successfully on d5a9c91


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

avatar brianteeman brianteeman - alter_testresult - 15 Apr 2016 - Luchen6: Tested successfully
avatar brianteeman brianteeman - alter_testresult - 15 Apr 2016 - luchen6: Tested successfully
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:

Add a Comment

Login with GitHub to post a comment