Bug 182822 - [DataBinding] AggregateValidationStatus assumes a default Realm has been set
Summary: [DataBinding] AggregateValidationStatus assumes a default Realm has been set
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: 3.4 M1   Edit
Assignee: Brad Reynolds CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-04-17 16:31 EDT by Brad Reynolds CLA
Modified: 2007-07-29 07:38 EDT (History)
3 users (show)

See Also:

Attachments
Add constructor that takes a realm to allow using AggregateValidationStatus without using Realm.setDefault(). (1.56 KB, patch)
2007-07-18 11:33 EDT, Matt Carter CLA 		no flags Details | Diff
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 Brad Reynolds CLA 2007-04-17 16:31:21 EDT 
The ComputedValue used in AggregateValidationStatus uses the no Realm constructor thus it assumes a default Realm has been set.  We need to expose a constructor from AggregateValidationStatus that accepts a Realm for this to run in.
Comment 1 Matt Carter CLA 2007-07-18 11:33:09 EDT 
Created attachment 74066 [details]
Add constructor that takes a realm to allow using AggregateValidationStatus without using Realm.setDefault().

Patch to add a constructor that takes a realm to AggregateValidationStatus (to allow it to be used without NPEs in an environment that's no longer using Realm.setDefault().)
Comment 2 Boris Bokowski CLA 2007-07-18 12:17:38 EDT 
Thanks for the patch. It shows that AggregateValidationStatus was added at the very last minute during 3.3...
Comment 3 Brad Reynolds CLA 2007-07-28 18:23:48 EDT 
FIXED > 20070728.

Patch 74066 applied to HEAD.
Comment 4 Brad Reynolds CLA 2007-07-28 18:34:08 EDT 
Matt, on your patches can you please put your name in the Copyright block?  The copyright block for this patch should look like the following.  This needs to be done for legal reasons.  Once you become a committer this is no longer necessary.

/*******************************************************************************
 * Copyright (c) 2005, 2007 IBM Corporation and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/legal/epl-v10.html
 *
 * Contributors:
 *     IBM Corporation - initial API and implementation
 *     Matt Carter - bug 182822
 *******************************************************************************/
Comment 5 Matt Carter CLA 2007-07-29 07:38:37 EDT 
Sure Brad, will do.