[drupal-devel] [task] Display referer with "page not found" warnings

[kbahey]

Issue status update for 
http://drupal.org/node/23651
Post a follow up: 
http://drupal.org/project/comments/add/23651

 Project:      Drupal
 Version:      4.6.2
 Component:    watchdog.module
 Category:     tasks
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Thox
 Updated by:   kbahey
 Status:       patch

+1 on functionality too.




kbahey



Previous comments:
------------------------------------------------------------------------

Thu, 26 May 2005 11:05:16 +0000 : Thox

I get plenty of "page not found" warnings when I look at the admin page,
but I have no way to fix them. The pages (rightfully) no longer exist,
so I'd simply like to stop links pointing there. If the referering page
was part of the message, that would be far more helpful.




------------------------------------------------------------------------

Tue, 26 Jul 2005 07:52:54 +0000 : ahoeben

Referer / referrer loggin would be usefull for a more than just page not
found warnings. I'ld say include them where available.




------------------------------------------------------------------------

Tue, 26 Jul 2005 09:20:13 +0000 : ahoeben

This requires minor changes in bootstrap.inc (watchdog() function), the
watchdog module, as well as an alteration to the watchdog table. I'll
try to make a patch




------------------------------------------------------------------------

Wed, 27 Jul 2005 21:06:52 +0000 : ahoeben

Attachment: http://drupal.org/files/issues/watchdog-referrer.patch (1.6 KB)

Here's the patch against 4.6.2.


First, you need to execute this SQL statement to alter the watchdog
table:
ALTER TABLE `watchdog` ADD COLUMN `referer` varchar (128)  NOT NULL 


Next, the patch includes two one-line changes, one each for
includes/bootstrap.inc and modules/watchdog.module
(you may have to split up the patch file and run them one by one or
manually)




------------------------------------------------------------------------

Wed, 27 Jul 2005 21:23:07 +0000 : drumm

+1 for this functionality, but it needs to be a patch against Drupal
CVS.