? Success

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
10 Feb 2015

This removes unnecessary code that thinks it needs to differentiate between SEF and non-SEF URLs. That is actually the job of the router and not of the URL-requesting extension. The router will automatically remove all query elements that are identical with a menu item. This removes that code.

Pathway

Since we are always setting $url, we don't have to create that as an empty string in the beginning.
The changes around line 80ff simply remove the unnecessary condition for SEF or non-SEF URLs.

Menu Module

As I wrote above, the router automatically removes URL parameters that are part of the menu item. Thus, this whole check is useless.

How to test

  • See behavior of mod_menu and the breadcrumbs in Joomla
  • Apply patch
  • See that nothing changed.
avatar Hackwar Hackwar - open - 10 Feb 2015
avatar zero-24 zero-24 - change - 10 Feb 2015
Easy No Yes
avatar joomla-cms-bot joomla-cms-bot - change - 10 Feb 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 12 Feb 2015
Category Router / SEF
avatar bertmert
bertmert - comment - 9 Mar 2015

@test Success
No changed menu links and breadcrumbs links before and after patch. All SEF and NON-SEF options tested.

avatar Bakual
Bakual - comment - 9 Mar 2015

Tested and looks fine.
I also take the blame for that crappy suffix code. Somehow I remember that didn't work back then properly but now it certainly works without issues when taken out.

Setting to RTC.

avatar zero-24 zero-24 - change - 9 Mar 2015
Status Pending Ready to Commit
avatar Bakual Bakual - change - 9 Mar 2015
Labels Added: ?
avatar zero-24 zero-24 - alter_testresult - 10 Mar 2015 - bertmert: Tested successfully
avatar zero-24 zero-24 - alter_testresult - 10 Mar 2015 - Bakual: Tested successfully
avatar zero-24 zero-24 - close - 17 Mar 2015
avatar wilsonge wilsonge - change - 17 Mar 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-03-17 20:36:51
avatar wilsonge wilsonge - close - 17 Mar 2015
avatar wilsonge wilsonge - reference | - 17 Mar 15
avatar wilsonge wilsonge - merge - 17 Mar 2015
avatar wilsonge wilsonge - close - 17 Mar 2015
avatar wilsonge wilsonge - change - 17 Mar 2015
Milestone Added:
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?
avatar Hackwar Hackwar - head_ref_deleted - 6 Jan 2016

Add a Comment

Login with GitHub to post a comment