This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 231068 - Regressions in code since fix for issue #228448
Summary: Regressions in code since fix for issue #228448
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Window System (show other bugs)
Version: 7.3
Hardware: All All
: P2 normal (vote)
Assignee: Theofanis Oikonomou
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-11 08:50 UTC by Martin Fousek
Modified: 2013-06-14 02:01 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
MultiViewCloneableTopComponent exception (2.21 KB, text/plain)
2013-06-11 08:50 UTC, Martin Fousek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Fousek 2013-06-11 08:50:15 UTC
Created attachment 135623 [details]
MultiViewCloneableTopComponent exception

Due to failing Multiview tests in our EE area I found out that the fix for issue #228448 probably made several regression in the Multiview code which should be fixed.

- our DDDataObject calls openView() method which should get the index of opened page... but the index doesn't correspond any more since XmlMultiViewEditorSupport#openView looks for available Perspectives where are all of them twice obtained twice - the original reason is likely inside MultiViewPeer#setMimeLookup()

- if I force correct index to be chosen, I'm getting NPE I attached from MultiViewCloneableTopComponent where is no mention about our DD loaders so I suppose that the issue lies somewhere in the multiview too

Related tests are the ddloaders ones shown in every our build:
http://bertram2.netbeans.org:8080/job/web-main-tests/lastCompletedBuild/testReport/

Please evaluate. Thanks a lot.
Comment 1 Theofanis Oikonomou 2013-06-12 12:26:38 UTC
No visible difference in behavior. Modified the fixed MVElements' indexes to make the tests pass. Fixed: http://hg.netbeans.org/core-main/rev/a9f31c416a16
Comment 2 Martin Fousek 2013-06-12 12:48:13 UTC
Ok. Thanks for quick fix.
Comment 3 Quality Engineering 2013-06-14 02:01:03 UTC
Integrated into 'main-golden', will be available in build *201306132301* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/a9f31c416a16
User: Theofanis Oikonomou <theofanis@netbeans.org>
Log: #231068 - Regressions in code since fix for issue #228448