? Success

User tests: Successful: Unsuccessful:

avatar Kubik-Rubik
Kubik-Rubik
15 Apr 2014

This pull request makes the collection of URLs optionally. This is useful to avoid unnecessary entries in the database.

JoomlaCode entry: [#33617] New option for redirect plugin: Collect URLs

http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33617

avatar Kubik-Rubik Kubik-Rubik - open - 15 Apr 2014
avatar Kubik-Rubik
Kubik-Rubik - comment - 15 Apr 2014

JoomlaCode entry: [#33617] New option for redirect plugin: Collect URLs

http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33617

avatar zero-24
zero-24 - comment - 15 Apr 2014

tested patch successful with com_patchtester :+1:

Thanks @Kubik-Rubik

avatar Kubik-Rubik Kubik-Rubik - change - 15 Apr 2014
The description was changed
Description <p>This pull request makes the collection of URLs optionally. This is useful to avoid unnecessary entries in the database.</p> <p>This pull request makes the collection of URLs optionally. This is useful to avoid unnecessary entries in the database.</p> <p>JoomlaCode entry: [#33617] New option for redirect plugin: Collect URLs</p> <p><a href="http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&amp;tracker_item_id=33617">http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&amp;tracker_item_id=33617</a></p>
Labels Added: ? ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 15 Apr 2014

@zero-24 Thanks for testing! Also changed the language strings according to your proposal. ;-)

avatar zero-24
zero-24 - comment - 15 Apr 2014

@Kubik-Rubik I like this :+1:

avatar Kubik-Rubik Kubik-Rubik - change - 16 Apr 2014
Title
New option for redirect plugin: Collect URLs
[#33617] New option for redirect plugin: Collect URLs
avatar Kubik-Rubik Kubik-Rubik - change - 16 Apr 2014
Title
New option for redirect plugin: Collect URLs
[#33617] New option for redirect plugin: Collect URLs
avatar Kubik-Rubik
Kubik-Rubik - comment - 16 Apr 2014

Updated pull request with a notification message in the component for the new option "Collect URLs" in the plugin. The message is only displayed if the plugin is enabled.

avatar Bakual
Bakual - comment - 17 Apr 2014

This has to wait till after the release of 3.3.0 since it introduces new language strings and we are currently in language freeze mode.
Otherwise I think it looks fine.

avatar infograf768
infograf768 - comment - 21 Apr 2014

This will have to be redone as it fails applying
Also, please alpha order the new language strings.

avatar Kubik-Rubik
Kubik-Rubik - comment - 21 Apr 2014

@infograf768 What exactly fails? Do have an error message? I used the Joomla! CodeSniffer file and the PHP Storm code styling file (by Matt Thomas).
Will reorder the language strings.

avatar Bakual
Bakual - comment - 21 Apr 2014

Apparently there is a conflct in some files and the PR can't be merged in current stage. You can rebase it to current staging and resolve the conflicts.

avatar Kubik-Rubik
Kubik-Rubik - comment - 21 Apr 2014

@infograf768 Solved conflicts, should be able to be applied.

avatar Bakual Bakual - reference | 581a03e - 5 May 14
avatar Bakual Bakual - close - 5 May 2014
avatar Bakual
Bakual - comment - 5 May 2014

Merged into 3.4-dev branch since it's a new feature.

avatar Bakual Bakual - change - 5 May 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-05-05 18:35:34
avatar Bakual Bakual - close - 5 May 2014

Add a Comment

Login with GitHub to post a comment