Attachment #212326 for bug #264246

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




		final String replacement = getReplacementString();
		final IDocument document = viewer.getDocument();
		final int length = replacement.length();
		boolean inAttribute = false, hasGroup = false;
		int offset = 0;
		char attType = 0;
		int exitPosition = -1;

						}
						break;
					case '/':
							if (!inAttribute) {
								inEndTag = i > 0 && replacement.charAt(i - 1) == '<';
							}
						break;
					case '>':
						if (!inAttribute) {
							if (!inAttribute && exitPosition == -1) {
								exitPosition = getReplacementOffset() + i + 1;
								if (!inEndTag) { // Don't add a position within the end-tag
									addPosition(model, document, getReplacementOffset() + i, 0); // position within start tag
									hasGroup = true;
								}
							}
							else {
								addPosition(model, document, getReplacementOffset() + i, 0); // position within start tag
								addPosition(model, document, getReplacementOffset() + i + 1, 0); // position after start tag
								hasGroup = true;
							}
						}
						break;

Return to bug 264246

Lines 60-66 Link Here

(-)src/org/eclipse/wst/xml/ui/internal/contentassist/MarkupCompletionProposal.java (-14 / +2 lines)
60	final String replacement = getReplacementString();	60	final String replacement = getReplacementString();
61	final IDocument document = viewer.getDocument();	61	final IDocument document = viewer.getDocument();
62	final int length = replacement.length();	62	final int length = replacement.length();
63	boolean inAttribute = false, hasGroup = false, inEndTag = false;	63	boolean inAttribute = false, hasGroup = false;
64	int offset = 0;	64	int offset = 0;
65	char attType = 0;	65	char attType = 0;
66	int exitPosition = -1;	66	int exitPosition = -1;
Lines 89-111 Link Here
89	}	89	}
90	break;	90	break;
91	case '/':	91	case '/':
92	if (!inAttribute) {
93	inEndTag = i > 0 && replacement.charAt(i - 1) == '<';
94	}
95	break;	92	break;
96	case '>':	93	case '>':
97	if (!inAttribute) {	94	if (!inAttribute) {
98	if (i == length - 1) {	95	if (!inAttribute && exitPosition == -1) {
99	exitPosition = getReplacementOffset() + i + 1;	96	exitPosition = getReplacementOffset() + i + 1;
100	if (!inEndTag) { // Don't add a position within the end-tag
101	addPosition(model, document, getReplacementOffset() + i, 0); // position within start tag
102	hasGroup = true;
103	}
104	}
105	else {
106	addPosition(model, document, getReplacementOffset() + i, 0); // position within start tag
107	addPosition(model, document, getReplacementOffset() + i + 1, 0); // position after start tag
108	hasGroup = true;
109	}	97	}
110	}	98	}
111	break;	99	break;