Attachment 258350 Details for Bug 482577 – Fix1

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Fix1

manifest_option1.patch (text/plain), 3.93 KB, created by Vikas Chandra

on 2015-11-30 03:26:59 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Vikas Chandra

Created: 2015-11-30 03:26:59 EST

Size: 3.93 KB

patch

obsolete

>diff --git a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/PDEUIMessages.java b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/PDEUIMessages.java
>index f354d3c..6aad132 100644
>--- a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/PDEUIMessages.java
>+++ b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/PDEUIMessages.java
>@@ -790,6 +790,10 @@
> 
> 	// Plug-in Manifest Editor ########################
> 
>+	public static String ManifestEditor_AddingSourcePages;
>+
>+	public static String ManifestEditor_SettingActivePage;
>+
> 	public static String ManifestEditor_DetailExtensionPointSection_title;
> 	public static String ManifestEditor_DetailExtensionPointSection_new;
> 	public static String ManifestEditor_DetailExtensionPointSection_newExtensionPoint;
>diff --git a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/ManifestEditor.java b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/ManifestEditor.java
>index aee4197..811b63f 100644
>--- a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/ManifestEditor.java
>+++ b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/ManifestEditor.java
>@@ -17,7 +17,8 @@
> import org.eclipse.core.filesystem.EFS;
> import org.eclipse.core.filesystem.IFileStore;
> import org.eclipse.core.resources.*;
>-import org.eclipse.core.runtime.CoreException;
>+import org.eclipse.core.runtime.*;
>+import org.eclipse.core.runtime.jobs.JobChangeAdapter;
> import org.eclipse.core.runtime.preferences.IEclipsePreferences;
> import org.eclipse.jface.action.IToolBarManager;
> import org.eclipse.jface.preference.IPreferenceStore;
>@@ -42,6 +43,7 @@
> import org.eclipse.ui.*;
> import org.eclipse.ui.ide.FileStoreEditorInput;
> import org.eclipse.ui.part.FileEditorInput;
>+import org.eclipse.ui.progress.UIJob;
> import org.eclipse.ui.views.properties.IPropertySheetPage;
> import org.osgi.service.prefs.BackingStoreException;
> 
>@@ -452,9 +454,36 @@
> 		} catch (PartInitException e) {
> 			PDEPlugin.logException(e);
> 		}
>-		addSourcePage(BundleInputContext.CONTEXT_ID);
>-		addSourcePage(PluginInputContext.CONTEXT_ID);
>-		addSourcePage(BuildInputContext.CONTEXT_ID);
>+
>+		UIJob addSourcePageJob = new UIJob(PDEUIMessages.ManifestEditor_AddingSourcePages) {
>+			@Override
>+			public IStatus runInUIThread(IProgressMonitor monitor) {
>+				addSourcePage(BundleInputContext.CONTEXT_ID);
>+				addSourcePage(PluginInputContext.CONTEXT_ID);
>+				addSourcePage(BuildInputContext.CONTEXT_ID);
>+				return Status.OK_STATUS;
>+			}
>+		};
>+		addSourcePageJob.addJobChangeListener(new JobChangeAdapter() {
>+			@Override
>+			public void done(org.eclipse.core.runtime.jobs.IJobChangeEvent event) {
>+				UIJob job = new UIJob(PDEUIMessages.ManifestEditor_SettingActivePage) {
>+					@Override
>+					public IStatus runInUIThread(IProgressMonitor monitor) {
>+						String pageToShow = computeInitialPageId();
>+						// have a check here
>+						if (pageToShow != null)
>+							setActivePage(pageToShow);
>+						return Status.OK_STATUS;
>+					}
>+				};
>+				job.setSystem(true);
>+				job.schedule();
>+			}
>+		});
>+		addSourcePageJob.schedule();
>+
>+
> 	}
> 
> 	private boolean isSourcePageID(String pageID) {
>diff --git a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/pderesources.properties b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/pderesources.properties
>index d257e78..574859b 100644
>--- a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/pderesources.properties
>+++ b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/pderesources.properties
>@@ -275,6 +275,8 @@
> FeatureOutlinePage_discoverUrls = Sites to Visit
> 
> ###### Plug-in Manifest Editor ########################
>+ManifestEditor_AddingSourcePages=Adding Source Pages
>+ManifestEditor_SettingActivePage=Setting Active Page
> ManifestEditor_DetailExtensionPointSection_title = All Extension Points
> ManifestEditor_DetailExtensionPointSection_new = Add...
> ManifestPackageRenameParticipant_packageRename=Rename packages referenced in plug-in manifest files

Actions: View | Diff

Attachments on bug 482577: 258350 | 258351