Defined a limit for code completion list entries PHPeclipsePlugin.MAX_PROPOSALS

[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpdt / internal / ui / text / template / DeclarationProposal.java

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/ui/text/template/DeclarationProposal.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/ui/text/template/DeclarationProposal.java
index c448d44..41c9f61 100644 (file)
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/ui/text/template/DeclarationProposal.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/ui/text/template/DeclarationProposal.java
@@ -4,13 +4,16 @@
  */
 package net.sourceforge.phpdt.internal.ui.text.template;
 
+import net.sourceforge.phpdt.internal.corext.phpdoc.PHPDocUtil;
 import net.sourceforge.phpdt.internal.corext.template.TemplateContext;
 import net.sourceforge.phpdt.internal.corext.template.TemplateMessages;
 import net.sourceforge.phpdt.internal.corext.template.php.PHPUnitContext;
+import net.sourceforge.phpdt.internal.ui.PHPUiImages;
 import net.sourceforge.phpdt.internal.ui.text.java.IPHPCompletionProposal;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionManager;
 import net.sourceforge.phpdt.internal.ui.text.link.LinkedPositionUI;
 import net.sourceforge.phpeclipse.PHPeclipsePlugin;
+import net.sourceforge.phpeclipse.builder.PHPIdentifierLocation;
 
 import org.eclipse.core.runtime.CoreException;
 import org.eclipse.jface.dialogs.MessageDialog;
@@ -32,10 +35,11 @@ import org.eclipse.swt.widgets.Shell;
 public class DeclarationProposal implements IPHPCompletionProposal {
 
   private final String fTemplate;
+  private final PHPIdentifierLocation fLocation;
   private final TemplateContext fContext;
   private final ITextViewer fViewer;
-  private final Image fImage_fun;
-  private final Image fImage_var;
+  //  private final Image fImage_fun;
+  //  private final Image fImage_var;
   private final IRegion fRegion;
 
   //private TemplateBuffer fTemplateBuffer;
@@ -48,17 +52,20 @@ public class DeclarationProposal implements IPHPCompletionProposal {
    * @param context   the context in which the template was requested.
    * @param image     the icon of the proposal.
    */
-  public DeclarationProposal(String template, TemplateContext context, IRegion region, ITextViewer viewer, Image image_fun, Image image_var) {
-    //         Assert.isNotNull(template);
-    //         Assert.isNotNull(context);
-    //         Assert.isNotNull(region);
-    //         Assert.isNotNull(viewer);
-
+  public DeclarationProposal(
+    String template,
+    PHPIdentifierLocation location,
+    TemplateContext context,
+    IRegion region,
+    ITextViewer viewer) {
+    //    Image image_fun,
+    //    Image image_var) {
     fTemplate = template;
+    fLocation = location;
     fContext = context;
     fViewer = viewer;
-       fImage_fun= image_fun;
-    fImage_var= image_var;
+    //    fImage_fun = image_fun;
+    //    fImage_var = image_var;
     fRegion = region;
   }
 
@@ -74,7 +81,7 @@ public class DeclarationProposal implements IPHPCompletionProposal {
       int end = fRegion.getOffset() + fRegion.getLength();
 
       // insert template string
-    //  String templateString = fTemplate; // fTemplateBuffer.getString();     
+      //  String templateString = fTemplate; // fTemplateBuffer.getString();   
       document.replace(start, end - start, fTemplate);
 
       // translate positions
@@ -94,8 +101,8 @@ public class DeclarationProposal implements IPHPCompletionProposal {
       //                       }
 
       LinkedPositionUI editor = new LinkedPositionUI(fViewer, manager);
-      editor.setFinalCaretOffset(fTemplate.length()+start);
-   //   editor.setFinalCaretOffset(getCaretOffset(fTemplateBuffer) + start);
+      editor.setFinalCaretOffset(fTemplate.length() + start);
+      //   editor.setFinalCaretOffset(getCaretOffset(fTemplateBuffer) + start);
       editor.enter();
 
       fSelectedRegion = editor.getSelectedRegion();
@@ -110,17 +117,17 @@ public class DeclarationProposal implements IPHPCompletionProposal {
     //     }       
   }
 
-//  private static int getCaretOffset(TemplateBuffer buffer) {
-//    TemplatePosition[] variables = buffer.getVariables();
-//    for (int i = 0; i != variables.length; i++) {
-//      TemplatePosition variable = variables[i];
-//
-//      if (variable.getName().equals(JavaTemplateMessages.getString("GlobalVariables.variable.name.cursor"))) //$NON-NLS-1$
-//        return variable.getOffsets()[0];
-//    }
-//
-//    return buffer.getString().length();
-//  }
+  //  private static int getCaretOffset(TemplateBuffer buffer) {
+  //    TemplatePosition[] variables = buffer.getVariables();
+  //    for (int i = 0; i != variables.length; i++) {
+  //      TemplatePosition variable = variables[i];
+  //
+  //      if (variable.getName().equals(JavaTemplateMessages.getString("GlobalVariables.variable.name.cursor"))) //$NON-NLS-1$
+  //        return variable.getOffsets()[0];
+  //    }
+  //
+  //    return buffer.getString().length();
+  //  }
 
   /*
    * @see ICompletionProposal#getSelection(IDocument)
@@ -134,16 +141,11 @@ public class DeclarationProposal implements IPHPCompletionProposal {
    * @see ICompletionProposal#getAdditionalProposalInfo()
    */
   public String getAdditionalProposalInfo() {
-    //     try {
-    //                 if (fTemplateBuffer == null)
-    //                         fTemplateBuffer= fContext.evaluate(fTemplate);
-
-    return textToHTML(fTemplate); // fTemplateBuffer.getString());
-
-    //     } catch (CoreException e) {
-    //                 handleException(e);                 
-    //                 return null;
-    //     }
+    StringBuffer hoverInfoBuffer = new StringBuffer();
+    String workspaceLocation = PHPeclipsePlugin.getWorkspace().getRoot().getLocation().toString();
+    String filename = workspaceLocation + fLocation.getFilename();
+    PHPDocUtil.appendPHPDoc(hoverInfoBuffer, filename, fLocation);
+    return textToHTML(hoverInfoBuffer.toString());
   }
 
   /*
@@ -158,10 +160,19 @@ public class DeclarationProposal implements IPHPCompletionProposal {
    * @see ICompletionProposal#getImage()
    */
   public Image getImage() {
-    if (fTemplate.charAt(0)=='$') {
-      return fImage_var;
+    switch (fLocation.getType()) {
+      case PHPIdentifierLocation.FUNCTION :
+        return PHPUiImages.get(PHPUiImages.IMG_FUN);
+                       case PHPIdentifierLocation.CLASS :
+                               return PHPUiImages.get(PHPUiImages.IMG_CLASS);
+                       case PHPIdentifierLocation.METHOD :
+                               return PHPUiImages.get(PHPUiImages.IMG_FUN);
+                       case PHPIdentifierLocation.DEFINE :
+                               return PHPUiImages.get(PHPUiImages.IMG_DEFINE);
+                       case PHPIdentifierLocation.VARIABLE :
+                               return PHPUiImages.get(PHPUiImages.IMG_VAR);
     }
-               return fImage_fun;
+    return PHPUiImages.get(PHPUiImages.IMG_FUN);
   }
 
   /*