Better event handling for browser preview refreshs in Wikipedia plugin
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpeclipse / PHPeclipsePlugin.java
index b0b2811..814b2e5 100644 (file)
@@ -5,9 +5,6 @@
  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
  **********************************************************************/
 package net.sourceforge.phpeclipse;
 
@@ -55,6 +52,7 @@ import net.sourceforge.phpeclipse.builder.FileStorage;
 import net.sourceforge.phpeclipse.builder.IdentifierIndexManager;
 import net.sourceforge.phpeclipse.phpeditor.CustomBufferFactory;
 import net.sourceforge.phpeclipse.phpeditor.DocumentAdapter;
+import net.sourceforge.phpeclipse.phpeditor.ICompilationUnitDocumentProvider;
 import net.sourceforge.phpeclipse.phpeditor.PHPDocumentProvider;
 import net.sourceforge.phpeclipse.phpeditor.PHPSyntaxRdr;
 import net.sourceforge.phpeclipse.phpeditor.WorkingCopyManager;
@@ -107,7 +105,6 @@ import org.eclipse.ui.editors.text.templates.ContributionContextTypeRegistry;
 import org.eclipse.ui.editors.text.templates.ContributionTemplateStore;
 import org.eclipse.ui.ide.IDE;
 import org.eclipse.ui.plugin.AbstractUIPlugin;
-import org.eclipse.ui.texteditor.AbstractDecoratedTextEditorPreferenceConstants;
 import org.eclipse.ui.texteditor.ChainedPreferenceStore;
 import org.eclipse.ui.texteditor.ConfigurationElementSorter;
 import org.eclipse.ui.texteditor.IDocumentProvider;
@@ -118,8 +115,7 @@ import org.osgi.framework.BundleContext;
 /**
  * The main plugin class to be used in the desktop.
  */
-public class PHPeclipsePlugin extends AbstractUIPlugin implements
-    IPreferenceConstants {
+public class PHPeclipsePlugin extends AbstractUIPlugin implements IPreferenceConstants {
   /**
    * The id of the PHP plugin (value <code>"net.sourceforge.phpeclipse"</code>).
    */
@@ -132,6 +128,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
 
   //public static final String BUILDER_INDEX_ID = PLUGIN_ID + ".indexbuilder";
   /** General debug flag */
+  
   public static final boolean DEBUG = false;
 
   /**
@@ -140,26 +137,32 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   public final static int MAX_PROPOSALS = 200;
 
   /**
-        * The key to store customized templates. 
-        * @since 3.0
-        */
-       private static final String TEMPLATES_KEY= "net.sourceforge.phpdt.ui.text.custom_templates"; //$NON-NLS-1$
-       /**
-        * The key to store customized code templates. 
-        * @since 3.0
-        */
-       private static final String CODE_TEMPLATES_KEY= "net.sourceforge.phpdt.ui.text.custom_code_templates"; //$NON-NLS-1$
-       /**
-        * The key to store whether the legacy templates have been migrated 
-        * @since 3.0
-        */
-//     private static final String TEMPLATES_MIGRATION_KEY= "net.sourceforge.phpdt.ui.text.templates_migrated"; //$NON-NLS-1$
-       /**
-        * The key to store whether the legacy code templates have been migrated 
-        * @since 3.0
-        */
-//     private static final String CODE_TEMPLATES_MIGRATION_KEY= "net.sourceforge.phpdt.ui.text.code_templates_migrated"; //$NON-NLS-1$
-       
+   * The key to store customized templates.
+   * 
+   * @since 3.0
+   */
+  private static final String TEMPLATES_KEY = "net.sourceforge.phpdt.ui.text.custom_templates"; //$NON-NLS-1$
+
+  /**
+   * The key to store customized code templates.
+   * 
+   * @since 3.0
+   */
+  private static final String CODE_TEMPLATES_KEY = "net.sourceforge.phpdt.ui.text.custom_code_templates"; //$NON-NLS-1$
+
+  /**
+   * The key to store whether the legacy templates have been migrated
+   * 
+   * @since 3.0
+   */
+  //   private static final String TEMPLATES_MIGRATION_KEY= "net.sourceforge.phpdt.ui.text.templates_migrated"; //$NON-NLS-1$
+  /**
+   * The key to store whether the legacy code templates have been migrated
+   * 
+   * @since 3.0
+   */
+  //   private static final String CODE_TEMPLATES_MIGRATION_KEY= "net.sourceforge.phpdt.ui.text.code_templates_migrated";
+  // //$NON-NLS-1$
 
   private static ExternalToolsPlugin externalTools;
 
@@ -185,8 +188,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
 
   // public static final String PHP_RESOURCES_VIEW_ID = PLUGIN_ID +
   // ".resourcesview.ViewPHPResources"; //$NON-NLS-1$
-  public static final String PHP_CODING_ACTION_SET_ID = PLUGIN_ID
-      + ".ui.CodingActionSet"; //$NON-NLS-1$
+  public static final String PHP_CODING_ACTION_SET_ID = PLUGIN_ID + ".ui.CodingActionSet"; //$NON-NLS-1$
 
   public final static String PHP_NATURE_ID = PLUGIN_ID + ".phpnature";
 
@@ -200,28 +202,34 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   //The shared instance.
   private static PHPeclipsePlugin plugin;
 
-  /** 
-        * The template context type registry for the java editor. 
-        * @since 3.0
-        */
-       private ContextTypeRegistry fContextTypeRegistry;
-       /** 
-        * The code template context type registry for the java editor. 
-        * @since 3.0
-        */
-       private ContextTypeRegistry fCodeTemplateContextTypeRegistry;
-       
-       /**
-        * The template store for the java editor. 
-        * @since 3.0
-        */
-       private TemplateStore fTemplateStore;
-       /**
-        * The coded template store for the java editor. 
-        * @since 3.0
-        */
-       private TemplateStore fCodeTemplateStore;
-       
+  /**
+   * The template context type registry for the java editor.
+   * 
+   * @since 3.0
+   */
+  private ContextTypeRegistry fContextTypeRegistry;
+
+  /**
+   * The code template context type registry for the java editor.
+   * 
+   * @since 3.0
+   */
+  private ContextTypeRegistry fCodeTemplateContextTypeRegistry;
+
+  /**
+   * The template store for the java editor.
+   * 
+   * @since 3.0
+   */
+  private TemplateStore fTemplateStore;
+
+  /**
+   * The coded template store for the java editor.
+   * 
+   * @since 3.0
+   */
+  private TemplateStore fCodeTemplateStore;
+
   /** Windows 9x */
   private static final int WINDOWS_9x = 6;
 
@@ -236,7 +244,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
 
   private IBufferFactory fBufferFactory;
 
-  private PHPDocumentProvider fCompilationUnitDocumentProvider;
+  private ICompilationUnitDocumentProvider fCompilationUnitDocumentProvider;
 
   private JavaTextTools fJavaTextTools;
 
@@ -262,7 +270,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
    * 
    * @since 3.0
    */
-  private IPropertyChangeListener fPropertyChangeListener;
+//  private IPropertyChangeListener fPropertyChangeListener;
 
   /**
    * The combined preference store.
@@ -272,17 +280,14 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   private IPreferenceStore fCombinedPreferenceStore;
 
   /**
-   * The extension point registry for the
-   * <code>net.sourceforge.phpdt.ui.javaFoldingStructureProvider</code> extension
-   * point.
+   * The extension point registry for the <code>net.sourceforge.phpdt.ui.javaFoldingStructureProvider</code> extension point.
    * 
    * @since 3.0
    */
   private JavaFoldingStructureProviderRegistry fFoldingStructureProviderRegistry;
 
   /**
-   * Mockup preference store for firing events and registering listeners on
-   * project setting changes. FIXME: Temporary solution.
+   * Mockup preference store for firing events and registering listeners on project setting changes. FIXME: Temporary solution.
    * 
    * @since 3.0
    */
@@ -296,6 +301,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
     plugin = this;
     setJVM();
     externalTools = new ExternalToolsPlugin();
+    
     //    try {
     //      resourceBundle =
     // ResourceBundle.getBundle("net.sourceforge.PHPeclipsePluginResources");
@@ -325,15 +331,13 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
    */
   public JavaEditorTextHoverDescriptor[] getJavaEditorTextHoverDescriptors() {
     if (fJavaEditorTextHoverDescriptors == null) {
-      fJavaEditorTextHoverDescriptors = JavaEditorTextHoverDescriptor
-          .getContributedHovers();
+      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();
+          return ((JavaEditorTextHoverDescriptor) object).getConfigurationElement();
         }
       };
       sorter.sort(fJavaEditorTextHoverDescriptors);
@@ -357,13 +361,11 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
         if (first == -1)
           first = i;
 
-        if (fJavaEditorTextHoverDescriptors[i].getId().equals(
-            "net.sourceforge.phpdt.ui.AnnotationHover")) { //$NON-NLS-1$
+        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$
+        if (fJavaEditorTextHoverDescriptors[i].getId().equals("net.sourceforge.phpdt.ui.ProblemHover")) { //$NON-NLS-1$
           problemHoverIndex = i;
           continue;
         }
@@ -391,8 +393,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
 
       // Move Best Match hover to front
       for (int i = 0; i < fJavaEditorTextHoverDescriptors.length - 1; i++) {
-        if (PreferenceConstants.ID_BESTMATCH_HOVER
-            .equals(fJavaEditorTextHoverDescriptors[i].getId())) {
+        if (PreferenceConstants.ID_BESTMATCH_HOVER.equals(fJavaEditorTextHoverDescriptors[i].getId())) {
           hoverDescriptor = fJavaEditorTextHoverDescriptors[i];
           for (int j = i; j > 0; j--)
             fJavaEditorTextHoverDescriptors[j] = fJavaEditorTextHoverDescriptors[j - 1];
@@ -409,8 +410,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   /**
    * Resets the Java editor text hovers contributed to the workbench.
    * <p>
-   * This will force a rebuild of the descriptors the next time a client asks
-   * for them.
+   * This will force a rebuild of the descriptors the next time a client asks for them.
    * </p>
    * 
    * @return an array of JavaEditorTextHoverDescriptor
@@ -448,9 +448,8 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Returns an array of all editors that have an unsaved content. If the
-   * identical content is presented in more than one editor, only one of those
-   * editor parts is part of the result.
+   * Returns an array of all editors that have an unsaved content. If the identical content is presented in more than one editor,
+   * only one of those editor parts is part of the result.
    * 
    * @return an array of all dirty editor parts.
    */
@@ -504,9 +503,8 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Returns the standard display to be used. The method first checks, if the
-   * thread calling this method has an associated display. If so, this display
-   * is returned. Otherwise the method returns the default display.
+   * Returns the standard display to be used. The method first checks, if the thread calling this method has an associated display.
+   * If so, this display is returned. Otherwise the method returns the default display.
    */
   public static Display getStandardDisplay() {
     Display display = Display.getCurrent();
@@ -561,13 +559,11 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   public static void log(Throwable e) {
-    log(new Status(IStatus.ERROR, PLUGIN_ID, IStatus.ERROR,
-        "PHPeclipsePlugin.internalErrorOccurred", e)); //$NON-NLS-1$
+    log(new Status(IStatus.ERROR, PLUGIN_ID, IStatus.ERROR, "PHPeclipsePlugin.internalErrorOccurred", e)); //$NON-NLS-1$
   }
 
   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));
   }
 
   private static void setJVM() {
@@ -599,7 +595,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   // TODO: refactor this into a better method name !
-  public synchronized PHPDocumentProvider getCompilationUnitDocumentProvider() {
+  public synchronized ICompilationUnitDocumentProvider getCompilationUnitDocumentProvider() {
     if (fCompilationUnitDocumentProvider == null)
       fCompilationUnitDocumentProvider = new PHPDocumentProvider();
     return fCompilationUnitDocumentProvider;
@@ -613,10 +609,8 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
    * @return
    */
   public IdentifierIndexManager getIndexManager(IProject iProject) {
-    String indexFilename = iProject.getLocation() + File.separator
-        + "project.index";
-    IdentifierIndexManager indexManager = (IdentifierIndexManager) fIndexManagerMap
-        .get(indexFilename);
+    String indexFilename = iProject.getLocation() + File.separator + "project.index";
+    IdentifierIndexManager indexManager = (IdentifierIndexManager) fIndexManagerMap.get(indexFilename);
     if (indexManager == null) {
       indexManager = new IdentifierIndexManager(indexFilename);
       fIndexManagerMap.put(indexFilename, indexManager);
@@ -626,10 +620,10 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
 
   public synchronized IWorkingCopyManager getWorkingCopyManager() {
     if (fWorkingCopyManager == null) {
-      PHPDocumentProvider provider = getCompilationUnitDocumentProvider();
-      fWorkingCopyManager = new WorkingCopyManager(provider);
-    }
-    return fWorkingCopyManager;
+               ICompilationUnitDocumentProvider provider= getCompilationUnitDocumentProvider();
+               fWorkingCopyManager= new WorkingCopyManager(provider);
+       }
+       return fWorkingCopyManager;
   }
 
   public synchronized MembersOrderPreferenceCache getMemberOrderPreferenceCache() {
@@ -639,8 +633,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Returns the mockup preference store for firing events and registering
-   * listeners on project setting changes. Temporary solution.
+   * Returns the mockup preference store for firing events and registering listeners on project setting changes. Temporary solution.
    */
   public MockupPreferenceStore getMockupPreferenceStore() {
     if (fMockupPreferenceStore == null)
@@ -662,8 +655,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   //  }
   public synchronized JavaTextTools getJavaTextTools() {
     if (fJavaTextTools == null)
-      fJavaTextTools = new JavaTextTools(getPreferenceStore(), JavaCore
-          .getPlugin().getPluginPreferences());
+      fJavaTextTools = new JavaTextTools(getPreferenceStore(), JavaCore.getPlugin().getPluginPreferences());
     return fJavaTextTools;
   }
 
@@ -672,8 +664,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Returns the string from the plugin's resource bundle, or 'key' if not
-   * found.
+   * Returns the string from the plugin's resource bundle, or 'key' if not found.
    */
   //  public static String getResourceString(String key) {
   //    ResourceBundle bundle = PHPeclipsePlugin.getDefault().getResourceBundle();
@@ -694,37 +685,62 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
     //    store.setDefault(LOCALHOST_PREF, "http://localhost");
     //    store.setDefault(DOCUMENTROOT_PREF,
     // getWorkspace().getRoot().getLocation().toString());
-//    store.setDefault(PHP_LOCALHOST_PREF, "http://localhost");
-//    store.setDefault(PHP_DOCUMENTROOT_PREF, getWorkspace().getRoot()
-//        .getLocation().toString());
+    //    store.setDefault(PHP_LOCALHOST_PREF, "http://localhost");
+    //    store.setDefault(PHP_DOCUMENTROOT_PREF, getWorkspace().getRoot()
+    //        .getLocation().toString());
+    
 
-    //    store.setDefault(SHOW_EXTERNAL_PREVIEW_PREF, "true");
-    store.setDefault(USE_EXTERNAL_BROWSER_PREF, "false");
+//    store.setDefault(SHOW_EXTERNAL_PREVIEW_PREF, "true");
+//    store.setDefault(USE_EXTERNAL_BROWSER_PREF, "false");
     store.setDefault(SHOW_OUTPUT_IN_CONSOLE, "true");
     store.setDefault(PHP_OBFUSCATOR_DEFAULT, "c:\\temp");
-//    store.setDefault(PHP_BOOKMARK_DEFAULT, "");
-//
-//    store.setDefault(PHP_AUTO_PREVIEW_DEFAULT, "true");
-//    store.setDefault(PHP_BRING_TO_TOP_PREVIEW_DEFAULT, "true");
-//    store.setDefault(PHP_SHOW_HTML_FILES_LOCAL, "true");
+    //    store.setDefault(PHP_BOOKMARK_DEFAULT, "");
+    //
+    //    store.setDefault(PHP_AUTO_PREVIEW_DEFAULT, "true");
+    //    store.setDefault(PHP_BRING_TO_TOP_PREVIEW_DEFAULT, "true");
+    //    store.setDefault(PHP_SHOW_HTML_FILES_LOCAL, "true");
 
     String windowsSystem = Platform.getWS();
     if (jvm == WINDOWS_9x) {
-      store.setDefault(EXTERNAL_BROWSER_PREF,
-          "command.com /c start iexplore {0}");
+//      store.setDefault(EXTERNAL_BROWSER_PREF, "command.com /c start iexplore {0}");
     } else if (windowsSystem.equals(Platform.WS_WIN32)) {
-      store.setDefault(EXTERNAL_BROWSER_PREF,
-          "rundll32 url.dll,FileProtocolHandler {0}");
+//      store.setDefault(EXTERNAL_BROWSER_PREF, "rundll32 url.dll,FileProtocolHandler {0}");
     } else if (windowsSystem.equals(Platform.WS_CARBON)) {
       // TODO How do we start Safari on Mac OS X ?
-      store.setDefault(EXTERNAL_BROWSER_PREF, "netscape {0}");
+//      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}");
     }
     //  if ((jvm == WINDOWS_9x) || (jvm == WINDOWS_NT)) {
     // 
     if (windowsSystem.equals(Platform.WS_WIN32)) {
+//      String workspaceLocation = getWorkspace().getRoot().getLocation().toString();
+//      String baseLocation = null;
+//      int index = workspaceLocation.lastIndexOf('/');
+//      if (index>0) {
+//        index = workspaceLocation.lastIndexOf('/', index-1);
+//        if (index>0) {
+//          baseLocation = workspaceLocation.substring(0,index);
+//        }
+//      }
+//      if (baseLocation!=null) {
+//        String xampp_start = baseLocation + "/xampp_start.exe";
+//        String xampp_stop = baseLocation + "/xampp_stop.exe";
+//        File testFile = new File(xampp_start);
+//        if (testFile.exists()) {
+//          System.out.println(xampp_start);
+//          System.out.println(xampp_stop);
+//        } else {
+//          xampp_start = "c:\\xampp\\xampp_start.exe";
+//          xampp_stop = "c:\\xampp\\xampp_stop.exe";
+//          testFile = new File(xampp_start);
+//          if (testFile.exists()) {
+//            System.out.println(xampp_start);
+//            System.out.println(xampp_stop);
+//          }
+//        }
+//      }
       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");
@@ -735,6 +751,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
       store.setDefault(MYSQL_RUN_PREF, "/apache/mysql/bin/mysqld");
       store.setDefault(APACHE_RUN_PREF, "/apache/apache");
     }
+    store.setDefault(PHP_EXTENSION_PREFS, "php,php3,php4,php5,phtml,inc,module,class");
     store.setDefault(MYSQL_PREF, "--standalone");
     store.setDefault(APACHE_START_PREF, "-c \"DocumentRoot \"{0}\"\"");
     store.setDefault(APACHE_STOP_PREF, "-k shutdown");
@@ -753,42 +770,32 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
     // 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_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_FUNCTIONNAME,
-        PHPColorProvider.FUNCTION_NAME);
-    PreferenceConverter.setDefault(store, PHP_CONSTANT,
-        PHPColorProvider.CONSTANT);
+    PreferenceConverter.setDefault(store, PHP_KEYWORD, PHPColorProvider.KEYWORD);
+    PreferenceConverter.setDefault(store, PHP_VARIABLE, 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, PHPColorProvider.STRING);
-    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));
+    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));
+    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,
@@ -829,7 +836,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
     //    store.setDefault(RESOURCE_BUNDLE_DE, "false");
     //    store.setDefault(RESOURCE_BUNDLE_FR, "false");
     //    store.setDefault(RESOURCE_BUNDLE_ES, "false");
-//    TemplatePreferencePage.initDefaults(store);
+    //    TemplatePreferencePage.initDefaults(store);
     //this will initialize the static fields in the syntaxrdr class
     new PHPSyntaxRdr();
     JavaCore.initializeDefaultPluginPreferences();
@@ -852,8 +859,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Open a file in the Workbench that may or may not exist in the workspace.
-   * Must be run on the UI thread.
+   * Open a file in the Workbench that may or may not exist in the workspace. Must be run on the UI thread.
    * 
    * @param filename
    * @throws CoreException
@@ -899,25 +905,21 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Open a file in the Workbench that may or may not exist in the workspace.
-   * Must be run on the UI thread.
+   * Open a file in the Workbench that may or may not exist in the workspace. Must be run on the UI thread.
    * 
    * @param filename
    * @param line
    * @throws CoreException
    */
-  public void openFileAndGotoLine(String filename, int line)
-      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));
+          IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
+          textEditor.selectAndReveal(document.getLineOffset(line), document.getLineLength(line));
         } catch (BadLocationException e) {
           // invalid text position -> do nothing
         }
@@ -926,35 +928,30 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Open a file in the Workbench that may or may not exist in the workspace.
-   * Must be run on the UI thread.
+   * Open a file in the Workbench that may or may not exist in the workspace. Must be run on the UI thread.
    * 
    * @param filename
    * @param offset
    * @throws CoreException
    */
-  public void openFileAndGotoOffset(String filename, int offset, int length)
-      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());
+        IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
         textEditor.selectAndReveal(offset, length);
       }
     }
   }
 
-  public void openFileAndFindString(String filename, String findString)
-      throws CoreException {
+  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());
+          IDocument document = textEditor.getDocumentProvider().getDocument(textEditor.getEditorInput());
           int offset = document.search(0, findString, true, false, true);
           textEditor.selectAndReveal(offset, findString.length());
         } catch (BadLocationException e) {
@@ -976,8 +973,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
       //                       JavaCore.stop(this, context);
       plugin.savePluginPreferences();
       IWorkspace workspace = ResourcesPlugin.getWorkspace();
-      workspace.removeResourceChangeListener(JavaModelManager
-          .getJavaModelManager().deltaState);
+      workspace.removeResourceChangeListener(JavaModelManager.getJavaModelManager().deltaState);
       workspace.removeSaveParticipant(plugin);
 
       JavaModelManager.getJavaModelManager().shutdown();
@@ -1081,72 +1077,59 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   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.
+     * 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().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));
+        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.
+     * 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));
+    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));
+          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.
+     * 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);
+//    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.
@@ -1160,9 +1143,8 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
    * Uninstalls backwards compatibility for the preference store.
    */
   private void uninstallPreferenceStoreBackwardsCompatibility() {
-    JFaceResources.getFontRegistry()
-        .removeListener(fFontPropertyChangeListener);
-    getPreferenceStore().removePropertyChangeListener(fPropertyChangeListener);
+    JFaceResources.getFontRegistry().removeListener(fFontPropertyChangeListener);
+//    getPreferenceStore().removePropertyChangeListener(fPropertyChangeListener);
   }
 
   /*
@@ -1185,46 +1167,41 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
       //                       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();
+      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(ProjectPrefUtil.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;
@@ -1293,7 +1270,7 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   //      // JavaModelManager.PluginPreferencesListener());
   //      getPluginPreferences().addPropertyChangeListener(
   //          new JavaModelManager.PluginPreferencesListener());
-  //      // TODO khartlage temp-del
+  //      // TODO temp-del
   //      // manager.loadVariablesAndContainers();
   //      final IWorkspace workspace = ResourcesPlugin.getWorkspace();
   //           workspace.addResourceChangeListener(
@@ -1360,11 +1337,8 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   public IPreferenceStore getCombinedPreferenceStore() {
     if (fCombinedPreferenceStore == null) {
       IPreferenceStore generalTextStore = EditorsUI.getPreferenceStore();
-      fCombinedPreferenceStore = new ChainedPreferenceStore(
-          new IPreferenceStore[] {
-              getPreferenceStore(),
-              new PreferencesAdapter(PHPeclipsePlugin.getDefault()
-                  .getPluginPreferences()), generalTextStore });
+      fCombinedPreferenceStore = new ChainedPreferenceStore(new IPreferenceStore[] { getPreferenceStore(),
+          new PreferencesAdapter(PHPeclipsePlugin.getDefault().getPluginPreferences()), generalTextStore });
     }
     return fCombinedPreferenceStore;
   }
@@ -1376,12 +1350,10 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   }
 
   /**
-   * Returns the registry of the extensions to the
-   * <code>net.sourceforge.phpdt.ui.javaFoldingStructureProvider</code> extension
+   * Returns the registry of the extensions to the <code>net.sourceforge.phpdt.ui.javaFoldingStructureProvider</code> extension
    * point.
    * 
-   * @return the registry of contributed
-   *         <code>IJavaFoldingStructureProvider</code>
+   * @return the registry of contributed <code>IJavaFoldingStructureProvider</code>
    * @since 3.0
    */
   public synchronized JavaFoldingStructureProviderRegistry getFoldingStructureProviderRegistry() {
@@ -1393,177 +1365,160 @@ public class PHPeclipsePlugin extends AbstractUIPlugin implements
   /**
    * Runs the given action as an atomic Java model operation.
    * <p>
-   * After running a method that modifies java elements, registered listeners
-   * receive after-the-fact notification of what just transpired, in the form of
-   * a element changed event. This method allows clients to call a number of
-   * methods that modify java elements and only have element changed event
-   * notifications reported at the end of the entire batch.
+   * After running a method that modifies java elements, registered listeners receive after-the-fact notification of what just
+   * transpired, in the form of a element changed event. This method allows clients to call a number of methods that modify java
+   * elements and only have element changed event notifications reported at the end of the entire batch.
    * </p>
    * <p>
-   * If this method is called outside the dynamic scope of another such call,
-   * this method runs the action and then reports a single element changed event
-   * describing the net effect of all changes done to java elements by the
-   * action.
+   * If this method is called outside the dynamic scope of another such call, this method runs the action and then reports a single
+   * element changed event describing the net effect of all changes done to java elements by the action.
    * </p>
    * <p>
-   * If this method is called in the dynamic scope of another such call, this
-   * method simply runs the action.
+   * If this method is called in the dynamic scope of another such call, this method simply runs the action.
    * </p>
    * 
    * @param action
    *          the action to perform
    * @param monitor
-   *          a progress monitor, or <code>null</code> if progress reporting
-   *          and cancellation are not desired
+   *          a progress monitor, or <code>null</code> if progress reporting and cancellation are not desired
    * @exception CoreException
    *              if the operation failed.
    * @since 2.1
    */
-  public static void run(IWorkspaceRunnable action, IProgressMonitor monitor)
-      throws CoreException {
+  public static void run(IWorkspaceRunnable action, IProgressMonitor monitor) throws CoreException {
     run(action, ResourcesPlugin.getWorkspace().getRoot(), monitor);
   }
 
   /**
    * Runs the given action as an atomic Java model operation.
    * <p>
-   * After running a method that modifies java elements, registered listeners
-   * receive after-the-fact notification of what just transpired, in the form of
-   * a element changed event. This method allows clients to call a number of
-   * methods that modify java elements and only have element changed event
-   * notifications reported at the end of the entire batch.
+   * After running a method that modifies java elements, registered listeners receive after-the-fact notification of what just
+   * transpired, in the form of a element changed event. This method allows clients to call a number of methods that modify java
+   * elements and only have element changed event notifications reported at the end of the entire batch.
    * </p>
    * <p>
-   * If this method is called outside the dynamic scope of another such call,
-   * this method runs the action and then reports a single element changed event
-   * describing the net effect of all changes done to java elements by the
-   * action.
+   * If this method is called outside the dynamic scope of another such call, this method runs the action and then reports a single
+   * element changed event describing the net effect of all changes done to java elements by the action.
    * </p>
    * <p>
-   * If this method is called in the dynamic scope of another such call, this
-   * method simply runs the action.
+   * If this method is called in the dynamic scope of another such call, this method simply runs the action.
    * </p>
    * <p>
-   * The supplied scheduling rule is used to determine whether this operation
-   * can be run simultaneously with workspace changes in other threads. See
-   * <code>IWorkspace.run(...)</code> for more details.
+   * The supplied scheduling rule is used to determine whether this operation can be run simultaneously with workspace changes in
+   * other threads. See <code>IWorkspace.run(...)</code> for more details.
    * </p>
    * 
    * @param action
    *          the action to perform
    * @param rule
-   *          the scheduling rule to use when running this operation, or
-   *          <code>null</code> if there are no scheduling restrictions for
-   *          this operation.
+   *          the scheduling rule to use when running this operation, or <code>null</code> if there are no scheduling restrictions
+   *          for this operation.
    * @param monitor
-   *          a progress monitor, or <code>null</code> if progress reporting
-   *          and cancellation are not desired
+   *          a progress monitor, or <code>null</code> if progress reporting and cancellation are not desired
    * @exception CoreException
    *              if the operation failed.
    * @since 3.0
    */
-  public static void run(IWorkspaceRunnable action, ISchedulingRule rule,
-      IProgressMonitor monitor) throws CoreException {
+  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);
+      workspace.run(new BatchOperation(action), rule, IWorkspace.AVOID_UPDATE, monitor);
     }
   }
-  
+
   /**
-        * Returns the template context type registry for the java plugin.
-        * 
-        * @return the template context type registry for the java plugin
-        * @since 3.0
-        */
-       public ContextTypeRegistry getTemplateContextRegistry() {
-               if (fContextTypeRegistry == null) {
-                       fContextTypeRegistry= new ContributionContextTypeRegistry();
-                       
-                       fContextTypeRegistry.addContextType(new JavaContextType());
-                       fContextTypeRegistry.addContextType(new JavaDocContextType());
-                       fContextTypeRegistry.addContextType(new HTMLContextType());
-               }
-
-               return fContextTypeRegistry;
-       }
-       
-       /**
-        * Returns the template store for the java editor templates.
-        * 
-        * @return the template store for the java editor templates
-        * @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;
-       }
+   * Returns the template context type registry for the java plugin.
+   * 
+   * @return the template context type registry for the java plugin
+   * @since 3.0
+   */
+  public ContextTypeRegistry getTemplateContextRegistry() {
+    if (fContextTypeRegistry == null) {
+      fContextTypeRegistry = new ContributionContextTypeRegistry();
 
-       /**
-        * Returns the template context type registry for the code generation
-        * templates.
-        * 
-        * @return the template context type registry for the code generation
-        *         templates
-        * @since 3.0
-        */
-       public ContextTypeRegistry getCodeTemplateContextRegistry() {
-               if (fCodeTemplateContextTypeRegistry == null) {
-                       fCodeTemplateContextTypeRegistry= new ContributionContextTypeRegistry();
-                       
-                       CodeTemplateContextType.registerContextTypes(fCodeTemplateContextTypeRegistry);
-               }
-
-               return fCodeTemplateContextTypeRegistry;
-       }
-       
-       /**
-        * Returns the template store for the code generation templates.
-        * 
-        * @return the template store for the code generation templates
-        * @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;
-       }
+      fContextTypeRegistry.addContextType(new JavaContextType());
+      fContextTypeRegistry.addContextType(new JavaDocContextType());
+      fContextTypeRegistry.addContextType(new HTMLContextType());
+    }
+
+    return fContextTypeRegistry;
+  }
+
+  /**
+   * Returns the template store for the java editor templates.
+   * 
+   * @return the template store for the java editor templates
+   * @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;
+  }
+
+  /**
+   * Returns the template context type registry for the code generation templates.
+   * 
+   * @return the template context type registry for the code generation templates
+   * @since 3.0
+   */
+  public ContextTypeRegistry getCodeTemplateContextRegistry() {
+    if (fCodeTemplateContextTypeRegistry == null) {
+      fCodeTemplateContextTypeRegistry = new ContributionContextTypeRegistry();
+
+      CodeTemplateContextType.registerContextTypes(fCodeTemplateContextTypeRegistry);
+    }
+
+    return fCodeTemplateContextTypeRegistry;
+  }
+
+  /**
+   * Returns the template store for the code generation templates.
+   * 
+   * @return the template store for the code generation templates
+   * @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;
+  }
 }
\ No newline at end of file