343558 – JEM needs to handle a flush better

Bug 343558 - JEM needs to handle a flush better

Summary: JEM needs to handle a flush better

Status:	RESOLVED FIXED

Alias:	None

Product:	WTP Java EE Tools
Classification:	WebTools
Component:	jst.jem (show other bugs)
Version:	1.5.5
Hardware:	PC Windows XP

Importance:	P2 major (vote)
Target Milestone:	1.5.5 P
Assignee:	Carl Anderson
QA Contact:	Chuck Bridgham

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	343559 343562
	Show dependency tree

Reported:	2011-04-21 10:38 EDT by Carl Anderson
Modified:	2011-08-05 16:13 EDT (History)
CC List:	0 users

See Also:

Flags:	cbridgha: review+

Attachments
Keep the current Methods "valid" (2.71 KB, patch) 2011-04-21 10:41 EDT, Carl Anderson	no flags	Details \| Diff
Keep the current Methods "valid", and add a null check (6.20 KB, patch) 2011-06-03 10:15 EDT, Carl Anderson	no flags	Details \| Diff
Keep the current Methods "valid", and add a null check (4.44 KB, patch) 2011-06-03 10:21 EDT, Carl Anderson	no flags	Details \| Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Carl Anderson

2011-04-21 10:38:15 EDT

Currently, if JEM is gathering (or has gathered) the Methods from a (JEM) JavaClass, and a flush() occurs, the Methods are removed from their parents and are invalid, resulting in NPEs and other errors without any warning.

There are things we can do, such as replacing the parent JavaClass with a JavaClassRef, that will at least keep the current Methods useful, while still allowing the flush to proceed.

Comment 1 Carl Anderson

2011-04-21 10:41:21 EDT

Created attachment 193836 [details]
Keep the current Methods "valid"

Comment 2 Carl Anderson

2011-04-21 10:43:21 EDT

This is causing (intermittent) major problems, including unnecessary termination, of various "headless" utilities based on WTP.

Comment 3 Chuck Bridgham

2011-04-22 09:48:48 EDT

approved

Comment 4 Carl Anderson

2011-06-03 10:15:34 EDT

Created attachment 197303 [details]
Keep the current Methods "valid", and add a null check

Comment 5 Carl Anderson

2011-06-03 10:21:56 EDT

Created attachment 197305 [details]
Keep the current Methods "valid", and add a null check

Removed extraneous (and accidentally included) System.out.println() calls.

Comment 6 Carl Anderson

2011-08-05 16:13:35 EDT

Committed to R1_5_5_patches