210998 - process bug spews out unitialized value error messages from about line 368

Status: RESOLVED DUPLICATE of bug 223706

(Bugzilla :: Creating/Changing Bugs, defect)

Description

[Jason Pyeron]

User-Agent:       Mozilla/4.0 (compatible; MSIE 5.5; Windows NT 5.0; .NET CLR 1.0.3705)
Build Identifier: 

have patch, did not try to chase down the root cause.

It happened when the oldvalue was undef and newvalue was TestComponent.

it was durring a multiple bug move to TestProduct with no comment.

Reproducible: Always

Steps to Reproduce:

Comment 1

[Jason Pyeron]

Attachment: patch

Comment 2

[Jason Pyeron]

Can someone verify, and check in this patch?

Comment 3

[Zach Lipton [:zach]]

Jason,

In bugzilla, we only mark bugs as fixed once they have actually been fixed, ie
the problem is gone in the cvs tip of Bugzilla. In this case, you have a first
patch attempt, but the problem is not actually fixed yet. 

Reopening

Comment 4

[Dave Miller [:justdave]]

What Zach actually means is the bug isn't marked fixed until the fix is checked
into cvs.

Comment 5

[Jason Pyeron]

Attachment: bugzilla-210998-v1.02.diff

fixed some bit rot

Comment 6

[Jason Pyeron]

Comment on attachment 160896 [details] [diff] [review]
bugzilla-210998-v1.02.diff

this is over 1.5 years old, please review

I TOOK THE TIME to update it

Comment 7

[Gervase Markham [:gerv]]

Comment on attachment 160896 [details] [diff] [review]
bugzilla-210998-v1.02.diff

I think your comment points out the correct fix - we should track down why
oldvalue is undefined, because it shouldn't be.

Gerv

Comment 8

[Marc Schumann [:Wurblzap]]

In bug 223706, comment 4 Dave says why oldvalue can be undefined.

Comment 9

[Jason Pyeron]

overworked underpaid, and this is what you get.
I filled two bugs for the same issue.

*** This bug has been marked as a duplicate of 223706 ***