Debug info was lost

andypost - July 4, 2009 - 17:06
Project:Project Issue File Review
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:boombatower
Status:closed
Description

Everytime when exception happens on run.review watchdog receives ONLY internal $code but whole message (reason) is lost.

Same with rpc call - on debug stage we should have ability to view rpc.response data passed.

This could be very helpful when debuging both client and server side

AttachmentSize
pifr2_debug.patch1.54 KB

#1

andypost - July 4, 2009 - 18:08

XML-RPC result should posted after watchdog

AttachmentSize
pifr2_debug.patch 1.8 KB

#2

boombatower - July 9, 2009 - 00:58

Changed to straight up watchdog() calls.

AttachmentSize
510176-debug.patch 1.78 KB

#3

boombatower - July 17, 2009 - 19:52
Assigned to:Anonymous» boombatower
Status:needs review» fixed

This was committed, forgot to update status.

#4

andypost - July 24, 2009 - 00:29
Status:fixed» needs work

@boombatower Logging all XML-RPC into watchdog is expensive so better leave this for pifr_debug()!

So when debug is disabled (production) watchdog stays clean.

#5

andypost - July 24, 2009 - 00:38
Status:needs work» needs review

Another opinion - responce could be big and make bot slower

AttachmentSize
pifr_debug.patch 1.16 KB

#6

boombatower - July 24, 2009 - 00:56

Question to make sure then since PIFR_DEBUG still makes it run a single test...is this what you want?

#7

boombatower - October 28, 2009 - 04:25

Final patch.

AttachmentSize
510176-debug.patch 1.82 KB

#8

boombatower - October 28, 2009 - 04:26
Status:needs review» fixed

Committed.

#9

System Message - November 11, 2009 - 04:30
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.