Prepared better HEREDOC support; see comment for bug #1319276
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpeclipse / phpeditor / PHPUnitEditor.java
index a7cc596..39cbeae 100644 (file)
@@ -2,11 +2,14 @@ package net.sourceforge.phpeclipse.phpeditor;
 
 import java.text.MessageFormat;
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Map;
 
 import net.sourceforge.phpdt.core.ICompilationUnit;
 import net.sourceforge.phpdt.core.IJavaElement;
+import net.sourceforge.phpdt.core.IJavaProject;
 import net.sourceforge.phpdt.core.IMember;
 import net.sourceforge.phpdt.core.ISourceRange;
 import net.sourceforge.phpdt.core.ISourceReference;
@@ -14,11 +17,16 @@ import net.sourceforge.phpdt.core.JavaCore;
 import net.sourceforge.phpdt.core.JavaModelException;
 import net.sourceforge.phpdt.core.dom.CompilationUnit;
 import net.sourceforge.phpdt.internal.compiler.parser.Scanner;
+import net.sourceforge.phpdt.internal.ui.actions.AddBlockCommentAction;
 import net.sourceforge.phpdt.internal.ui.actions.CompositeActionGroup;
+import net.sourceforge.phpdt.internal.ui.actions.IndentAction;
+import net.sourceforge.phpdt.internal.ui.actions.RemoveBlockCommentAction;
 import net.sourceforge.phpdt.internal.ui.text.ContentAssistPreference;
 import net.sourceforge.phpdt.internal.ui.text.IPHPPartitions;
 import net.sourceforge.phpdt.internal.ui.text.PHPPairMatcher;
 import net.sourceforge.phpdt.internal.ui.text.SmartBackspaceManager;
+import net.sourceforge.phpdt.internal.ui.text.SmartSemicolonAutoEditStrategy;
+import net.sourceforge.phpdt.internal.ui.text.comment.CommentFormattingContext;
 import net.sourceforge.phpdt.internal.ui.text.java.IJavaReconcilingListener;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionManager;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionUI;
@@ -28,6 +36,8 @@ import net.sourceforge.phpdt.ui.PreferenceConstants;
 import net.sourceforge.phpdt.ui.actions.GenerateActionGroup;
 import net.sourceforge.phpdt.ui.text.JavaTextTools;
 import net.sourceforge.phpeclipse.PHPeclipsePlugin;
+import net.sourceforge.phpeclipse.phpeditor.actions.RTrimAction;
+import net.sourceforge.phpeclipse.ui.editor.ShowExternalPreviewAction;
 
 import org.eclipse.core.internal.runtime.ListenerList;
 import org.eclipse.core.resources.IFile;
@@ -48,6 +58,7 @@ import org.eclipse.jface.preference.IPreferenceStore;
 import org.eclipse.jface.preference.PreferenceConverter;
 import org.eclipse.jface.text.BadLocationException;
 import org.eclipse.jface.text.DocumentCommand;
+import org.eclipse.jface.text.IAutoEditStrategy;
 import org.eclipse.jface.text.IDocument;
 import org.eclipse.jface.text.ILineTracker;
 import org.eclipse.jface.text.IRegion;
@@ -57,6 +68,8 @@ import org.eclipse.jface.text.ITypedRegion;
 import org.eclipse.jface.text.IWidgetTokenKeeper;
 import org.eclipse.jface.text.contentassist.ContentAssistant;
 import org.eclipse.jface.text.contentassist.IContentAssistant;
+import org.eclipse.jface.text.formatter.FormattingContextProperties;
+import org.eclipse.jface.text.formatter.IFormattingContext;
 import org.eclipse.jface.text.source.IOverviewRuler;
 import org.eclipse.jface.text.source.ISourceViewer;
 import org.eclipse.jface.text.source.IVerticalRuler;
@@ -94,9 +107,9 @@ import org.eclipse.ui.texteditor.TextOperationAction;
  * program and the accompanying materials are made available under the terms of
  * the Common Public License v1.0 which accompanies this distribution, and is
  * available at http://www.eclipse.org/legal/cpl-v10.html
- * 
- * Contributors: IBM Corporation - Initial implementation Klaus Hartlage -
- * www.eclipseproject.de
+ *
+ * Contributors: IBM Corporation - Initial implementation
+ * www.phpeclipse.de
  ******************************************************************************/
 /**
  * PHP specific text editor.
@@ -107,93 +120,25 @@ public class PHPUnitEditor extends PHPEditor { //implements
     void customizeDocumentCommand(IDocument document, DocumentCommand command);
   };
 
-  //  class AdaptedRulerLayout extends Layout {
-  //
-  //    protected int fGap;
-  //    protected AdaptedSourceViewer fAdaptedSourceViewer;
-  //
-  //    protected AdaptedRulerLayout(int gap, AdaptedSourceViewer asv) {
-  //      fGap = gap;
-  //      fAdaptedSourceViewer = asv;
-  //    }
-  //
-  //    protected Point computeSize(Composite composite, int wHint, int hHint,
-  // boolean flushCache) {
-  //      Control[] children = composite.getChildren();
-  //      Point s = children[children.length - 1].computeSize(SWT.DEFAULT,
-  // SWT.DEFAULT, flushCache);
-  //      if (fAdaptedSourceViewer.isVerticalRulerVisible())
-  //        s.x += fAdaptedSourceViewer.getVerticalRuler().getWidth() + fGap;
-  //      return s;
-  //    }
-  //
-  //    protected void layout(Composite composite, boolean flushCache) {
-  //      Rectangle clArea = composite.getClientArea();
-  //      if (fAdaptedSourceViewer.isVerticalRulerVisible()) {
-  //
-  //        StyledText textWidget = fAdaptedSourceViewer.getTextWidget();
-  //        Rectangle trim = textWidget.computeTrim(0, 0, 0, 0);
-  //        int scrollbarHeight = trim.height;
-  //
-  //        IVerticalRuler vr = fAdaptedSourceViewer.getVerticalRuler();
-  //        int vrWidth = vr.getWidth();
-  //
-  //        int orWidth = 0;
-  //        if (fAdaptedSourceViewer.isOverviewRulerVisible()) {
-  //          OverviewRuler or = fAdaptedSourceViewer.getOverviewRuler();
-  //          orWidth = or.getWidth();
-  //          or.getControl().setBounds(clArea.width - orWidth, scrollbarHeight,
-  // orWidth, clArea.height - 3 * scrollbarHeight);
-  //        }
-  //
-  //        textWidget.setBounds(vrWidth + fGap, 0, clArea.width - vrWidth - orWidth
-  // -
-  // 2 * fGap, clArea.height);
-  //        vr.getControl().setBounds(0, 0, vrWidth, clArea.height -
-  // scrollbarHeight);
-  //
-  //      } else {
-  //        StyledText textWidget = fAdaptedSourceViewer.getTextWidget();
-  //        textWidget.setBounds(0, 0, clArea.width, clArea.height);
-  //      }
-  //    }
-  //  };
-  //
-  //  class AdaptedSourceViewer extends SourceViewer { // extends
-  // JavaCorrectionSourceViewer {
-  //
+  //  class AdaptedSourceViewer extends JavaSourceViewer {
   //    private List fTextConverters;
   //
-  //    private OverviewRuler fOverviewRuler;
-  //    private boolean fIsOverviewRulerVisible;
-  //    /** The viewer's overview ruler hovering controller */
-  //    private AbstractHoverInformationControlManager
-  // fOverviewRulerHoveringController;
-  //
   //    private boolean fIgnoreTextConverters = false;
   //
-  //    private IVerticalRuler fCachedVerticalRuler;
-  //    private boolean fCachedIsVerticalRulerVisible;
-  //
-  //    public AdaptedSourceViewer(Composite parent, IVerticalRuler ruler, int
-  // styles) {
-  //      super(parent, ruler, styles); //, CompilationUnitEditor.this);
-  //
-  //      fCachedVerticalRuler = ruler;
-  //      fCachedIsVerticalRulerVisible = (ruler != null);
-  //      fOverviewRuler = new OverviewRuler(VERTICAL_RULER_WIDTH);
-  //
-  //      delayedCreateControl(parent, styles);
-  //    }
-  //
-  //    /*
-  //     * @see ISourceViewer#showAnnotations(boolean)
-  //     */
-  //    public void showAnnotations(boolean show) {
-  //      fCachedIsVerticalRulerVisible = (show && fCachedVerticalRuler != null);
-  //      // super.showAnnotations(show);
+  //    // private JavaCorrectionAssistant fCorrectionAssistant;
+  //    public AdaptedSourceViewer(Composite parent, IVerticalRuler verticalRuler,
+  //        IOverviewRuler overviewRuler, boolean showAnnotationsOverview,
+  //        int styles, IPreferenceStore store) {
+  //      super(parent, verticalRuler, overviewRuler, showAnnotationsOverview,
+  //          styles, store);
   //    }
   //
+  //    // public AdaptedSourceViewer(Composite parent,
+  //    // IVerticalRuler verticalRuler, IOverviewRuler overviewRuler,
+  //    // boolean showAnnotationsOverview, int styles) {
+  //    // super(parent, verticalRuler, overviewRuler,
+  //    // showAnnotationsOverview, styles);
+  //    // }
   //    public IContentAssistant getContentAssistant() {
   //      return fContentAssistant;
   //    }
@@ -202,28 +147,47 @@ public class PHPUnitEditor extends PHPEditor { //implements
   //     * @see ITextOperationTarget#doOperation(int)
   //     */
   //    public void doOperation(int operation) {
-  //
   //      if (getTextWidget() == null)
   //        return;
-  //
   //      switch (operation) {
-  //        case CONTENTASSIST_PROPOSALS :
-  //          String msg = fContentAssistant.showPossibleCompletions();
-  //          setStatusLineErrorMessage(msg);
-  //          return;
-  //        case UNDO :
-  //          fIgnoreTextConverters = true;
-  //          break;
-  //        case REDO :
-  //          fIgnoreTextConverters = true;
-  //          break;
+  //      case CONTENTASSIST_PROPOSALS:
+  //        String msg = fContentAssistant.showPossibleCompletions();
+  //        setStatusLineErrorMessage(msg);
+  //        return;
+  //      // case CORRECTIONASSIST_PROPOSALS:
+  //      // fCorrectionAssistant.showPossibleCompletions();
+  //      // return;
+  //      case UNDO:
+  //        fIgnoreTextConverters = true;
+  //        break;
+  //      case REDO:
+  //        fIgnoreTextConverters = true;
+  //        break;
   //      }
-  //
   //      super.doOperation(operation);
   //    }
   //
-  //    public void insertTextConverter(ITextConverter textConverter, int index)
-  // {
+  //    /*
+  //     * @see ITextOperationTarget#canDoOperation(int)
+  //     */
+  //    public boolean canDoOperation(int operation) {
+  //      // if (operation == CORRECTIONASSIST_PROPOSALS)
+  //      // return isEditable();
+  //      return super.canDoOperation(operation);
+  //    }
+  //
+  //    /*
+  //     * @see TextViewer#handleDispose()
+  //     */
+  //    protected void handleDispose() {
+  //      // if (fCorrectionAssistant != null) {
+  //      // fCorrectionAssistant.uninstall();
+  //      // fCorrectionAssistant= null;
+  //      // }
+  //      super.handleDispose();
+  //    }
+  //
+  //    public void insertTextConverter(ITextConverter textConverter, int index) {
   //      throw new UnsupportedOperationException();
   //    }
   //
@@ -250,92 +214,12 @@ public class PHPUnitEditor extends PHPEditor { //implements
   //      super.customizeDocumentCommand(command);
   //      if (!fIgnoreTextConverters && fTextConverters != null) {
   //        for (Iterator e = fTextConverters.iterator(); e.hasNext();)
-  //           ((ITextConverter) e.next()).customizeDocumentCommand(getDocument(),
-  // command);
+  //          ((ITextConverter) e.next()).customizeDocumentCommand(getDocument(),
+  //              command);
   //      }
   //      fIgnoreTextConverters = false;
   //    }
   //
-  //    public IVerticalRuler getVerticalRuler() {
-  //      return fCachedVerticalRuler;
-  //    }
-  //
-  //    public boolean isVerticalRulerVisible() {
-  //      return fCachedIsVerticalRulerVisible;
-  //    }
-  //
-  //    public OverviewRuler getOverviewRuler() {
-  //      return fOverviewRuler;
-  //    }
-  //
-  //    /*
-  //     * @see TextViewer#createControl(Composite, int)
-  //     */
-  //    protected void createControl(Composite parent, int styles) {
-  //      // do nothing here
-  //    }
-  //
-  //    protected void delayedCreateControl(Composite parent, int styles) {
-  //      //create the viewer
-  //      super.createControl(parent, styles);
-  //
-  //      Control control = getControl();
-  //      if (control instanceof Composite) {
-  //        Composite composite = (Composite) control;
-  //        composite.setLayout(new AdaptedRulerLayout(GAP_SIZE, this));
-  //        fOverviewRuler.createControl(composite, this);
-  //      }
-  //    }
-  //    protected void ensureOverviewHoverManagerInstalled() {
-  //      if (fOverviewRulerHoveringController == null && fAnnotationHover != null
-  // && fHoverControlCreator != null) {
-  //        fOverviewRulerHoveringController =
-  //          new OverviewRulerHoverManager(fOverviewRuler, this, fAnnotationHover,
-  // fHoverControlCreator);
-  //        fOverviewRulerHoveringController.install(fOverviewRuler.getControl());
-  //      }
-  //    }
-  //
-  //    public void hideOverviewRuler() {
-  //      fIsOverviewRulerVisible = false;
-  //      Control control = getControl();
-  //      if (control instanceof Composite) {
-  //        Composite composite = (Composite) control;
-  //        composite.layout();
-  //      }
-  //      if (fOverviewRulerHoveringController != null) {
-  //        fOverviewRulerHoveringController.dispose();
-  //        fOverviewRulerHoveringController = null;
-  //      }
-  //    }
-  //
-  //    public void showOverviewRuler() {
-  //      fIsOverviewRulerVisible = true;
-  //      Control control = getControl();
-  //      if (control instanceof Composite) {
-  //        Composite composite = (Composite) control;
-  //        composite.layout();
-  //      }
-  //      ensureOverviewHoverManagerInstalled();
-  //    }
-  //
-  //    public boolean isOverviewRulerVisible() {
-  //      return fIsOverviewRulerVisible;
-  //    }
-  //
-  //    /*
-  //     * @see ISourceViewer#setDocument(IDocument, IAnnotationModel, int, int)
-  //     */
-  //    public void setDocument(
-  //      IDocument document,
-  //      IAnnotationModel annotationModel,
-  //      int visibleRegionOffset,
-  //      int visibleRegionLength) {
-  //      super.setDocument(document, annotationModel, visibleRegionOffset,
-  // visibleRegionLength);
-  //      fOverviewRuler.setModel(annotationModel);
-  //    }
-  //
   //    // http://dev.eclipse.org/bugs/show_bug.cgi?id=19270
   //    public void updateIndentationPrefixes() {
   //      SourceViewerConfiguration configuration = getSourceViewerConfiguration();
@@ -356,47 +240,42 @@ public class PHPUnitEditor extends PHPEditor { //implements
   //      return super.requestWidgetToken(requester);
   //    }
   //
-  //    /*
-  //     * @see
-  // org.eclipse.jface.text.source.ISourceViewer#configure(org.eclipse.jface.text.source.SourceViewerConfiguration)
-  //     */
+  //// /*
+  //// * @see org.eclipse.jface.text.source.ISourceViewer#configure(org.eclipse.jface.text.source.SourceViewerConfiguration)
+  //// */
+  //// public void configure(SourceViewerConfiguration configuration) {
+  //// super.configure(configuration);
+  //// // fCorrectionAssistant= new
+  //// // JavaCorrectionAssistant(CompilationUnitEditor.this);
+  //// // fCorrectionAssistant.install(this);
+  //// //TODO install SmartBracesAutoEditStrategy
+  //// // prependAutoEditStrategy(new SmartBracesAutoEditStrategy(this),
+  //// // IDocument.DEFAULT_CONTENT_TYPE);
+  //// }
   //    public void configure(SourceViewerConfiguration configuration) {
-  //      super.configure(configuration);
-  //      // prependAutoEditStrategy(new SmartBracesAutoEditStrategy(this),
-  // IDocument.DEFAULT_CONTENT_TYPE);
-  //    }
-  //
-  //    protected void handleDispose() {
-  //      fOverviewRuler = null;
-  //
-  //      if (fOverviewRulerHoveringController != null) {
-  //        fOverviewRulerHoveringController.dispose();
-  //        fOverviewRulerHoveringController = null;
-  //      }
-  //
-  //      super.handleDispose();
-  //    }
-  //
+  //           super.configure(configuration);
+  //// fCorrectionAssistant= new JavaCorrectionAssistant(CompilationUnitEditor.this);
+  //// fCorrectionAssistant.install(this);
+  //           IAutoEditStrategy smartSemi= new SmartSemicolonAutoEditStrategy(IPHPPartitions.PHP_PARTITIONING);
+  //           prependAutoEditStrategy(smartSemi, IDocument.DEFAULT_CONTENT_TYPE);
+  //           prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_DQ);
+  //           prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_SQ);
+  //// prependAutoEditStrategy(smartSemi, IPHPPartitions.JAVA_CHARACTER);
+  //   }
   //  };
   class AdaptedSourceViewer extends JavaSourceViewer {
+
     private List fTextConverters;
 
     private boolean fIgnoreTextConverters = false;
 
-    //                 private JavaCorrectionAssistant fCorrectionAssistant;
-    public AdaptedSourceViewer(Composite parent, IVerticalRuler verticalRuler,
-        IOverviewRuler overviewRuler, boolean showAnnotationsOverview,
-        int styles, IPreferenceStore store) {
-      super(parent, verticalRuler, overviewRuler, showAnnotationsOverview,
-          styles, store);
+    //    private JavaCorrectionAssistant fCorrectionAssistant;
+
+    public AdaptedSourceViewer(Composite parent, IVerticalRuler verticalRuler, IOverviewRuler overviewRuler,
+        boolean showAnnotationsOverview, int styles, IPreferenceStore store) {
+      super(parent, verticalRuler, overviewRuler, showAnnotationsOverview, styles, store);
     }
 
-    //         public AdaptedSourceViewer(Composite parent,
-    //                         IVerticalRuler verticalRuler, IOverviewRuler overviewRuler,
-    //                         boolean showAnnotationsOverview, int styles) {
-    //                 super(parent, verticalRuler, overviewRuler,
-    //                                 showAnnotationsOverview, styles);
-    //         }
     public IContentAssistant getContentAssistant() {
       return fContentAssistant;
     }
@@ -405,23 +284,31 @@ public class PHPUnitEditor extends PHPEditor { //implements
      * @see ITextOperationTarget#doOperation(int)
      */
     public void doOperation(int operation) {
+
       if (getTextWidget() == null)
         return;
+
       switch (operation) {
       case CONTENTASSIST_PROPOSALS:
         String msg = fContentAssistant.showPossibleCompletions();
         setStatusLineErrorMessage(msg);
         return;
-      //                                       case CORRECTIONASSIST_PROPOSALS:
-      //                                               fCorrectionAssistant.showPossibleCompletions();
-      //                                               return;
+      //      case CORRECTIONASSIST_PROPOSALS:
+      //        msg = fCorrectionAssistant.showPossibleCompletions();
+      //        setStatusLineErrorMessage(msg);
+      //        return;
       case UNDO:
         fIgnoreTextConverters = true;
-        break;
+        super.doOperation(operation);
+        fIgnoreTextConverters = false;
+        return;
       case REDO:
         fIgnoreTextConverters = true;
-        break;
+        super.doOperation(operation);
+        fIgnoreTextConverters = false;
+        return;
       }
+
       super.doOperation(operation);
     }
 
@@ -429,20 +316,22 @@ public class PHPUnitEditor extends PHPEditor { //implements
      * @see ITextOperationTarget#canDoOperation(int)
      */
     public boolean canDoOperation(int operation) {
-      //                               if (operation == CORRECTIONASSIST_PROPOSALS)
-      //                                       return isEditable();
+      //      if (operation == CORRECTIONASSIST_PROPOSALS)
+      //        return isEditable();
+
       return super.canDoOperation(operation);
     }
 
     /*
-     * @see TextViewer#handleDispose()
+     * @see org.eclipse.jface.text.source.ISourceViewerExtension2#unconfigure()
+     * @since 3.0
      */
-    protected void handleDispose() {
-      //                               if (fCorrectionAssistant != null) {
-      //                                       fCorrectionAssistant.uninstall();
-      //                                       fCorrectionAssistant= null;
-      //                               }
-      super.handleDispose();
+    public void unconfigure() {
+      //      if (fCorrectionAssistant != null) {
+      //        fCorrectionAssistant.uninstall();
+      //        fCorrectionAssistant = null;
+      //      }
+      super.unconfigure();
     }
 
     public void insertTextConverter(ITextConverter textConverter, int index) {
@@ -472,10 +361,8 @@ public class PHPUnitEditor extends PHPEditor { //implements
       super.customizeDocumentCommand(command);
       if (!fIgnoreTextConverters && fTextConverters != null) {
         for (Iterator e = fTextConverters.iterator(); e.hasNext();)
-          ((ITextConverter) e.next()).customizeDocumentCommand(getDocument(),
-              command);
+          ((ITextConverter) e.next()).customizeDocumentCommand(getDocument(), command);
       }
-      fIgnoreTextConverters = false;
     }
 
     // http://dev.eclipse.org/bugs/show_bug.cgi?id=19270
@@ -499,370 +386,417 @@ public class PHPUnitEditor extends PHPEditor { //implements
     }
 
     /*
+     * @see IWidgetTokenOwnerExtension#requestWidgetToken(IWidgetTokenKeeper, int)
+     * @since 3.0
+     */
+    public boolean requestWidgetToken(IWidgetTokenKeeper requester, int priority) {
+      if (WorkbenchHelp.isContextHelpDisplayed())
+        return false;
+      return super.requestWidgetToken(requester, priority);
+    }
+
+    /*
      * @see org.eclipse.jface.text.source.ISourceViewer#configure(org.eclipse.jface.text.source.SourceViewerConfiguration)
      */
     public void configure(SourceViewerConfiguration configuration) {
       super.configure(configuration);
-      //                               fCorrectionAssistant= new
-      // JavaCorrectionAssistant(CompilationUnitEditor.this);
-      //                               fCorrectionAssistant.install(this);
-      //TODO install SmartBracesAutoEditStrategy
-      //                               prependAutoEditStrategy(new SmartBracesAutoEditStrategy(this),
-      // IDocument.DEFAULT_CONTENT_TYPE);
+      //      fCorrectionAssistant = new JavaCorrectionAssistant(CompilationUnitEditor.this);
+      //      fCorrectionAssistant.install(this);
+      IAutoEditStrategy smartSemi = new SmartSemicolonAutoEditStrategy(IPHPPartitions.PHP_PARTITIONING);
+      prependAutoEditStrategy(smartSemi, IDocument.DEFAULT_CONTENT_TYPE);
+      prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_DQ);
+      prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_SQ);
+      prependAutoEditStrategy(smartSemi, IPHPPartitions.PHP_STRING_HEREDOC);
     }
-  };
+
+    /*
+     * @see org.eclipse.jface.text.source.SourceViewer#createFormattingContext()
+     * @since 3.0
+     */
+    public IFormattingContext createFormattingContext() {
+      IFormattingContext context = new CommentFormattingContext();
+
+      Map preferences;
+      IJavaElement inputJavaElement = getInputJavaElement();
+      IJavaProject javaProject = inputJavaElement != null ? inputJavaElement.getJavaProject() : null;
+      if (javaProject == null)
+        preferences = new HashMap(JavaCore.getOptions());
+      else
+        preferences = new HashMap(javaProject.getOptions(true));
+
+      context.storeToMap(PreferenceConstants.getPreferenceStore(), preferences, false);
+      context.setProperty(FormattingContextProperties.CONTEXT_PREFERENCES, preferences);
+
+      return context;
+    }
+  }
+
   /**
-        * Remembers data related to the current selection to be able to
-        * restore it later.
-        * 
-        * @since 3.0
-        */
-       private class RememberedSelection {
-               /** The remembered selection start. */
-               private RememberedOffset fStartOffset= new RememberedOffset();
-               /** The remembered selection end. */
-               private RememberedOffset fEndOffset= new RememberedOffset();
-
-               /**
-                * Remember current selection.
-                */
-               public void remember() {
-                       /* https://bugs.eclipse.org/bugs/show_bug.cgi?id=52257
-                        * This method may be called inside an async call posted
-                        * to the UI thread, so protect against intermediate disposal
-                        * of the editor. 
-                        */
-                       ISourceViewer viewer= getSourceViewer();
-                       if (viewer != null) {
-                               IRegion selection= getSignedSelection(viewer);
-                               int startOffset= selection.getOffset();
-                               int endOffset= startOffset + selection.getLength();
-                               
-                               fStartOffset.setOffset(startOffset);
-                               fEndOffset.setOffset(endOffset);
-                       }
-               }
-
-               /**
-                * Restore remembered selection.
-                */
-               public void restore() {
-                       /* https://bugs.eclipse.org/bugs/show_bug.cgi?id=52257
-                        * This method may be called inside an async call posted
-                        * to the UI thread, so protect against intermediate disposal
-                        * of the editor.
-                        */
-                       if (getSourceViewer() == null)
-                               return;
-                       
-                       try {
-                               
-                               int startOffset, endOffset;
-                               int revealStartOffset, revealEndOffset; 
-                               if (showsHighlightRangeOnly()) {
-                                       IJavaElement newStartElement= fStartOffset.getElement();
-                                       startOffset= fStartOffset.getRememberedOffset(newStartElement);
-                                       revealStartOffset= fStartOffset.getRevealOffset(newStartElement, startOffset);
-                                       if (revealStartOffset == -1)
-                                               startOffset= -1;
-                                       
-                                       IJavaElement newEndElement= fEndOffset.getElement();
-                                       endOffset= fEndOffset.getRememberedOffset(newEndElement);
-                                       revealEndOffset= fEndOffset.getRevealOffset(newEndElement, endOffset);
-                                       if (revealEndOffset == -1)
-                                               endOffset= -1;
-                               } else {
-                                       startOffset= fStartOffset.getOffset();
-                                       revealStartOffset= startOffset;
-                                       endOffset= fEndOffset.getOffset();
-                                       revealEndOffset= endOffset;
-                               }
-                               
-                               if (startOffset == -1) {
-                                       startOffset= endOffset; // fallback to caret offset
-                                       revealStartOffset= revealEndOffset;
-                               }
-                               
-                               if (endOffset == -1) {
-                                       endOffset= startOffset; // fallback to other offset
-                                       revealEndOffset= revealStartOffset;
-                               }
-                               
-                               IJavaElement element;
-                               if (endOffset == -1) {
-                                        // fallback to element selection
-                                       element= fEndOffset.getElement();
-                                       if (element == null)
-                                               element= fStartOffset.getElement();
-                                       if (element != null)
-                                               setSelection(element);
-                                       return;
-                               }
-                                                       
-                               if (isValidSelection(revealStartOffset, revealEndOffset - revealStartOffset) && isValidSelection(startOffset, endOffset - startOffset))
-                                       selectAndReveal(startOffset, endOffset - startOffset, revealStartOffset, revealEndOffset - revealStartOffset);
-                       } finally {
-                               fStartOffset.clear();
-                               fEndOffset.clear();
-                       }
-               }
-
-               private boolean isValidSelection(int offset, int length) {
-                       IDocumentProvider provider= getDocumentProvider();
-                       if (provider != null) {
-                               IDocument document= provider.getDocument(getEditorInput());
-                               if (document != null) {
-                                       int end= offset + length;
-                                       int documentLength= document.getLength();
-                                       return 0 <= offset  && offset <= documentLength && 0 <= end && end <= documentLength;
-                               }
-                       }
-                       return false;
-               }
-               
-       }
-
-       /**
-        * Remembers additional data for a given
-        * offset to be able restore it later.
-        * 
-        * @since 3.0
-        */
-       private class RememberedOffset {
-               /** Remembered line for the given offset */
-               private int fLine;
-               /** Remembered column for the given offset*/
-               private int fColumn;
-               /** Remembered Java element for the given offset*/
-               private IJavaElement fElement;
-               /** Remembered Java element line for the given offset*/
-               private int fElementLine;
-               
-               /**
-                * Store visual properties of the given offset.  
-                * 
-                * @param offset Offset in the document
-                */
-               public void setOffset(int offset) {
-                       try {
-                               IDocument document= getSourceViewer().getDocument();
-                               fLine= document.getLineOfOffset(offset);
-                               fColumn= offset - document.getLineOffset(fLine);
-                               fElement= getElementAt(offset, true);
-
-                               fElementLine= -1;
-                               if (fElement instanceof IMember) {
-                                       ISourceRange range= ((IMember) fElement).getNameRange();
-                                       if (range != null)
-                                               fElementLine= document.getLineOfOffset(range.getOffset());
-                               }
-                               if (fElementLine == -1)
-                                       fElementLine= document.getLineOfOffset(getOffset(fElement));
-                       } catch (BadLocationException e) {
-                               // should not happen
-                               PHPeclipsePlugin.log(e);
-                               clear();
-                       } catch (JavaModelException e) {
-                               // should not happen
-                         PHPeclipsePlugin.log(e.getStatus());
-                               clear();
-                       }
-               }
-
-               /**
-                * Return offset recomputed from stored visual properties.  
-                * 
-                * @return Offset in the document
-                */
-               public int getOffset() {
-                       IJavaElement newElement= getElement();
-                       
-                       int offset= getRememberedOffset(newElement);
-                       
-                       if (offset != -1 && !containsOffset(newElement, offset) && (offset == 0 || !containsOffset(newElement, offset - 1)))
-                               return -1;
-                       
-                       return offset;
-               }
-               
-               /**
-                * Return offset recomputed from stored visual properties.  
-                * 
-                * @param newElement Enclosing element
-                * @return Offset in the document
-                */
-               public int getRememberedOffset(IJavaElement newElement) {
-                       try {
-                               if (newElement == null)
-                                       return -1;
-                               
-                               IDocument document= getSourceViewer().getDocument();
-                               int newElementLine= -1;
-                               if (newElement instanceof IMember) {
-                                       ISourceRange range= ((IMember) newElement).getNameRange();
-                                       if (range != null)
-                                               newElementLine= document.getLineOfOffset(range.getOffset());
-                               }
-                               if (newElementLine == -1)
-                                       newElementLine= document.getLineOfOffset(getOffset(newElement));
-                               if (newElementLine == -1)
-                                       return -1;
-
-                               int newLine= fLine + newElementLine - fElementLine;
-                               if (newLine < 0 || newLine >= document.getNumberOfLines())
-                                       return -1;
-                               int maxColumn= document.getLineLength(newLine);
-                               String lineDelimiter= document.getLineDelimiter(newLine);
-                               if (lineDelimiter != null)
-                                       maxColumn= maxColumn - lineDelimiter.length();
-                               int offset;
-                               if (fColumn > maxColumn)
-                                       offset= document.getLineOffset(newLine) + maxColumn;
-                               else
-                                       offset= document.getLineOffset(newLine) + fColumn;
-
-                               return offset;
-                       } catch (BadLocationException e) {
-                               // should not happen
-                         PHPeclipsePlugin.log(e);
-                               return -1;
-                       } catch (JavaModelException e) {
-                               // should not happen
-                         PHPeclipsePlugin.log(e.getStatus());
-                               return -1;
-                       }
-               }
-               
-               /**
-                * Returns the offset used to reveal the given element based on the given selection offset.
-                * @param element the element
-                * @param offset the selection offset
-                * @return the offset to reveal the given element based on the given selection offset
-                */
-               public int getRevealOffset(IJavaElement element, int offset) {
-                       if (element == null || offset == -1)
-                               return -1;
-
-                       if (containsOffset(element, offset)) {
-                               if (offset > 0) {
-                                       IJavaElement alternateElement= getElementAt(offset, false);
-                                       if (element.getHandleIdentifier().equals(alternateElement.getParent().getHandleIdentifier()))
-                                               return offset - 1; // Solves test case 2 from https://bugs.eclipse.org/bugs/show_bug.cgi?id=47727#c3
-                               }
-                               return offset;
-                       } else if (offset > 0 && containsOffset(element, offset - 1))
-                               return offset - 1; // Solves test case 1 from https://bugs.eclipse.org/bugs/show_bug.cgi?id=47727#c3
-                       
-                       return -1;
-               }
-
-               /**
-                * Return Java element recomputed from stored visual properties.  
-                * 
-                * @return Java element
-                */
-               public IJavaElement getElement() {
-                       if (fElement == null)
-                               return null;
-                       
-                       return findElement(fElement);
-               }
-
-               /**
-                * Clears the stored position 
-                */
-               public void clear() {
-                       fLine= -1;
-                       fColumn= -1;
-                       fElement= null;
-                       fElementLine= -1;
-               }
-               
-               /**
-                * Does the given Java element contain the given offset?
-                * @param element Java element
-                * @param offset Offset
-                * @return <code>true</code> iff the Java element contains the offset
-                */
-               private boolean containsOffset(IJavaElement element, int offset) {
-                       int elementOffset= getOffset(element);
-                       int elementLength= getLength(element);
-                       return (elementOffset > -1 && elementLength > -1) ? (offset >= elementOffset && offset < elementOffset + elementLength) : false;
-               }
-               /**
-                * Returns the offset of the given Java element.
-                * 
-                * @param element       Java element
-                * @return Offset of the given Java element
-                */
-               private int getOffset(IJavaElement element) {
-                       if (element instanceof ISourceReference) {
-                               ISourceReference sr= (ISourceReference) element;
-                               try {
-                                       ISourceRange srcRange= sr.getSourceRange();
-                                       if (srcRange != null)
-                                               return srcRange.getOffset();
-                               } catch (JavaModelException e) {
-                               }
-                       }
-                       return -1;      
-               }
-               
-               /**
-                * Returns the length of the given Java element.
-                * 
-                * @param element       Java element
-                * @return Length of the given Java element
-                */
-               private int getLength(IJavaElement element) {
-                       if (element instanceof ISourceReference) {
-                               ISourceReference sr= (ISourceReference) element;
-                               try {
-                                       ISourceRange srcRange= sr.getSourceRange();
-                                       if (srcRange != null)
-                                               return srcRange.getLength();
-                               } catch (JavaModelException e) {
-                               }
-                       }
-                       return -1;      
-               }
-               
-               /**
-                * Returns the updated java element for the old java element.
-                * 
-                * @param element Old Java element
-                * @return Updated Java element
-                */
-               private IJavaElement findElement(IJavaElement element) {
-                       
-                       if (element == null)
-                               return null;
-                       
-                       IWorkingCopyManager manager= PHPeclipsePlugin.getDefault().getWorkingCopyManager();
-                       ICompilationUnit unit= manager.getWorkingCopy(getEditorInput());
-                       
-                       if (unit != null) {
-                               try {
-                                       
-                                       synchronized (unit) {
-//                                             unit.reconcile(ICompilationUnit.NO_AST, false, null, null);
-                                               unit.reconcile();
-                                       }
-                                       IJavaElement[] findings= unit.findElements(element);
-                                       if (findings != null && findings.length > 0)
-                                               return findings[0];
-                               
-                               } catch (JavaModelException x) {
-                                 PHPeclipsePlugin.log(x.getStatus());
-                                       // nothing found, be tolerant and go on
-                               }
-                       }
-                       
-                       return null;
-               }
-               
-       }
-       
+   * Remembers data related to the current selection to be able to restore it later.
+   *
+   * @since 3.0
+   */
+  private class RememberedSelection {
+    /** The remembered selection start. */
+    private RememberedOffset fStartOffset = new RememberedOffset();
+
+    /** The remembered selection end. */
+    private RememberedOffset fEndOffset = new RememberedOffset();
+
+    /**
+     * Remember current selection.
+     */
+    public void remember() {
+      /*
+       * https://bugs.eclipse.org/bugs/show_bug.cgi?id=52257 This method may be called inside an async call posted to the UI thread,
+       * so protect against intermediate disposal of the editor.
+       */
+      ISourceViewer viewer = getSourceViewer();
+      if (viewer != null) {
+        IRegion selection = getSignedSelection(viewer);
+        int startOffset = selection.getOffset();
+        int endOffset = startOffset + selection.getLength();
+
+        fStartOffset.setOffset(startOffset);
+        fEndOffset.setOffset(endOffset);
+      }
+    }
+
+    /**
+     * Restore remembered selection.
+     */
+    public void restore() {
+      /*
+       * https://bugs.eclipse.org/bugs/show_bug.cgi?id=52257 This method may be called inside an async call posted to the UI thread,
+       * so protect against intermediate disposal of the editor.
+       */
+      if (getSourceViewer() == null)
+        return;
+
+      try {
+
+        int startOffset, endOffset;
+        int revealStartOffset, revealEndOffset;
+        if (showsHighlightRangeOnly()) {
+          IJavaElement newStartElement = fStartOffset.getElement();
+          startOffset = fStartOffset.getRememberedOffset(newStartElement);
+          revealStartOffset = fStartOffset.getRevealOffset(newStartElement, startOffset);
+          if (revealStartOffset == -1)
+            startOffset = -1;
+
+          IJavaElement newEndElement = fEndOffset.getElement();
+          endOffset = fEndOffset.getRememberedOffset(newEndElement);
+          revealEndOffset = fEndOffset.getRevealOffset(newEndElement, endOffset);
+          if (revealEndOffset == -1)
+            endOffset = -1;
+        } else {
+          startOffset = fStartOffset.getOffset();
+          revealStartOffset = startOffset;
+          endOffset = fEndOffset.getOffset();
+          revealEndOffset = endOffset;
+        }
+
+        if (startOffset == -1) {
+          startOffset = endOffset; // fallback to caret offset
+          revealStartOffset = revealEndOffset;
+        }
+
+        if (endOffset == -1) {
+          endOffset = startOffset; // fallback to other offset
+          revealEndOffset = revealStartOffset;
+        }
+
+        IJavaElement element;
+        if (endOffset == -1) {
+          // fallback to element selection
+          element = fEndOffset.getElement();
+          if (element == null)
+            element = fStartOffset.getElement();
+          if (element != null)
+            setSelection(element);
+          return;
+        }
+
+        if (isValidSelection(revealStartOffset, revealEndOffset - revealStartOffset)
+            && isValidSelection(startOffset, endOffset - startOffset))
+          selectAndReveal(startOffset, endOffset - startOffset, revealStartOffset, revealEndOffset - revealStartOffset);
+      } finally {
+        fStartOffset.clear();
+        fEndOffset.clear();
+      }
+    }
+
+    private boolean isValidSelection(int offset, int length) {
+      IDocumentProvider provider = getDocumentProvider();
+      if (provider != null) {
+        IDocument document = provider.getDocument(getEditorInput());
+        if (document != null) {
+          int end = offset + length;
+          int documentLength = document.getLength();
+          return 0 <= offset && offset <= documentLength && 0 <= end && end <= documentLength;
+        }
+      }
+      return false;
+    }
+
+  }
+
+  /**
+   * Remembers additional data for a given offset to be able restore it later.
+   *
+   * @since 3.0
+   */
+  private class RememberedOffset {
+    /** Remembered line for the given offset */
+    private int fLine;
+
+    /** Remembered column for the given offset */
+    private int fColumn;
+
+    /** Remembered Java element for the given offset */
+    private IJavaElement fElement;
+
+    /** Remembered Java element line for the given offset */
+    private int fElementLine;
+
+    /**
+     * Store visual properties of the given offset.
+     *
+     * @param offset
+     *          Offset in the document
+     */
+    public void setOffset(int offset) {
+      try {
+        IDocument document = getSourceViewer().getDocument();
+        fLine = document.getLineOfOffset(offset);
+        fColumn = offset - document.getLineOffset(fLine);
+        fElement = getElementAt(offset, true);
+
+        fElementLine = -1;
+        if (fElement instanceof IMember) {
+          ISourceRange range = ((IMember) fElement).getNameRange();
+          if (range != null)
+            fElementLine = document.getLineOfOffset(range.getOffset());
+        }
+        if (fElementLine == -1)
+          fElementLine = document.getLineOfOffset(getOffset(fElement));
+      } catch (BadLocationException e) {
+        // should not happen
+        PHPeclipsePlugin.log(e);
+        clear();
+      } catch (JavaModelException e) {
+        // should not happen
+        PHPeclipsePlugin.log(e.getStatus());
+        clear();
+      }
+    }
+
+    /**
+     * Return offset recomputed from stored visual properties.
+     *
+     * @return Offset in the document
+     */
+    public int getOffset() {
+      IJavaElement newElement = getElement();
+
+      int offset = getRememberedOffset(newElement);
+
+      if (offset != -1 && !containsOffset(newElement, offset) && (offset == 0 || !containsOffset(newElement, offset - 1)))
+        return -1;
+
+      return offset;
+    }
+
+    /**
+     * Return offset recomputed from stored visual properties.
+     *
+     * @param newElement
+     *          Enclosing element
+     * @return Offset in the document
+     */
+    public int getRememberedOffset(IJavaElement newElement) {
+      try {
+        if (newElement == null)
+          return -1;
+
+        IDocument document = getSourceViewer().getDocument();
+        int newElementLine = -1;
+        if (newElement instanceof IMember) {
+          ISourceRange range = ((IMember) newElement).getNameRange();
+          if (range != null)
+            newElementLine = document.getLineOfOffset(range.getOffset());
+        }
+        if (newElementLine == -1)
+          newElementLine = document.getLineOfOffset(getOffset(newElement));
+        if (newElementLine == -1)
+          return -1;
+
+        int newLine = fLine + newElementLine - fElementLine;
+        if (newLine < 0 || newLine >= document.getNumberOfLines())
+          return -1;
+        int maxColumn = document.getLineLength(newLine);
+        String lineDelimiter = document.getLineDelimiter(newLine);
+        if (lineDelimiter != null)
+          maxColumn = maxColumn - lineDelimiter.length();
+        int offset;
+        if (fColumn > maxColumn)
+          offset = document.getLineOffset(newLine) + maxColumn;
+        else
+          offset = document.getLineOffset(newLine) + fColumn;
+
+        return offset;
+      } catch (BadLocationException e) {
+        // should not happen
+        PHPeclipsePlugin.log(e);
+        return -1;
+      } catch (JavaModelException e) {
+        // should not happen
+        PHPeclipsePlugin.log(e.getStatus());
+        return -1;
+      }
+    }
+
+    /**
+     * Returns the offset used to reveal the given element based on the given selection offset.
+     *
+     * @param element
+     *          the element
+     * @param offset
+     *          the selection offset
+     * @return the offset to reveal the given element based on the given selection offset
+     */
+    public int getRevealOffset(IJavaElement element, int offset) {
+      if (element == null || offset == -1)
+        return -1;
+
+      if (containsOffset(element, offset)) {
+        if (offset > 0) {
+          IJavaElement alternateElement = getElementAt(offset, false);
+          if (element.getHandleIdentifier().equals(alternateElement.getParent().getHandleIdentifier()))
+            return offset - 1; // Solves test case 2 from https://bugs.eclipse.org/bugs/show_bug.cgi?id=47727#c3
+        }
+        return offset;
+      } else if (offset > 0 && containsOffset(element, offset - 1))
+        return offset - 1; // Solves test case 1 from https://bugs.eclipse.org/bugs/show_bug.cgi?id=47727#c3
+
+      return -1;
+    }
+
+    /**
+     * Return Java element recomputed from stored visual properties.
+     *
+     * @return Java element
+     */
+    public IJavaElement getElement() {
+      if (fElement == null)
+        return null;
+
+      return findElement(fElement);
+    }
+
+    /**
+     * Clears the stored position
+     */
+    public void clear() {
+      fLine = -1;
+      fColumn = -1;
+      fElement = null;
+      fElementLine = -1;
+    }
+
+    /**
+     * Does the given Java element contain the given offset?
+     *
+     * @param element
+     *          Java element
+     * @param offset
+     *          Offset
+     * @return <code>true</code> iff the Java element contains the offset
+     */
+    private boolean containsOffset(IJavaElement element, int offset) {
+      int elementOffset = getOffset(element);
+      int elementLength = getLength(element);
+      return (elementOffset > -1 && elementLength > -1) ? (offset >= elementOffset && offset < elementOffset + elementLength)
+          : false;
+    }
+
+    /**
+     * Returns the offset of the given Java element.
+     *
+     * @param element
+     *          Java element
+     * @return Offset of the given Java element
+     */
+    private int getOffset(IJavaElement element) {
+      if (element instanceof ISourceReference) {
+        ISourceReference sr = (ISourceReference) element;
+        try {
+          ISourceRange srcRange = sr.getSourceRange();
+          if (srcRange != null)
+            return srcRange.getOffset();
+        } catch (JavaModelException e) {
+        }
+      }
+      return -1;
+    }
+
+    /**
+     * Returns the length of the given Java element.
+     *
+     * @param element
+     *          Java element
+     * @return Length of the given Java element
+     */
+    private int getLength(IJavaElement element) {
+      if (element instanceof ISourceReference) {
+        ISourceReference sr = (ISourceReference) element;
+        try {
+          ISourceRange srcRange = sr.getSourceRange();
+          if (srcRange != null)
+            return srcRange.getLength();
+        } catch (JavaModelException e) {
+        }
+      }
+      return -1;
+    }
+
+    /**
+     * Returns the updated java element for the old java element.
+     *
+     * @param element
+     *          Old Java element
+     * @return Updated Java element
+     */
+    private IJavaElement findElement(IJavaElement element) {
+
+      if (element == null)
+        return null;
+
+      IWorkingCopyManager manager = PHPeclipsePlugin.getDefault().getWorkingCopyManager();
+      ICompilationUnit unit = manager.getWorkingCopy(getEditorInput());
+
+      if (unit != null) {
+        try {
+
+          synchronized (unit) {
+            //                                         unit.reconcile(ICompilationUnit.NO_AST, false, null, null);
+            unit.reconcile();
+          }
+          IJavaElement[] findings = unit.findElements(element);
+          if (findings != null && findings.length > 0)
+            return findings[0];
+
+        } catch (JavaModelException x) {
+          PHPeclipsePlugin.log(x.getStatus());
+          // nothing found, be tolerant and go on
+        }
+      }
+
+      return null;
+    }
+
+  }
+
   static class TabConverter implements ITextConverter {
     private int fTabRatio;
 
@@ -889,8 +823,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
       return remainder;
     }
 
-    public void customizeDocumentCommand(IDocument document,
-        DocumentCommand command) {
+    public void customizeDocumentCommand(IDocument document, DocumentCommand command) {
       String text = command.text;
       if (text == null)
         return;
@@ -906,8 +839,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
             String line = text.substring(offset, endOffset);
             int position = 0;
             if (i == 0) {
-              IRegion firstLine = document
-                  .getLineInformationOfOffset(command.offset);
+              IRegion firstLine = document.getLineInformationOfOffset(command.offset);
               position = command.offset - firstLine.getOffset();
             }
             int length = line.length();
@@ -939,12 +871,10 @@ public class PHPUnitEditor extends PHPEditor { //implements
      * @see org.phpeclipse.phpdt.internal.ui.text.link.LinkedPositionUI.ExitPolicy#doExit(org.phpeclipse.phpdt.internal.ui.text.link.LinkedPositionManager,
      *      org.eclipse.swt.events.VerifyEvent, int, int)
      */
-    public ExitFlags doExit(LinkedPositionManager manager, VerifyEvent event,
-        int offset, int length) {
+    public ExitFlags doExit(LinkedPositionManager manager, VerifyEvent event, int offset, int length) {
       if (event.character == fExitCharacter) {
         if (manager.anyPositionIncludes(offset, length))
-          return new ExitFlags(LinkedPositionUI.COMMIT
-              | LinkedPositionUI.UPDATE_CARET, false);
+          return new ExitFlags(LinkedPositionUI.COMMIT | LinkedPositionUI.UPDATE_CARET, false);
         else
           return new ExitFlags(LinkedPositionUI.COMMIT, true);
       }
@@ -973,11 +903,12 @@ public class PHPUnitEditor extends PHPEditor { //implements
     LinkedPositionUI fEditor;
   };
 
-  private class BracketInserter implements VerifyKeyListener,
-      LinkedPositionUI.ExitListener {
+  private class BracketInserter implements VerifyKeyListener, LinkedPositionUI.ExitListener {
     private boolean fCloseBracketsPHP = true;
 
-    private boolean fCloseStringsPHP = true;
+    private boolean fCloseStringsPHPDQ = true;
+
+    private boolean fCloseStringsPHPSQ = true;
 
     private boolean fCloseBracketsHTML = true;
 
@@ -991,8 +922,12 @@ public class PHPUnitEditor extends PHPEditor { //implements
       fCloseBracketsPHP = enabled;
     }
 
-    public void setCloseStringsPHPEnabled(boolean enabled) {
-      fCloseStringsPHP = enabled;
+    public void setCloseStringsPHPDQEnabled(boolean enabled) {
+      fCloseStringsPHPDQ = enabled;
+    }
+
+    public void setCloseStringsPHPSQEnabled(boolean enabled) {
+      fCloseStringsPHPSQ = enabled;
     }
 
     public void setCloseBracketsHTMLEnabled(boolean enabled) {
@@ -1010,8 +945,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
         int maxEnd = endLine.getOffset() + endLine.getLength();
         while (end != maxEnd && Character.isWhitespace(document.getChar(end)))
           ++end;
-        return end != maxEnd
-            && Scanner.isPHPIdentifierPart(document.getChar(end));
+        return end != maxEnd && Scanner.isPHPIdentifierPart(document.getChar(end));
       } catch (BadLocationException e) {
         // be conservative
         return true;
@@ -1023,18 +957,15 @@ public class PHPUnitEditor extends PHPEditor { //implements
         int start = offset;
         IRegion startLine = document.getLineInformationOfOffset(start);
         int minStart = startLine.getOffset();
-        while (start != minStart
-            && Character.isWhitespace(document.getChar(start - 1)))
+        while (start != minStart && Character.isWhitespace(document.getChar(start - 1)))
           --start;
-        return start != minStart
-            && Scanner.isPHPIdentifierPart(document.getChar(start - 1));
+        return start != minStart && Scanner.isPHPIdentifierPart(document.getChar(start - 1));
       } catch (BadLocationException e) {
         return true;
       }
     }
 
-    private boolean hasCharacterToTheRight(IDocument document, int offset,
-        char character) {
+    private boolean hasCharacterToTheRight(IDocument document, int offset, char character) {
       try {
         int end = offset;
         IRegion endLine = document.getLineInformationOfOffset(end);
@@ -1062,7 +993,8 @@ public class PHPUnitEditor extends PHPEditor { //implements
       try {
         ITypedRegion partition = document.getPartition(offset);
         String type = partition.getType();
-        if (type.equals(IPHPPartitions.PHP_PARTITIONING)) {
+        if (type.equals(IPHPPartitions.PHP_PARTITIONING) || type.equals(IDocument.DEFAULT_CONTENT_TYPE)) {
+          // you will get IDocument.DEFAULT_CONTENT_TYPE for both PHP and HTML area
           switch (event.character) {
           case '(':
             if (hasCharacterToTheRight(document, offset + length, '('))
@@ -1076,7 +1008,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
           // fall through
           case '"':
             if (event.character == '"') {
-              if (!fCloseStringsPHP)
+              if (!fCloseStringsPHPDQ)
                 return;
               // changed for statements like echo "" print ""
               //    if (hasIdentifierToTheLeft(document, offset)
@@ -1093,46 +1025,36 @@ public class PHPUnitEditor extends PHPEditor { //implements
             // &&
             // (partition.getOffset() != offset))
             //         return;
-            final char character = event.character;
-            final char closingCharacter = getPeerCharacter(character);
-            final StringBuffer buffer = new StringBuffer();
-            buffer.append(character);
-            buffer.append(closingCharacter);
-            document.replace(offset, length, buffer.toString());
-            LinkedPositionManager manager = new LinkedPositionManager(document);
-            manager.addPosition(offset + 1, 0);
+            final char characterDQ = event.character;
+            final char closingCharacterDQ = getPeerCharacter(characterDQ);
+            final StringBuffer bufferDQ = new StringBuffer();
+            bufferDQ.append(characterDQ);
+            bufferDQ.append(closingCharacterDQ);
+            document.replace(offset, length, bufferDQ.toString());
+            LinkedPositionManager managerDQ = new LinkedPositionManager(document);
+            managerDQ.addPosition(offset + 1, 0);
             fOffset = offset;
             fLength = 2;
-            LinkedPositionUI editor = new LinkedPositionUI(sourceViewer,
-                manager);
-            editor.setCancelListener(this);
-            editor.setExitPolicy(new ExitPolicy(closingCharacter));
-            editor.setFinalCaretOffset(offset + 2);
-            editor.enter();
-            IRegion newSelection = editor.getSelectedRegion();
-            sourceViewer.setSelectedRange(newSelection.getOffset(),
-                newSelection.getLength());
+            LinkedPositionUI editorDQ = new LinkedPositionUI(sourceViewer, managerDQ);
+            editorDQ.setCancelListener(this);
+            editorDQ.setExitPolicy(new ExitPolicy(closingCharacterDQ));
+            editorDQ.setFinalCaretOffset(offset + 2);
+            editorDQ.enter();
+            IRegion newSelectionDQ = editorDQ.getSelectedRegion();
+            sourceViewer.setSelectedRange(newSelectionDQ.getOffset(), newSelectionDQ.getLength());
             event.doit = false;
-          }
-        } else if (type.equals(IPHPPartitions.HTML)
-            || type.equals(IDocument.DEFAULT_CONTENT_TYPE)) {
-          switch (event.character) {
-          case '(':
-            if (hasCharacterToTheRight(document, offset + length, '('))
-              return;
-          // fall through
-          case '[':
-            if (!fCloseBracketsHTML)
-              return;
-            if (hasIdentifierToTheRight(document, offset + length))
-              return;
-          // fall through
-          case '"':
-            if (event.character == '"') {
-              if (!fCloseStringsHTML)
+            break;
+          //          fall through
+          case '\'':
+            if (event.character == '\'') {
+              if (!fCloseStringsPHPSQ)
                 return;
-              if (hasIdentifierToTheLeft(document, offset)
-                  || hasIdentifierToTheRight(document, offset + length))
+              // changed for statements like echo "" print ""
+              //    if (hasIdentifierToTheLeft(document, offset)
+              // ||
+              // hasIdentifierToTheRight(document, offset +
+              // length))
+              if (hasIdentifierToTheRight(document, offset + length))
                 return;
             }
             //     ITypedRegion partition=
@@ -1142,28 +1064,75 @@ public class PHPUnitEditor extends PHPEditor { //implements
             // &&
             // (partition.getOffset() != offset))
             //         return;
-            final char character = event.character;
-            final char closingCharacter = getPeerCharacter(character);
-            final StringBuffer buffer = new StringBuffer();
-            buffer.append(character);
-            buffer.append(closingCharacter);
-            document.replace(offset, length, buffer.toString());
-            LinkedPositionManager manager = new LinkedPositionManager(document);
-            manager.addPosition(offset + 1, 0);
+            final char characterSQ = event.character;
+            final char closingCharacterSQ = getPeerCharacter(characterSQ);
+            final StringBuffer bufferSQ = new StringBuffer();
+            bufferSQ.append(characterSQ);
+            bufferSQ.append(closingCharacterSQ);
+            document.replace(offset, length, bufferSQ.toString());
+            LinkedPositionManager managerSQ = new LinkedPositionManager(document);
+            managerSQ.addPosition(offset + 1, 0);
             fOffset = offset;
             fLength = 2;
-            LinkedPositionUI editor = new LinkedPositionUI(sourceViewer,
-                manager);
-            editor.setCancelListener(this);
-            editor.setExitPolicy(new ExitPolicy(closingCharacter));
-            editor.setFinalCaretOffset(offset + 2);
-            editor.enter();
-            IRegion newSelection = editor.getSelectedRegion();
-            sourceViewer.setSelectedRange(newSelection.getOffset(),
-                newSelection.getLength());
+            LinkedPositionUI editorSQ = new LinkedPositionUI(sourceViewer, managerSQ);
+            editorSQ.setCancelListener(this);
+            editorSQ.setExitPolicy(new ExitPolicy(closingCharacterSQ));
+            editorSQ.setFinalCaretOffset(offset + 2);
+            editorSQ.enter();
+            IRegion newSelectionSQ = editorSQ.getSelectedRegion();
+            sourceViewer.setSelectedRange(newSelectionSQ.getOffset(), newSelectionSQ.getLength());
             event.doit = false;
           }
         }
+        //        } else if (type.equals(IPHPPartitions.HTML)) {
+        //          switch (event.character) {
+        //          case '(':
+        //            if (hasCharacterToTheRight(document, offset + length, '('))
+        //              return;
+        //          // fall through
+        //          case '[':
+        //            if (!fCloseBracketsHTML)
+        //              return;
+        //            if (hasIdentifierToTheRight(document, offset + length))
+        //              return;
+        //          // fall through
+        //          case '"':
+        //            if (event.character == '"') {
+        //              if (!fCloseStringsHTML)
+        //                return;
+        //              if (hasIdentifierToTheLeft(document, offset)
+        //                  || hasIdentifierToTheRight(document, offset + length))
+        //                return;
+        //            }
+        //            // ITypedRegion partition=
+        //            // document.getPartition(offset);
+        //            // if (!
+        //            // IDocument.DEFAULT_CONTENT_TYPE.equals(partition.getType())
+        //            // &&
+        //            // (partition.getOffset() != offset))
+        //            // return;
+        //            final char character = event.character;
+        //            final char closingCharacter = getPeerCharacter(character);
+        //            final StringBuffer buffer = new StringBuffer();
+        //            buffer.append(character);
+        //            buffer.append(closingCharacter);
+        //            document.replace(offset, length, buffer.toString());
+        //            LinkedPositionManager manager = new LinkedPositionManager(document);
+        //            manager.addPosition(offset + 1, 0);
+        //            fOffset = offset;
+        //            fLength = 2;
+        //            LinkedPositionUI editor = new LinkedPositionUI(sourceViewer,
+        //                manager);
+        //            editor.setCancelListener(this);
+        //            editor.setExitPolicy(new ExitPolicy(closingCharacter));
+        //            editor.setFinalCaretOffset(offset + 2);
+        //            editor.enter();
+        //            IRegion newSelection = editor.getSelectedRegion();
+        //            sourceViewer.setSelectedRange(newSelection.getOffset(),
+        //                newSelection.getLength());
+        //            event.doit = false;
+        //          }
+        //        }
       } catch (BadLocationException e) {
       }
     }
@@ -1188,25 +1157,21 @@ public class PHPUnitEditor extends PHPEditor { //implements
   protected ISavePolicy fSavePolicy;
 
   /**
-   * Listener to annotation model changes that updates the error tick in the tab
-   * image
+   * Listener to annotation model changes that updates the error tick in the tab image
    */
   private JavaEditorErrorTickUpdater fJavaEditorErrorTickUpdater;
 
   /** The editor's paint manager */
   //  private PaintManager fPaintManager;
   /** The editor's bracket painter */
-  private BracketPainter fBracketPainter;
-
+  //  private BracketPainter fBracketPainter;
   /** The editor's bracket matcher */
   private PHPPairMatcher fBracketMatcher;
 
   /** The editor's line painter */
-  private LinePainter fLinePainter;
-
+  //  private LinePainter fLinePainter;
   /** The editor's print margin ruler painter */
-  private PrintMarginPainter fPrintMarginPainter;
-
+  //  private PrintMarginPainter fPrintMarginPainter;
   /** The editor's problem painter */
   //  private ProblemPainter fProblemPainter;
   /** The editor's tab converter */
@@ -1221,11 +1186,11 @@ public class PHPUnitEditor extends PHPEditor { //implements
   private IJavaElement fRememberedElement;
 
   /**
-        * The remembered selection.
-        * @since 3.0
-        */
- private RememberedSelection fRememberedSelection= new RememberedSelection();
-       
+   * The remembered selection.
+   *
+   * @since 3.0
+   */
+  private RememberedSelection fRememberedSelection = new RememberedSelection();
 
   /** The remembered php element offset */
   private int fRememberedElementOffset;
@@ -1252,26 +1217,19 @@ public class PHPUnitEditor extends PHPEditor { //implements
   private final static String CODE_FORMATTER_TAB_SIZE = JavaCore.FORMATTER_TAB_SIZE;
 
   /** Preference key for matching brackets */
-  private final static String MATCHING_BRACKETS = PreferenceConstants.EDITOR_MATCHING_BRACKETS;
-
+  //  private final static String MATCHING_BRACKETS = PreferenceConstants.EDITOR_MATCHING_BRACKETS;
   /** Preference key for matching brackets color */
-  private final static String MATCHING_BRACKETS_COLOR = PreferenceConstants.EDITOR_MATCHING_BRACKETS_COLOR;
-
+  //  private final static String MATCHING_BRACKETS_COLOR = PreferenceConstants.EDITOR_MATCHING_BRACKETS_COLOR;
   /** Preference key for highlighting current line */
-  private final static String CURRENT_LINE = PreferenceConstants.EDITOR_CURRENT_LINE;
-
+  //  private final static String CURRENT_LINE = PreferenceConstants.EDITOR_CURRENT_LINE;
   /** Preference key for highlight color of current line */
-  private final static String CURRENT_LINE_COLOR = PreferenceConstants.EDITOR_CURRENT_LINE_COLOR;
-
+  //  private final static String CURRENT_LINE_COLOR = PreferenceConstants.EDITOR_CURRENT_LINE_COLOR;
   /** Preference key for showing print marging ruler */
-  private final static String PRINT_MARGIN = PreferenceConstants.EDITOR_PRINT_MARGIN;
-
+  //  private final static String PRINT_MARGIN = PreferenceConstants.EDITOR_PRINT_MARGIN;
   /** Preference key for print margin ruler color */
-  private final static String PRINT_MARGIN_COLOR = PreferenceConstants.EDITOR_PRINT_MARGIN_COLOR;
-
+  //  private final static String PRINT_MARGIN_COLOR = PreferenceConstants.EDITOR_PRINT_MARGIN_COLOR;
   /** Preference key for print margin ruler column */
-  private final static String PRINT_MARGIN_COLUMN = PreferenceConstants.EDITOR_PRINT_MARGIN_COLUMN;
-
+  //  private final static String PRINT_MARGIN_COLUMN = PreferenceConstants.EDITOR_PRINT_MARGIN_COLUMN;
   /** Preference key for inserting spaces rather than tabs */
   private final static String SPACES_FOR_TABS = PreferenceConstants.EDITOR_SPACES_FOR_TABS;
 
@@ -1335,12 +1293,14 @@ public class PHPUnitEditor extends PHPEditor { //implements
   /** Preference key for unknown annotation indication in overview ruler */
   private final static String UNKNOWN_INDICATION_IN_OVERVIEW_RULER = PreferenceConstants.EDITOR_UNKNOWN_INDICATION_IN_OVERVIEW_RULER;
 
-  /** Preference key for automatically closing strings */
-  private final static String CLOSE_STRINGS_PHP = PreferenceConstants.EDITOR_CLOSE_STRINGS_PHP;
+  /** Preference key for automatically closing double quoted strings */
+  private final static String CLOSE_STRINGS_DQ_PHP = PreferenceConstants.EDITOR_CLOSE_STRINGS_DQ_PHP;
 
-  /** Preference key for automatically wrapping Java strings */
-  private final static String WRAP_STRINGS = PreferenceConstants.EDITOR_WRAP_STRINGS;
+  /** Preference key for automatically closing single quoted strings */
+  private final static String CLOSE_STRINGS_SQ_PHP = PreferenceConstants.EDITOR_CLOSE_STRINGS_SQ_PHP;
 
+  /** Preference key for automatically wrapping Java strings */
+  //  private final static String WRAP_STRINGS = PreferenceConstants.EDITOR_WRAP_STRINGS_DQ;
   /** Preference key for automatically closing brackets and parenthesis */
   private final static String CLOSE_BRACKETS_PHP = PreferenceConstants.EDITOR_CLOSE_BRACKETS_PHP;
 
@@ -1351,8 +1311,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
   private final static String ADD_JAVADOC_TAGS = PreferenceConstants.EDITOR_ADD_JAVADOC_TAGS;
 
   /** Preference key for automatically formatting phpdocs */
-  private final static String FORMAT_JAVADOCS = PreferenceConstants.EDITOR_FORMAT_JAVADOCS;
-
+  //  private final static String FORMAT_JAVADOCS = PreferenceConstants.EDITOR_FORMAT_JAVADOCS;
   /** Preference key for automatically closing strings */
   private final static String CLOSE_STRINGS_HTML = PreferenceConstants.EDITOR_CLOSE_STRINGS_HTML;
 
@@ -1422,26 +1381,25 @@ public class PHPUnitEditor extends PHPEditor { //implements
   /**
    * Creates a new php unit editor.
    */
-  
+
   /**
-        * Reconciling listeners.
-        * @since 3.0
-        */
-       private ListenerList fReconcilingListeners= new ListenerList();
-
-       /**
-        * Mutex for the reconciler. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=63898
-        * for a description of the problem.
-        * <p>
-        * TODO remove once the underlying problem is solved.
-        * </p>
-        */
-       private final Object fReconcilerLock= new Object();
-  
+   * Reconciling listeners.
+   *
+   * @since 3.0
+   */
+  private ListenerList fReconcilingListeners = new ListenerList();
+
+  /**
+   * Mutex for the reconciler. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=63898 for a description of the problem.
+   * <p>
+   * TODO remove once the underlying problem is solved.
+   * </p>
+   */
+  private final Object fReconcilerLock = new Object();
+
   public PHPUnitEditor() {
     super();
-    setDocumentProvider(PHPeclipsePlugin.getDefault()
-        .getCompilationUnitDocumentProvider());
+    setDocumentProvider(PHPeclipsePlugin.getDefault().getCompilationUnitDocumentProvider());
     setEditorContextMenuId("#PHPEditorContext"); //$NON-NLS-1$
     setRulerContextMenuId("#PHPRulerContext"); //$NON-NLS-1$
     setOutlinerContextMenuId("#PHPOutlinerContext"); //$NON-NLS-1$
@@ -1466,10 +1424,8 @@ public class PHPUnitEditor extends PHPEditor { //implements
     // //$NON-NLS-1$
     //// WorkbenchHelp.setHelp(action,
     // IJavaHelpContextIds.QUICK_FIX_ACTION);
-    action = new ContentAssistAction(PHPEditorMessages.getResourceBundle(),
-        "ContentAssistProposal.", this); //$NON-NLS-1$
-    action
-        .setActionDefinitionId(PHPEditorActionDefinitionIds.CONTENT_ASSIST_PROPOSALS);
+    action = new ContentAssistAction(PHPEditorMessages.getResourceBundle(), "ContentAssistProposal.", this); //$NON-NLS-1$
+    action.setActionDefinitionId(PHPEditorActionDefinitionIds.CONTENT_ASSIST_PROPOSALS);
     setAction("ContentAssistProposal", action); //$NON-NLS-1$
     markAsStateDependentAction("ContentAssistProposal", true); //$NON-NLS-1$
     //         WorkbenchHelp.setHelp(action,
@@ -1494,52 +1450,48 @@ public class PHPUnitEditor extends PHPEditor { //implements
     // //$NON-NLS-1$
     //// WorkbenchHelp.setHelp(action,
     // IJavaHelpContextIds.PARAMETER_HINTS_ACTION);
-    action = new TextOperationAction(PHPEditorMessages.getResourceBundle(),
-        "Comment.", this, ITextOperationTarget.PREFIX); //$NON-NLS-1$
+    action = new TextOperationAction(PHPEditorMessages.getResourceBundle(), "Comment.", this, ITextOperationTarget.PREFIX); //$NON-NLS-1$
     action.setActionDefinitionId(PHPEditorActionDefinitionIds.COMMENT);
     setAction("Comment", action); //$NON-NLS-1$
     markAsStateDependentAction("Comment", true); //$NON-NLS-1$
     //         WorkbenchHelp.setHelp(action, IJavaHelpContextIds.COMMENT_ACTION);
-    action = new TextOperationAction(PHPEditorMessages.getResourceBundle(),
-        "Uncomment.", this, ITextOperationTarget.STRIP_PREFIX); //$NON-NLS-1$
+    action = new TextOperationAction(PHPEditorMessages.getResourceBundle(), "Uncomment.", this, ITextOperationTarget.STRIP_PREFIX); //$NON-NLS-1$
     action.setActionDefinitionId(PHPEditorActionDefinitionIds.UNCOMMENT);
     setAction("Uncomment", action); //$NON-NLS-1$
     markAsStateDependentAction("Uncomment", true); //$NON-NLS-1$
     //         WorkbenchHelp.setHelp(action, IJavaHelpContextIds.UNCOMMENT_ACTION);
 
-    action = new ToggleCommentAction(PHPEditorMessages.getResourceBundle(),
-        "ToggleComment.", this); //$NON-NLS-1$
+    action = new ToggleCommentAction(PHPEditorMessages.getResourceBundle(), "ToggleComment.", this); //$NON-NLS-1$
     action.setActionDefinitionId(PHPEditorActionDefinitionIds.TOGGLE_COMMENT);
     setAction("ToggleComment", action); //$NON-NLS-1$
     markAsStateDependentAction("ToggleComment", true); //$NON-NLS-1$
     //WorkbenchHelp.setHelp(action,
     // IJavaHelpContextIds.TOGGLE_COMMENT_ACTION);
     configureToggleCommentAction();
-       
-    action = new TextOperationAction(PHPEditorMessages.getResourceBundle(),
-        "Format.", this, ISourceViewer.FORMAT); //$NON-NLS-1$
+
+    action = new TextOperationAction(PHPEditorMessages.getResourceBundle(), "Format.", this, ISourceViewer.FORMAT); //$NON-NLS-1$
     action.setActionDefinitionId(PHPEditorActionDefinitionIds.FORMAT);
     setAction("Format", action); //$NON-NLS-1$
     markAsStateDependentAction("Format", true); //$NON-NLS-1$
-    markAsSelectionDependentAction("Format", true); //$NON-NLS-1$              
+    markAsSelectionDependentAction("Format", true); //$NON-NLS-1$
     //         WorkbenchHelp.setHelp(action, IJavaHelpContextIds.FORMAT_ACTION);
 
-//    action = new AddBlockCommentAction(PHPEditorMessages.getResourceBundle(),
-//        "AddBlockComment.", this); //$NON-NLS-1$
-//    action
-//        .setActionDefinitionId(PHPEditorActionDefinitionIds.ADD_BLOCK_COMMENT);
-//    setAction("AddBlockComment", action); //$NON-NLS-1$
-//    markAsStateDependentAction("AddBlockComment", true); //$NON-NLS-1$
-//    markAsSelectionDependentAction("AddBlockComment", true); //$NON-NLS-1$           
-//    //               WorkbenchHelp.setHelp(action,
-//    // IJavaHelpContextIds.ADD_BLOCK_COMMENT_ACTION);
-//    action = new RemoveBlockCommentAction(
-//        PHPEditorMessages.getResourceBundle(), "RemoveBlockComment.", this); //$NON-NLS-1$
-//    action
-//        .setActionDefinitionId(PHPEditorActionDefinitionIds.REMOVE_BLOCK_COMMENT);
-//    setAction("RemoveBlockComment", action); //$NON-NLS-1$
-//    markAsStateDependentAction("RemoveBlockComment", true); //$NON-NLS-1$
-//    markAsSelectionDependentAction("RemoveBlockComment", true); //$NON-NLS-1$                
+    //    action = new AddBlockCommentAction(PHPEditorMessages.getResourceBundle(),
+    //        "AddBlockComment.", this); //$NON-NLS-1$
+    //    action
+    //        .setActionDefinitionId(PHPEditorActionDefinitionIds.ADD_BLOCK_COMMENT);
+    //    setAction("AddBlockComment", action); //$NON-NLS-1$
+    //    markAsStateDependentAction("AddBlockComment", true); //$NON-NLS-1$
+    //    markAsSelectionDependentAction("AddBlockComment", true); //$NON-NLS-1$
+    //    // WorkbenchHelp.setHelp(action,
+    //    // IJavaHelpContextIds.ADD_BLOCK_COMMENT_ACTION);
+    //    action = new RemoveBlockCommentAction(
+    //        PHPEditorMessages.getResourceBundle(), "RemoveBlockComment.", this); //$NON-NLS-1$
+    //    action
+    //        .setActionDefinitionId(PHPEditorActionDefinitionIds.REMOVE_BLOCK_COMMENT);
+    //    setAction("RemoveBlockComment", action); //$NON-NLS-1$
+    //    markAsStateDependentAction("RemoveBlockComment", true); //$NON-NLS-1$
+    //    markAsSelectionDependentAction("RemoveBlockComment", true); //$NON-NLS-1$
     //         WorkbenchHelp.setHelp(action,
     // IJavaHelpContextIds.ADD_BLOCK_COMMENT_ACTION);
     //         action= new IndentAction(PHPEditorMessages.getResourceBundle(),
@@ -1549,31 +1501,57 @@ public class PHPUnitEditor extends PHPEditor { //implements
     //         markAsStateDependentAction("Indent", true); //$NON-NLS-1$
     //         markAsSelectionDependentAction("Indent", true); //$NON-NLS-1$
     //// WorkbenchHelp.setHelp(action, IJavaHelpContextIds.INDENT_ACTION);
-    //         
+    //
     //         action= new IndentAction(PHPEditorMessages.getResourceBundle(),
     // "Indent.", this, true); //$NON-NLS-1$
     //         setAction("IndentOnTab", action); //$NON-NLS-1$
     //         markAsStateDependentAction("IndentOnTab", true); //$NON-NLS-1$
     //         markAsSelectionDependentAction("IndentOnTab", true); //$NON-NLS-1$
-    //         
+    //
+
+    action = new AddBlockCommentAction(PHPEditorMessages.getResourceBundle(), "AddBlockComment.", this); //$NON-NLS-1$
+    action.setActionDefinitionId(PHPEditorActionDefinitionIds.ADD_BLOCK_COMMENT);
+    setAction("AddBlockComment", action); //$NON-NLS-1$
+    markAsStateDependentAction("AddBlockComment", true); //$NON-NLS-1$
+    markAsSelectionDependentAction("AddBlockComment", true); //$NON-NLS-1$
+    // WorkbenchHelp.setHelp(action, IJavaHelpContextIds.ADD_BLOCK_COMMENT_ACTION);
+
+    action = new RemoveBlockCommentAction(PHPEditorMessages.getResourceBundle(), "RemoveBlockComment.", this); //$NON-NLS-1$
+    action.setActionDefinitionId(PHPEditorActionDefinitionIds.REMOVE_BLOCK_COMMENT);
+    setAction("RemoveBlockComment", action); //$NON-NLS-1$
+    markAsStateDependentAction("RemoveBlockComment", true); //$NON-NLS-1$
+    markAsSelectionDependentAction("RemoveBlockComment", true); //$NON-NLS-1$
+    // WorkbenchHelp.setHelp(action, IJavaHelpContextIds.REMOVE_BLOCK_COMMENT_ACTION);
+
+    // action= new IndentAction(PHPEditorMessages.getResourceBundle(), "Indent.", this, false); //$NON-NLS-1$
+    // action.setActionDefinitionId(PHPEditorActionDefinitionIds.INDENT);
+    // setAction("Indent", action); //$NON-NLS-1$
+    // markAsStateDependentAction("Indent", true); //$NON-NLS-1$
+    // markAsSelectionDependentAction("Indent", true); //$NON-NLS-1$
+    //// WorkbenchHelp.setHelp(action, IJavaHelpContextIds.INDENT_ACTION);
+    //
+    action = new IndentAction(PHPEditorMessages.getResourceBundle(), "Indent.", this, true); //$NON-NLS-1$
+    setAction("IndentOnTab", action); //$NON-NLS-1$
+    markAsStateDependentAction("IndentOnTab", true); //$NON-NLS-1$
+    markAsSelectionDependentAction("IndentOnTab", true); //$NON-NLS-1$
+
     if (getPreferenceStore().getBoolean(PreferenceConstants.EDITOR_SMART_TAB)) {
       // don't replace Shift Right - have to make sure their enablement is
       // mutually exclusive
       //                       removeActionActivationCode(ITextEditorActionConstants.SHIFT_RIGHT);
       setActionActivationCode("IndentOnTab", '\t', -1, SWT.NONE); //$NON-NLS-1$
     }
-    fGenerateActionGroup = new GenerateActionGroup(this,
-        ITextEditorActionConstants.GROUP_EDIT);
-    fActionGroups = new CompositeActionGroup(
-        new ActionGroup[] { fGenerateActionGroup });
-    //   We have to keep the context menu group separate to have better
-    // control
-    // over positioning
-    fContextMenuGroup = new CompositeActionGroup(
-        new ActionGroup[] { fGenerateActionGroup });
-    //      rg,
-    //      new LocalHistoryActionGroup(this,
-    // ITextEditorActionConstants.GROUP_EDIT)});
+    fGenerateActionGroup = new GenerateActionGroup(this, ITextEditorActionConstants.GROUP_EDIT);
+    //    ActionGroup rg= new RefactorActionGroup(this, ITextEditorActionConstants.GROUP_EDIT);
+
+    // fActionGroups.addGroup(rg);
+    fActionGroups.addGroup(fGenerateActionGroup);
+
+    // We have to keep the context menu group separate to have better control over positioning
+    fContextMenuGroup = new CompositeActionGroup(new ActionGroup[] { fGenerateActionGroup
+    //         rg,
+        //             new LocalHistoryActionGroup(this, ITextEditorActionConstants.GROUP_EDIT)
+        });
 
   }
 
@@ -1585,20 +1563,17 @@ public class PHPUnitEditor extends PHPEditor { //implements
   }
 
   /**
-   * Returns the most narrow element including the given offset. If
-   * <code>reconcile</code> is <code>true</code> the editor's input element
-   * is reconciled in advance. If it is <code>false</code> this method only
-   * returns a result if the editor's input element does not need to be
-   * reconciled.
-   * 
+   * Returns the most narrow element including the given offset. If <code>reconcile</code> is <code>true</code> the editor's
+   * input element is reconciled in advance. If it is <code>false</code> this method only returns a result if the editor's input
+   * element does not need to be reconciled.
+   *
    * @param offset
    *          the offset included by the retrieved element
    * @param reconcile
    *          <code>true</code> if working copy should be reconciled
    */
   protected IJavaElement getElementAt(int offset, boolean reconcile) {
-    IWorkingCopyManager manager = PHPeclipsePlugin.getDefault()
-        .getWorkingCopyManager();
+    IWorkingCopyManager manager = PHPeclipsePlugin.getDefault().getWorkingCopyManager();
     ICompilationUnit unit = manager.getWorkingCopy(getEditorInput());
     if (unit != null) {
       try {
@@ -1635,15 +1610,14 @@ public class PHPUnitEditor extends PHPEditor { //implements
     //    fPaintManager = new PaintManager(getSourceViewer());
     LinePainter linePainter;
     linePainter = new LinePainter(getSourceViewer());
-    linePainter
-        .setHighlightColor(new Color(Display.getCurrent(), 225, 235, 224));
+    linePainter.setHighlightColor(new Color(Display.getCurrent(), 225, 235, 224));
     //    fPaintManager.addPainter(linePainter);
-    if (isBracketHighlightingEnabled())
-      startBracketHighlighting();
-    if (isLineHighlightingEnabled())
-      startLineHighlighting();
-    if (isPrintMarginVisible())
-      showPrintMargin();
+    //    if (isBracketHighlightingEnabled())
+    //      startBracketHighlighting();
+    //    if (isLineHighlightingEnabled())
+    //      startLineHighlighting();
+    //    if (isPrintMarginVisible())
+    //      showPrintMargin();
     //    Iterator e = ANNOTATION_MAP.keySet().iterator();
     //    while (e.hasNext()) {
     //      AnnotationType type = (AnnotationType) e.next();
@@ -1660,17 +1634,18 @@ public class PHPUnitEditor extends PHPEditor { //implements
     //    preferences.addPropertyChangeListener(fPropertyChangeListener);
     IPreferenceStore preferenceStore = getPreferenceStore();
     boolean closeBracketsPHP = preferenceStore.getBoolean(CLOSE_BRACKETS_PHP);
-    boolean closeStringsPHP = preferenceStore.getBoolean(CLOSE_STRINGS_PHP);
+    boolean closeStringsPHPDQ = preferenceStore.getBoolean(CLOSE_STRINGS_DQ_PHP);
+    boolean closeStringsPHPSQ = preferenceStore.getBoolean(CLOSE_STRINGS_SQ_PHP);
     boolean closeBracketsHTML = preferenceStore.getBoolean(CLOSE_BRACKETS_HTML);
     boolean closeStringsHTML = preferenceStore.getBoolean(CLOSE_STRINGS_HTML);
     fBracketInserter.setCloseBracketsPHPEnabled(closeBracketsPHP);
-    fBracketInserter.setCloseStringsPHPEnabled(closeStringsPHP);
+    fBracketInserter.setCloseStringsPHPDQEnabled(closeStringsPHPDQ);
+    fBracketInserter.setCloseStringsPHPSQEnabled(closeStringsPHPSQ);
     fBracketInserter.setCloseBracketsHTMLEnabled(closeBracketsHTML);
     fBracketInserter.setCloseStringsHTMLEnabled(closeStringsHTML);
     ISourceViewer sourceViewer = getSourceViewer();
     if (sourceViewer instanceof ITextViewerExtension)
-      ((ITextViewerExtension) sourceViewer)
-          .prependVerifyKeyListener(fBracketInserter);
+      ((ITextViewerExtension) sourceViewer).prependVerifyKeyListener(fBracketInserter);
   }
 
   private static char getPeerCharacter(char character) {
@@ -1685,157 +1660,84 @@ public class PHPUnitEditor extends PHPEditor { //implements
       return '[';
     case '"':
       return character;
+    case '\'':
+      return character;
     default:
       throw new IllegalArgumentException();
     }
   }
 
-  private void startBracketHighlighting() {
-    if (fBracketPainter == null) {
-      ISourceViewer sourceViewer = getSourceViewer();
-      fBracketPainter = new BracketPainter(sourceViewer);
-      fBracketPainter.setHighlightColor(getColor(MATCHING_BRACKETS_COLOR));
-      //      fPaintManager.addPainter(fBracketPainter);
-    }
-  }
-
-  private void stopBracketHighlighting() {
-    if (fBracketPainter != null) {
-      //      fPaintManager.removePainter(fBracketPainter);
-      fBracketPainter.deactivate(true);
-      fBracketPainter.dispose();
-      fBracketPainter = null;
-    }
-  }
-
-  private boolean isBracketHighlightingEnabled() {
-    IPreferenceStore store = getPreferenceStore();
-    return store.getBoolean(MATCHING_BRACKETS);
-  }
-
-  private void startLineHighlighting() {
-    if (fLinePainter == null) {
-      ISourceViewer sourceViewer = getSourceViewer();
-      fLinePainter = new LinePainter(sourceViewer);
-      fLinePainter.setHighlightColor(getColor(CURRENT_LINE_COLOR));
-      //      fPaintManager.addPainter(fLinePainter);
-    }
-  }
-
-  private void stopLineHighlighting() {
-    if (fLinePainter != null) {
-      //      fPaintManager.removePainter(fLinePainter);
-      fLinePainter.deactivate(true);
-      fLinePainter.dispose();
-      fLinePainter = null;
-    }
-  }
-
-  private boolean isLineHighlightingEnabled() {
-    IPreferenceStore store = getPreferenceStore();
-    return store.getBoolean(CURRENT_LINE);
-  }
-
-  private void showPrintMargin() {
-    if (fPrintMarginPainter == null) {
-      fPrintMarginPainter = new PrintMarginPainter(getSourceViewer());
-      fPrintMarginPainter.setMarginRulerColor(getColor(PRINT_MARGIN_COLOR));
-      fPrintMarginPainter.setMarginRulerColumn(getPreferenceStore().getInt(
-          PRINT_MARGIN_COLUMN));
-      //      fPaintManager.addPainter(fPrintMarginPainter);
-    }
-  }
-
-  private void hidePrintMargin() {
-    if (fPrintMarginPainter != null) {
-      //      fPaintManager.removePainter(fPrintMarginPainter);
-      fPrintMarginPainter.deactivate(true);
-      fPrintMarginPainter.dispose();
-      fPrintMarginPainter = null;
-    }
-  }
-
-  private boolean isPrintMarginVisible() {
-    IPreferenceStore store = getPreferenceStore();
-    return store.getBoolean(PRINT_MARGIN);
-  }
-
-  //  private void startAnnotationIndication(AnnotationType annotationType) {
-  //    if (fProblemPainter == null) {
-  //      fProblemPainter = new ProblemPainter(this, getSourceViewer());
-  //// fPaintManager.addPainter(fProblemPainter);
+  //  private void startBracketHighlighting() {
+  //    if (fBracketPainter == null) {
+  //      ISourceViewer sourceViewer = getSourceViewer();
+  //      fBracketPainter = new BracketPainter(sourceViewer);
+  //      fBracketPainter.setHighlightColor(getColor(MATCHING_BRACKETS_COLOR));
+  //      // fPaintManager.addPainter(fBracketPainter);
   //    }
-  //    fProblemPainter.setColor(annotationType, getColor(annotationType));
-  //    fProblemPainter.paintAnnotations(annotationType, true);
-  //    fProblemPainter.paint(IPainter.CONFIGURATION);
   //  }
   //
-  //  private void shutdownAnnotationIndication() {
-  //    if (fProblemPainter != null) {
-  //
-  //      if (!fProblemPainter.isPaintingAnnotations()) {
-  //// fPaintManager.removePainter(fProblemPainter);
-  //        fProblemPainter.deactivate(true);
-  //        fProblemPainter.dispose();
-  //        fProblemPainter = null;
-  //      } else {
-  //        fProblemPainter.paint(IPainter.CONFIGURATION);
-  //      }
+  //  private void stopBracketHighlighting() {
+  //    if (fBracketPainter != null) {
+  //      // fPaintManager.removePainter(fBracketPainter);
+  //      fBracketPainter.deactivate(true);
+  //      fBracketPainter.dispose();
+  //      fBracketPainter = null;
   //    }
   //  }
-  //
-  //  private void stopAnnotationIndication(AnnotationType annotationType) {
-  //    if (fProblemPainter != null) {
-  //      fProblemPainter.paintAnnotations(annotationType, false);
-  //      shutdownAnnotationIndication();
+
+  //  private boolean isBracketHighlightingEnabled() {
+  //    IPreferenceStore store = getPreferenceStore();
+  //    return store.getBoolean(MATCHING_BRACKETS);
+  //  }
+
+  //  private void startLineHighlighting() {
+  //    if (fLinePainter == null) {
+  //      ISourceViewer sourceViewer = getSourceViewer();
+  //      fLinePainter = new LinePainter(sourceViewer);
+  //      fLinePainter.setHighlightColor(getColor(CURRENT_LINE_COLOR));
+  //      // fPaintManager.addPainter(fLinePainter);
   //    }
   //  }
-  //
-  //  private boolean isAnnotationIndicationEnabled(AnnotationType
-  // annotationType) {
-  //    IPreferenceStore store = getPreferenceStore();
-  //    AnnotationInfo info = (AnnotationInfo)
-  // ANNOTATION_MAP.get(annotationType);
-  //    if (info != null)
-  //      return store.getBoolean(info.fEditorPreference);
-  //    return false;
+
+  //  private void stopLineHighlighting() {
+  //    if (fLinePainter != null) {
+  //      // fPaintManager.removePainter(fLinePainter);
+  //      fLinePainter.deactivate(true);
+  //      fLinePainter.dispose();
+  //      fLinePainter = null;
+  //    }
   //  }
-  //
-  //  private boolean
-  // isAnnotationIndicationInOverviewRulerEnabled(AnnotationType
-  // annotationType) {
+
+  //  private boolean isLineHighlightingEnabled() {
   //    IPreferenceStore store = getPreferenceStore();
-  //    AnnotationInfo info = (AnnotationInfo)
-  // ANNOTATION_MAP.get(annotationType);
-  //    if (info != null)
-  //      return store.getBoolean(info.fOverviewRulerPreference);
-  //    return false;
+  //    return store.getBoolean(CURRENT_LINE);
   //  }
-  //
-  //  private void showAnnotationIndicationInOverviewRuler(AnnotationType
-  // annotationType, boolean show) {
-  //    AdaptedSourceViewer asv = (AdaptedSourceViewer) getSourceViewer();
-  //    OverviewRuler ruler = asv.getOverviewRuler();
-  //    if (ruler != null) {
-  //      ruler.setColor(annotationType, getColor(annotationType));
-  //      ruler.showAnnotation(annotationType, show);
-  //      ruler.update();
+
+  //  private void showPrintMargin() {
+  //    if (fPrintMarginPainter == null) {
+  //      fPrintMarginPainter = new PrintMarginPainter(getSourceViewer());
+  //      fPrintMarginPainter.setMarginRulerColor(getColor(PRINT_MARGIN_COLOR));
+  //      fPrintMarginPainter.setMarginRulerColumn(getPreferenceStore().getInt(PRINT_MARGIN_COLUMN));
+  //      // fPaintManager.addPainter(fPrintMarginPainter);
   //    }
   //  }
-  //
-  //  private void setColorInOverviewRuler(AnnotationType annotationType, Color
-  // color) {
-  //    AdaptedSourceViewer asv = (AdaptedSourceViewer) getSourceViewer();
-  //    OverviewRuler ruler = asv.getOverviewRuler();
-  //    if (ruler != null) {
-  //      ruler.setColor(annotationType, color);
-  //      ruler.update();
+
+  //  private void hidePrintMargin() {
+  //    if (fPrintMarginPainter != null) {
+  //      // fPaintManager.removePainter(fPrintMarginPainter);
+  //      fPrintMarginPainter.deactivate(true);
+  //      fPrintMarginPainter.dispose();
+  //      fPrintMarginPainter = null;
   //    }
   //  }
 
+  //  private boolean isPrintMarginVisible() {
+  //    IPreferenceStore store = getPreferenceStore();
+  //    return store.getBoolean(PRINT_MARGIN);
+  //  }
+
   private int getTabSize() {
-    Preferences preferences = PHPeclipsePlugin.getDefault()
-        .getPluginPreferences();
+    Preferences preferences = PHPeclipsePlugin.getDefault().getPluginPreferences();
     return preferences.getInt(CODE_FORMATTER_TAB_SIZE);
   }
 
@@ -1844,30 +1746,6 @@ public class PHPUnitEditor extends PHPEditor { //implements
     return store.getBoolean(SPACES_FOR_TABS);
   }
 
-  //  private void showOverviewRuler() {
-  //    AdaptedSourceViewer asv = (AdaptedSourceViewer) getSourceViewer();
-  //    asv.showOverviewRuler();
-  //
-  //    OverviewRuler overviewRuler = asv.getOverviewRuler();
-  //    if (overviewRuler != null) {
-  //      for (int i = 0; i < ANNOTATION_LAYERS.length; i++) {
-  //        AnnotationType type = ANNOTATION_LAYERS[i];
-  //        overviewRuler.setLayer(type, i);
-  //        if (isAnnotationIndicationInOverviewRulerEnabled(type))
-  //          showAnnotationIndicationInOverviewRuler(type, true);
-  //      }
-  //    }
-  //  }
-  //
-  //  private void hideOverviewRuler() {
-  //    AdaptedSourceViewer asv = (AdaptedSourceViewer) getSourceViewer();
-  //    asv.hideOverviewRuler();
-  //  }
-  //
-  //  private boolean isOverviewRulerVisible() {
-  //    IPreferenceStore store = getPreferenceStore();
-  //    return store.getBoolean(OVERVIEW_RULER);
-  //  }
   private Color getColor(String key) {
     RGB rgb = PreferenceConverter.getColor(getPreferenceStore(), key);
     return getColor(rgb);
@@ -1888,8 +1766,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
   public void dispose() {
     ISourceViewer sourceViewer = getSourceViewer();
     if (sourceViewer instanceof ITextViewerExtension)
-      ((ITextViewerExtension) sourceViewer)
-          .removeVerifyKeyListener(fBracketInserter);
+      ((ITextViewerExtension) sourceViewer).removeVerifyKeyListener(fBracketInserter);
     //    if (fPropertyChangeListener != null) {
     //      Preferences preferences =
     // PHPeclipsePlugin.getDefault().getPluginPreferences();
@@ -1936,23 +1813,23 @@ public class PHPUnitEditor extends PHPEditor { //implements
       if (asv != null) {
         String p = event.getProperty();
         if (CLOSE_BRACKETS_PHP.equals(p)) {
-          fBracketInserter.setCloseBracketsPHPEnabled(getPreferenceStore()
-              .getBoolean(p));
+          fBracketInserter.setCloseBracketsPHPEnabled(getPreferenceStore().getBoolean(p));
           return;
         }
-        if (CLOSE_STRINGS_PHP.equals(p)) {
-          fBracketInserter.setCloseStringsPHPEnabled(getPreferenceStore()
-              .getBoolean(p));
+        if (CLOSE_STRINGS_DQ_PHP.equals(p)) {
+          fBracketInserter.setCloseStringsPHPDQEnabled(getPreferenceStore().getBoolean(p));
+          return;
+        }
+        if (CLOSE_STRINGS_SQ_PHP.equals(p)) {
+          fBracketInserter.setCloseStringsPHPSQEnabled(getPreferenceStore().getBoolean(p));
           return;
         }
         if (CLOSE_BRACKETS_HTML.equals(p)) {
-          fBracketInserter.setCloseBracketsHTMLEnabled(getPreferenceStore()
-              .getBoolean(p));
+          fBracketInserter.setCloseBracketsHTMLEnabled(getPreferenceStore().getBoolean(p));
           return;
         }
         if (CLOSE_STRINGS_HTML.equals(p)) {
-          fBracketInserter.setCloseStringsHTMLEnabled(getPreferenceStore()
-              .getBoolean(p));
+          fBracketInserter.setCloseStringsHTMLEnabled(getPreferenceStore().getBoolean(p));
           return;
         }
         if (SPACES_FOR_TABS.equals(p)) {
@@ -1962,52 +1839,49 @@ public class PHPUnitEditor extends PHPEditor { //implements
             stopTabConversion();
           return;
         }
-        if (MATCHING_BRACKETS.equals(p)) {
-          if (isBracketHighlightingEnabled())
-            startBracketHighlighting();
-          else
-            stopBracketHighlighting();
-          return;
-        }
-        if (MATCHING_BRACKETS_COLOR.equals(p)) {
-          if (fBracketPainter != null)
-            fBracketPainter
-                .setHighlightColor(getColor(MATCHING_BRACKETS_COLOR));
-          return;
-        }
-        if (CURRENT_LINE.equals(p)) {
-          if (isLineHighlightingEnabled())
-            startLineHighlighting();
-          else
-            stopLineHighlighting();
-          return;
-        }
-        if (CURRENT_LINE_COLOR.equals(p)) {
-          if (fLinePainter != null) {
-            stopLineHighlighting();
-            startLineHighlighting();
-          }
-          return;
-        }
-        if (PRINT_MARGIN.equals(p)) {
-          if (isPrintMarginVisible())
-            showPrintMargin();
-          else
-            hidePrintMargin();
-          return;
-        }
-        if (PRINT_MARGIN_COLOR.equals(p)) {
-          if (fPrintMarginPainter != null)
-            fPrintMarginPainter
-                .setMarginRulerColor(getColor(PRINT_MARGIN_COLOR));
-          return;
-        }
-        if (PRINT_MARGIN_COLUMN.equals(p)) {
-          if (fPrintMarginPainter != null)
-            fPrintMarginPainter.setMarginRulerColumn(getPreferenceStore()
-                .getInt(PRINT_MARGIN_COLUMN));
-          return;
-        }
+        //        if (MATCHING_BRACKETS.equals(p)) {
+        //          if (isBracketHighlightingEnabled())
+        //            startBracketHighlighting();
+        //          else
+        //            stopBracketHighlighting();
+        //          return;
+        //        }
+        //        if (MATCHING_BRACKETS_COLOR.equals(p)) {
+        //          if (fBracketPainter != null)
+        //            fBracketPainter.setHighlightColor(getColor(MATCHING_BRACKETS_COLOR));
+        //          return;
+        //        }
+        //        if (CURRENT_LINE.equals(p)) {
+        //          if (isLineHighlightingEnabled())
+        //            startLineHighlighting();
+        //          else
+        //            stopLineHighlighting();
+        //          return;
+        //        }
+        //        if (CURRENT_LINE_COLOR.equals(p)) {
+        //          if (fLinePainter != null) {
+        //            stopLineHighlighting();
+        //            startLineHighlighting();
+        //          }
+        //          return;
+        //        }
+        //        if (PRINT_MARGIN.equals(p)) {
+        //          if (isPrintMarginVisible())
+        //            showPrintMargin();
+        //          else
+        //            hidePrintMargin();
+        //          return;
+        //        }
+        //        if (PRINT_MARGIN_COLOR.equals(p)) {
+        //          if (fPrintMarginPainter != null)
+        //            fPrintMarginPainter.setMarginRulerColor(getColor(PRINT_MARGIN_COLOR));
+        //          return;
+        //        }
+        //        if (PRINT_MARGIN_COLUMN.equals(p)) {
+        //          if (fPrintMarginPainter != null)
+        //            fPrintMarginPainter.setMarginRulerColumn(getPreferenceStore().getInt(PRINT_MARGIN_COLUMN));
+        //          return;
+        //        }
         //        if (OVERVIEW_RULER.equals(p)) {
         //          if (isOverviewRulerVisible())
         //            showOverviewRuler();
@@ -2048,8 +1922,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
         //        }
         IContentAssistant c = asv.getContentAssistant();
         if (c instanceof ContentAssistant)
-          ContentAssistPreference.changeConfiguration((ContentAssistant) c,
-              getPreferenceStore(), event);
+          ContentAssistPreference.changeConfiguration((ContentAssistant) c, getPreferenceStore(), event);
       }
     } finally {
       super.handlePreferenceStoreChanged(event);
@@ -2057,10 +1930,9 @@ public class PHPUnitEditor extends PHPEditor { //implements
   }
 
   /*
-   * @see org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#handlePreferencePropertyChanged(org.eclipse.core.runtime.Preferences.PropertyChangeEvent)
+   * @see net.sourceforge.phpdt.internal.ui.javaeditor.JavaEditor#handlePreferencePropertyChanged(org.eclipse.core.runtime.Preferences.PropertyChangeEvent)
    */
-  protected void handlePreferencePropertyChanged(
-      org.eclipse.core.runtime.Preferences.PropertyChangeEvent event) {
+  protected void handlePreferencePropertyChanged(org.eclipse.core.runtime.Preferences.PropertyChangeEvent event) {
     AdaptedSourceViewer asv = (AdaptedSourceViewer) getSourceViewer();
     if (asv != null) {
       String p = event.getProperty();
@@ -2074,9 +1946,9 @@ public class PHPUnitEditor extends PHPEditor { //implements
   }
 
   /**
-   * Handles a property change event describing a change of the php core's
-   * preferences and updates the preference related editor properties.
-   * 
+   * Handles a property change event describing a change of the php core's preferences and updates the preference related editor
+   * properties.
+   *
    * @param event
    *          the property change event
    */
@@ -2094,15 +1966,12 @@ public class PHPUnitEditor extends PHPEditor { //implements
   //    }
   //  }
   /*
-   * @see org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#createJavaSourceViewer(org.eclipse.swt.widgets.Composite,
-   *      org.eclipse.jface.text.source.IVerticalRuler,
-   *      org.eclipse.jface.text.source.IOverviewRuler, boolean, int)
+   * @see net.sourceforge.phpdt.internal.ui.javaeditor.JavaEditor#createJavaSourceViewer(org.eclipse.swt.widgets.Composite,
+   *      org.eclipse.jface.text.source.IVerticalRuler, org.eclipse.jface.text.source.IOverviewRuler, boolean, int)
    */
-  protected ISourceViewer createJavaSourceViewer(Composite parent,
-      IVerticalRuler verticalRuler, IOverviewRuler overviewRuler,
+  protected ISourceViewer createJavaSourceViewer(Composite parent, IVerticalRuler verticalRuler, IOverviewRuler overviewRuler,
       boolean isOverviewRulerVisible, int styles, IPreferenceStore store) {
-    return new AdaptedSourceViewer(parent, verticalRuler, overviewRuler,
-        isOverviewRulerVisible, styles, store);
+    return new AdaptedSourceViewer(parent, verticalRuler, overviewRuler, isOverviewRulerVisible, styles, store);
   }
 
   //  protected ISourceViewer createJavaSourceViewer(Composite parent,
@@ -2116,26 +1985,25 @@ public class PHPUnitEditor extends PHPEditor { //implements
       if (document != null) {
         int end = offset + length;
         int documentLength = document.getLength();
-        return 0 <= offset && offset <= documentLength && 0 <= end
-            && end <= documentLength;
+        return 0 <= offset && offset <= documentLength && 0 <= end && end <= documentLength;
       }
     }
     return false;
   }
 
   /*
-        * @see org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#getInputElement()
-        */
-       protected IJavaElement getInputJavaElement() {
-               return PHPeclipsePlugin.getDefault().getWorkingCopyManager().getWorkingCopy(getEditorInput());
-       }
+   * @see net.sourceforge.phpdt.internal.ui.javaeditor.JavaEditor#getInputElement()
+   */
+  protected IJavaElement getInputJavaElement() {
+    return PHPeclipsePlugin.getDefault().getWorkingCopyManager().getWorkingCopy(getEditorInput());
+  }
+
   /*
    * @see AbstractTextEditor#editorContextMenuAboutToShow(IMenuManager)
    */
   public void editorContextMenuAboutToShow(IMenuManager menu) {
     super.editorContextMenuAboutToShow(menu);
-    ActionContext context = new ActionContext(getSelectionProvider()
-        .getSelection());
+    ActionContext context = new ActionContext(getSelectionProvider().getSelection());
     fContextMenuGroup.setContext(context);
     fContextMenuGroup.fillContextMenu(menu);
     fContextMenuGroup.setContext(null);
@@ -2146,16 +2014,13 @@ public class PHPUnitEditor extends PHPEditor { //implements
    */
   protected void setOutlinePageInput(JavaOutlinePage page, IEditorInput input) {
     if (page != null) {
-      IWorkingCopyManager manager = PHPeclipsePlugin.getDefault()
-          .getWorkingCopyManager();
+      IWorkingCopyManager manager = PHPeclipsePlugin.getDefault().getWorkingCopyManager();
       page.setInput(manager.getWorkingCopy(input));
     }
   }
-  
 
   /*
-   * @see AbstractTextEditor#performSaveOperation(WorkspaceModifyOperation,
-   *      IProgressMonitor)
+   * @see AbstractTextEditor#performSaveOperation(WorkspaceModifyOperation, IProgressMonitor)
    */
   //  protected void performSaveOperation(WorkspaceModifyOperation operation,
   // IProgressMonitor progressMonitor) {
@@ -2190,36 +2055,36 @@ public class PHPUnitEditor extends PHPEditor { //implements
       if (isSaveAsAllowed()) {
 
         /*
-         * 1GEUSSR: ITPUI:ALL - User should never loose changes made in the
-         * editors. Changed Behavior to make sure that if called inside a
-         * regular save (because of deletion of input element) there is a way to
-         * report back to the caller.
+         * 1GEUSSR: ITPUI:ALL - User should never loose changes made in the editors. Changed Behavior to make sure that if called
+         * inside a regular save (because of deletion of input element) there is a way to report back to the caller.
          */
         performSaveAs(progressMonitor);
 
       } else {
 
         /*
-         * 1GF5YOX: ITPJUI:ALL - Save of delete file claims it's still there
-         * Missing resources.
+         * 1GF5YOX: ITPJUI:ALL - Save of delete file claims it's still there Missing resources.
          */
         Shell shell = getSite().getShell();
         MessageDialog
             .openError(
                 shell,
-                PHPEditorMessages
-                    .getString("PHPUnitEditor.error.saving.title1"), PHPEditorMessages.getString("PHPUnitEditor.error.saving.message1")); //$NON-NLS-1$ //$NON-NLS-2$
+                PHPEditorMessages.getString("PHPUnitEditor.error.saving.title1"), PHPEditorMessages.getString("PHPUnitEditor.error.saving.message1")); //$NON-NLS-1$ //$NON-NLS-2$
       }
 
     } else {
+      if (getPreferenceStore().getBoolean(PreferenceConstants.EDITOR_P_RTRIM_ON_SAVE)) {
+        RTrimAction trimAction = new RTrimAction();
+        trimAction.setActiveEditor(null, getSite().getPage().getActiveEditor());
+        trimAction.run(null);
+      }
 
       setStatusLineErrorMessage(null);
 
       updateState(getEditorInput());
       validateState(getEditorInput());
 
-      IWorkingCopyManager manager = PHPeclipsePlugin.getDefault()
-          .getWorkingCopyManager();
+      IWorkingCopyManager manager = PHPeclipsePlugin.getDefault().getWorkingCopyManager();
       ICompilationUnit unit = manager.getWorkingCopy(getEditorInput());
 
       if (unit != null) {
@@ -2229,10 +2094,6 @@ public class PHPUnitEditor extends PHPEditor { //implements
       } else
         performSave(false, progressMonitor);
     }
-    ShowExternalPreviewAction a = ShowExternalPreviewAction.getInstance();
-    if (a != null) {
-      a.refresh();
-    }
   }
 
   public boolean isSaveAsAllowed() {
@@ -2240,11 +2101,9 @@ public class PHPUnitEditor extends PHPEditor { //implements
   }
 
   /**
-   * The compilation unit editor implementation of this
-   * <code>AbstractTextEditor</code> method asks the user for the workspace
-   * path of a file resource and saves the document there. See
-   * http://dev.eclipse.org/bugs/show_bug.cgi?id=6295
-   * 
+   * The compilation unit editor implementation of this <code>AbstractTextEditor</code> method asks the user for the workspace
+   * path of a file resource and saves the document there. See http://dev.eclipse.org/bugs/show_bug.cgi?id=6295
+   *
    * @param progressMonitor
    *          the progress monitor
    */
@@ -2255,9 +2114,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
 
     SaveAsDialog dialog = new SaveAsDialog(shell);
 
-    IFile original = (input instanceof IFileEditorInput) ? ((IFileEditorInput) input)
-        .getFile()
-        : null;
+    IFile original = (input instanceof IFileEditorInput) ? ((IFileEditorInput) input).getFile() : null;
     if (original != null)
       dialog.setOriginalFile(original);
 
@@ -2270,9 +2127,8 @@ public class PHPUnitEditor extends PHPEditor { //implements
     }
 
     if (provider.isDeleted(input) && original != null) {
-      String message = PHPEditorMessages
-          .getFormattedString(
-              "CompilationUnitEditor.warning.save.delete", new Object[] { original.getName() }); //$NON-NLS-1$
+      String message = PHPEditorMessages.getFormattedString(
+          "CompilationUnitEditor.warning.save.delete", new Object[] { original.getName() }); //$NON-NLS-1$
       dialog.setErrorMessage(null);
       dialog.setMessage(message, IMessageProvider.WARNING);
     }
@@ -2298,8 +2154,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
     try {
 
       provider.aboutToChange(newInput);
-      getDocumentProvider().saveDocument(progressMonitor, newInput,
-          getDocumentProvider().getDocument(getEditorInput()), true);
+      getDocumentProvider().saveDocument(progressMonitor, newInput, getDocumentProvider().getDocument(getEditorInput()), true);
       success = true;
 
     } catch (CoreException x) {
@@ -2308,8 +2163,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
         ErrorDialog
             .openError(
                 shell,
-                PHPEditorMessages
-                    .getString("CompilationUnitEditor.error.saving.title2"), PHPEditorMessages.getString("CompilationUnitEditor.error.saving.message2"), x.getStatus()); //$NON-NLS-1$ //$NON-NLS-2$
+                PHPEditorMessages.getString("CompilationUnitEditor.error.saving.title2"), PHPEditorMessages.getString("CompilationUnitEditor.error.saving.message2"), x.getStatus()); //$NON-NLS-1$ //$NON-NLS-2$
     } finally {
       provider.changed(newInput);
       if (success)
@@ -2331,7 +2185,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
 
   //   /*
   //    * @see
-  // org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#installOverrideIndicator(boolean)
+  // net.sourceforge.phpdt.internal.ui.javaeditor.JavaEditor#installOverrideIndicator(boolean)
   //    * @since 3.0
   //    */
   //   protected void installOverrideIndicator(boolean waitForReconcilation) {
@@ -2344,16 +2198,16 @@ public class PHPUnitEditor extends PHPEditor { //implements
   //                   IJavaElement inputElement= getInputJavaElement();
   //                   if (model == null || inputElement == null)
   //                           return;
-  //                   
+  //
   //                   fOverrideIndicatorManager= new OverrideIndicatorManager(model,
   // inputElement, null);
   //                   addReconcileListener(fOverrideIndicatorManager);
   //           }
   //   }
-  //   
+  //
   //   /*
   //    * @see
-  // org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#uninstallOverrideIndicator()
+  // net.sourceforge.phpdt.internal.ui.javaeditor.JavaEditor#uninstallOverrideIndicator()
   //    * @since 3.0
   //    */
   //   protected void uninstallOverrideIndicator() {
@@ -2364,7 +2218,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
 
   /**
    * Configures the toggle comment action
-   * 
+   *
    * @since 3.0
    */
   private void configureToggleCommentAction() {
@@ -2376,11 +2230,20 @@ public class PHPUnitEditor extends PHPEditor { //implements
     }
   }
 
+  //  private void configureTabConverter() {
+  //    if (fTabConverter != null) {
+  //      IDocumentProvider provider = getDocumentProvider();
+  //      if (provider instanceof PHPDocumentProvider) {
+  //        PHPDocumentProvider cup = (PHPDocumentProvider) provider;
+  //        fTabConverter.setLineTracker(cup.createLineTracker(getEditorInput()));
+  //      }
+  //    }
+  //  }
   private void configureTabConverter() {
     if (fTabConverter != null) {
       IDocumentProvider provider = getDocumentProvider();
-      if (provider instanceof PHPDocumentProvider) {
-        PHPDocumentProvider cup = (PHPDocumentProvider) provider;
+      if (provider instanceof ICompilationUnitDocumentProvider) {
+        ICompilationUnitDocumentProvider cup = (ICompilationUnitDocumentProvider) provider;
         fTabConverter.setLineTracker(cup.createLineTracker(getEditorInput()));
       }
     }
@@ -2409,20 +2272,33 @@ public class PHPUnitEditor extends PHPEditor { //implements
   }
 
   /*
-   * @see org.eclipse.ui.texteditor.AbstractTextEditor#performSave(boolean,
-   *      org.eclipse.core.runtime.IProgressMonitor)
+   * @see org.eclipse.ui.texteditor.AbstractTextEditor#performSave(boolean, org.eclipse.core.runtime.IProgressMonitor)
    */
   protected void performSave(boolean overwrite, IProgressMonitor progressMonitor) {
+    //    IDocumentProvider p = getDocumentProvider();
+    //    if (p instanceof PHPDocumentProvider) {
+    //      PHPDocumentProvider cp = (PHPDocumentProvider) p;
+    //      cp.setSavePolicy(fSavePolicy);
+    //    }
+    //    try {
+    //      super.performSave(overwrite, progressMonitor);
+    //    } finally {
+    //      if (p instanceof PHPDocumentProvider) {
+    //        PHPDocumentProvider cp = (PHPDocumentProvider) p;
+    //        cp.setSavePolicy(null);
+    //      }
+    //    }
+
     IDocumentProvider p = getDocumentProvider();
-    if (p instanceof PHPDocumentProvider) {
-      PHPDocumentProvider cp = (PHPDocumentProvider) p;
+    if (p instanceof ICompilationUnitDocumentProvider) {
+      ICompilationUnitDocumentProvider cp = (ICompilationUnitDocumentProvider) p;
       cp.setSavePolicy(fSavePolicy);
     }
     try {
       super.performSave(overwrite, progressMonitor);
     } finally {
-      if (p instanceof PHPDocumentProvider) {
-        PHPDocumentProvider cp = (PHPDocumentProvider) p;
+      if (p instanceof ICompilationUnitDocumentProvider) {
+        ICompilationUnitDocumentProvider cp = (ICompilationUnitDocumentProvider) p;
         cp.setSavePolicy(null);
       }
     }
@@ -2439,7 +2315,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
 
   /**
    * Asks the user if it is ok to store in non-workbench encoding.
-   * 
+   *
    * @return <true>if the user wants to continue
    */
   private boolean askIfNonWorkbenchEncodingIsOk() {
@@ -2451,72 +2327,61 @@ public class PHPUnitEditor extends PHPEditor { //implements
       String defaultEncoding = storageProvider.getDefaultEncoding();
       if (encoding != null && !encoding.equals(defaultEncoding)) {
         Shell shell = getSite().getShell();
-        String title = PHPEditorMessages
-            .getString("PHPUnitEditor.warning.save.nonWorkbenchEncoding.title"); //$NON-NLS-1$
+        String title = PHPEditorMessages.getString("PHPUnitEditor.warning.save.nonWorkbenchEncoding.title"); //$NON-NLS-1$
         String msg;
         if (input != null)
-          msg = MessageFormat
-              .format(
-                  PHPEditorMessages
-                      .getString("PHPUnitEditor.warning.save.nonWorkbenchEncoding.message1"),
-                  new String[] { input.getName(), encoding }); //$NON-NLS-1$
+          msg = MessageFormat.format(PHPEditorMessages.getString("PHPUnitEditor.warning.save.nonWorkbenchEncoding.message1"),
+              new String[] { input.getName(), encoding }); //$NON-NLS-1$
         else
-          msg = MessageFormat
-              .format(
-                  PHPEditorMessages
-                      .getString("PHPUnitEditor.warning.save.nonWorkbenchEncoding.message2"),
-                  new String[] { encoding }); //$NON-NLS-1$
+          msg = MessageFormat.format(PHPEditorMessages.getString("PHPUnitEditor.warning.save.nonWorkbenchEncoding.message2"),
+              new String[] { encoding }); //$NON-NLS-1$
         return MessageDialog.openQuestion(shell, title, msg);
       }
     }
     return true;
   }
 
-       /*
-        * @see org.eclipse.jdt.internal.ui.text.java.IJavaReconcilingListener#aboutToBeReconciled()
-        * @since 3.0
-        */
-       public void aboutToBeReconciled() {
-
-               // Notify AST provider
-//             PHPeclipsePlugin.getDefault().getASTProvider().aboutToBeReconciled(getInputJavaElement());
-               
-               // Notify listeners
-               Object[] listeners = fReconcilingListeners.getListeners();
-               for (int i = 0, length= listeners.length; i < length; ++i)
-                       ((IJavaReconcilingListener)listeners[i]).aboutToBeReconciled();
-       }
-       
   /*
-        * @see org.eclipse.jdt.internal.ui.text.java.IJavaReconcilingListener#reconciled(CompilationUnit, boolean, IProgressMonitor)
-        * @since 3.0
-        */
-       public void reconciled(CompilationUnit ast, boolean forced, IProgressMonitor progressMonitor) {
-
-               // Always notify AST provider
-//             PHPeclipsePlugin.getDefault().getASTProvider().reconciled(ast, getInputJavaElement());
-               
-               // Notify listeners
-//             Object[] listeners = fReconcilingListeners.getListeners();
-//             for (int i = 0, length= listeners.length; i < length; ++i)
-//                     ((IJavaReconcilingListener)listeners[i]).reconciled(ast, forced, progressMonitor);
-
-               // Update Java Outline page selection
-               if (!forced && !progressMonitor.isCanceled()) {
-                       Shell shell= getSite().getShell();
-                       if (shell != null && !shell.isDisposed()) {
-                               shell.getDisplay().asyncExec(new Runnable() {
-                                       public void run() {
-                                               selectionChanged();
-                                       }
-                               });
-                       }
-               }
-       }
-
-  private boolean synchronizeOutlineOnCursorMove() {
-    return PreferenceConstants.getPreferenceStore().getBoolean(
-        PreferenceConstants.EDITOR_SYNC_OUTLINE_ON_CURSOR_MOVE);
+   * @see net.sourceforge.phpdt.internal.ui.text.java.IJavaReconcilingListener#aboutToBeReconciled()
+   * @since 3.0
+   */
+  public void aboutToBeReconciled() {
+
+    // Notify AST provider
+    //         PHPeclipsePlugin.getDefault().getASTProvider().aboutToBeReconciled(getInputJavaElement());
+
+    // Notify listeners
+    Object[] listeners = fReconcilingListeners.getListeners();
+    for (int i = 0, length = listeners.length; i < length; ++i)
+      ((IJavaReconcilingListener) listeners[i]).aboutToBeReconciled();
+  }
+
+  /*
+   * @see net.sourceforge.phpdt.internal.ui.text.java.IJavaReconcilingListener#reconciled(CompilationUnit, boolean,
+   *      IProgressMonitor)
+   * @since 3.0
+   */
+  public void reconciled(CompilationUnit ast, boolean forced, IProgressMonitor progressMonitor) {
+
+    // Always notify AST provider
+    //         PHPeclipsePlugin.getDefault().getASTProvider().reconciled(ast, getInputJavaElement());
+
+    // Notify listeners
+    //         Object[] listeners = fReconcilingListeners.getListeners();
+    //         for (int i = 0, length= listeners.length; i < length; ++i)
+    //                 ((IJavaReconcilingListener)listeners[i]).reconciled(ast, forced, progressMonitor);
+
+    // Update Java Outline page selection
+    if (!forced && !progressMonitor.isCanceled()) {
+      Shell shell = getSite().getShell();
+      if (shell != null && !shell.isDisposed()) {
+        shell.getDisplay().asyncExec(new Runnable() {
+          public void run() {
+            selectionChanged();
+          }
+        });
+      }
+    }
   }
 
   /**
@@ -2525,8 +2390,7 @@ public class PHPUnitEditor extends PHPEditor { //implements
   private IJavaElement findElement(IJavaElement element) {
     if (element == null)
       return null;
-    IWorkingCopyManager manager = PHPeclipsePlugin.getDefault()
-        .getWorkingCopyManager();
+    IWorkingCopyManager manager = PHPeclipsePlugin.getDefault().getWorkingCopyManager();
     ICompilationUnit unit = manager.getWorkingCopy(getEditorInput());
     if (unit != null) {
       try {
@@ -2560,147 +2424,157 @@ public class PHPUnitEditor extends PHPEditor { //implements
     return -1;
   }
 
-  
-
   /*
    * @see AbstractTextEditor#restoreSelection()
    */
-//  protected void restoreSelection() {
-//    try {
-//      if (getSourceViewer() == null || fRememberedSelection == null)
-//        return;
-//      IJavaElement newElement = findElement(fRememberedElement);
-//      int newOffset = getOffset(newElement);
-//      int delta = (newOffset > -1 && fRememberedElementOffset > -1) ? newOffset
-//          - fRememberedElementOffset : 0;
-//      if (isValidSelection(delta + fRememberedSelection.getOffset(),
-//          fRememberedSelection.getLength()))
-//        selectAndReveal(delta + fRememberedSelection.getOffset(),
-//            fRememberedSelection.getLength());
-//    } finally {
-//      fRememberedSelection = null;
-//      fRememberedElement = null;
-//      fRememberedElementOffset = -1;
-//    }
-//  }
+  //  protected void restoreSelection() {
+  //    try {
+  //      if (getSourceViewer() == null || fRememberedSelection == null)
+  //        return;
+  //      IJavaElement newElement = findElement(fRememberedElement);
+  //      int newOffset = getOffset(newElement);
+  //      int delta = (newOffset > -1 && fRememberedElementOffset > -1) ? newOffset
+  //          - fRememberedElementOffset : 0;
+  //      if (isValidSelection(delta + fRememberedSelection.getOffset(),
+  //          fRememberedSelection.getLength()))
+  //        selectAndReveal(delta + fRememberedSelection.getOffset(),
+  //            fRememberedSelection.getLength());
+  //    } finally {
+  //      fRememberedSelection = null;
+  //      fRememberedElement = null;
+  //      fRememberedElementOffset = -1;
+  //    }
+  //  }
+  /**
+   * Tells whether this is the active editor in the active page.
+   *
+   * @return <code>true</code> if this is the active editor in the active page
+   * @see IWorkbenchPage#getActiveEditor();
+   */
+  protected final boolean isActiveEditor() {
+    IWorkbenchWindow window = getSite().getWorkbenchWindow();
+    IWorkbenchPage page = window.getActivePage();
+    if (page == null)
+      return false;
+    IEditorPart activeEditor = page.getActiveEditor();
+    return activeEditor != null && activeEditor.equals(this);
+  }
 
   /**
-        * Tells whether this is the active editor in the active page.
-        *
-        * @return <code>true</code> if this is the active editor in the active page
-        * @see IWorkbenchPage#getActiveEditor();
-        */
-       protected final boolean isActiveEditor() {
-               IWorkbenchWindow window= getSite().getWorkbenchWindow();
-               IWorkbenchPage page= window.getActivePage();
-               if (page == null)
-                       return false;
-               IEditorPart activeEditor= page.getActiveEditor();
-               return activeEditor != null && activeEditor.equals(this);
-       }
-       
-       /**
-        * Adds the given listener.
-        * Has no effect if an identical listener was not already registered.
-        * 
-        * @param listener      The reconcile listener to be added
-        * @since 3.0
-        */
-       final void addReconcileListener(IJavaReconcilingListener listener) {
-               synchronized (fReconcilingListeners) {
-                       fReconcilingListeners.add(listener);
-               }
-       }
-       
-       /**
-        * Removes the given listener.
-        * Has no effect if an identical listener was not already registered.
-        * 
-        * @param listener      the reconcile listener to be removed
-        * @since 3.0
-        */
-       final void removeReconcileListener(IJavaReconcilingListener listener) {
-               synchronized (fReconcilingListeners) {
-                       fReconcilingListeners.remove(listener);
-               }
-       }
-               
-       protected void updateStateDependentActions() {
-               super.updateStateDependentActions();
-               fGenerateActionGroup.editorStateChanged();
-       }
-       
-       /*
-        * @see AbstractTextEditor#rememberSelection()
-        */
-       protected void rememberSelection() {
-               fRememberedSelection.remember();
-       }
-       
-       /*
-        * @see AbstractTextEditor#restoreSelection()
-        */
-       protected void restoreSelection() {
-               fRememberedSelection.restore();
-       }
-       
-       /*
-        * @see AbstractTextEditor#canHandleMove(IEditorInput, IEditorInput)
-        */
-       protected boolean canHandleMove(IEditorInput originalElement, IEditorInput movedElement) {
-               
-               String oldExtension= ""; //$NON-NLS-1$
-               if (originalElement instanceof IFileEditorInput) {
-                       IFile file= ((IFileEditorInput) originalElement).getFile();
-                       if (file != null) {
-                               String ext= file.getFileExtension();
-                               if (ext != null)
-                                       oldExtension= ext;
-                       }
-               }
-               
-               String newExtension= ""; //$NON-NLS-1$
-               if (movedElement instanceof IFileEditorInput) {
-                       IFile file= ((IFileEditorInput) movedElement).getFile();
-                       if (file != null)
-                               newExtension= file.getFileExtension();
-               }
-               
-               return oldExtension.equals(newExtension);
-       }
-
-       /*
-        * @see org.eclipse.ui.texteditor.AbstractDecoratedTextEditor#isPrefQuickDiffAlwaysOn()
-        */
-       protected boolean isPrefQuickDiffAlwaysOn() {
-               // reestablishes the behaviour from AbstractDecoratedTextEditor which was hacked by JavaEditor
-               // to disable the change bar for the class file (attached source) java editor.
-               IPreferenceStore store= getPreferenceStore();
-               return store.getBoolean(AbstractDecoratedTextEditorPreferenceConstants.QUICK_DIFF_ALWAYS_ON);
-       }
-       
-       /*
-        * @see org.eclipse.jdt.internal.ui.javaeditor.JavaEditor#getAdapter(java.lang.Class)
-        */
-       public Object getAdapter(Class required) {
-               if (SmartBackspaceManager.class.equals(required)) {
-                       if (getSourceViewer() instanceof JavaSourceViewer) {
-                               return ((JavaSourceViewer) getSourceViewer()).getBackspaceManager();
-                       }
-               }
-
-               return super.getAdapter(required);
-       }
+   * Adds the given listener. Has no effect if an identical listener was not already registered.
+   *
+   * @param listener
+   *          The reconcile listener to be added
+   * @since 3.0
+   */
+  final void addReconcileListener(IJavaReconcilingListener listener) {
+    synchronized (fReconcilingListeners) {
+      fReconcilingListeners.add(listener);
+    }
+  }
+
+  /**
+   * Removes the given listener. Has no effect if an identical listener was not already registered.
+   *
+   * @param listener
+   *          the reconcile listener to be removed
+   * @since 3.0
+   */
+  final void removeReconcileListener(IJavaReconcilingListener listener) {
+    synchronized (fReconcilingListeners) {
+      fReconcilingListeners.remove(listener);
+    }
+  }
+
+  protected void updateStateDependentActions() {
+    super.updateStateDependentActions();
+    fGenerateActionGroup.editorStateChanged();
+  }
+
+  /*
+   * @see AbstractTextEditor#rememberSelection()
+   */
+  protected void rememberSelection() {
+    fRememberedSelection.remember();
+  }
+
+  /*
+   * @see AbstractTextEditor#restoreSelection()
+   */
+  protected void restoreSelection() {
+    fRememberedSelection.restore();
+  }
+
+  /*
+   * @see AbstractTextEditor#canHandleMove(IEditorInput, IEditorInput)
+   */
+  protected boolean canHandleMove(IEditorInput originalElement, IEditorInput movedElement) {
+
+    String oldExtension = ""; //$NON-NLS-1$
+    if (originalElement instanceof IFileEditorInput) {
+      IFile file = ((IFileEditorInput) originalElement).getFile();
+      if (file != null) {
+        String ext = file.getFileExtension();
+        if (ext != null)
+          oldExtension = ext;
+      }
+    }
+
+    String newExtension = ""; //$NON-NLS-1$
+    if (movedElement instanceof IFileEditorInput) {
+      IFile file = ((IFileEditorInput) movedElement).getFile();
+      if (file != null)
+        newExtension = file.getFileExtension();
+    }
+
+    return oldExtension.equals(newExtension);
+  }
+
+  /*
+   * @see org.eclipse.ui.texteditor.AbstractDecoratedTextEditor#isPrefQuickDiffAlwaysOn()
+   */
+  protected boolean isPrefQuickDiffAlwaysOn() {
+    // reestablishes the behaviour from AbstractDecoratedTextEditor which was hacked by JavaEditor
+    // to disable the change bar for the class file (attached source) java editor.
+    IPreferenceStore store = getPreferenceStore();
+    return store.getBoolean(AbstractDecoratedTextEditorPreferenceConstants.QUICK_DIFF_ALWAYS_ON);
+  }
+
+  /*
+   * @see net.sourceforge.phpdt.internal.ui.javaeditor.JavaEditor#getAdapter(java.lang.Class)
+   */
+  public Object getAdapter(Class required) {
+    if (SmartBackspaceManager.class.equals(required)) {
+      if (getSourceViewer() instanceof JavaSourceViewer) {
+        return ((JavaSourceViewer) getSourceViewer()).getBackspaceManager();
+      }
+    }
+
+    return super.getAdapter(required);
+  }
+
   /**
-        * Returns the mutex for the reconciler. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=63898
-        * for a description of the problem.
-        * <p>
-        * TODO remove once the underlying problem is solved.
-        * </p>
-        * @return the lock reconcilers may use to synchronize on
-        */
-       public Object getReconcilerLock() {
-               return fReconcilerLock;
-       }
-       
+   * Returns the mutex for the reconciler. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=63898 for a description of the problem.
+   * <p>
+   * TODO remove once the underlying problem is solved.
+   * </p>
+   *
+   * @return the lock reconcilers may use to synchronize on
+   */
+  public Object getReconcilerLock() {
+    return fReconcilerLock;
+  }
+
+  /*
+   * (non-Javadoc)
+   *
+   * @see org.eclipse.ui.texteditor.AbstractTextEditor#editorSaved()
+   */
+  protected void editorSaved() {
+    super.editorSaved();
+    ShowExternalPreviewAction a = ShowExternalPreviewAction.getInstance();
+    if (a != null) {
+      a.refresh(ShowExternalPreviewAction.PHP_TYPE);
+    }
+  }
 }
\ No newline at end of file