View | Details | Raw Unified | Return to bug 95580 | Differences between
and this patch

Collapse All | Expand All

(-)model/org/eclipse/jdt/internal/core/CreateFieldOperation.java (-4 / +13 lines)
Lines 11-16 Link Here
11
package org.eclipse.jdt.internal.core;
11
package org.eclipse.jdt.internal.core;
12
12
13
import org.eclipse.jdt.core.ICompilationUnit;
13
import org.eclipse.jdt.core.ICompilationUnit;
14
import org.eclipse.jdt.core.IField;
14
import org.eclipse.jdt.core.IJavaElement;
15
import org.eclipse.jdt.core.IJavaElement;
15
import org.eclipse.jdt.core.IJavaModelStatus;
16
import org.eclipse.jdt.core.IJavaModelStatus;
16
import org.eclipse.jdt.core.IJavaModelStatusConstants;
17
import org.eclipse.jdt.core.IJavaModelStatusConstants;
Lines 71-81 Link Here
71
protected void initializeDefaultPosition() {
72
protected void initializeDefaultPosition() {
72
	IType parentElement = getType();
73
	IType parentElement = getType();
73
	try {
74
	try {
74
		IJavaElement[] elements = parentElement.getFields();
75
		IField[] fields = parentElement.getFields();
75
		if (elements != null && elements.length > 0) {
76
		if (fields != null && fields.length > 0) {
76
			createAfter(elements[elements.length - 1]);
77
			final IField lastField = fields[fields.length - 1];
78
			if (parentElement.isEnum()) {
79
				IField field = lastField;
80
				if (!field.isEnumConstant()) {
81
					createAfter(lastField);
82
				}
83
			} else {
84
				createAfter(lastField);
85
			}
77
		} else {
86
		} else {
78
			elements = parentElement.getChildren();
87
			IJavaElement[] elements = parentElement.getChildren();
79
			if (elements != null && elements.length > 0) {
88
			if (elements != null && elements.length > 0) {
80
				createBefore(elements[0]);
89
				createBefore(elements[0]);
81
			}
90
			}

Return to bug 95580