Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#7773] - Issue #7771 - router.php changed to handle SEO links for Archive view

? Success

User tests: Successful: Unsuccessful:

avatar schnuti
schnuti
26 Aug 2015

Tested with Archive module and standard search.

With a Menu item for Archived articles
link: "to-your-joomla/archived-articles/2010/12"

Without a Menu item for Archived articles I have 2 cases
1. The active menu item is not a Content menu.
link: "to-your-joomla/component/content/archive/2010/12"
2. The active menu item is a Content menu.
link: "to-your-joomla/component/content/archive/2010/12?Itemid=101
i.e. sometimes the Itemid is added automaticly

avatar schnuti schnuti - open - 26 Aug 2015
avatar schnuti schnuti - change - 26 Aug 2015
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Aug 2015
Labels Added: ?
avatar schnuti schnuti - reference | 5bc5fc2 - 26 Aug 15
avatar schnuti
schnuti - comment - 26 Aug 2015

Reference #7771

avatar schnuti schnuti - change - 26 Aug 2015
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2015-08-26 16:30:44
Closed_By ⇒ schnuti
avatar schnuti schnuti - close - 26 Aug 2015
avatar schnuti schnuti - close - 26 Aug 2015
avatar schnuti schnuti - change - 26 Aug 2015
Status Closed ⇒ New
Closed_Date 2015-08-26 16:30:44 ⇒
Closed_By schnuti ⇒
avatar schnuti schnuti - reopen - 26 Aug 2015
avatar schnuti schnuti - reopen - 26 Aug 2015
avatar zero-24 zero-24 - change - 26 Aug 2015
Category ⇒ Router / SEF
avatar zero-24 zero-24 - change - 26 Aug 2015
Status New ⇒ Pending
avatar snarf007 snarf007 - test_item - 10 Oct 2015 - Tested successfully
avatar snarf007
snarf007 - comment - 10 Oct 2015

I have tested this item :white_check_mark: successfully on cada7d0

Created a menu item with a single article and load the following in the article: {loadmodule articles_archive,Archived Articles}
Then clicked one of the items listed under "Archived Articles". My results:
Patch not applied:archived-articles/2011?view=archive&month=1
Patch applied: archived-articles/2011/1
Seems a improvement to CEO

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

avatar dam-man dam-man - test_item - 10 Oct 2015 - Tested successfully
avatar dam-man
dam-man - comment - 10 Oct 2015

I have tested this item :white_check_mark: successfully on cada7d0

Ran the above test too and seems to be working as expected - Seo improved from:

Patch not applied:archived-articles/2011?view=archive&month=1
Patch applied: archived-articles/2011/1

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

avatar zero-24 zero-24 - change - 10 Oct 2015
Status Pending ⇒ Ready to Commit
avatar zero-24 zero-24 - change - 10 Oct 2015
Milestone Added:
avatar zero-24
zero-24 - comment - 10 Oct 2015

RTC @joomla-cms-bot

avatar joomla-cms-bot joomla-cms-bot - change - 10 Oct 2015
Labels Added: ?
avatar rdeutz
rdeutz - comment - 11 Oct 2015

@Hackwar can you have a look if that will be a problem when merging the Router stuff in 3.5, Thanks!

avatar Hackwar
Hackwar - comment - 11 Oct 2015

I personally don't like changing the old router code, but this would not be a big problem when merging the router stuff. We need to update the legacrouter.php file for this to be adopted when merging the changes.

avatar rdeutz rdeutz - change - 11 Oct 2015
Status Ready to Commit ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2015-10-11 13:02:31
Closed_By ⇒ rdeutz
avatar rdeutz rdeutz - close - 11 Oct 2015
avatar joomla-cms-bot joomla-cms-bot - close - 11 Oct 2015
avatar rdeutz rdeutz - reference | 3c848a3 - 11 Oct 15
avatar rdeutz rdeutz - merge - 11 Oct 2015
avatar rdeutz rdeutz - close - 11 Oct 2015
avatar joomla-cms-bot joomla-cms-bot - change - 11 Oct 2015
Labels Removed: ?
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone Added:
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone Added:
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone

Add a Comment

Login with GitHub to post a comment