removed ColorManager, not used anywhere
authorbananeweizen <bananeweizen>
Sat, 4 Feb 2006 15:47:51 +0000 (15:47 +0000)
committerbananeweizen <bananeweizen>
Sat, 4 Feb 2006 15:47:51 +0000 (15:47 +0000)
this reduces dependencies on .externaltools

net.sourceforge.phpeclipse.externaltools/src/net/sourceforge/phpdt/externaltools/internal/model/ColorManager.java [deleted file]
net.sourceforge.phpeclipse.externaltools/src/net/sourceforge/phpeclipse/externaltools/ExternalToolsPlugin.java
net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/PHPeclipsePlugin.java

diff --git a/net.sourceforge.phpeclipse.externaltools/src/net/sourceforge/phpdt/externaltools/internal/model/ColorManager.java b/net.sourceforge.phpeclipse.externaltools/src/net/sourceforge/phpdt/externaltools/internal/model/ColorManager.java
deleted file mode 100644 (file)
index c82cdb4..0000000
+++ /dev/null
@@ -1,53 +0,0 @@
-package net.sourceforge.phpdt.externaltools.internal.model;
-
-/**********************************************************************
-Copyright (c) 2000, 2002 IBM Corp.  All rights reserved.
-This file is 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
-**********************************************************************/
-
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.Map;
-
-import org.eclipse.swt.graphics.Color;
-import org.eclipse.swt.graphics.RGB;
-import org.eclipse.swt.widgets.Display;
-
-/**
- * Generic color manager.
- */
-public class ColorManager {    
-       
-       private static ColorManager fgColorManager;
-       
-       private ColorManager() {
-       }
-       
-       public static ColorManager getDefault() {
-               if (fgColorManager == null) {
-                       fgColorManager= new ColorManager();
-               }
-               return fgColorManager;
-       }
-       
-       protected Map fColorTable= new HashMap(10);
-       
-       public Color getColor(RGB rgb) {
-               Color color= (Color) fColorTable.get(rgb);
-               if (color == null) {
-                       color= new Color(Display.getCurrent(), rgb);
-                       fColorTable.put(rgb, color);
-               }
-               return color;
-       }
-       
-       public void dispose() {
-               Iterator e= fColorTable.values().iterator();
-               while (e.hasNext())
-                       ((Color) e.next()).dispose();
-       }
-}
-
-
index b805a73..df3f577 100644 (file)
@@ -15,7 +15,6 @@ import java.net.URL;
 import java.util.Enumeration;
 import java.util.PropertyResourceBundle;
 
-import net.sourceforge.phpdt.externaltools.internal.model.ColorManager;
 import net.sourceforge.phpdt.externaltools.internal.model.ExternalToolsImages;
 import net.sourceforge.phpdt.externaltools.internal.model.IPreferenceConstants;
 import net.sourceforge.phpdt.externaltools.internal.model.VariableContextManager;
@@ -33,7 +32,6 @@ import org.eclipse.jface.preference.IPreferenceStore;
 import org.eclipse.jface.preference.PreferenceConverter;
 import org.eclipse.jface.resource.ImageDescriptor;
 import org.eclipse.jface.resource.ImageRegistry;
-import org.eclipse.swt.graphics.Color;
 import org.eclipse.swt.graphics.RGB;
 import org.eclipse.swt.widgets.Display;
 import org.eclipse.ui.IWorkbenchWindow;
@@ -98,65 +96,65 @@ public final class ExternalToolsPlugin extends AbstractUIPlugin {
    * This version is recommended for eclipse3.0 and above
    */
   public ExternalToolsPlugin() {
-    super();
-    plugin = this;
+       super();
+       plugin = this;
   }
 
   /**
    * Returns the default instance of the receiver. This represents the runtime plugin.
    */
   public static ExternalToolsPlugin getDefault() {
-    return plugin;
+       return plugin;
   }
 
   /**
    * Returns a new <code>IStatus</code> for this plug-in
    */
   public static IStatus newErrorStatus(String message, Throwable exception) {
-    return new Status(Status.ERROR, IExternalToolConstants.PLUGIN_ID, 0, message, exception);
+       return new Status(Status.ERROR, IExternalToolConstants.PLUGIN_ID, 0, message, exception);
   }
 
   /**
    * Returns a new <code>CoreException</code> for this plug-in
    */
   public static CoreException newError(String message, Throwable exception) {
-    return new CoreException(new Status(Status.ERROR, IExternalToolConstants.PLUGIN_ID, 0, message, exception));
+       return new CoreException(new Status(Status.ERROR, IExternalToolConstants.PLUGIN_ID, 0, message, exception));
   }
 
   /**
    * Returns the registry of refresh scope variables.
    */
   public ArgumentVariableRegistry getArgumentVariableRegistry() {
-    if (argumentVarRegistry == null)
-      argumentVarRegistry = new ArgumentVariableRegistry();
-    return argumentVarRegistry;
+       if (argumentVarRegistry == null)
+         argumentVarRegistry = new ArgumentVariableRegistry();
+       return argumentVarRegistry;
   }
 
   /**
    * Returns the registry of directory location variables.
    */
   public PathLocationVariableRegistry getDirectoryLocationVariableRegistry() {
-    if (dirLocVarRegistry == null)
-      dirLocVarRegistry = new PathLocationVariableRegistry(IExternalToolConstants.EXTENSION_POINT_DIRECTORY_VARIABLES);
-    return dirLocVarRegistry;
+       if (dirLocVarRegistry == null)
+         dirLocVarRegistry = new PathLocationVariableRegistry(IExternalToolConstants.EXTENSION_POINT_DIRECTORY_VARIABLES);
+       return dirLocVarRegistry;
   }
 
   /**
    * Returns the registry of file location variables.
    */
   public PathLocationVariableRegistry getFileLocationVariableRegistry() {
-    if (fileLocVarRegistry == null)
-      fileLocVarRegistry = new PathLocationVariableRegistry(IExternalToolConstants.EXTENSION_POINT_FILE_VARIABLES);
-    return fileLocVarRegistry;
+       if (fileLocVarRegistry == null)
+         fileLocVarRegistry = new PathLocationVariableRegistry(IExternalToolConstants.EXTENSION_POINT_FILE_VARIABLES);
+       return fileLocVarRegistry;
   }
 
   /**
    * Returns the registry of refresh scope variables.
    */
   public RefreshScopeVariableRegistry getRefreshVariableRegistry() {
-    if (refreshVarRegistry == null)
-      refreshVarRegistry = new RefreshScopeVariableRegistry();
-    return refreshVarRegistry;
+       if (refreshVarRegistry == null)
+         refreshVarRegistry = new RefreshScopeVariableRegistry();
+       return refreshVarRegistry;
   }
 
   /**
@@ -166,13 +164,13 @@ public final class ExternalToolsPlugin extends AbstractUIPlugin {
    *          the text to write to the log
    */
   public void log(String message, Throwable exception) {
-    IStatus status = newErrorStatus(message, exception);
-    // getLog().log(status);
-    ExternalToolsPlugin.log(status);
+       IStatus status = newErrorStatus(message, exception);
+       //      getLog().log(status);
+       ExternalToolsPlugin.log(status);
   }
 
   public static void log(IStatus status) {
-    getDefault().getLog().log(status);
+       getDefault().getLog().log(status);
   }
 
   /**
@@ -181,14 +179,14 @@ public final class ExternalToolsPlugin extends AbstractUIPlugin {
    * @return the ImageDescriptor object
    */
   public ImageDescriptor getImageDescriptor(String path) {
-    try {
-      Bundle bundle = ExternalToolsPlugin.getDefault().getBundle();
-      URL installURL = bundle.getEntry("/"); //$NON-NLS-1$
-      URL url = new URL(installURL, path);
-      return ImageDescriptor.createFromURL(url);
-    } catch (MalformedURLException e) {
-      return null;
-    }
+       try {
+         Bundle bundle = ExternalToolsPlugin.getDefault().getBundle();
+         URL installURL = bundle.getEntry("/"); //$NON-NLS-1$
+         URL url = new URL(installURL, path);
+         return ImageDescriptor.createFromURL(url);
+       } catch (MalformedURLException e) {
+         return null;
+       }
   }
 
   /*
@@ -196,59 +194,59 @@ public final class ExternalToolsPlugin extends AbstractUIPlugin {
    */
 
   protected void initializeDefaultPreferences(IPreferenceStore store) {
-    String operatingSystem = Platform.getOS();
-    // maxosx, linux, solaris, win32,...
-    try {
-      InputStream is = getDefault().openStream(new Path("prefs/default_" + operatingSystem + ".properties"));
-      PropertyResourceBundle resourceBundle = new PropertyResourceBundle(is);
-      Enumeration e = resourceBundle.getKeys();
-      String key;
-      while (e.hasMoreElements()) {
-        key = (String) e.nextElement();
-        store.setDefault(key, resourceBundle.getString(key));
-      }
-    } catch (Exception e) {
-      // no default properties found
-      if (operatingSystem.equals(Platform.OS_WIN32)) {
-        store.setDefault(PHP_RUN_PREF, "c:\\apache\\php\\php.exe");
-        store.setDefault(EXTERNAL_PARSER_PREF, "c:\\apache\\php\\php -l -f {0}");
-        store.setDefault(MYSQL_RUN_PREF, "c:\\apache\\mysql\\bin\\mysqld-nt.exe");
-        store.setDefault(APACHE_RUN_PREF, "c:\\apache\\apache.exe");
-        store.setDefault(XAMPP_START_PREF, "c:\\xampp\\xampp_start.exe");
-        store.setDefault(XAMPP_STOP_PREF, "c:\\xampp\\xampp_stop.exe");
-        store.setDefault(ETC_HOSTS_PATH_PREF, "c:\\windows\\system32\\drivers\\etc\\hosts");
-      } else {
-        store.setDefault(PHP_RUN_PREF, "/apache/php/php");
-        store.setDefault(EXTERNAL_PARSER_PREF, "/apache/php/php -l -f {0}");
-        store.setDefault(MYSQL_RUN_PREF, "/apache/mysql/bin/mysqld");
-        store.setDefault(APACHE_RUN_PREF, "/apache/apache");
-        store.setDefault(XAMPP_START_PREF, "xamp/xampp_start");
-        store.setDefault(XAMPP_STOP_PREF, "xampp/xampp_stop");
-      }
-      store.setDefault(MYSQL_PREF, "--standalone");
-      store.setDefault(APACHE_START_PREF, "-c \"DocumentRoot \"{0}\"\"");
-      store.setDefault(APACHE_STOP_PREF, "-k shutdown");
-      store.setDefault(APACHE_RESTART_PREF, "-k restart");
-      store.setDefault(MYSQL_START_BACKGROUND, "true");
-      store.setDefault(APACHE_START_BACKGROUND, "true");
-      store.setDefault(APACHE_STOP_BACKGROUND, "true");
-      store.setDefault(APACHE_RESTART_BACKGROUND, "true");
-    }
-
-    //    store.setDefault(SHOW_OUTPUT_IN_CONSOLE, "true");
-
-    store.setDefault(IPreferenceConstants.PROMPT_FOR_MIGRATION, true);
-
-    PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_ERROR_RGB, new RGB(255, 0, 0)); // red - exactly the same as
-    // debug Console
-    PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_WARNING_RGB, new RGB(255, 100, 0)); // orange
-    PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_INFO_RGB, new RGB(0, 0, 255)); // blue
-    PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_VERBOSE_RGB, new RGB(0, 200, 125)); // green
-    PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_DEBUG_RGB, new RGB(0, 0, 0)); // black
+       String operatingSystem = Platform.getOS();
+       // maxosx, linux, solaris, win32,...
+       try {
+         InputStream is = getDefault().openStream(new Path("prefs/default_" + operatingSystem + ".properties"));
+         PropertyResourceBundle resourceBundle = new PropertyResourceBundle(is);
+         Enumeration e = resourceBundle.getKeys();
+         String key;
+         while (e.hasMoreElements()) {
+               key = (String) e.nextElement();
+               store.setDefault(key, resourceBundle.getString(key));
+         }
+       } catch (Exception e) {
+         // no default properties found
+         if (operatingSystem.equals(Platform.OS_WIN32)) {
+               store.setDefault(PHP_RUN_PREF, "c:\\apache\\php\\php.exe");
+               store.setDefault(EXTERNAL_PARSER_PREF, "c:\\apache\\php\\php -l -f {0}");
+               store.setDefault(MYSQL_RUN_PREF, "c:\\apache\\mysql\\bin\\mysqld-nt.exe");
+               store.setDefault(APACHE_RUN_PREF, "c:\\apache\\apache.exe");
+               store.setDefault(XAMPP_START_PREF, "c:\\xampp\\xampp_start.exe");
+               store.setDefault(XAMPP_STOP_PREF, "c:\\xampp\\xampp_stop.exe");
+               store.setDefault(ETC_HOSTS_PATH_PREF, "c:\\windows\\system32\\drivers\\etc\\hosts");
+         } else {
+               store.setDefault(PHP_RUN_PREF, "/apache/php/php");
+               store.setDefault(EXTERNAL_PARSER_PREF, "/apache/php/php -l -f {0}");
+               store.setDefault(MYSQL_RUN_PREF, "/apache/mysql/bin/mysqld");
+               store.setDefault(APACHE_RUN_PREF, "/apache/apache");
+               store.setDefault(XAMPP_START_PREF, "xamp/xampp_start");
+               store.setDefault(XAMPP_STOP_PREF, "xampp/xampp_stop");
+         }
+         store.setDefault(MYSQL_PREF, "--standalone");
+         store.setDefault(APACHE_START_PREF, "-c \"DocumentRoot \"{0}\"\"");
+         store.setDefault(APACHE_STOP_PREF, "-k shutdown");
+         store.setDefault(APACHE_RESTART_PREF, "-k restart");
+         store.setDefault(MYSQL_START_BACKGROUND, "true");
+         store.setDefault(APACHE_START_BACKGROUND, "true");
+         store.setDefault(APACHE_STOP_BACKGROUND, "true");
+         store.setDefault(APACHE_RESTART_BACKGROUND, "true");
+       }
+
+       //    store.setDefault(SHOW_OUTPUT_IN_CONSOLE, "true");
+
+       store.setDefault(IPreferenceConstants.PROMPT_FOR_MIGRATION, true);
+
+       PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_ERROR_RGB, new RGB(255, 0, 0)); // red - exactly the same as
+       // debug Console
+       PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_WARNING_RGB, new RGB(255, 100, 0)); // orange
+       PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_INFO_RGB, new RGB(0, 0, 255)); // blue
+       PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_VERBOSE_RGB, new RGB(0, 200, 125)); // green
+       PreferenceConverter.setDefault(store, IPreferenceConstants.CONSOLE_DEBUG_RGB, new RGB(0, 0, 0)); // black
   }
 
   public static IWorkbenchWindow getActiveWorkbenchWindow() {
-    return ExternalToolsPlugin.getDefault().getWorkbench().getActiveWorkbenchWindow();
+       return ExternalToolsPlugin.getDefault().getWorkbench().getActiveWorkbenchWindow();
   }
 
   /**
@@ -256,18 +254,18 @@ public final class ExternalToolsPlugin extends AbstractUIPlugin {
    * If so, this display is returned. Otherwise the method returns the default display.
    */
   public static Display getStandardDisplay() {
-    Display display = Display.getCurrent();
-    if (display == null) {
-      display = Display.getDefault();
-    }
-    return display;
+       Display display = Display.getCurrent();
+       if (display == null) {
+         display = Display.getDefault();
+       }
+       return display;
   }
 
   /**
    * @see org.eclipse.ui.plugin.AbstractUIPlugin#createImageRegistry()
    */
   protected ImageRegistry createImageRegistry() {
-    return ExternalToolsImages.initializeImageRegistry();
+       return ExternalToolsImages.initializeImageRegistry();
   }
 
   /**
@@ -275,29 +273,12 @@ public final class ExternalToolsPlugin extends AbstractUIPlugin {
  * @see org.eclipse.core.runtime.Plugin#start(BundleContext context)
    */
   public void start(BundleContext context) throws Exception {
-    super.start(context);
-    getStandardDisplay().asyncExec(new Runnable() {
-      public void run() {
-        //initialize the variable context manager
-        VariableContextManager.getDefault();
-      }
-    });
-  }
-
-  /**
-   * @throws Exception
- * @see org.eclipse.core.runtime.Plugin#stop(BundleContext context)
-   */
-  public void stop(BundleContext context) throws Exception {
-    ColorManager.getDefault().dispose();
-    super.stop(context);
-  }
-
-  /**
-   * Returns the preference color, identified by the given preference.
-   */
-  public static Color getPreferenceColor(String pref) {
-    return ColorManager.getDefault().getColor(
-        PreferenceConverter.getColor(ExternalToolsPlugin.getDefault().getPreferenceStore(), pref));
+       super.start(context);
+       getStandardDisplay().asyncExec(new Runnable() {
+         public void run() {
+               //initialize the variable context manager
+               VariableContextManager.getDefault();
+         }
+       });
   }
 }
\ No newline at end of file
index 6eeecfb..f35f9f1 100644 (file)
@@ -27,7 +27,6 @@ import net.sourceforge.phpdt.core.ICompilationUnit;
 import net.sourceforge.phpdt.core.IJavaElement;
 import net.sourceforge.phpdt.core.JavaCore;
 import net.sourceforge.phpdt.core.WorkingCopyOwner;
-import net.sourceforge.phpdt.externaltools.internal.model.ColorManager;
 import net.sourceforge.phpdt.internal.core.BatchOperation;
 import net.sourceforge.phpdt.internal.core.JavaModelManager;
 import net.sourceforge.phpdt.internal.core.util.Util;
@@ -300,16 +299,16 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * The constructor.
    */
   public PHPeclipsePlugin() {
-    super();
-    plugin = this;
+       super();
+       plugin = this;
 //    externalTools = new ExternalToolsPlugin();
 
-    //    try {
-    //      resourceBundle =
-    // ResourceBundle.getBundle("net.sourceforge.PHPeclipsePluginResources");
-    //    } catch (MissingResourceException x) {
-    //      resourceBundle = null;
-    //    }
+       //    try {
+       //      resourceBundle =
+       // ResourceBundle.getBundle("net.sourceforge.PHPeclipsePluginResources");
+       //    } catch (MissingResourceException x) {
+       //      resourceBundle = null;
+       //    }
   }
 
   //  /**
@@ -332,81 +331,81 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 2.1
    */
   public JavaEditorTextHoverDescriptor[] getJavaEditorTextHoverDescriptors() {
-    if (fJavaEditorTextHoverDescriptors == null) {
-      fJavaEditorTextHoverDescriptors = JavaEditorTextHoverDescriptor.getContributedHovers();
-      ConfigurationElementSorter sorter = new ConfigurationElementSorter() {
-        /*
-         * @see org.eclipse.ui.texteditor.ConfigurationElementSorter#getConfigurationElement(java.lang.Object)
-         */
-        public IConfigurationElement getConfigurationElement(Object object) {
-          return ((JavaEditorTextHoverDescriptor) object).getConfigurationElement();
-        }
-      };
-      sorter.sort(fJavaEditorTextHoverDescriptors);
-
-      // The Problem hover has to be the first and the Annotation hover has to
-      // be the last one in the JDT UI's hover list
-      int length = fJavaEditorTextHoverDescriptors.length;
-      int first = -1;
-      int last = length - 1;
-      int problemHoverIndex = -1;
-      int annotationHoverIndex = -1;
-      for (int i = 0; i < length; i++) {
-        if (!fJavaEditorTextHoverDescriptors[i].getId().startsWith(PLUGIN_ID)) {
-          if (problemHoverIndex == -1 || annotationHoverIndex == -1)
-            continue;
-          else {
-            last = i - 1;
-            break;
-          }
-        }
-        if (first == -1)
-          first = i;
-
-        if (fJavaEditorTextHoverDescriptors[i].getId().equals("net.sourceforge.phpdt.ui.AnnotationHover")) { //$NON-NLS-1$
-          annotationHoverIndex = i;
-          continue;
-        }
-        if (fJavaEditorTextHoverDescriptors[i].getId().equals("net.sourceforge.phpdt.ui.ProblemHover")) { //$NON-NLS-1$
-          problemHoverIndex = i;
-          continue;
-        }
-      }
-
-      JavaEditorTextHoverDescriptor hoverDescriptor = null;
-
-      if (first > -1 && problemHoverIndex > -1 && problemHoverIndex != first) {
-        // move problem hover to beginning
-        hoverDescriptor = fJavaEditorTextHoverDescriptors[first];
-        fJavaEditorTextHoverDescriptors[first] = fJavaEditorTextHoverDescriptors[problemHoverIndex];
-        fJavaEditorTextHoverDescriptors[problemHoverIndex] = hoverDescriptor;
-
-        // update annotation hover index if needed
-        if (annotationHoverIndex == first)
-          annotationHoverIndex = problemHoverIndex;
-      }
-
-      if (annotationHoverIndex > -1 && annotationHoverIndex != last) {
-        // move annotation hover to end
-        hoverDescriptor = fJavaEditorTextHoverDescriptors[last];
-        fJavaEditorTextHoverDescriptors[last] = fJavaEditorTextHoverDescriptors[annotationHoverIndex];
-        fJavaEditorTextHoverDescriptors[annotationHoverIndex] = hoverDescriptor;
-      }
-
-      // Move Best Match hover to front
-      for (int i = 0; i < fJavaEditorTextHoverDescriptors.length - 1; i++) {
-        if (PreferenceConstants.ID_BESTMATCH_HOVER.equals(fJavaEditorTextHoverDescriptors[i].getId())) {
-          hoverDescriptor = fJavaEditorTextHoverDescriptors[i];
-          for (int j = i; j > 0; j--)
-            fJavaEditorTextHoverDescriptors[j] = fJavaEditorTextHoverDescriptors[j - 1];
-          fJavaEditorTextHoverDescriptors[0] = hoverDescriptor;
-          break;
-        }
-
-      }
-    }
-
-    return fJavaEditorTextHoverDescriptors;
+       if (fJavaEditorTextHoverDescriptors == null) {
+         fJavaEditorTextHoverDescriptors = JavaEditorTextHoverDescriptor.getContributedHovers();
+         ConfigurationElementSorter sorter = new ConfigurationElementSorter() {
+               /*
+                * @see org.eclipse.ui.texteditor.ConfigurationElementSorter#getConfigurationElement(java.lang.Object)
+                */
+               public IConfigurationElement getConfigurationElement(Object object) {
+                 return ((JavaEditorTextHoverDescriptor) object).getConfigurationElement();
+               }
+         };
+         sorter.sort(fJavaEditorTextHoverDescriptors);
+
+         // The Problem hover has to be the first and the Annotation hover has to
+         // be the last one in the JDT UI's hover list
+         int length = fJavaEditorTextHoverDescriptors.length;
+         int first = -1;
+         int last = length - 1;
+         int problemHoverIndex = -1;
+         int annotationHoverIndex = -1;
+         for (int i = 0; i < length; i++) {
+               if (!fJavaEditorTextHoverDescriptors[i].getId().startsWith(PLUGIN_ID)) {
+                 if (problemHoverIndex == -1 || annotationHoverIndex == -1)
+                       continue;
+                 else {
+                       last = i - 1;
+                       break;
+                 }
+               }
+               if (first == -1)
+                 first = i;
+
+               if (fJavaEditorTextHoverDescriptors[i].getId().equals("net.sourceforge.phpdt.ui.AnnotationHover")) { //$NON-NLS-1$
+                 annotationHoverIndex = i;
+                 continue;
+               }
+               if (fJavaEditorTextHoverDescriptors[i].getId().equals("net.sourceforge.phpdt.ui.ProblemHover")) { //$NON-NLS-1$
+                 problemHoverIndex = i;
+                 continue;
+               }
+         }
+
+         JavaEditorTextHoverDescriptor hoverDescriptor = null;
+
+         if (first > -1 && problemHoverIndex > -1 && problemHoverIndex != first) {
+               // move problem hover to beginning
+               hoverDescriptor = fJavaEditorTextHoverDescriptors[first];
+               fJavaEditorTextHoverDescriptors[first] = fJavaEditorTextHoverDescriptors[problemHoverIndex];
+               fJavaEditorTextHoverDescriptors[problemHoverIndex] = hoverDescriptor;
+
+               // update annotation hover index if needed
+               if (annotationHoverIndex == first)
+                 annotationHoverIndex = problemHoverIndex;
+         }
+
+         if (annotationHoverIndex > -1 && annotationHoverIndex != last) {
+               // move annotation hover to end
+               hoverDescriptor = fJavaEditorTextHoverDescriptors[last];
+               fJavaEditorTextHoverDescriptors[last] = fJavaEditorTextHoverDescriptors[annotationHoverIndex];
+               fJavaEditorTextHoverDescriptors[annotationHoverIndex] = hoverDescriptor;
+         }
+
+         // Move Best Match hover to front
+         for (int i = 0; i < fJavaEditorTextHoverDescriptors.length - 1; i++) {
+               if (PreferenceConstants.ID_BESTMATCH_HOVER.equals(fJavaEditorTextHoverDescriptors[i].getId())) {
+                 hoverDescriptor = fJavaEditorTextHoverDescriptors[i];
+                 for (int j = i; j > 0; j--)
+                       fJavaEditorTextHoverDescriptors[j] = fJavaEditorTextHoverDescriptors[j - 1];
+                 fJavaEditorTextHoverDescriptors[0] = hoverDescriptor;
+                 break;
+               }
+
+         }
+       }
+
+       return fJavaEditorTextHoverDescriptors;
   }
 
   /**
@@ -419,34 +418,34 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 2.1
    */
   public void resetJavaEditorTextHoverDescriptors() {
-    fJavaEditorTextHoverDescriptors = null;
+       fJavaEditorTextHoverDescriptors = null;
   }
 
   /**
    * Creates the PHP plugin standard groups in a context menu.
    */
   public static void createStandardGroups(IMenuManager menu) {
-    if (!menu.isEmpty())
-      return;
-    menu.add(new Separator(IContextMenuConstants.GROUP_NEW));
-    menu.add(new GroupMarker(IContextMenuConstants.GROUP_GOTO));
-    menu.add(new Separator(IContextMenuConstants.GROUP_OPEN));
-    menu.add(new GroupMarker(IContextMenuConstants.GROUP_SHOW));
-    menu.add(new Separator(IContextMenuConstants.GROUP_REORGANIZE));
-    menu.add(new Separator(IContextMenuConstants.GROUP_GENERATE));
-    menu.add(new Separator(IContextMenuConstants.GROUP_SEARCH));
-    menu.add(new Separator(IContextMenuConstants.GROUP_BUILD));
-    menu.add(new Separator(IContextMenuConstants.GROUP_ADDITIONS));
-    menu.add(new Separator(IContextMenuConstants.GROUP_VIEWER_SETUP));
-    menu.add(new Separator(IContextMenuConstants.GROUP_PROPERTIES));
+       if (!menu.isEmpty())
+         return;
+       menu.add(new Separator(IContextMenuConstants.GROUP_NEW));
+       menu.add(new GroupMarker(IContextMenuConstants.GROUP_GOTO));
+       menu.add(new Separator(IContextMenuConstants.GROUP_OPEN));
+       menu.add(new GroupMarker(IContextMenuConstants.GROUP_SHOW));
+       menu.add(new Separator(IContextMenuConstants.GROUP_REORGANIZE));
+       menu.add(new Separator(IContextMenuConstants.GROUP_GENERATE));
+       menu.add(new Separator(IContextMenuConstants.GROUP_SEARCH));
+       menu.add(new Separator(IContextMenuConstants.GROUP_BUILD));
+       menu.add(new Separator(IContextMenuConstants.GROUP_ADDITIONS));
+       menu.add(new Separator(IContextMenuConstants.GROUP_VIEWER_SETUP));
+       menu.add(new Separator(IContextMenuConstants.GROUP_PROPERTIES));
   }
 
   public static IWorkbenchPage getActivePage() {
-    return getDefault().internalGetActivePage();
+       return getDefault().internalGetActivePage();
   }
 
   public static Shell getActiveWorkbenchShell() {
-    return getActiveWorkbenchWindow().getShell();
+       return getActiveWorkbenchWindow().getShell();
   }
 
   /**
@@ -456,44 +455,44 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @return an array of all dirty editor parts.
    */
   public static IEditorPart[] getDirtyEditors() {
-    Set inputs = new HashSet();
-    List result = new ArrayList(0);
-    IWorkbench workbench = getDefault().getWorkbench();
-    IWorkbenchWindow[] windows = workbench.getWorkbenchWindows();
-    for (int i = 0; i < windows.length; i++) {
-      IWorkbenchPage[] pages = windows[i].getPages();
-      for (int x = 0; x < pages.length; x++) {
-        IEditorPart[] editors = pages[x].getDirtyEditors();
-        for (int z = 0; z < editors.length; z++) {
-          IEditorPart ep = editors[z];
-          IEditorInput input = ep.getEditorInput();
-          if (!inputs.contains(input)) {
-            inputs.add(input);
-            result.add(ep);
-          }
-        }
-      }
-    }
-    return (IEditorPart[]) result.toArray(new IEditorPart[result.size()]);
+       Set inputs = new HashSet();
+       List result = new ArrayList(0);
+       IWorkbench workbench = getDefault().getWorkbench();
+       IWorkbenchWindow[] windows = workbench.getWorkbenchWindows();
+       for (int i = 0; i < windows.length; i++) {
+         IWorkbenchPage[] pages = windows[i].getPages();
+         for (int x = 0; x < pages.length; x++) {
+               IEditorPart[] editors = pages[x].getDirtyEditors();
+               for (int z = 0; z < editors.length; z++) {
+                 IEditorPart ep = editors[z];
+                 IEditorInput input = ep.getEditorInput();
+                 if (!inputs.contains(input)) {
+                       inputs.add(input);
+                       result.add(ep);
+                 }
+               }
+         }
+       }
+       return (IEditorPart[]) result.toArray(new IEditorPart[result.size()]);
   }
 
   public static IWorkbenchWindow getActiveWorkbenchWindow() {
-    return getDefault().getWorkbench().getActiveWorkbenchWindow();
+       return getDefault().getWorkbench().getActiveWorkbenchWindow();
   }
 
   /**
    * Returns the shared instance.
    */
   public static PHPeclipsePlugin getDefault() {
-    return plugin;
+       return plugin;
   }
 
   public static ImageDescriptorRegistry getImageDescriptorRegistry() {
-    return getDefault().internalGetImageDescriptorRegistry();
+       return getDefault().internalGetImageDescriptorRegistry();
   }
 
   static IPath getInstallLocation() {
-    return new Path(getDefault().getBundle().getEntry("/").getFile());
+       return new Path(getDefault().getBundle().getEntry("/").getFile());
   }
 
   //  public static int getJVM() {
@@ -501,7 +500,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
   //  }
 
   public static String getPluginId() {
-    return getDefault().getBundle().getSymbolicName() ;
+       return getDefault().getBundle().getSymbolicName() ;
   }
 
   /**
@@ -509,11 +508,11 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * If so, this display is returned. Otherwise the method returns the default display.
    */
   public static Display getStandardDisplay() {
-    Display display = Display.getCurrent();
-    if (display == null) {
-      display = Display.getDefault();
-    }
-    return display;
+       Display display = Display.getCurrent();
+       if (display == null) {
+         display = Display.getDefault();
+       }
+       return display;
   }
 
   //  public static ExternalToolsPlugin getExternalTools() {
@@ -523,11 +522,11 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * Returns the workspace instance.
    */
   public static IWorkspace getWorkspace() {
-    return ResourcesPlugin.getWorkspace();
+       return ResourcesPlugin.getWorkspace();
   }
 
   public static boolean isDebug() {
-    return getDefault().isDebugging();
+       return getDefault().isDebugging();
   }
 
   //  public static void logErrorMessage(String message) {
@@ -552,12 +551,12 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
   // JavaUIMessages.getString("JavaPlugin.internal_error"), e)); //$NON-NLS-1$
   //  }
   public static void log(int severity, String message) {
-    Status status = new Status(severity, PLUGIN_ID, IStatus.OK, message, null);
-    log(status);
+       Status status = new Status(severity, PLUGIN_ID, IStatus.OK, message, null);
+       log(status);
   }
 
   public static void log(IStatus status) {
-    getDefault().getLog().log(status);
+       getDefault().getLog().log(status);
   }
 
   public static void log(Throwable t) {
@@ -565,11 +564,11 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
   }
 
   public static void log(String message, Throwable t) {
-    log(error(message, t));
+       log(error(message, t));
   }
 
   public static void logErrorMessage(String message) {
-    log(new Status(IStatus.ERROR, getPluginId(), IJavaStatusConstants.INTERNAL_ERROR, message, null));
+       log(new Status(IStatus.ERROR, getPluginId(), IJavaStatusConstants.INTERNAL_ERROR, message, null));
   }
 
   public static IStatus error(Throwable t) {
@@ -610,9 +609,9 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
 
   // TODO: refactor this into a better method name !
   public synchronized ICompilationUnitDocumentProvider getCompilationUnitDocumentProvider() {
-    if (fCompilationUnitDocumentProvider == null)
-      fCompilationUnitDocumentProvider = new PHPDocumentProvider();
-    return fCompilationUnitDocumentProvider;
+       if (fCompilationUnitDocumentProvider == null)
+         fCompilationUnitDocumentProvider = new PHPDocumentProvider();
+       return fCompilationUnitDocumentProvider;
   }
 
   /**
@@ -623,9 +622,9 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @return
    */
   public IdentifierIndexManager getIndexManager(IProject iProject) {
-    IPath path = iProject.getWorkingLocation(PHPeclipsePlugin.PLUGIN_ID);
-    path = path.append("project.index");
-    String indexFilename = path.toString();
+       IPath path = iProject.getWorkingLocation(PHPeclipsePlugin.PLUGIN_ID);
+       path = path.append("project.index");
+       String indexFilename = path.toString();
 //    try {
 //      IdentDB db = IdentDB.getInstance();
 //    } catch (ClassNotFoundException e) {
@@ -633,42 +632,42 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
 //    } catch (SQLException e) {
 //      e.printStackTrace();
 //    }
-    IdentifierIndexManager indexManager = (IdentifierIndexManager) fIndexManagerMap.get(indexFilename);
-    if (indexManager == null) {
-      indexManager = new IdentifierIndexManager(indexFilename);
-      fIndexManagerMap.put(indexFilename, indexManager);
-    }
-    return indexManager;
+       IdentifierIndexManager indexManager = (IdentifierIndexManager) fIndexManagerMap.get(indexFilename);
+       if (indexManager == null) {
+         indexManager = new IdentifierIndexManager(indexFilename);
+         fIndexManagerMap.put(indexFilename, indexManager);
+       }
+       return indexManager;
   }
 
   public synchronized IWorkingCopyManager getWorkingCopyManager() {
-    if (fWorkingCopyManager == null) {
-      ICompilationUnitDocumentProvider provider = getCompilationUnitDocumentProvider();
-      fWorkingCopyManager = new WorkingCopyManager(provider);
-    }
-    return fWorkingCopyManager;
+       if (fWorkingCopyManager == null) {
+         ICompilationUnitDocumentProvider provider = getCompilationUnitDocumentProvider();
+         fWorkingCopyManager = new WorkingCopyManager(provider);
+       }
+       return fWorkingCopyManager;
   }
 
   public synchronized MembersOrderPreferenceCache getMemberOrderPreferenceCache() {
-    if (fMembersOrderPreferenceCache == null)
-      fMembersOrderPreferenceCache = new MembersOrderPreferenceCache();
-    return fMembersOrderPreferenceCache;
+       if (fMembersOrderPreferenceCache == null)
+         fMembersOrderPreferenceCache = new MembersOrderPreferenceCache();
+       return fMembersOrderPreferenceCache;
   }
 
   /**
    * Returns the mockup preference store for firing events and registering listeners on project setting changes. Temporary solution.
    */
   public MockupPreferenceStore getMockupPreferenceStore() {
-    if (fMockupPreferenceStore == null)
-      fMockupPreferenceStore = new MockupPreferenceStore();
+       if (fMockupPreferenceStore == null)
+         fMockupPreferenceStore = new MockupPreferenceStore();
 
-    return fMockupPreferenceStore;
+       return fMockupPreferenceStore;
   }
 
   public synchronized ProblemMarkerManager getProblemMarkerManager() {
-    if (fProblemMarkerManager == null)
-      fProblemMarkerManager = new ProblemMarkerManager();
-    return fProblemMarkerManager;
+       if (fProblemMarkerManager == null)
+         fProblemMarkerManager = new ProblemMarkerManager();
+       return fProblemMarkerManager;
   }
 
   //  public synchronized JavaTextTools getJavaTextTools() {
@@ -677,13 +676,13 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
   //    return fJavaTextTools;
   //  }
   public synchronized JavaTextTools getJavaTextTools() {
-    if (fJavaTextTools == null)
-      fJavaTextTools = new JavaTextTools(getPreferenceStore(), JavaCore.getPlugin().getPluginPreferences());
-    return fJavaTextTools;
+       if (fJavaTextTools == null)
+         fJavaTextTools = new JavaTextTools(getPreferenceStore(), JavaCore.getPlugin().getPluginPreferences());
+       return fJavaTextTools;
   }
 
   public IFile getLastEditorFile() {
-    return fLastEditorFile;
+       return fLastEditorFile;
   }
 
   /**
@@ -704,21 +703,21 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
   //    return resourceBundle;
   //  }
   protected void initializeDefaultPreferences(IPreferenceStore store) {
-    String operatingSystem = Platform.getOS();
-    // maxosx, linux, solaris, win32,...
-    try {
-      InputStream is = getDefault().openStream(
-               new Path("prefs/default_" + operatingSystem + ".properties"));
-      PropertyResourceBundle resourceBundle = new PropertyResourceBundle(is);
-      Enumeration e = resourceBundle.getKeys();
-      String key;
-      while (e.hasMoreElements()) {
-        key = (String)e.nextElement();
-        store.setDefault(key, resourceBundle.getString( key ));
-      }
-    } catch (Exception e) {
-      // no default properties found
-      if (operatingSystem.equals(Platform.OS_WIN32)) {
+       String operatingSystem = Platform.getOS();
+       // maxosx, linux, solaris, win32,...
+       try {
+         InputStream is = getDefault().openStream(
+                         new Path("prefs/default_" + operatingSystem + ".properties"));
+         PropertyResourceBundle resourceBundle = new PropertyResourceBundle(is);
+         Enumeration e = resourceBundle.getKeys();
+         String key;
+         while (e.hasMoreElements()) {
+               key = (String)e.nextElement();
+               store.setDefault(key, resourceBundle.getString( key ));
+         }
+       } catch (Exception e) {
+         // no default properties found
+         if (operatingSystem.equals(Platform.OS_WIN32)) {
 //        store.setDefault(PHP_RUN_PREF, "c:\\apache\\php\\php.exe");
 //        store.setDefault(EXTERNAL_PARSER_PREF, "c:\\apache\\php\\php -l -f {0}");
 //        store.setDefault(MYSQL_RUN_PREF, "c:\\apache\\mysql\\bin\\mysqld-nt.exe");
@@ -727,14 +726,14 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
 //        store.setDefault(XAMPP_STOP_PREF, "c:\\xampp\\xampp_stop.exe");
 //        store.setDefault(
 //             ETC_HOSTS_PATH_PREF, "c:\\windows\\system32\\drivers\\etc\\hosts");
-      } else {
+         } else {
 //        store.setDefault(PHP_RUN_PREF, "/apache/php/php");
 //        store.setDefault(EXTERNAL_PARSER_PREF, "/apache/php/php -l -f {0}");
 //        store.setDefault(MYSQL_RUN_PREF, "/apache/mysql/bin/mysqld");
 //        store.setDefault(APACHE_RUN_PREF, "/apache/apache");
 //        store.setDefault(XAMPP_START_PREF, "xamp/xampp_start");
 //        store.setDefault(XAMPP_STOP_PREF, "xampp/xampp_stop");
-      }
+         }
 //      store.setDefault(MYSQL_PREF, "--standalone");
 //      store.setDefault(APACHE_START_PREF, "-c \"DocumentRoot \"{0}\"\"");
 //      store.setDefault(APACHE_STOP_PREF, "-k shutdown");
@@ -743,122 +742,122 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
 //      store.setDefault(APACHE_START_BACKGROUND, "true");
 //      store.setDefault(APACHE_STOP_BACKGROUND, "true");
 //      store.setDefault(APACHE_RESTART_BACKGROUND, "true");
-    }
+       }
 
 //    store.setDefault(SHOW_OUTPUT_IN_CONSOLE, "true");
 //    store.setDefault(PHP_OBFUSCATOR_DEFAULT, "c:\\temp");
 //    if (operatingSystem.equals(Platform.OS_WIN32)) {
-      //    store.setDefault(EXTERNAL_BROWSER_PREF, "rundll32 url.dll,FileProtocolHandler {0}");
+         //    store.setDefault(EXTERNAL_BROWSER_PREF, "rundll32 url.dll,FileProtocolHandler {0}");
 //    } else if (operatingSystem.equals(Platform.OS_MACOSX)) {
 //      store.setDefault(PHP_OBFUSCATOR_DEFAULT, "/tmp");
-      // TODO How do we start Safari on Mac OS X ?
-      //    store.setDefault(EXTERNAL_BROWSER_PREF, "netscape {0}");
+         // TODO How do we start Safari on Mac OS X ?
+         //    store.setDefault(EXTERNAL_BROWSER_PREF, "netscape {0}");
 //    } else {
 //      store.setDefault(PHP_OBFUSCATOR_DEFAULT, "/tmp");
-      //    store.setDefault(EXTERNAL_BROWSER_PREF, "netscape {0}");
+         //    store.setDefault(EXTERNAL_BROWSER_PREF, "netscape {0}");
 //    }
 
 //    store.setDefault(PHP_EXTENSION_PREFS, "php,php3,php4,php5,phtml,inc,module,class");
 
-    //    store.setDefault(PHP_PARSER_DEFAULT, PHP_EXTERNAL_PARSER);
-    //    store.setDefault(PHP_INTERNAL_PARSER, "false");
-    //    store.setDefault(PHP_EXTERNAL_PARSER, "true");
-    //    store.setDefault(PHP_PARSE_ON_SAVE, "true");
-    // show line numbers:
-    //   store.setDefault(LINE_NUMBER_RULER, "false");
-    //    store.setDefault(FORMATTER_TAB_SIZE, "4");
-    // php syntax highlighting
-    store.setDefault(PHP_USERDEF_XMLFILE, "");
-    //assume there is none chooA
+       //    store.setDefault(PHP_PARSER_DEFAULT, PHP_EXTERNAL_PARSER);
+       //    store.setDefault(PHP_INTERNAL_PARSER, "false");
+       //    store.setDefault(PHP_EXTERNAL_PARSER, "true");
+       //    store.setDefault(PHP_PARSE_ON_SAVE, "true");
+       // show line numbers:
+       //   store.setDefault(LINE_NUMBER_RULER, "false");
+       //    store.setDefault(FORMATTER_TAB_SIZE, "4");
+       // php syntax highlighting
+       store.setDefault(PHP_USERDEF_XMLFILE, "");
+       //assume there is none chooA
 //    PreferenceConverter.setDefault(store, PHP_MULTILINE_COMMENT, PHPColorProvider.MULTI_LINE_COMMENT);
 //    PreferenceConverter.setDefault(store, PHP_SINGLELINE_COMMENT, PHPColorProvider.SINGLE_LINE_COMMENT);
-    PreferenceConverter.setDefault(store, PHP_TAG, PHPColorProvider.TAG);
-    PreferenceConverter.setDefault(store, PHP_KEYWORD, PHPColorProvider.KEYWORD);
-    PreferenceConverter.setDefault(store, PHP_VARIABLE, PHPColorProvider.VARIABLE);
-    PreferenceConverter.setDefault(store, PHP_VARIABLE_DOLLAR, PHPColorProvider.VARIABLE);
-    PreferenceConverter.setDefault(store, PHP_FUNCTIONNAME, PHPColorProvider.FUNCTION_NAME);
-    PreferenceConverter.setDefault(store, PHP_CONSTANT, PHPColorProvider.CONSTANT);
-    PreferenceConverter.setDefault(store, PHP_TYPE, PHPColorProvider.TYPE);
+       PreferenceConverter.setDefault(store, PHP_TAG, PHPColorProvider.TAG);
+       PreferenceConverter.setDefault(store, PHP_KEYWORD, PHPColorProvider.KEYWORD);
+       PreferenceConverter.setDefault(store, PHP_VARIABLE, PHPColorProvider.VARIABLE);
+       PreferenceConverter.setDefault(store, PHP_VARIABLE_DOLLAR, PHPColorProvider.VARIABLE);
+       PreferenceConverter.setDefault(store, PHP_FUNCTIONNAME, PHPColorProvider.FUNCTION_NAME);
+       PreferenceConverter.setDefault(store, PHP_CONSTANT, PHPColorProvider.CONSTANT);
+       PreferenceConverter.setDefault(store, PHP_TYPE, PHPColorProvider.TYPE);
 //    PreferenceConverter.setDefault(store, PHP_STRING_DQ, PHPColorProvider.STRING_DQ);
 //    store.setDefault(PHP_STRING_BOLD_DQ, true);
 //    PreferenceConverter.setDefault(store, PHP_STRING_SQ, PHPColorProvider.STRING_SQ);
-    PreferenceConverter.setDefault(store, PHP_DEFAULT, PHPColorProvider.DEFAULT);
-    PreferenceConverter.setDefault(store, PHPDOC_KEYWORD, PHPColorProvider.PHPDOC_KEYWORD);
-    PreferenceConverter.setDefault(store, PHPDOC_TAG, PHPColorProvider.PHPDOC_TAG);
-    PreferenceConverter.setDefault(store, PHPDOC_LINK, PHPColorProvider.PHPDOC_LINK);
-    PreferenceConverter.setDefault(store, PHPDOC_DEFAULT, PHPColorProvider.PHPDOC_DEFAULT);
-
-    PreferenceConverter.setDefault(store, EDITOR_PHP_KEYWORD_RETURN_COLOR, new RGB(127, 0, 85));
-    store.setDefault(EDITOR_PHP_KEYWORD_RETURN_BOLD, true);
-    store.setDefault(EDITOR_PHP_KEYWORD_RETURN_ITALIC, false);
-
-    PreferenceConverter.setDefault(store, EDITOR_PHP_OPERATOR_COLOR, new RGB(0, 0, 0));
-    store.setDefault(EDITOR_PHP_OPERATOR_BOLD, false);
-    store.setDefault(EDITOR_PHP_OPERATOR_ITALIC, false);
-
-    PreferenceConverter.setDefault(store, EDITOR_PHP_BRACE_OPERATOR_COLOR, new RGB(0, 0, 0));
-    store.setDefault(EDITOR_PHP_BRACE_OPERATOR_BOLD, false);
-    store.setDefault(EDITOR_PHP_BRACE_OPERATOR_ITALIC, false);
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      PHP_EDITOR_BACKGROUND,
-    //      PHPColorProvider.BACKGROUND);
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      LINKED_POSITION_COLOR,
-    //      PHPColorProvider.LINKED_POSITION_COLOR);
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      LINE_NUMBER_COLOR,
-    //      PHPColorProvider.LINE_NUMBER_COLOR);
-    //    // set default PHPDoc colors:
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      PHPDOC_KEYWORD,
-    //      PHPColorProvider.PHPDOC_KEYWORD);
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      PHPDOC_LINK,
-    //      PHPColorProvider.PHPDOC_LINK);
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      PHPDOC_DEFAULT,
-    //      PHPColorProvider.PHPDOC_DEFAULT);
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      PHPDOC_TAG,
-    //      PHPColorProvider.PHPDOC_TAG);
-    //    store.setDefault(PREFERENCE_COLOR_BACKGROUND_SYSTEM_DEFAULT, "true");
-    //    PreferenceConverter.setDefault(
-    //      store,
-    //      PREFERENCE_COLOR_BACKGROUND,
-    //      PHPColorProvider.BACKGROUND_COLOR);
-    //language stuff
-    //    store.setDefault(RESOURCE_BUNDLE, LANGUAGE_DEFAULT);
-    //    store.setDefault(RESOURCE_BUNDLE_EN_GB, "true");
-    //    store.setDefault(RESOURCE_BUNDLE_DE, "false");
-    //    store.setDefault(RESOURCE_BUNDLE_FR, "false");
-    //    store.setDefault(RESOURCE_BUNDLE_ES, "false");
-    //    TemplatePreferencePage.initDefaults(store);
-    //this will initialize the static fields in the syntaxrdr class
-    new PHPSyntaxRdr();
-    JavaCore.initializeDefaultPluginPreferences();
-    PreferenceConstants.initializeDefaultValues(store);
+       PreferenceConverter.setDefault(store, PHP_DEFAULT, PHPColorProvider.DEFAULT);
+       PreferenceConverter.setDefault(store, PHPDOC_KEYWORD, PHPColorProvider.PHPDOC_KEYWORD);
+       PreferenceConverter.setDefault(store, PHPDOC_TAG, PHPColorProvider.PHPDOC_TAG);
+       PreferenceConverter.setDefault(store, PHPDOC_LINK, PHPColorProvider.PHPDOC_LINK);
+       PreferenceConverter.setDefault(store, PHPDOC_DEFAULT, PHPColorProvider.PHPDOC_DEFAULT);
+
+       PreferenceConverter.setDefault(store, EDITOR_PHP_KEYWORD_RETURN_COLOR, new RGB(127, 0, 85));
+       store.setDefault(EDITOR_PHP_KEYWORD_RETURN_BOLD, true);
+       store.setDefault(EDITOR_PHP_KEYWORD_RETURN_ITALIC, false);
+
+       PreferenceConverter.setDefault(store, EDITOR_PHP_OPERATOR_COLOR, new RGB(0, 0, 0));
+       store.setDefault(EDITOR_PHP_OPERATOR_BOLD, false);
+       store.setDefault(EDITOR_PHP_OPERATOR_ITALIC, false);
+
+       PreferenceConverter.setDefault(store, EDITOR_PHP_BRACE_OPERATOR_COLOR, new RGB(0, 0, 0));
+       store.setDefault(EDITOR_PHP_BRACE_OPERATOR_BOLD, false);
+       store.setDefault(EDITOR_PHP_BRACE_OPERATOR_ITALIC, false);
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      PHP_EDITOR_BACKGROUND,
+       //      PHPColorProvider.BACKGROUND);
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      LINKED_POSITION_COLOR,
+       //      PHPColorProvider.LINKED_POSITION_COLOR);
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      LINE_NUMBER_COLOR,
+       //      PHPColorProvider.LINE_NUMBER_COLOR);
+       //    // set default PHPDoc colors:
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      PHPDOC_KEYWORD,
+       //      PHPColorProvider.PHPDOC_KEYWORD);
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      PHPDOC_LINK,
+       //      PHPColorProvider.PHPDOC_LINK);
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      PHPDOC_DEFAULT,
+       //      PHPColorProvider.PHPDOC_DEFAULT);
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      PHPDOC_TAG,
+       //      PHPColorProvider.PHPDOC_TAG);
+       //    store.setDefault(PREFERENCE_COLOR_BACKGROUND_SYSTEM_DEFAULT, "true");
+       //    PreferenceConverter.setDefault(
+       //      store,
+       //      PREFERENCE_COLOR_BACKGROUND,
+       //      PHPColorProvider.BACKGROUND_COLOR);
+       //language stuff
+       //    store.setDefault(RESOURCE_BUNDLE, LANGUAGE_DEFAULT);
+       //    store.setDefault(RESOURCE_BUNDLE_EN_GB, "true");
+       //    store.setDefault(RESOURCE_BUNDLE_DE, "false");
+       //    store.setDefault(RESOURCE_BUNDLE_FR, "false");
+       //    store.setDefault(RESOURCE_BUNDLE_ES, "false");
+       //    TemplatePreferencePage.initDefaults(store);
+       //this will initialize the static fields in the syntaxrdr class
+       new PHPSyntaxRdr();
+       JavaCore.initializeDefaultPluginPreferences();
+       PreferenceConstants.initializeDefaultValues(store);
 //    externalTools.initializeDefaultPreferences(store);
-    MarkerAnnotationPreferences.initializeDefaultValues(store);
+       MarkerAnnotationPreferences.initializeDefaultValues(store);
   }
 
   private IWorkbenchPage internalGetActivePage() {
-    IWorkbenchWindow window = getWorkbench().getActiveWorkbenchWindow();
-    if (window != null)
-      return window.getActivePage();
-    return null;
+       IWorkbenchWindow window = getWorkbench().getActiveWorkbenchWindow();
+       if (window != null)
+         return window.getActivePage();
+       return null;
   }
 
   private ImageDescriptorRegistry internalGetImageDescriptorRegistry() {
-    if (fImageDescriptorRegistry == null)
-      fImageDescriptorRegistry = new ImageDescriptorRegistry();
-    return fImageDescriptorRegistry;
+       if (fImageDescriptorRegistry == null)
+         fImageDescriptorRegistry = new ImageDescriptorRegistry();
+       return fImageDescriptorRegistry;
   }
 
   /**
@@ -868,43 +867,43 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @throws CoreException
    */
   public ITextEditor openFileInTextEditor(String filename) throws CoreException {
-    // reject directories
-    if (new File(filename).isDirectory())
-      return null;
-    IWorkbench workbench = PlatformUI.getWorkbench();
-    IWorkbenchWindow window = workbench.getWorkbenchWindows()[0];
-    IWorkbenchPage page = window.getActivePage();
-    IPath path = new Path(filename);
-    // If the file exists in the workspace, open it
-    IFile file = getWorkspace().getRoot().getFileForLocation(path);
-    IEditorPart editor;
-    ITextEditor textEditor;
-    if (file != null && file.exists()) {
-      editor = IDE.openEditor(page, file, true);
-      textEditor = (ITextEditor) editor.getAdapter(ITextEditor.class);
-    } else {
-      // Otherwise open the stream directly
-      if (page == null)
-        return null;
-      FileStorage storage = new FileStorage(path);
-      IEditorRegistry registry = getWorkbench().getEditorRegistry();
-      IEditorDescriptor desc = registry.getDefaultEditor(filename);
-      if (desc == null) {
-        desc = registry.findEditor(IEditorRegistry.SYSTEM_EXTERNAL_EDITOR_ID);
-        //   desc = registry.getDefaultEditor();
-      }
-      IEditorInput input = new ExternalEditorInput(storage);
-      editor = page.openEditor(input, desc.getId());
-      textEditor = (ITextEditor) editor.getAdapter(ITextEditor.class);
-      // If the storage provider is not ours, we can't guarantee read/write.
-      if (textEditor != null) {
-        IDocumentProvider documentProvider = textEditor.getDocumentProvider();
-        if (!(documentProvider instanceof ExternalStorageDocumentProvider)) {
-          storage.setReadOnly();
-        }
-      }
-    }
-    return textEditor;
+       //      reject directories
+       if (new File(filename).isDirectory())
+         return null;
+       IWorkbench workbench = PlatformUI.getWorkbench();
+       IWorkbenchWindow window = workbench.getWorkbenchWindows()[0];
+       IWorkbenchPage page = window.getActivePage();
+       IPath path = new Path(filename);
+       // If the file exists in the workspace, open it
+       IFile file = getWorkspace().getRoot().getFileForLocation(path);
+       IEditorPart editor;
+       ITextEditor textEditor;
+       if (file != null && file.exists()) {
+         editor = IDE.openEditor(page, file, true);
+         textEditor = (ITextEditor) editor.getAdapter(ITextEditor.class);
+       } else {
+         // Otherwise open the stream directly
+         if (page == null)
+               return null;
+         FileStorage storage = new FileStorage(path);
+         IEditorRegistry registry = getWorkbench().getEditorRegistry();
+         IEditorDescriptor desc = registry.getDefaultEditor(filename);
+         if (desc == null) {
+               desc = registry.findEditor(IEditorRegistry.SYSTEM_EXTERNAL_EDITOR_ID);
+               //   desc = registry.getDefaultEditor();
+         }
+         IEditorInput input = new ExternalEditorInput(storage);
+         editor = page.openEditor(input, desc.getId());
+         textEditor = (ITextEditor) editor.getAdapter(ITextEditor.class);
+         // If the storage provider is not ours, we can't guarantee read/write.
+         if (textEditor != null) {
+               IDocumentProvider documentProvider = textEditor.getDocumentProvider();
+               if (!(documentProvider instanceof ExternalStorageDocumentProvider)) {
+                 storage.setReadOnly();
+               }
+         }
+       }
+       return textEditor;
   }
 
   /**
@@ -915,19 +914,19 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @throws CoreException
    */
   public void openFileAndGotoLine(String filename, int line) throws CoreException {
-    ITextEditor textEditor = openFileInTextEditor(filename);
-    if (textEditor != null) {
-      // If a line number was given, go to it
-      if (line > 0) {
-        try {
-          line--; // document is 0 based
-          IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
-          textEditor.selectAndReveal(document.getLineOffset(line), document.getLineLength(line));
-        } catch (BadLocationException e) {
-          // invalid text position -> do nothing
-        }
-      }
-    }
+       ITextEditor textEditor = openFileInTextEditor(filename);
+       if (textEditor != null) {
+         // If a line number was given, go to it
+         if (line > 0) {
+               try {
+                 line--; // document is 0 based
+                 IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
+                 textEditor.selectAndReveal(document.getLineOffset(line), document.getLineLength(line));
+               } catch (BadLocationException e) {
+                 // invalid text position -> do nothing
+               }
+         }
+       }
   }
 
   /**
@@ -938,95 +937,94 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @throws CoreException
    */
   public void openFileAndGotoOffset(String filename, int offset, int length) throws CoreException {
-    ITextEditor textEditor = openFileInTextEditor(filename);
-    if (textEditor != null) {
-      // If a line number was given, go to it
-      if (offset >= 0) {
-        IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
-        textEditor.selectAndReveal(offset, length);
-      }
-    }
+       ITextEditor textEditor = openFileInTextEditor(filename);
+       if (textEditor != null) {
+         // If a line number was given, go to it
+         if (offset >= 0) {
+               IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
+               textEditor.selectAndReveal(offset, length);
+         }
+       }
   }
 
   public void openFileAndFindString(String filename, String findString) throws CoreException {
-    ITextEditor textEditor = openFileInTextEditor(filename);
-    if (textEditor != null) {
-      //               If a string was given, go to it
-      if (findString != null) {
-        try {
-          IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
-          int offset = document.search(0, findString, true, false, true);
-          textEditor.selectAndReveal(offset, findString.length());
-        } catch (BadLocationException e) {
-          // invalid text position -> do nothing
-        }
-      }
-    }
+       ITextEditor textEditor = openFileInTextEditor(filename);
+       if (textEditor != null) {
+         //            If a string was given, go to it
+         if (findString != null) {
+               try {
+                 IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
+                 int offset = document.search(0, findString, true, false, true);
+                 textEditor.selectAndReveal(offset, findString.length());
+               } catch (BadLocationException e) {
+                 // invalid text position -> do nothing
+               }
+         }
+       }
   }
 
   public void setLastEditorFile(IFile textEditor) {
-    this.fLastEditorFile = textEditor;
+       this.fLastEditorFile = textEditor;
   }
 
   /*
    * @see org.eclipse.core.runtime.Plugin#stop
    */
   public void stop(BundleContext context) throws Exception {
-    try {
-      //                       JavaCore.stop(this, context);
-      plugin.savePluginPreferences();
-      IWorkspace workspace = ResourcesPlugin.getWorkspace();
-      workspace.removeResourceChangeListener(JavaModelManager.getJavaModelManager().deltaState);
-      workspace.removeSaveParticipant(plugin);
-
-      JavaModelManager.getJavaModelManager().shutdown();
-
-      ColorManager.getDefault().dispose();
-      // save the information from the php index files if necessary
-      Collection collection = fIndexManagerMap.values();
-      Iterator iterator = collection.iterator();
-      IdentifierIndexManager indexManager = null;
-      while (iterator.hasNext()) {
-        indexManager = (IdentifierIndexManager) iterator.next();
-        indexManager.writeFile();
-      }
-      if (fImageDescriptorRegistry != null)
-        fImageDescriptorRegistry.dispose();
-
-      //                       AllTypesCache.terminate();
-
-      if (fImageDescriptorRegistry != null)
-        fImageDescriptorRegistry.dispose();
-
-      unregisterAdapters();
-
-      //                       if (fASTProvider != null) {
-      //                               fASTProvider.dispose();
-      //                               fASTProvider= null;
-      //                       }
-
-      if (fWorkingCopyManager != null) {
-        fWorkingCopyManager.shutdown();
-        fWorkingCopyManager = null;
-      }
-
-      if (fCompilationUnitDocumentProvider != null) {
-        fCompilationUnitDocumentProvider.shutdown();
-        fCompilationUnitDocumentProvider = null;
-      }
-
-      if (fJavaTextTools != null) {
-        fJavaTextTools.dispose();
-        fJavaTextTools = null;
-      }
-      //                       JavaDocLocations.shutdownJavadocLocations();
-
-      uninstallPreferenceStoreBackwardsCompatibility();
-
-      //                       RefactoringCore.getUndoManager().shutdown();
-    } finally {
-      super.stop(context);
-    }
+       try {
+         //                    JavaCore.stop(this, context);
+         plugin.savePluginPreferences();
+         IWorkspace workspace = ResourcesPlugin.getWorkspace();
+         workspace.removeResourceChangeListener(JavaModelManager.getJavaModelManager().deltaState);
+         workspace.removeSaveParticipant(plugin);
+
+         JavaModelManager.getJavaModelManager().shutdown();
+
+         // save the information from the php index files if necessary
+         Collection collection = fIndexManagerMap.values();
+         Iterator iterator = collection.iterator();
+         IdentifierIndexManager indexManager = null;
+         while (iterator.hasNext()) {
+               indexManager = (IdentifierIndexManager) iterator.next();
+               indexManager.writeFile();
+         }
+         if (fImageDescriptorRegistry != null)
+               fImageDescriptorRegistry.dispose();
+
+         //                    AllTypesCache.terminate();
+
+         if (fImageDescriptorRegistry != null)
+               fImageDescriptorRegistry.dispose();
+
+         unregisterAdapters();
+
+         //                    if (fASTProvider != null) {
+         //                            fASTProvider.dispose();
+         //                            fASTProvider= null;
+         //                    }
+
+         if (fWorkingCopyManager != null) {
+               fWorkingCopyManager.shutdown();
+               fWorkingCopyManager = null;
+         }
+
+         if (fCompilationUnitDocumentProvider != null) {
+               fCompilationUnitDocumentProvider.shutdown();
+               fCompilationUnitDocumentProvider = null;
+         }
+
+         if (fJavaTextTools != null) {
+               fJavaTextTools.dispose();
+               fJavaTextTools = null;
+         }
+         //                    JavaDocLocations.shutdownJavadocLocations();
+
+         uninstallPreferenceStoreBackwardsCompatibility();
+
+         //                    RefactoringCore.getUndoManager().shutdown();
+       } finally {
+         super.stop(context);
+       }
   }
 
   /**
@@ -1079,148 +1077,148 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    */
   private void installPreferenceStoreBackwardsCompatibility() {
 
-    /*
-     * Installs backwards compatibility: propagate the Java editor font from a pre-2.1 plug-in to the Platform UI's preference store
-     * to preserve the Java editor font from a pre-2.1 workspace. This is done only once.
-     */
-    String fontPropagatedKey = "fontPropagated"; //$NON-NLS-1$
-    if (getPreferenceStore().contains(JFaceResources.TEXT_FONT) && !getPreferenceStore().isDefault(JFaceResources.TEXT_FONT)) {
-      if (!getPreferenceStore().getBoolean(fontPropagatedKey))
-        PreferenceConverter.setValue(PlatformUI.getWorkbench().getPreferenceStore(), PreferenceConstants.EDITOR_TEXT_FONT,
-            PreferenceConverter.getFontDataArray(getPreferenceStore(), JFaceResources.TEXT_FONT));
-    }
-    getPreferenceStore().setValue(fontPropagatedKey, true);
-
-    /*
-     * Backwards compatibility: set the Java editor font in this plug-in's preference store to let older versions access it. Since
-     * 2.1 the Java editor font is managed by the workbench font preference page.
-     */
-    PreferenceConverter.putValue(getPreferenceStore(), JFaceResources.TEXT_FONT, JFaceResources.getFontRegistry().getFontData(
-        PreferenceConstants.EDITOR_TEXT_FONT));
-
-    fFontPropertyChangeListener = new IPropertyChangeListener() {
-      public void propertyChange(PropertyChangeEvent event) {
-        if (PreferenceConstants.EDITOR_TEXT_FONT.equals(event.getProperty()))
-          PreferenceConverter.putValue(getPreferenceStore(), JFaceResources.TEXT_FONT, JFaceResources.getFontRegistry()
-              .getFontData(PreferenceConstants.EDITOR_TEXT_FONT));
-      }
-    };
-    JFaceResources.getFontRegistry().addListener(fFontPropertyChangeListener);
-
-    /*
-     * Backwards compatibility: propagate the Java editor tab width from a pre-3.0 plug-in to the new preference key. This is done
-     * only once.
-     */
-    //    final String oldTabWidthKey = PreferenceConstants.EDITOR_TAB_WIDTH;
-    //    final String newTabWidthKey = AbstractDecoratedTextEditorPreferenceConstants.EDITOR_TAB_WIDTH;
-    //    String tabWidthPropagatedKey = "tabWidthPropagated"; //$NON-NLS-1$
-    //    if (getPreferenceStore().contains(oldTabWidthKey) && !getPreferenceStore().isDefault(oldTabWidthKey)) {
-    //      if (!getPreferenceStore().getBoolean(tabWidthPropagatedKey))
-    //        getPreferenceStore().setValue(newTabWidthKey, getPreferenceStore().getInt(oldTabWidthKey));
-    //    }
-    //    getPreferenceStore().setValue(tabWidthPropagatedKey, true);
-    //
-    //    /*
-    //     * Backwards compatibility: set the Java editor tab width in this plug-in's preference store with the old key to let older
-    //     * versions access it. Since 3.0 the tab width is managed by the extended texteditor and uses a new key.
-    //     */
-    //    getPreferenceStore().putValue(oldTabWidthKey, getPreferenceStore().getString(newTabWidthKey));
-    //    fPropertyChangeListener = new IPropertyChangeListener() {
-    //      public void propertyChange(PropertyChangeEvent event) {
-    //        if (newTabWidthKey.equals(event.getProperty()))
-    //          getPreferenceStore().putValue(oldTabWidthKey, getPreferenceStore().getString(newTabWidthKey));
-    //      }
-    //    };
-    //    getPreferenceStore().addPropertyChangeListener(fPropertyChangeListener);
-    /*
-     * Backward compatibility for the refactoring preference key.
-     */
-    //         getPreferenceStore().setValue(
-    //                 PreferenceConstants.REFACTOR_ERROR_PAGE_SEVERITY_THRESHOLD,
-    //                 RefactoringCore.getConditionCheckingFailedSeverity());
+       /*
+        * Installs backwards compatibility: propagate the Java editor font from a pre-2.1 plug-in to the Platform UI's preference store
+        * to preserve the Java editor font from a pre-2.1 workspace. This is done only once.
+        */
+       String fontPropagatedKey = "fontPropagated"; //$NON-NLS-1$
+       if (getPreferenceStore().contains(JFaceResources.TEXT_FONT) && !getPreferenceStore().isDefault(JFaceResources.TEXT_FONT)) {
+         if (!getPreferenceStore().getBoolean(fontPropagatedKey))
+               PreferenceConverter.setValue(PlatformUI.getWorkbench().getPreferenceStore(), PreferenceConstants.EDITOR_TEXT_FONT,
+                       PreferenceConverter.getFontDataArray(getPreferenceStore(), JFaceResources.TEXT_FONT));
+       }
+       getPreferenceStore().setValue(fontPropagatedKey, true);
+
+       /*
+        * Backwards compatibility: set the Java editor font in this plug-in's preference store to let older versions access it. Since
+        * 2.1 the Java editor font is managed by the workbench font preference page.
+        */
+       PreferenceConverter.putValue(getPreferenceStore(), JFaceResources.TEXT_FONT, JFaceResources.getFontRegistry().getFontData(
+               PreferenceConstants.EDITOR_TEXT_FONT));
+
+       fFontPropertyChangeListener = new IPropertyChangeListener() {
+         public void propertyChange(PropertyChangeEvent event) {
+               if (PreferenceConstants.EDITOR_TEXT_FONT.equals(event.getProperty()))
+                 PreferenceConverter.putValue(getPreferenceStore(), JFaceResources.TEXT_FONT, JFaceResources.getFontRegistry()
+                         .getFontData(PreferenceConstants.EDITOR_TEXT_FONT));
+         }
+       };
+       JFaceResources.getFontRegistry().addListener(fFontPropertyChangeListener);
+
+       /*
+        * Backwards compatibility: propagate the Java editor tab width from a pre-3.0 plug-in to the new preference key. This is done
+        * only once.
+        */
+       //    final String oldTabWidthKey = PreferenceConstants.EDITOR_TAB_WIDTH;
+       //    final String newTabWidthKey = AbstractDecoratedTextEditorPreferenceConstants.EDITOR_TAB_WIDTH;
+       //    String tabWidthPropagatedKey = "tabWidthPropagated"; //$NON-NLS-1$
+       //    if (getPreferenceStore().contains(oldTabWidthKey) && !getPreferenceStore().isDefault(oldTabWidthKey)) {
+       //      if (!getPreferenceStore().getBoolean(tabWidthPropagatedKey))
+       //        getPreferenceStore().setValue(newTabWidthKey, getPreferenceStore().getInt(oldTabWidthKey));
+       //    }
+       //    getPreferenceStore().setValue(tabWidthPropagatedKey, true);
+       //
+       //    /*
+       //     * Backwards compatibility: set the Java editor tab width in this plug-in's preference store with the old key to let older
+       //     * versions access it. Since 3.0 the tab width is managed by the extended texteditor and uses a new key.
+       //     */
+       //    getPreferenceStore().putValue(oldTabWidthKey, getPreferenceStore().getString(newTabWidthKey));
+       //    fPropertyChangeListener = new IPropertyChangeListener() {
+       //      public void propertyChange(PropertyChangeEvent event) {
+       //        if (newTabWidthKey.equals(event.getProperty()))
+       //          getPreferenceStore().putValue(oldTabWidthKey, getPreferenceStore().getString(newTabWidthKey));
+       //      }
+       //    };
+       //    getPreferenceStore().addPropertyChangeListener(fPropertyChangeListener);
+       /*
+        * Backward compatibility for the refactoring preference key.
+        */
+       //              getPreferenceStore().setValue(
+       //                      PreferenceConstants.REFACTOR_ERROR_PAGE_SEVERITY_THRESHOLD,
+       //                      RefactoringCore.getConditionCheckingFailedSeverity());
   }
 
   /**
    * Uninstalls backwards compatibility for the preference store.
    */
   private void uninstallPreferenceStoreBackwardsCompatibility() {
-    JFaceResources.getFontRegistry().removeListener(fFontPropertyChangeListener);
-    //    getPreferenceStore().removePropertyChangeListener(fPropertyChangeListener);
+       JFaceResources.getFontRegistry().removeListener(fFontPropertyChangeListener);
+       //    getPreferenceStore().removePropertyChangeListener(fPropertyChangeListener);
   }
 
   /*
    * (non - Javadoc) Method declared in Plugin
    */
   public void start(BundleContext context) throws Exception {
-    super.start(context);
-
-    //         JavaCore.start(this, context);
-    final JavaModelManager manager = JavaModelManager.getJavaModelManager();
-    try {
-      manager.configurePluginDebugOptions();
-
-      // request state folder creation (workaround 19885)
-      //      JavaCore.getPlugin().getStateLocation();
-      getStateLocation();
-      // retrieve variable values
-      PHPeclipsePlugin.getDefault().getPluginPreferences().addPropertyChangeListener(
-          new JavaModelManager.PluginPreferencesListener());
-      //                       manager.loadVariablesAndContainers();
-
-      final IWorkspace workspace = ResourcesPlugin.getWorkspace();
-      workspace.addResourceChangeListener(manager.deltaState, IResourceChangeEvent.PRE_BUILD | IResourceChangeEvent.POST_BUILD
-          | IResourceChangeEvent.POST_CHANGE | IResourceChangeEvent.PRE_DELETE | IResourceChangeEvent.PRE_CLOSE);
-
-      //               startIndexing();
-      ISavedState savedState = workspace.addSaveParticipant(PHPeclipsePlugin.this, manager);
-
-      // process deltas since last activated in indexer thread so that indexes are up-to-date.
-      // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=38658
-      Job processSavedState = new Job(Util.bind("savedState.jobName")) { //$NON-NLS-1$
-        protected IStatus run(IProgressMonitor monitor) {
-          try {
-            // add save participant and process delta atomically
-            // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=59937
-            workspace.run(new IWorkspaceRunnable() {
-              public void run(IProgressMonitor progress) throws CoreException {
-                ISavedState savedState = workspace.addSaveParticipant(PHPeclipsePlugin.this, manager);
-                if (savedState != null) {
-                  // the event type coming from the saved state is always POST_AUTO_BUILD
-                  // force it to be POST_CHANGE so that the delta processor can handle it
-                  manager.deltaState.getDeltaProcessor().overridenEventType = IResourceChangeEvent.POST_CHANGE;
-                  savedState.processResourceChangeEvents(manager.deltaState);
-                }
-              }
-            }, monitor);
-          } catch (CoreException e) {
-            return e.getStatus();
-          }
-          return Status.OK_STATUS;
-        }
-      };
-      processSavedState.setSystem(true);
-      processSavedState.setPriority(Job.SHORT); // process asap
-      processSavedState.schedule();
-    } catch (RuntimeException e) {
-      manager.shutdown();
-      throw e;
-    }
-
-    registerAdapters();
-
-    // if (USE_WORKING_COPY_OWNERS) {
-    WorkingCopyOwner.setPrimaryBufferProvider(new WorkingCopyOwner() {
-      public IBuffer createBuffer(ICompilationUnit workingCopy) {
-        ICompilationUnit original = workingCopy.getPrimary();
-        IResource resource = original.getResource();
-        if (resource instanceof IFile)
-          return new DocumentAdapter(workingCopy, (IFile) resource);
-        return DocumentAdapter.NULL;
-      }
-    });
-    // }
-
-    installPreferenceStoreBackwardsCompatibility();
+       super.start(context);
+
+       //              JavaCore.start(this, context);
+       final JavaModelManager manager = JavaModelManager.getJavaModelManager();
+       try {
+         manager.configurePluginDebugOptions();
+
+         // request state folder creation (workaround 19885)
+         //      JavaCore.getPlugin().getStateLocation();
+         getStateLocation();
+         // retrieve variable values
+         PHPeclipsePlugin.getDefault().getPluginPreferences().addPropertyChangeListener(
+                 new JavaModelManager.PluginPreferencesListener());
+         //                            manager.loadVariablesAndContainers();
+
+         final IWorkspace workspace = ResourcesPlugin.getWorkspace();
+         workspace.addResourceChangeListener(manager.deltaState, IResourceChangeEvent.PRE_BUILD | IResourceChangeEvent.POST_BUILD
+                 | IResourceChangeEvent.POST_CHANGE | IResourceChangeEvent.PRE_DELETE | IResourceChangeEvent.PRE_CLOSE);
+
+         //            startIndexing();
+         ISavedState savedState = workspace.addSaveParticipant(PHPeclipsePlugin.this, manager);
+
+         // process deltas since last activated in indexer thread so that indexes are up-to-date.
+         // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=38658
+         Job processSavedState = new Job(Util.bind("savedState.jobName")) { //$NON-NLS-1$
+               protected IStatus run(IProgressMonitor monitor) {
+                 try {
+                       // add save participant and process delta atomically
+                       // see https://bugs.eclipse.org/bugs/show_bug.cgi?id=59937
+                       workspace.run(new IWorkspaceRunnable() {
+                         public void run(IProgressMonitor progress) throws CoreException {
+                               ISavedState savedState = workspace.addSaveParticipant(PHPeclipsePlugin.this, manager);
+                               if (savedState != null) {
+                                 // the event type coming from the saved state is always POST_AUTO_BUILD
+                                 // force it to be POST_CHANGE so that the delta processor can handle it
+                                 manager.deltaState.getDeltaProcessor().overridenEventType = IResourceChangeEvent.POST_CHANGE;
+                                 savedState.processResourceChangeEvents(manager.deltaState);
+                               }
+                         }
+                       }, monitor);
+                 } catch (CoreException e) {
+                       return e.getStatus();
+                 }
+                 return Status.OK_STATUS;
+               }
+         };
+         processSavedState.setSystem(true);
+         processSavedState.setPriority(Job.SHORT); // process asap
+         processSavedState.schedule();
+       } catch (RuntimeException e) {
+         manager.shutdown();
+         throw e;
+       }
+
+       registerAdapters();
+
+       //      if (USE_WORKING_COPY_OWNERS) {
+       WorkingCopyOwner.setPrimaryBufferProvider(new WorkingCopyOwner() {
+         public IBuffer createBuffer(ICompilationUnit workingCopy) {
+               ICompilationUnit original = workingCopy.getPrimary();
+               IResource resource = original.getResource();
+               if (resource instanceof IFile)
+                 return new DocumentAdapter(workingCopy, (IFile) resource);
+               return DocumentAdapter.NULL;
+         }
+       });
+       //      }
+
+       installPreferenceStoreBackwardsCompatibility();
 
   }
 
@@ -1302,28 +1300,28 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
   //  }
 
   private void registerAdapters() {
-    fJavaElementAdapterFactory = new JavaElementAdapterFactory();
-    //         fMarkerAdapterFactory= new MarkerAdapterFactory();
-    //         fEditorInputAdapterFactory= new EditorInputAdapterFactory();
-    fResourceAdapterFactory = new ResourceAdapterFactory();
-    //         fLogicalPackageAdapterFactory= new LogicalPackageAdapterFactory();
-
-    IAdapterManager manager = Platform.getAdapterManager();
-    manager.registerAdapters(fJavaElementAdapterFactory, IJavaElement.class);
-    //         manager.registerAdapters(fMarkerAdapterFactory, IMarker.class);
-    //         manager.registerAdapters(fEditorInputAdapterFactory, IEditorInput.class);
-    manager.registerAdapters(fResourceAdapterFactory, IResource.class);
-    //         manager.registerAdapters(fLogicalPackageAdapterFactory,
-    // LogicalPackage.class);
+       fJavaElementAdapterFactory = new JavaElementAdapterFactory();
+       //              fMarkerAdapterFactory= new MarkerAdapterFactory();
+       //              fEditorInputAdapterFactory= new EditorInputAdapterFactory();
+       fResourceAdapterFactory = new ResourceAdapterFactory();
+       //              fLogicalPackageAdapterFactory= new LogicalPackageAdapterFactory();
+
+       IAdapterManager manager = Platform.getAdapterManager();
+       manager.registerAdapters(fJavaElementAdapterFactory, IJavaElement.class);
+       //              manager.registerAdapters(fMarkerAdapterFactory, IMarker.class);
+       //              manager.registerAdapters(fEditorInputAdapterFactory, IEditorInput.class);
+       manager.registerAdapters(fResourceAdapterFactory, IResource.class);
+       //              manager.registerAdapters(fLogicalPackageAdapterFactory,
+       // LogicalPackage.class);
   }
 
   private void unregisterAdapters() {
-    IAdapterManager manager = Platform.getAdapterManager();
-    manager.unregisterAdapters(fJavaElementAdapterFactory);
-    //         manager.unregisterAdapters(fMarkerAdapterFactory);
-    //         manager.unregisterAdapters(fEditorInputAdapterFactory);
-    manager.unregisterAdapters(fResourceAdapterFactory);
-    //         manager.unregisterAdapters(fLogicalPackageAdapterFactory);
+       IAdapterManager manager = Platform.getAdapterManager();
+       manager.unregisterAdapters(fJavaElementAdapterFactory);
+       //              manager.unregisterAdapters(fMarkerAdapterFactory);
+       //              manager.unregisterAdapters(fEditorInputAdapterFactory);
+       manager.unregisterAdapters(fResourceAdapterFactory);
+       //              manager.unregisterAdapters(fLogicalPackageAdapterFactory);
   }
 
   /**
@@ -1334,20 +1332,20 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public IPreferenceStore getCombinedPreferenceStore() {
-    if (fCombinedPreferenceStore == null) {
-      IPreferenceStore generalTextStore = EditorsUI.getPreferenceStore();
-      fCombinedPreferenceStore = new ChainedPreferenceStore(new IPreferenceStore[] {
-          getPreferenceStore(),
-          new PreferencesAdapter(PHPeclipsePlugin.getDefault().getPluginPreferences()),
-          generalTextStore });
-    }
-    return fCombinedPreferenceStore;
+       if (fCombinedPreferenceStore == null) {
+         IPreferenceStore generalTextStore = EditorsUI.getPreferenceStore();
+         fCombinedPreferenceStore = new ChainedPreferenceStore(new IPreferenceStore[] {
+                 getPreferenceStore(),
+                 new PreferencesAdapter(PHPeclipsePlugin.getDefault().getPluginPreferences()),
+                 generalTextStore });
+       }
+       return fCombinedPreferenceStore;
   }
 
   public synchronized IBufferFactory getBufferFactory() {
-    if (fBufferFactory == null)
-      fBufferFactory = new CustomBufferFactory();
-    return fBufferFactory;
+       if (fBufferFactory == null)
+         fBufferFactory = new CustomBufferFactory();
+       return fBufferFactory;
   }
 
   /**
@@ -1358,9 +1356,9 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public synchronized JavaFoldingStructureProviderRegistry getFoldingStructureProviderRegistry() {
-    if (fFoldingStructureProviderRegistry == null)
-      fFoldingStructureProviderRegistry = new JavaFoldingStructureProviderRegistry();
-    return fFoldingStructureProviderRegistry;
+       if (fFoldingStructureProviderRegistry == null)
+         fFoldingStructureProviderRegistry = new JavaFoldingStructureProviderRegistry();
+       return fFoldingStructureProviderRegistry;
   }
 
   /**
@@ -1387,7 +1385,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 2.1
    */
   public static void run(IWorkspaceRunnable action, IProgressMonitor monitor) throws CoreException {
-    run(action, ResourcesPlugin.getWorkspace().getRoot(), monitor);
+       run(action, ResourcesPlugin.getWorkspace().getRoot(), monitor);
   }
 
   /**
@@ -1421,14 +1419,14 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public static void run(IWorkspaceRunnable action, ISchedulingRule rule, IProgressMonitor monitor) throws CoreException {
-    IWorkspace workspace = ResourcesPlugin.getWorkspace();
-    if (workspace.isTreeLocked()) {
-      new BatchOperation(action).run(monitor);
-    } else {
-      // use IWorkspace.run(...) to ensure that a build will be done in
-      // autobuild mode
-      workspace.run(new BatchOperation(action), rule, IWorkspace.AVOID_UPDATE, monitor);
-    }
+       IWorkspace workspace = ResourcesPlugin.getWorkspace();
+       if (workspace.isTreeLocked()) {
+         new BatchOperation(action).run(monitor);
+       } else {
+         // use IWorkspace.run(...) to ensure that a build will be done in
+         // autobuild mode
+         workspace.run(new BatchOperation(action), rule, IWorkspace.AVOID_UPDATE, monitor);
+       }
   }
 
   /**
@@ -1438,15 +1436,15 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public ContextTypeRegistry getTemplateContextRegistry() {
-    if (fContextTypeRegistry == null) {
-      fContextTypeRegistry = new ContributionContextTypeRegistry();
+       if (fContextTypeRegistry == null) {
+         fContextTypeRegistry = new ContributionContextTypeRegistry();
 
-      fContextTypeRegistry.addContextType(new JavaContextType());
-      fContextTypeRegistry.addContextType(new JavaDocContextType());
-      fContextTypeRegistry.addContextType(new HTMLContextType());
-    }
+         fContextTypeRegistry.addContextType(new JavaContextType());
+         fContextTypeRegistry.addContextType(new JavaDocContextType());
+         fContextTypeRegistry.addContextType(new HTMLContextType());
+       }
 
-    return fContextTypeRegistry;
+       return fContextTypeRegistry;
   }
 
   /**
@@ -1456,24 +1454,24 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public TemplateStore getTemplateStore() {
-    if (fTemplateStore == null) {
-      //                       boolean alreadyMigrated= getPreferenceStore().getBoolean(TEMPLATES_MIGRATION_KEY);
-      //                       if (alreadyMigrated)
-      fTemplateStore = new ContributionTemplateStore(getTemplateContextRegistry(), getPreferenceStore(), TEMPLATES_KEY);
-      //                       else {
-      //                               fTemplateStore= new CompatibilityTemplateStore(getTemplateContextRegistry(), getPreferenceStore(), TEMPLATES_KEY,
-      // Templates.getInstance());
-      //                               getPreferenceStore().setValue(TEMPLATES_MIGRATION_KEY, true);
-      //                       }
-
-      try {
-        fTemplateStore.load();
-      } catch (IOException e) {
-        log(e);
-      }
-    }
-
-    return fTemplateStore;
+       if (fTemplateStore == null) {
+         //                    boolean alreadyMigrated= getPreferenceStore().getBoolean(TEMPLATES_MIGRATION_KEY);
+         //                    if (alreadyMigrated)
+         fTemplateStore = new ContributionTemplateStore(getTemplateContextRegistry(), getPreferenceStore(), TEMPLATES_KEY);
+         //                    else {
+         //                            fTemplateStore= new CompatibilityTemplateStore(getTemplateContextRegistry(), getPreferenceStore(), TEMPLATES_KEY,
+         // Templates.getInstance());
+         //                            getPreferenceStore().setValue(TEMPLATES_MIGRATION_KEY, true);
+         //                    }
+
+         try {
+               fTemplateStore.load();
+         } catch (IOException e) {
+               log(e);
+         }
+       }
+
+       return fTemplateStore;
   }
 
   /**
@@ -1483,13 +1481,13 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public ContextTypeRegistry getCodeTemplateContextRegistry() {
-    if (fCodeTemplateContextTypeRegistry == null) {
-      fCodeTemplateContextTypeRegistry = new ContributionContextTypeRegistry();
+       if (fCodeTemplateContextTypeRegistry == null) {
+         fCodeTemplateContextTypeRegistry = new ContributionContextTypeRegistry();
 
-      CodeTemplateContextType.registerContextTypes(fCodeTemplateContextTypeRegistry);
-    }
+         CodeTemplateContextType.registerContextTypes(fCodeTemplateContextTypeRegistry);
+       }
 
-    return fCodeTemplateContextTypeRegistry;
+       return fCodeTemplateContextTypeRegistry;
   }
 
   /**
@@ -1499,27 +1497,27 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceCon
    * @since 3.0
    */
   public TemplateStore getCodeTemplateStore() {
-    if (fCodeTemplateStore == null) {
-      //                       boolean alreadyMigrated= getPreferenceStore().getBoolean(CODE_TEMPLATES_MIGRATION_KEY);
-      //                       if (alreadyMigrated)
-      fCodeTemplateStore = new ContributionTemplateStore(getCodeTemplateContextRegistry(), getPreferenceStore(), CODE_TEMPLATES_KEY);
-      //                       else {
-      //                               fCodeTemplateStore= new CompatibilityTemplateStore(getCodeTemplateContextRegistry(), getPreferenceStore(),
-      // CODE_TEMPLATES_KEY, CodeTemplates.getInstance());
-      //                               getPreferenceStore().setValue(CODE_TEMPLATES_MIGRATION_KEY, true);
-      //                       }
-
-      try {
-        fCodeTemplateStore.load();
-      } catch (IOException e) {
-        log(e);
-      }
-
-      // compatibility / bug fixing code for duplicated templates
-      // TODO remove for 3.0
-      //                       CompatibilityTemplateStore.pruneDuplicates(fCodeTemplateStore, true);
-    }
-
-    return fCodeTemplateStore;
+       if (fCodeTemplateStore == null) {
+         //                    boolean alreadyMigrated= getPreferenceStore().getBoolean(CODE_TEMPLATES_MIGRATION_KEY);
+         //                    if (alreadyMigrated)
+         fCodeTemplateStore = new ContributionTemplateStore(getCodeTemplateContextRegistry(), getPreferenceStore(), CODE_TEMPLATES_KEY);
+         //                    else {
+         //                            fCodeTemplateStore= new CompatibilityTemplateStore(getCodeTemplateContextRegistry(), getPreferenceStore(),
+         // CODE_TEMPLATES_KEY, CodeTemplates.getInstance());
+         //                            getPreferenceStore().setValue(CODE_TEMPLATES_MIGRATION_KEY, true);
+         //                    }
+
+         try {
+               fCodeTemplateStore.load();
+         } catch (IOException e) {
+               log(e);
+         }
+
+         // compatibility / bug fixing code for duplicated templates
+         // TODO remove for 3.0
+         //                    CompatibilityTemplateStore.pruneDuplicates(fCodeTemplateStore, true);
+       }
+
+       return fCodeTemplateStore;
   }
 }
\ No newline at end of file