Fixed some bugs in the syntax editor preference page

[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpeclipse / phpeditor / PHPSourceViewerConfiguration.java

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPSourceViewerConfiguration.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPSourceViewerConfiguration.java
index c8e1090..1156dee 100644 (file)
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPSourceViewerConfiguration.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpeclipse/phpeditor/PHPSourceViewerConfiguration.java
@@ -14,7 +14,6 @@ package net.sourceforge.phpeclipse.phpeditor;
 import java.util.Vector;
 
 import net.sourceforge.phpdt.internal.ui.text.ContentAssistPreference;
-import net.sourceforge.phpdt.internal.ui.text.JavaColorManager;
 import net.sourceforge.phpdt.internal.ui.text.java.JavaFormattingStrategy;
 import net.sourceforge.phpdt.internal.ui.text.phpdoc.PHPDocCompletionProcessor;
 import net.sourceforge.phpdt.ui.PreferenceConstants;
@@ -29,6 +28,7 @@ import net.sourceforge.phpeclipse.phpeditor.php.PHPCompletionProcessor;
 import net.sourceforge.phpeclipse.phpeditor.php.PHPDoubleClickSelector;
 import net.sourceforge.phpeclipse.phpeditor.util.PHPColorProvider;
 
+import org.eclipse.core.resources.IFile;
 import org.eclipse.jface.preference.IPreferenceStore;
 import org.eclipse.jface.text.DefaultAutoIndentStrategy;
 import org.eclipse.jface.text.IAutoIndentStrategy;
@@ -52,28 +52,27 @@ import org.eclipse.jface.text.rules.Token;
 import org.eclipse.jface.text.source.IAnnotationHover;
 import org.eclipse.jface.text.source.ISourceViewer;
 import org.eclipse.jface.text.source.SourceViewerConfiguration;
+import org.eclipse.ui.IFileEditorInput;
 
 /**
  * Configuration for an <code>SourceViewer</code> which shows PHP code. 
  */
 public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
 
-       
   /** 
    * Preference key used to look up display tab width.
    * 
    * @since 2.0
    */
-  public final static String PREFERENCE_TAB_WIDTH= PreferenceConstants.EDITOR_TAB_WIDTH;
+  public final static String PREFERENCE_TAB_WIDTH = PreferenceConstants.EDITOR_TAB_WIDTH;
   /** 
    * Preference key for inserting spaces rather than tabs.
    * 
    * @since 2.0
    */
-  public final static String SPACES_FOR_TABS= PreferenceConstants.EDITOR_SPACES_FOR_TABS;
+  public final static String SPACES_FOR_TABS = PreferenceConstants.EDITOR_SPACES_FOR_TABS;
 
-
-//  public static final String HTML_DEFAULT = IPHPPartitionScannerConstants.HTML;
+  //  public static final String HTML_DEFAULT = IPHPPartitionScannerConstants.HTML;
   //IDocument.DEFAULT_CONTENT_TYPE;
 
   private JavaTextTools fJavaTextTools;
@@ -186,7 +185,7 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
   protected RuleBasedScanner getCodeScanner() {
     return fJavaTextTools.getCodeScanner();
   }
-  
+
   /**
    * Returns the HTML source code scanner for this configuration.
    *
@@ -195,7 +194,7 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
   protected RuleBasedScanner getHTMLScanner() {
     return fJavaTextTools.getHTMLScanner();
   }
-  
+
   /**
    * Returns the PHPDoc source code scanner for this configuration.
    *
@@ -204,20 +203,20 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
   protected RuleBasedScanner getPHPDocScanner() {
     return fJavaTextTools.getJavaDocScanner();
   }
-  
+
   /* (non-Javadoc)
    * Method declared on SourceViewerConfiguration
    */
   public String[] getConfiguredContentTypes(ISourceViewer sourceViewer) {
     return new String[] {
       IPHPPartitionScannerConstants.HTML,
-        IPHPPartitionScannerConstants.HTML_MULTILINE_COMMENT,
-        IPHPPartitionScannerConstants.PHP,
-        IPHPPartitionScannerConstants.PHP_MULTILINE_COMMENT,
-        IPHPPartitionScannerConstants.CSS,
-        IPHPPartitionScannerConstants.CSS_MULTILINE_COMMENT,
-        IPHPPartitionScannerConstants.JAVASCRIPT,
-        IPHPPartitionScannerConstants.JS_MULTILINE_COMMENT,
+      IPHPPartitionScannerConstants.HTML_MULTILINE_COMMENT,
+      IPHPPartitionScannerConstants.PHP,
+      IPHPPartitionScannerConstants.PHP_MULTILINE_COMMENT,
+      IPHPPartitionScannerConstants.CSS,
+      IPHPPartitionScannerConstants.CSS_MULTILINE_COMMENT,
+      IPHPPartitionScannerConstants.JAVASCRIPT,
+      IPHPPartitionScannerConstants.JS_MULTILINE_COMMENT,
       IDocument.DEFAULT_CONTENT_TYPE };
   }
 
@@ -289,10 +288,10 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
 
     // prefix[0] is either '\t' or ' ' x tabWidth, depending on useSpaces
 
-    final IPreferenceStore preferences= PHPeclipsePlugin.getDefault().getPreferenceStore();
-    int tabWidth= preferences.getInt(PHPCore.FORMATTER_TAB_SIZE);
-    boolean useSpaces= getPreferenceStore().getBoolean(SPACES_FOR_TABS);
-    
+    final IPreferenceStore preferences = PHPeclipsePlugin.getDefault().getPreferenceStore();
+    int tabWidth = preferences.getInt(PHPCore.FORMATTER_TAB_SIZE);
+    boolean useSpaces = getPreferenceStore().getBoolean(SPACES_FOR_TABS);
+
     for (int i = 0; i <= tabWidth; i++) {
       StringBuffer prefix = new StringBuffer();
 
@@ -322,7 +321,7 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
    */
   public IPresentationReconciler getPresentationReconciler(ISourceViewer sourceViewer) {
     //  PHPColorProvider provider = PHPEditorEnvironment.getPHPColorProvider();
-    JavaColorManager provider = PHPEditorEnvironment.getPHPColorProvider();
+    //    JavaColorManager provider = PHPEditorEnvironment.getPHPColorProvider();
     PresentationReconciler reconciler = new PresentationReconciler();
 
     DefaultDamagerRepairer dr = new DefaultDamagerRepairer(getHTMLScanner());
@@ -345,7 +344,8 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
     reconciler.setDamager(dr, IPHPPartitionScannerConstants.JS_MULTILINE_COMMENT);
     reconciler.setRepairer(dr, IPHPPartitionScannerConstants.JS_MULTILINE_COMMENT);
     dr =
-      new DefaultDamagerRepairer(new SingleTokenScanner(new TextAttribute(provider.getColor(PHPColorProvider.MULTI_LINE_COMMENT))));
+      new DefaultDamagerRepairer(
+        new SingleTokenScanner(new TextAttribute(fJavaTextTools.getColorManager().getColor(PHPColorProvider.MULTI_LINE_COMMENT))));
     reconciler.setDamager(dr, IPHPPartitionScannerConstants.HTML_MULTILINE_COMMENT);
     reconciler.setRepairer(dr, IPHPPartitionScannerConstants.HTML_MULTILINE_COMMENT);
 
@@ -371,6 +371,12 @@ public class PHPSourceViewerConfiguration extends SourceViewerConfiguration {
    * Method declared on SourceViewerConfiguration
    */
   public ITextHover getTextHover(ISourceViewer sourceViewer, String contentType) {
-    return new PHPTextHover();
+    try {
+      IFile f = ((IFileEditorInput) fEditor.getEditorInput()).getFile();
+      return new PHPTextHover(f.getProject());
+    } catch (NullPointerException e) {
+       // this exception occurs, if getTextHover is called by preference pages !
+    }
+    return new PHPTextHover(null);
   }
 }