Skip to content

Commit 44823f8

Browse files
committed
[GEF] Use EditPartViewer where possible
The IEditPartViewer and the EditPartViewer interfaces have been aligned close enough so that in several places, we can work directly on the GEF interface.
1 parent c6f3515 commit 44823f8

File tree

40 files changed

+126
-137
lines changed

40 files changed

+126
-137
lines changed

org.eclipse.wb.core.java/src/org/eclipse/wb/internal/core/editor/DesignContextMenuProvider.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*******************************************************************************
2-
* Copyright (c) 2011, 2024 Google, Inc and others
2+
* Copyright (c) 2011, 2026 Google, Inc and others
33
*
44
* This program and the accompanying materials are made available under the
55
* terms of the Eclipse Public License 2.0 which is available at
@@ -16,7 +16,6 @@
1616
import org.eclipse.wb.core.editor.IContextMenuConstants;
1717
import org.eclipse.wb.core.editor.constants.IEditorPreferenceConstants;
1818
import org.eclipse.wb.core.model.ObjectInfo;
19-
import org.eclipse.wb.gef.core.IEditPartViewer;
2019
import org.eclipse.wb.internal.core.editor.actions.DesignPageActions;
2120
import org.eclipse.wb.internal.core.utils.execution.ExecutionUtils;
2221
import org.eclipse.wb.internal.core.utils.execution.RunnableEx;
@@ -25,6 +24,7 @@
2524

2625
import org.eclipse.core.runtime.preferences.InstanceScope;
2726
import org.eclipse.gef.EditPart;
27+
import org.eclipse.gef.EditPartViewer;
2828
import org.eclipse.jface.action.IMenuManager;
2929
import org.eclipse.jface.action.Separator;
3030

@@ -72,7 +72,7 @@ public static void addGroups(IMenuManager manager) {
7272
// Constructor
7373
//
7474
////////////////////////////////////////////////////////////////////////////
75-
public DesignContextMenuProvider(IEditPartViewer viewer, DesignPageActions pageActions) {
75+
public DesignContextMenuProvider(EditPartViewer viewer, DesignPageActions pageActions) {
7676
super(viewer);
7777
m_pageActions = pageActions;
7878
}

org.eclipse.wb.core.java/src/org/eclipse/wb/internal/core/editor/JavaDesignToolbarHelper.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*******************************************************************************
2-
* Copyright (c) 2011 Google, Inc.
2+
* Copyright (c) 2011, 2026 Google, Inc. and others.
33
*
44
* This program and the accompanying materials are made available under the
55
* terms of the Eclipse Public License 2.0 which is available at
@@ -14,10 +14,10 @@
1414

1515
import org.eclipse.wb.core.model.JavaInfo;
1616
import org.eclipse.wb.core.model.ObjectInfo;
17-
import org.eclipse.wb.gef.core.IEditPartViewer;
1817
import org.eclipse.wb.internal.core.editor.actions.DesignPageActions;
1918
import org.eclipse.wb.internal.core.nls.ExternalizeStringsContributionItem;
2019

20+
import org.eclipse.gef.EditPartViewer;
2121
import org.eclipse.jface.action.Separator;
2222
import org.eclipse.jface.action.ToolBarManager;
2323
import org.eclipse.swt.widgets.ToolBar;
@@ -48,9 +48,9 @@ public JavaDesignToolbarHelper(ToolBar toolBar) {
4848
//
4949
////////////////////////////////////////////////////////////////////////////
5050
/**
51-
* Initializes with {@link DesignPageActions} and {@link IEditPartViewer}.
51+
* Initializes with {@link DesignPageActions} and {@link EditPartViewer}.
5252
*/
53-
public void initialize(DesignPageActions pageActions, IEditPartViewer viewer) {
53+
public void initialize(DesignPageActions pageActions, EditPartViewer viewer) {
5454
super.initialize(viewer);
5555
m_pageActions = pageActions;
5656
}

org.eclipse.wb.core.java/src/org/eclipse/wb/internal/core/editor/actions/DesignPageActions.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
package org.eclipse.wb.internal.core.editor.actions;
1414

1515
import org.eclipse.wb.core.model.JavaInfo;
16-
import org.eclipse.wb.gef.core.IEditPartViewer;
1716
import org.eclipse.wb.internal.core.editor.DesignPage;
1817
import org.eclipse.wb.internal.core.editor.actions.assistant.LayoutAssistantAction;
1918
import org.eclipse.wb.internal.core.editor.actions.errors.ErrorsAction;
@@ -60,7 +59,7 @@ public final class DesignPageActions {
6059
* Initializes new {@link DesignPageActions}.
6160
*
6261
* @param viewer
63-
* the {@link IEditPartViewer} of {@link DesignPage}.
62+
* the {@link EditPartViewer} of {@link DesignPage}.
6463
* @param actionBars
6564
* the editor site {@link IActionBars}.
6665
*/

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/core/tools/CreationTool.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,11 @@
1212
*******************************************************************************/
1313
package org.eclipse.wb.gef.core.tools;
1414

15-
import org.eclipse.wb.gef.core.IEditPartViewer;
1615
import org.eclipse.wb.gef.core.requests.CreateRequest;
1716
import org.eclipse.wb.gef.core.requests.ICreationFactory;
1817

1918
import org.eclipse.gef.EditPart;
19+
import org.eclipse.gef.EditPartViewer;
2020
import org.eclipse.gef.Request;
2121

2222
/**
@@ -85,7 +85,7 @@ protected void selectAddedObjects() {
8585
CreateRequest request = (CreateRequest) getTargetRequest();
8686
Object model = request.getSelectObject();
8787
if (model != null) {
88-
IEditPartViewer viewer = getCurrentViewer();
88+
EditPartViewer viewer = getCurrentViewer();
8989
if (viewer != null) {
9090
EditPart editPart = viewer.getEditPartRegistry().get(model);
9191
if (editPart != null) {

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/core/tools/DragEditPartTracker.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
*******************************************************************************/
1313
package org.eclipse.wb.gef.core.tools;
1414

15-
import org.eclipse.wb.gef.core.IEditPartViewer;
1615
import org.eclipse.wb.gef.core.requests.ChangeBoundsRequest;
1716
import org.eclipse.wb.gef.core.requests.DragPermissionRequest;
1817
import org.eclipse.wb.gef.graphical.GraphicalEditPart;
@@ -22,6 +21,7 @@
2221
import org.eclipse.draw2d.IFigure;
2322
import org.eclipse.draw2d.geometry.Point;
2423
import org.eclipse.gef.EditPart;
24+
import org.eclipse.gef.EditPartViewer;
2525
import org.eclipse.gef.EditPartViewer.Conditional;
2626
import org.eclipse.gef.Request;
2727
import org.eclipse.gef.RequestConstants;
@@ -255,7 +255,7 @@ private List<Object> getOperationSetModels() {
255255
*/
256256
private void restoreSelectionFromModels(List<Object> models) {
257257
if (models != null) {
258-
IEditPartViewer viewer = getCurrentViewer();
258+
EditPartViewer viewer = getCurrentViewer();
259259
// prepare new EditPart's
260260
List<EditPart> newEditParts = new ArrayList<>();
261261
for (Object model : models) {

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/core/tools/PasteTool.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@
1212
*******************************************************************************/
1313
package org.eclipse.wb.gef.core.tools;
1414

15-
import org.eclipse.wb.gef.core.IEditPartViewer;
1615
import org.eclipse.wb.gef.core.requests.PasteRequest;
1716

1817
import org.eclipse.gef.EditPart;
18+
import org.eclipse.gef.EditPartViewer;
1919
import org.eclipse.gef.Request;
2020
import org.eclipse.jface.viewers.StructuredSelection;
2121

@@ -68,7 +68,7 @@ protected Request createTargetRequest() {
6868

6969
@Override
7070
protected void selectAddedObjects() {
71-
final IEditPartViewer viewer = getCurrentViewer();
71+
final EditPartViewer viewer = getCurrentViewer();
7272
// prepare pasted EditPart's
7373
List<EditPart> editParts = new ArrayList<>();
7474
{

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/core/tools/SelectEditPartTracker.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,8 @@
1212
*******************************************************************************/
1313
package org.eclipse.wb.gef.core.tools;
1414

15-
import org.eclipse.wb.gef.core.IEditPartViewer;
16-
1715
import org.eclipse.gef.EditPart;
16+
import org.eclipse.gef.EditPartViewer;
1817
import org.eclipse.gef.RequestConstants;
1918
import org.eclipse.gef.requests.SelectionRequest;
2019
import org.eclipse.swt.SWT;
@@ -133,7 +132,7 @@ protected boolean handleDoubleClick(int button) {
133132
private void performSelection() {
134133
if (!m_isSelected) {
135134
m_isSelected = true;
136-
IEditPartViewer viewer = getCurrentViewer();
135+
EditPartViewer viewer = getCurrentViewer();
137136
//
138137
if ((m_stateMask & SWT.CONTROL) != 0) {
139138
if (viewer.getSelectedEditParts().contains(m_sourceEditPart)) {

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/core/tools/TargetingTool.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*******************************************************************************
2-
* Copyright (c) 2011, 2025 Google, Inc. and others.
2+
* Copyright (c) 2011, 2026 Google, Inc. and others.
33
*
44
* This program and the accompanying materials are made available under the
55
* terms of the Eclipse Public License 2.0 which is available at
@@ -12,10 +12,9 @@
1212
*******************************************************************************/
1313
package org.eclipse.wb.gef.core.tools;
1414

15-
import org.eclipse.wb.gef.core.IEditPartViewer;
16-
1715
import org.eclipse.draw2d.IFigure;
1816
import org.eclipse.gef.EditPart;
17+
import org.eclipse.gef.EditPartViewer;
1918
import org.eclipse.gef.EditPartViewer.Conditional;
2019
import org.eclipse.gef.Request;
2120
import org.eclipse.gef.commands.Command;
@@ -27,7 +26,7 @@
2726
* The base implementation for tools which perform targeting of editparts. Targeting tools may
2827
* operate using either mouse drags or just mouse moves. Targeting tools work with a <i>target</i>
2928
* request. This request is used along with the mouse location to obtain an active target from the
30-
* current {@link IEditPartViewer}. This target is then asked for the <code>{@link Command}</code>
29+
* current {@link EditPartViewer}. This target is then asked for the <code>{@link Command}</code>
3130
* that performs the given request. The target is also asked to show target feedback.
3231
*
3332
* @author lobas_av

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/core/tools/ToolUtilities.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*******************************************************************************
2-
* Copyright (c) 2011, 2024 Google, Inc. and others.
2+
* Copyright (c) 2011, 2026 Google, Inc. and others.
33
*
44
* This program and the accompanying materials are made available under the
55
* terms of the Eclipse Public License 2.0 which is available at
@@ -12,9 +12,8 @@
1212
*******************************************************************************/
1313
package org.eclipse.wb.gef.core.tools;
1414

15-
import org.eclipse.wb.gef.core.IEditPartViewer;
16-
1715
import org.eclipse.gef.EditPart;
16+
import org.eclipse.gef.EditPartViewer;
1817

1918
import java.util.ArrayList;
2019
import java.util.Collections;
@@ -31,7 +30,7 @@ public class ToolUtilities {
3130
* Returns a list containing the top level selected {@link EditPart}'s based on the viewer's
3231
* selection. If selection parents of edit parts is differed returns empty list.
3332
*/
34-
public static List<EditPart> getSelectionWithoutDependants(IEditPartViewer viewer) {
33+
public static List<EditPart> getSelectionWithoutDependants(EditPartViewer viewer) {
3534
List<EditPart> operationSet = new ArrayList<>();
3635
// add selected EditPart's only if their parent is not added yet
3736
{

org.eclipse.wb.core/src-gef/org/eclipse/wb/gef/graphical/tools/MarqueeSelectionTool.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414

1515
import org.eclipse.wb.draw2d.Figure;
1616
import org.eclipse.wb.draw2d.FigureUtils;
17-
import org.eclipse.wb.gef.core.IEditPartViewer;
1817
import org.eclipse.wb.gef.core.tools.Tool;
1918
import org.eclipse.wb.gef.graphical.GraphicalEditPart;
2019

@@ -24,6 +23,7 @@
2423
import org.eclipse.draw2d.IFigure;
2524
import org.eclipse.draw2d.geometry.Rectangle;
2625
import org.eclipse.gef.EditPart;
26+
import org.eclipse.gef.EditPartViewer;
2727
import org.eclipse.gef.LayerConstants;
2828
import org.eclipse.gef.Request;
2929
import org.eclipse.gef.RequestConstants;
@@ -164,7 +164,7 @@ protected boolean handleDragInProgress() {
164164
* the selection and select the new group
165165
*/
166166
private void performMarqueeSelect() {
167-
IEditPartViewer viewer = getCurrentViewer();
167+
EditPartViewer viewer = getCurrentViewer();
168168
List<EditPart> newSelections = calculateNewSelection();
169169
//
170170
if (m_selectionMode == APPEND_MODE) {

0 commit comments

Comments
 (0)