Problem/Motivation

The reports at /admin/reports/page-not-found and /admin/reports/access-denied would be more useful if they included the date and time of the most recent instance of each item.

Steps to reproduce

1. Enable dblog
2. Record some 404 or 403 errors
3. View the Top 'page not found' errors or Top 'access denied' errors report

Proposed resolution

Add a Latest instance column

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3291914

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

malcomio created an issue. See original summary.

malcomio’s picture

malcomio commented
Issue summary: View changes
cilefen’s picture

cilefen commented
Version: 10.1.x-dev » 9.5.x-dev

Feature requests can go in 9.5.x.

Version: 9.5.x-dev » 10.1.x-dev

Drupal 9.5.0-beta2 and Drupal 10.0.0-beta2 were released on September 29, 2022, which means new developments and disruptive changes should now be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 10.1.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch, which currently accepts only minor-version allowed changes. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

ressa’s picture

ressa
he/him
commented

Perhaps when #2401463: Make dblog entities is committed, adding this feature could be a potential follow up issue?

malcomio opened merge request !13477

malcomio’s picture

malcomio commented
Status: Active » Needs review

Seeing this update, I had a quick look at the code controlling these reports - seems like a relatively straightforward addition.

Obviously #2401463: Make dblog entities is a much bigger issue with more to discuss, but the change in MR 13477 seems to work.

ressa’s picture

ressa
he/him
commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks -- this small change will improve the usefulness of these pages a lot.

poker10’s picture

poker10 commented
Status: Reviewed & tested by the community » Postponed
Issue tags: +Usability
Related issues: +#2105431: Rename the table headers on the watchdog 403/404 controllers

Thanks for working on this.

I reviewed this issue and MR and I personally think, that we should not update these hardcoded tables in the current state, but instead we should try to convert them to views - which would give us much more options to customize them (also for site owners themself). I found that such issue already existed: #2986126: Convert "Top 'access denied' errors.", "Top 'page not found' errors" and Top search phrases to views .

Regarding this issue - since it is adding a new column to the table, I think that is an UI change and as per Usability core gate it needs screenshots in the IS and probably also a feedback from the UX Team about the new column title. There is also another similar issue to update the other column title here: #2105431: Rename the table headers on the watchdog 403/404 controllers

I am postponing this issue on #2986126: Convert "Top 'access denied' errors.", "Top 'page not found' errors" and Top search phrases to views. Thanks!

ressa’s picture

ressa
he/him
commented

Thanks for taking a look at this and the other issue @poker10, and even including an example. I agree that having this and related Views as configuration would be the preferred method, since it would allow for more features to be included, and allow customization.

At least, there's a patch here, in the meantime, until it is possible :)

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.