Remove two duplicate watchdog calls

jonathan1055 - August 28, 2009 - 19:47
Project:Web Links
Version:6.x-2.x-dev
Component:Contrib: Checker
Category:support request
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hi Nancy,

There are two watchdog calls in weblinks_checker.module which I modified and/or replaced as part of #472970: Enhancements to weblinks_checker but the originals got left in by mistake so they need to be deleted.

The first is immediately after the line setting $extra = $response->error, and is now covered by the two calls within the following conditional blocks which take into account the new messaging detail options.

The second is just after the $status == '301' testing, where the new call adds more detail.

I have attached a patch to delete these two lines, rolled against Dev of 28th Aug.

Jonathan

AttachmentSize
_weblinks_checker.module.remove_two_watchdogs.patch.txt1.21 KB

#1

NancyDru - August 28, 2009 - 23:49
Status:active» fixed

Thanks. Committed.

#2

NancyDru - August 29, 2009 - 01:48
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.