Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 9 commits into from
Closed

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

wants to merge 9 commits into from

Conversation

Kubik-Rubik
Copy link
Member

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

@Kubik-Rubik
Copy link
Member Author

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

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

@@ -5,3 +5,5 @@

PLG_REDIRECT_XML_DESCRIPTION="The system redirect plugin enables the Joomla Redirect system to catch missing pages and redirect users."
PLG_SYSTEM_REDIRECT="System - Redirect"
PLG_SYSTEM_REDIRECT_FIELD_COLLECT_URLS_LABEL="Collect URLs"
PLG_SYSTEM_REDIRECT_FIELD_COLLECT_URLS_DESC="This option disables the collection of URLs. This is useful to avoid unnecessary load on the database."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you think to say here: "This option controls the collection of URLs ...."?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, why not? :-)

@zero-24
Copy link
Contributor

zero-24 commented Apr 15, 2014

tested patch successful with com_patchtester 👍

Thanks @Kubik-Rubik

@Kubik-Rubik
Copy link
Member Author

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

@zero-24
Copy link
Contributor

zero-24 commented Apr 15, 2014

@Kubik-Rubik I like this 👍

Display message in the component about the option state "Collect URLs"
@Kubik-Rubik Kubik-Rubik changed the title New option for redirect plugin: Collect URLs [#33617] New option for redirect plugin: Collect URLs Apr 16, 2014
@Kubik-Rubik
Copy link
Member Author

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.

@Bakual
Copy link
Contributor

Bakual commented Apr 17, 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.

@infograf768
Copy link
Member

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

@Kubik-Rubik
Copy link
Member Author

@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.

@Bakual
Copy link
Contributor

Bakual commented Apr 21, 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.

Solved conflicts

Solved conflicts

Solved conflicts
@Kubik-Rubik
Copy link
Member Author

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

Bakual pushed a commit that referenced this pull request May 5, 2014
@Bakual
Copy link
Contributor

Bakual commented May 5, 2014

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

@Bakual Bakual closed this May 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants