Labels |
Added:
?
|
What is the purpose of the URL column?
The original site what that report is comming from.
Wouldn't that be the domain then and not the URL ?
Would it be better to say enabled/disabled instead of published/unpublished?
Whatever is best. Enabled / Disabled should work too IIRC published/unpublished is just the default :)
For me it is like plugins and should be enabled etc. I will prepare a PR for that
Wouldn't that be the domain then and not the URL ?
One site only has (usually) only one domain right? When the report comes from example.org/joomla/10-joomla than this is the URL? Or I'm missing something?
So then there is a bug as all my entries in the column are identical as shown in the screenshot
So then there is a bug as all my entries in the column are identical as shown in the screenshot
Well do you have other sub pages where specific stuff is loaded that is not loaded on the main page? For example a youtube page?
The site is installed at http://localhost/joomla-cms
Below is the results after having detect mode on for site and admin and then visiting multiple pages on both. As you can see they all have the identical data in the URI column - I was expecting the full link to the page so that I am able to see what is causing the report.
I was expecting the full link to the page so that I am able to see what is causing the report.
Yes that is or better was the intended behavior seems to be a bug to me. Also seems that I actually implemented that issue in there.
In the end that line should be something like:
$report->blocked_uri = $parsedBlockedUri['scheme'] . '://' . $parsedBlockedUri['host'] . '/' . $parsedBlockedUri['path'] . '/' . $parsedBlockedUri['query'];
Maybe we even better throw that into the Uri class?
Should we move this to discussions?
No - its a bug
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-12-18 10:33:16 |
Closed_By | ⇒ | rdeutz |
The original site what that report is comming from.
Whatever is best. Enabled / Disabled should work too IIRC published/unpublished is just the default :)