Set active class in mod_articles_categories when SEF is off

[csthomas]

Pull Request for Issue #19194 .

Summary of Changes

Do not replace & by & in URL in JRoute::_(...) when comparing to $_SERVER['REQUEST_URI']

Testing Instructions

Test issue #19194

Expected result

When SEF is off.
Active class will appear when we are on the category page.

Actual result

When SEF is off active class in (<li class="active">) is missing.

Documentation Changes Required

None

[joomdonation]

How do you think about using code like this:

1. Add the code to get necessary variables from input at the beginning of file:

$input  = JFactory::getApplication()->input;
$option = $input->getCmd('option');
$view   = $input->getCmd('view');
$id     = $input->getInt('id');

2. And then change if command to:

if ($option == 'com_content' && $view == 'category' && $id == $item->id) echo ' class="active"';

[csthomas]

@joomdonation Your suggestion is good, thanks.

[joomdonation]

I have tested this item ✅ successfully on f4dce78

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19197.}

[Quy]

Remove the space after <li since there is a space before class.

[csthomas]

Done.

[Quy]

I have tested this item ✅ successfully on fc0c98f

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19197.}

[ghost]

@joomdonation can you please retest?

[Quy]

@franz-wohlkoenig It is only coding style to remove an extra space. It does not affect test result.

[joomdonation]

The last change is just code style, so my test is still success. You can mark this PR as RTC

[ghost]

Ready to Commit after two successful tests.