Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Updated: Comment 0
Problem/Motivation
This 403/404 page should better describe that the message is actually the page on which the access denied/not found happened.
Proposed resolution
Remaining tasks
User interface changes
API changes
Related Issues
Comment | File | Size | Author |
---|---|---|---|
#6 | D8-Rename_watchdog_table_header_text-2105431.patch | 5.04 KB | Sagar Ramgade |
#3 | D8-Rename_watchdog_table_header_text-2105431.patch | 716 bytes | Sagar Ramgade |
#1 | D8-Rename_watchdog_table_header_text-2105431.patch | 674 bytes | Sagar Ramgade |
Comments
Comment #1
Sagar Ramgade CreditAttribution: Sagar Ramgade commentedHi,
Does it makes more sense to rename it to "URL", Attached patch does that.
Comment #2
dawehnerThis is also used for 'search', so maybe we should check whether we show 403/404 results.
Comment #3
Sagar Ramgade CreditAttribution: Sagar Ramgade commentedI didn't notice that, Here is the updated patch.
Comment #4
dawehnerCool
Comment #5
alexpottShouldn't we do this in a more elegant and extensible way? For example add:
And change
DbLogController::topLogMessages
to accept and row title?Comment #6
Sagar Ramgade CreditAttribution: Sagar Ramgade commentedHi Alex,
Here's the patch as per your suggestion.
Comment #10
dagmarMoved to
dblog.module
for future triage.Comment #11
dagmarThis needs to be updated to array
[]
syntax.This change should not be part of this patch.
This is breaking Backward Compatibility. You could use the info returned by topLogMessages() and change the header there.