? Pending

User tests: Successful: Unsuccessful:

avatar julienV
julienV
28 Apr 2020

Pull Request for Issue #28858 .
duplicate of #28859

Summary of Changes

use getId when parsing urls

Testing Instructions

you need a component with a router stripping ids, and way to get to a specific frontend view without a matching menu item

Expected result

urls are built and parsed properly

Actual result

urls were built using getSegment, but not parsed using getId

Documentation Changes Required

avatar julienV julienV - open - 28 Apr 2020
avatar julienV julienV - change - 28 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Apr 2020
Category Libraries
avatar jwaisner jwaisner - change - 28 Apr 2020
Status Pending Duplicate Report
Closed_Date 0000-00-00 00:00:00 2020-04-28 15:19:34
Closed_By jwaisner
avatar joomla-cms-bot joomla-cms-bot - change - 28 Apr 2020
Status Duplicate Report Closed
Closed_Date 2020-04-28 15:19:34 2020-04-28 15:19:35
Closed_By jwaisner joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 28 Apr 2020
avatar joomla-cms-bot
joomla-cms-bot - comment - 28 Apr 2020

Set to "closed" on behalf of @jwaisner by The JTracker Application at issues.joomla.org/joomla-cms/28860

avatar jwaisner
jwaisner - comment - 28 Apr 2020

Duplicate of #28859


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

avatar Quy
Quy - comment - 28 Apr 2020

This should be opened for staging that will be ported to v3.10 and close the other one.

avatar Quy Quy - change - 28 Apr 2020
Status Closed New
Closed_Date 2020-04-28 15:19:35
Closed_By joomla-cms-bot
Labels Added: ?
avatar Quy Quy - change - 28 Apr 2020
Status New Pending
avatar Quy Quy - reopen - 28 Apr 2020
avatar zero-24 zero-24 - change - 12 Jun 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-06-12 19:17:28
Closed_By zero-24
Labels Added: ?
Removed: ?
avatar zero-24
zero-24 - comment - 12 Jun 2022

I have just double checked this with @Hackwar and this is not an change we are going to accept into 3.10 for that reason I'm closing this PR. Thanks

avatar zero-24 zero-24 - close - 12 Jun 2022

Add a Comment

Login with GitHub to post a comment