JBoss Tools SVN: r17716 - trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/contentassist.
by jbosstools-commits＠lists.jboss.org
Author: vrubezhny
Date: 2009-09-24 09:53:59 -0400 (Thu, 24 Sep 2009)
New Revision: 17716
Modified:
trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/contentassist/AbstractXMLContentAssistProcessor.java
Log:
JBIDE-2332 CA with proposal list for comonent's atributes doesn't work before double quotes.
Issue is fixed
Modified: trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/contentassist/AbstractXMLContentAssistProcessor.java
===================================================================
--- trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/contentassist/AbstractXMLContentAssistProcessor.java 2009-09-24 13:37:24 UTC (rev 17715)
+++ trunk/jst/plugins/org.jboss.tools.jst.jsp/src/org/jboss/tools/jst/jsp/contentassist/AbstractXMLContentAssistProcessor.java 2009-09-24 13:53:59 UTC (rev 17716)
@@ -269,8 +269,24 @@
domnode = prevnode;
}
}
+
return super.getCompletionRegion(documentPosition, domnode);
}
+
+ protected ITextRegion getCompletionRegion(int offset, IStructuredDocumentRegion sdRegion) {
+ ITextRegion region = super.getCompletionRegion(offset, sdRegion);
+ if (region != null && region.getType() == DOMRegionContext.UNDEFINED) {
+ // FIX: JBIDE-2332 CA with proposal list for comonent's atributes doesn't work before double quotes.
+ // Sometimes, especially if we have a broken XML node, the region returned has UNDEFINED type.
+ // If so, we're try to use the prevoius region, which probably will be the region of type XML_TAG_NAME.
+
+ ITextRegion previousRegion = sdRegion.getRegionAtCharacterOffset(offset - 1);
+ if ((previousRegion != null) && (previousRegion != region) && (previousRegion.getTextLength() < previousRegion.getLength())) {
+ region = previousRegion;
+ }
+ }
+ return region;
+ }
/*
* (non-Javadoc)
* @see org.eclipse.wst.xml.ui.internal.contentassist.AbstractContentAssistProcessor#computeTagNameProposals(int, java.lang.String, org.eclipse.wst.sse.core.internal.provisional.text.ITextRegion, org.eclipse.wst.xml.core.internal.provisional.document.IDOMNode, org.eclipse.wst.xml.core.internal.provisional.document.IDOMNode)
15 years
JBoss Tools SVN: r17714 - workspace/qa and 1 other directory.
by jbosstools-commits＠lists.jboss.org
Author: aklimkovich
Date: 2009-09-24 07:23:24 -0400 (Thu, 24 Sep 2009)
New Revision: 17714
Added:
workspace/qa/TestWorkspaces.rar
Removed:
trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar
Log:
Moved remotely
Deleted: trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar
===================================================================
(Binary files differ)
Copied: workspace/qa/TestWorkspaces.rar (from rev 17713, trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar)
===================================================================
(Binary files differ)
15 years
JBoss Tools SVN: r17713 - workspace.
by jbosstools-commits＠lists.jboss.org
Author: aklimkovich
Date: 2009-09-24 05:33:20 -0400 (Thu, 24 Sep 2009)
New Revision: 17713
Added:
workspace/qa/
Log:
Created folder remotely
15 years
JBoss Tools SVN: r17712 - workspace.
by jbosstools-commits＠lists.jboss.org
Author: aklimkovich
Date: 2009-09-24 05:21:09 -0400 (Thu, 24 Sep 2009)
New Revision: 17712
Removed:
workspace/qa
Log:
Removed file/folder
Deleted: workspace/qa
===================================================================
(Binary files differ)
15 years
JBoss Tools SVN: r17711 - workspace.
by jbosstools-commits＠lists.jboss.org
Author: aklimkovich
Date: 2009-09-24 05:15:39 -0400 (Thu, 24 Sep 2009)
New Revision: 17711
Added:
workspace/qa
Log:
Copied remotely
Copied: workspace/qa (from rev 17710, trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar)
===================================================================
(Binary files differ)
15 years
JBoss Tools SVN: r17710 - trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard.
by jbosstools-commits＠lists.jboss.org
Author: akazakov
Date: 2009-09-24 03:48:50 -0400 (Thu, 24 Sep 2009)
New Revision: 17710
Modified:
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/SeamBaseOperation.java
Log:
https://jira.jboss.org/jira/browse/JBIDE-4944
Modified: trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/SeamBaseOperation.java
===================================================================
--- trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/SeamBaseOperation.java 2009-09-23 22:43:11 UTC (rev 17709)
+++ trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/SeamBaseOperation.java 2009-09-24 07:48:50 UTC (rev 17710)
@@ -110,7 +110,7 @@
int index=0;
for (String[] mapping : fileMappingCopy) {
file[index] = new File(mapping[1]);
- FileUtils.getFileUtils().copyFile(new File(mapping[0]), file[index],filters,true);
+ FileUtils.getFileUtils().copyFile(new File(mapping[0]), file[index],filters,false);
index++;
}
if(shouldTouchServer(seamPrjSet)) {
15 years
JBoss Tools SVN: r17707 - in trunk: jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/resources/jsfTest/WebContent/pages/JBIDE/4713 and 4 other directories.
by jbosstools-commits＠lists.jboss.org
Author: yradtsevich
Date: 2009-09-23 15:12:29 -0400 (Wed, 23 Sep 2009)
New Revision: 17707
Added:
trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/resources/jsfTest/WebContent/pages/JBIDE/4713/
trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/resources/jsfTest/WebContent/pages/JBIDE/4713/SelectWholeElement_JBIDE4713.html
trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/jbide/SelectWholeElement_JBIDE4713.java
Modified:
trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/JsfAllTests.java
trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/SelectionManager.java
trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/SelectionUtil.java
Log:
RESOLVED - issue JBIDE-4713: Visual selection isn't correct from selection bar.
https://jira.jboss.org/jira/browse/JBIDE-4713
- fixed
- added JUnit test
Added: trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/resources/jsfTest/WebContent/pages/JBIDE/4713/SelectWholeElement_JBIDE4713.html
===================================================================
--- trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/resources/jsfTest/WebContent/pages/JBIDE/4713/SelectWholeElement_JBIDE4713.html (rev 0)
+++ trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/resources/jsfTest/WebContent/pages/JBIDE/4713/SelectWholeElement_JBIDE4713.html 2009-09-23 19:12:29 UTC (rev 17707)
@@ -0,0 +1,9 @@
+<html>
+ <body>
+ <form>
+ BEFORE
+ <h1 id="selected-element">H1</h1>
+ AFTER
+ </form>
+ </body>
+</html>
Modified: trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/JsfAllTests.java
===================================================================
--- trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/JsfAllTests.java 2009-09-23 16:09:52 UTC (rev 17706)
+++ trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/JsfAllTests.java 2009-09-23 19:12:29 UTC (rev 17707)
@@ -77,6 +77,7 @@
import org.jboss.tools.jsf.vpe.jsf.test.jbide.JsfJbide2362Test;
import org.jboss.tools.jsf.vpe.jsf.test.jbide.OpenOnCssClassTest_JBIDE4775;
import org.jboss.tools.jsf.vpe.jsf.test.jbide.SelectAllAndCut_JBIDE4853;
+import org.jboss.tools.jsf.vpe.jsf.test.jbide.SelectWholeElement_JBIDE4713;
import org.jboss.tools.jsf.vpe.jsf.test.jbide.VPERefreshTest;
import org.jboss.tools.tests.ImportBean;
import org.jboss.tools.vpe.ui.test.VpeTestSetup;
@@ -104,6 +105,7 @@
suite.addTestSuite(JBIDE3519Test.class);
suite.addTestSuite(ContextMenuDoubleInsertionTest_JBIDE3888.class);
suite.addTestSuite(SelectAllAndCut_JBIDE4853.class);
+ suite.addTestSuite(SelectWholeElement_JBIDE4713.class);
suite.addTestSuite(JBIDE4037Test.class);
suite.addTestSuite(JBIDE3734Test.class);
suite.addTestSuite(JBIDE3617Test.class);
Added: trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/jbide/SelectWholeElement_JBIDE4713.java
===================================================================
--- trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/jbide/SelectWholeElement_JBIDE4713.java (rev 0)
+++ trunk/jsf/tests/org.jboss.tools.jsf.vpe.jsf.test/src/org/jboss/tools/jsf/vpe/jsf/test/jbide/SelectWholeElement_JBIDE4713.java 2009-09-23 19:12:29 UTC (rev 17707)
@@ -0,0 +1,70 @@
+/*******************************************************************************
+ * Copyright (c) 2007-2009 Red Hat, Inc.
+ * Distributed under license by Red Hat, Inc. All rights reserved.
+ * This program is made available under the terms of the
+ * Eclipse Public License v1.0 which accompanies this distribution,
+ * and is available at http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributor:
+ * Red Hat, Inc. - initial API and implementation
+ ******************************************************************************/
+package org.jboss.tools.jsf.vpe.jsf.test.jbide;
+
+import org.eclipse.swt.custom.StyledText;
+import org.eclipse.swt.graphics.Point;
+import org.eclipse.wst.sse.ui.internal.StructuredTextViewer;
+import org.jboss.tools.jsf.vpe.jsf.test.JsfAllTests;
+import org.jboss.tools.vpe.editor.VpeController;
+import org.jboss.tools.vpe.editor.util.HTML;
+import org.jboss.tools.vpe.editor.util.SelectionUtil;
+import org.jboss.tools.vpe.ui.test.TestUtil;
+import org.jboss.tools.vpe.ui.test.VpeTest;
+import org.mozilla.interfaces.nsIDOMElement;
+import org.mozilla.interfaces.nsIDOMNode;
+
+/**
+ * Tests for JIRA issue JBIDE-4713:
+ * Visual selection isn't correct from selection bar.
+ * (https://jira.jboss.org/jira/browse/JBIDE-4713 )
+ *
+ * @author yradtsevich
+ */
+public class SelectWholeElement_JBIDE4713 extends VpeTest {
+ private static final String TEST_PAGE_NAME
+ = "JBIDE/4713/SelectWholeElement_JBIDE4713.html"; //$NON-NLS-1$
+ private static final Point SELECTION_START = new Point(5, 13);
+ private static final Point SELECTION_END = new Point(5, 46);
+ private static final String SELECTED_ELEMENT_ID
+ = "selected-element"; //$NON-NLS-1$
+
+ public SelectWholeElement_JBIDE4713(String name) {
+ super(name);
+ }
+
+ public void testSelectWholeElement() throws Throwable {
+ VpeController vpeController = openInVpe(JsfAllTests.IMPORT_PROJECT_NAME,
+ TEST_PAGE_NAME);
+
+ StructuredTextViewer textViewer = vpeController.getSourceEditor()
+ .getTextViewer();
+ StyledText textWidget = textViewer.getTextWidget();
+
+ int selectionStartOffset = TestUtil.getLinePositionOffcet(
+ textViewer, SELECTION_START.x, SELECTION_START.y);
+ int selectionEndOffset = TestUtil.getLinePositionOffcet(
+ textViewer, SELECTION_END.x, SELECTION_END.y);
+
+ textViewer.setSelectedRange(selectionStartOffset,
+ selectionEndOffset - selectionStartOffset);
+ TestUtil.waitForIdle();
+
+ nsIDOMNode selectedNode = SelectionUtil.getLastSelectedNode(
+ vpeController.getPageContext());
+
+ assertTrue(selectedNode.getNodeType() == nsIDOMNode.ELEMENT_NODE);
+ nsIDOMElement element = (nsIDOMElement)
+ selectedNode.queryInterface(nsIDOMElement.NS_IDOMELEMENT_IID);
+
+ assertEquals(SELECTED_ELEMENT_ID, element.getAttribute(HTML.ATTR_ID));
+ }
+}
Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/SelectionManager.java
===================================================================
--- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/SelectionManager.java 2009-09-23 16:09:52 UTC (rev 17706)
+++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/SelectionManager.java 2009-09-23 19:12:29 UTC (rev 17707)
@@ -11,10 +11,7 @@
package org.jboss.tools.vpe.editor.template;
-import org.eclipse.jface.viewers.ISelection;
import org.eclipse.swt.graphics.Point;
-import org.eclipse.wst.sse.core.StructuredModelManager;
-import org.eclipse.wst.sse.core.internal.provisional.IStructuredModel;
import org.eclipse.wst.sse.ui.StructuredTextEditor;
import org.eclipse.wst.xml.core.internal.document.NodeImpl;
import org.jboss.tools.vpe.editor.context.VpePageContext;
@@ -252,91 +249,72 @@
*/
final public void refreshVisualSelection() {
- IStructuredModel model = null;
- try {
- // checks for null, for case when we close editor and background
- // update job is running
- if (getSourceEditor().getTextViewer() == null) {
+ // checks for null, for case when we close editor and background
+ // update job is running
+ if (getSourceEditor().getTextViewer() == null) {
- return;
- }
- // gets source model for read, model should be released see
- // JBIDE-2219
- model = StructuredModelManager.getModelManager()
- .getExistingModelForRead(
- getSourceEditor().getTextViewer().getDocument());
-
- //fix for JBIDE-3805, mareshkau
- if(model==null) {
- return;
- }
-
- Point range = SelectionUtil
- .getSourceSelectionRange(getSourceEditor());
+ return;
+ }
- if (range == null)
- return;
+ Point range = SelectionUtil
+ .getSourceSelectionRange(getSourceEditor());
- int focusOffcetInSourceDocument = range.x;
+ if (range == null)
+ return;
- int anchorOffcetInSourceDocument = focusOffcetInSourceDocument
- + range.y;
+ int focusOffcetInSourceDocument = range.x;
- // get element mapping
- VpeNodeMapping nodeMapping = SelectionUtil
- .getNodeMappingBySourceSelection(model, getDomMapping(),
- focusOffcetInSourceDocument,
- anchorOffcetInSourceDocument);
+ int anchorOffcetInSourceDocument = focusOffcetInSourceDocument
+ + range.y;
+ VpeNodeMapping nodeMapping = SelectionUtil
+ .getNodeMappingBySourceSelection(getSourceEditor(),
+ getDomMapping());
- if (nodeMapping == null)
- return;
+ if (nodeMapping == null) {
+ return;
+ }
- // visual node which will be selected
- nsIDOMNode targetVisualNode;
+ // visual node which will be selected
+ nsIDOMNode targetVisualNode;
- // int visualNodeOffcet =
- // TextUtil.visualPosition(((Node)targetSourceNode
- // ).getNodeValue(),offcetReferenceToSourceNode);
+ // int visualNodeOffcet =
+ // TextUtil.visualPosition(((Node)targetSourceNode
+ // ).getNodeValue(),offcetReferenceToSourceNode);
- // if mapping is elementMapping
-
- SelectionUtil.clearSelection(selectionController);
-
- if (nodeMapping instanceof VpeElementMapping) {
+ // if mapping is elementMapping
+
+ SelectionUtil.clearSelection(selectionController);
+
+ if (nodeMapping instanceof VpeElementMapping) {
- VpeElementMapping elementMapping = (VpeElementMapping) nodeMapping;
+ VpeElementMapping elementMapping = (VpeElementMapping) nodeMapping;
- VpeTemplate template = elementMapping.getTemplate();
+ VpeTemplate template = elementMapping.getTemplate();
- targetVisualNode = template.getVisualNodeBySourcePosition(
- elementMapping, focusOffcetInSourceDocument,
- anchorOffcetInSourceDocument, getDomMapping());
+ targetVisualNode = template.getVisualNodeBySourcePosition(
+ elementMapping, focusOffcetInSourceDocument,
+ anchorOffcetInSourceDocument, getDomMapping());
- NodeData nodeData = template.getNodeData(targetVisualNode,
- elementMapping.getElementData(), getDomMapping());
- // we can restore cursor position only if we have nodeData and
- // range.y==0
- if (nodeData != null) {
- // restore cursor position in source document
- restoreVisualCursorPosition(template, nodeData,
- focusOffcetInSourceDocument,
- anchorOffcetInSourceDocument);
- }
- } else {
+ NodeData nodeData = template.getNodeData(targetVisualNode,
+ elementMapping.getElementData(), getDomMapping());
+ // we can restore cursor position only if we have nodeData and
+ // range.y==0
+ if (nodeData != null) {
+ // restore cursor position in source document
+ restoreVisualCursorPosition(template, nodeData,
+ focusOffcetInSourceDocument,
+ anchorOffcetInSourceDocument);
+ }
+ } else {
- targetVisualNode = nodeMapping.getVisualNode();
- // restore cursor position for source node
+ targetVisualNode = nodeMapping.getVisualNode();
+ // restore cursor position for source node
// restoreVisualCursorPositionForTextNode(targetVisualNode,
// focusOffcetInSourceDocument, model);
- }
- // here we restore only highlight
- getPageContext().getVisualBuilder().setSelectionRectangle(
- targetVisualNode);
- } finally {
- if (model != null) {
- model.releaseFromRead();
- }
}
+ // here we restore only highlight
+ getPageContext().getVisualBuilder().setSelectionRectangle(
+ targetVisualNode);
}
/**
Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/SelectionUtil.java
===================================================================
--- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/SelectionUtil.java 2009-09-23 16:09:52 UTC (rev 17706)
+++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/SelectionUtil.java 2009-09-23 19:12:29 UTC (rev 17707)
@@ -119,6 +119,11 @@
offset, length);
}
+ /**
+ * @deprecated This method causes JBIDE-4713. Use
+ * {@link #getNodeMappingBySourceSelection(StructuredTextEditor, VpeDomMapping)}
+ * instead.
+ */
public static VpeNodeMapping getNodeMappingBySourceSelection(
IStructuredModel model, VpeDomMapping domMapping, int focus,
int anchor) {
@@ -228,8 +233,15 @@
IStructuredModel model = null;
try {
+ // gets source model for read, model should be released see
+ // JBIDE-2219
model = StructuredModelManager.getModelManager()
.getExistingModelForRead(document);
+
+ //fix for JBIDE-3805, mareshkau
+ if(model == null) {
+ return null;
+ }
int anchor = range.x;
int focus = range.x + range.y;
15 years
JBoss Tools SVN: r17706 - trunk/documentation/qa/docs/Test_Cases/Test_Projects.
by jbosstools-commits＠lists.jboss.org
Author: aklimkovich
Date: 2009-09-23 12:09:52 -0400 (Wed, 23 Sep 2009)
New Revision: 17706
Added:
trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar
Log:
Added a file remotely
Added: trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar
===================================================================
(Binary files differ)
Property changes on: trunk/documentation/qa/docs/Test_Cases/Test_Projects/TestWorkspaces.rar
___________________________________________________________________
Name: svn:mime-type
+ application/octet-stream
15 years