[#20199] - Category link with ?layout= when active Modern route
- Closed
- 18 Apr 2018
- Medium
- Build: staging
- # 20199
Steps to reproduce the issue
- Go to Articles: Options and select Modern as URL Routing
- If need create Category and Article
- Create Menu Item Category blog (not home)
- Go to page and see category link in article blog
For example category alias will be block
Expected result
site.com/blog
Actual result
site.com/blog?layout=
System information (as much as possible)
PHP 7.2.2 / 7.1.11
Joomla 3.8.7
Additional comments
In truth, I do not know in which update it appeared. This was noticed by one of the clients.
I understand that this is a Modern Route however it's still a bug
I watched the problem on 3 websites
| Labels |
Added:
?
|
||
| Title |
|
||||||
| Title |
|
||||||
| Category | ⇒ | Router / SEF |
Can't confirm on live 3.8.7-Site but its on php 7.0.4.
| Status | New | ⇒ | Discussion |
@brianteeman in clean site all fine, However, this site was created yesterday
https://dev.joomlazen.com/en/
In single language sites same problem
I try to reproduce the exact path that leads to the problem or find & fix the reason
@mbabker Yes #19681 is the reason when i comments these lines the link became normal
in my site where override com_content link was /blog/web.html?layout=septdircategory
when comment
// $jinput = JFactory::getApplication()->input;
// $layout = $jinput->get('layout');
//
// if ($layout !== '')
// {
// $link .= '&layout=' . $layout;
// }
link was has become /blog/web.html
In sites where com_content link become norlmal without ?layout
$layout = $jinput->get('layout');So if layout is null (not set)
you will get the layout variable appended as empty, right ? because null !== ''
Maybe fix with
$layout = $jinput->get('layout', '', 'CMD');or
if (!empty($layout))| Closed_By | Quy | ⇒ | joomla-cms-bot |
Quy
- | Status | Discussion | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-04-18 20:50:43 |
| Closed_By | ⇒ | Quy |
Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/20199
by one of or all of?
I cant replicate it myself on a clean install