Check getBundle() returns non-null after PDE Manifest-editor opened

Fixes #1374
eclipse-pde · Aug 15, 2024 · 969eb9b · 969eb9b
1 parent 9b87c23
commit 969eb9b
Showing 1 changed file with 15 additions and 12 deletions.
diff --git a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/GeneralInfoSection.java b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/GeneralInfoSection.java
@@ -354,19 +354,22 @@ public void refresh() {
 		// Select the corresponding 'Bundle-Shape' radio button (eventually)
 		Stream.of(fBundleShapeDefault, fBundleShapeJar, fBundleShapeDir).forEach(b -> b.setSelection(false));
 		Button selectedBundleShape = fBundleShapeDefault;
-		IManifestHeader header = getBundle().getManifestHeader(ICoreConstants.ECLIPSE_BUNDLE_SHAPE);
-		if (header != null) {
-			String value = header.getValue();
-			if (value != null) {
-				selectedBundleShape = switch (value) {
-					case ICoreConstants.SHAPE_JAR -> fBundleShapeJar;
-					case ICoreConstants.SHAPE_DIR -> fBundleShapeDir;
-					default -> null; // unsupported value
-				};
+		IBundle bundle = getBundle();
+		if (bundle != null) {
+			IManifestHeader header = bundle.getManifestHeader(ICoreConstants.ECLIPSE_BUNDLE_SHAPE);
+			if (header != null) {
+				String value = header.getValue();
+				if (value != null) {
+					selectedBundleShape = switch (value) {
+						case ICoreConstants.SHAPE_JAR -> fBundleShapeJar;
+						case ICoreConstants.SHAPE_DIR -> fBundleShapeDir;
+						default -> null; // unsupported value
+					};
+				}
+			}
+			if (selectedBundleShape != null) {
+				selectedBundleShape.setSelection(true);
 			}
-		}
-		if (selectedBundleShape != null) {
-			selectedBundleShape.setSelection(true);
 		}
 		super.refresh();
 	}