No Code Attached Yet
avatar brianteeman
brianteeman
8 Jan 2020

Steps to reproduce the issue

enable the redirects plugin and the collect urls option
visit several non-existing urls multiple times

com_redirects will now list all of the non-existing urls with a hit count

Take one of the non-existing urls and edit it so that it has a new url and enable the link

Revisit all of the same non-existing urls

Expected result

com_redirects increases the hit count for all the non-existing urls

Actual result

com_redirects increases the hit only for the disabled links

Additional comments

It seems wrong to me that we stop recording the hits because a link is being redirected.

I was looking at this to resolve the feature request #27436

After a site change a website may receive a number hits from cached Google results that will go away after Google's cache is updated. There is a 404 Hits column in the report, but adding a Last Hit Date column would help identify when a redirect was no longer needed.

avatar brianteeman brianteeman - open - 8 Jan 2020
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jan 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 8 Jan 2020
avatar HLeithner
HLeithner - comment - 8 Jan 2020

I found the same problem last year on a project and wondered way existing redirects aren't counted.

Have you checked whats the reason is?

Your linked issue seams unrelated?

avatar brianteeman
brianteeman - comment - 8 Jan 2020

The reason is that the code is not there.

The reason the linked issue is related was because the single line of code I added to make it log the last date didn't work because the enabled do not get updated.

It's an easy fix just dont know the reasoning. I would have thought it was useful

avatar alikon
alikon - comment - 8 Jan 2020

if i've understand correclty the issue please test #27440

avatar Quy Quy - change - 26 Aug 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-08-26 13:53:28
Closed_By Quy
Labels Added: No Code Attached Yet
Removed: ?
avatar Quy Quy - close - 26 Aug 2021
avatar Quy
Quy - comment - 26 Aug 2021

Please test PR #34640

Add a Comment

Login with GitHub to post a comment