fix #774 infinite loop in net.sourceforge.phpeclipse.builder.IdentifierIndexManager...
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpdt / internal / ui / text / template / BuiltInEngine.java
index 68d6a36..1ab738f 100644 (file)
@@ -6,38 +6,46 @@ package net.sourceforge.phpdt.internal.ui.text.template;
 
 import java.util.ArrayList;
 
-import net.sourceforge.phpdt.internal.corext.template.ContextType;
-import net.sourceforge.phpdt.internal.corext.template.java.CompilationUnitContext;
-import net.sourceforge.phpdt.internal.corext.template.java.CompilationUnitContextType;
-import net.sourceforge.phpdt.internal.ui.PHPUiImages;
-import net.sourceforge.phpdt.internal.ui.text.java.IJavaCompletionProposal;
+import net.sourceforge.phpdt.core.ICompilationUnit;
+import net.sourceforge.phpdt.internal.corext.template.php.CompilationUnitContextType;
+import net.sourceforge.phpdt.internal.corext.template.php.JavaContext;
+import net.sourceforge.phpdt.internal.corext.template.php.JavaContextType;
+import net.sourceforge.phpdt.internal.ui.text.java.IPHPCompletionProposal;
+import net.sourceforge.phpeclipse.PHPeclipsePlugin;
+import net.sourceforge.phpeclipse.phpeditor.php.PHPElement;
+import net.sourceforge.phpeclipse.phpeditor.php.PHPFunction;
+
+import org.eclipse.jface.text.BadLocationException;
 import org.eclipse.jface.text.IDocument;
 import org.eclipse.jface.text.IRegion;
 import org.eclipse.jface.text.ITextViewer;
 import org.eclipse.jface.text.Region;
-//import org.eclipse.jdt.internal.ui.text.link.LinkedPositionManager;
+import org.eclipse.swt.graphics.Point;
 
 public class BuiltInEngine {
 
        /** The context type. */
-       private ContextType fContextType;
+       private JavaContextType fContextType;
+
        /** The result proposals. */
-       private ArrayList fProposals= new ArrayList();
+       private ArrayList fProposals = new ArrayList();
 
        /**
-        * Creates the template engine for a particular context type.
-        * See <code>TemplateContext</code> for supported context types.
+        * Creates the template engine for a particular context type. See
+        * <code>TemplateContext</code> for supported context types.
         */
-       public BuiltInEngine(ContextType contextType) {
-       //      Assert.isNotNull(contextType);
-               fContextType= contextType;
+       public BuiltInEngine(JavaContextType contextType) {
+               // Assert.isNotNull(contextType);
+               fContextType = contextType;
        }
 
        /**
         * Empties the collector.
         * 
-        * @param viewer the text viewer  
-        * @param unit   the compilation unit (may be <code>null</code>)
+        * @param viewer
+        *            the text viewer
+        * @param unit
+        *            the compilation unit (may be <code>null</code>)
         */
        public void reset() {
                fProposals.clear();
@@ -46,45 +54,72 @@ public class BuiltInEngine {
        /**
         * Returns the array of matching templates.
         */
-       public IJavaCompletionProposal[] getResults() {
-               return (IJavaCompletionProposal[]) fProposals.toArray(new IJavaCompletionProposal[fProposals.size()]);
+       public IPHPCompletionProposal[] getResults() {
+               return (IPHPCompletionProposal[]) fProposals
+                               .toArray(new IPHPCompletionProposal[fProposals.size()]);
        }
 
        /**
-        * Inspects the context of the compilation unit around <code>completionPosition</code>
-        * and feeds the collector with proposals.
-        * @param viewer the text viewer
-        * @param completionPosition the context position in the document of the text viewer
-        * @param compilationUnit the compilation unit (may be <code>null</code>)
+        * Inspects the context of the compilation unit around
+        * <code>completionPosition</code> and feeds the collector with proposals.
+        * 
+        * @param viewer
+        *            the text viewer
+        * @param completionPosition
+        *            the context position in the document of the text viewer
+        * @param compilationUnit
+        *            the compilation unit (may be <code>null</code>)
         */
-       public void complete(ITextViewer viewer, int completionPosition, Object[] identifiers)
-  //,ICompilationUnit compilationUnit)
-       //hrows JavaModelException
+       public void complete(ITextViewer viewer, int completionPosition,
+                       ArrayList identifiers, ICompilationUnit compilationUnit)
+       // hrows JavaModelException
        {
-           IDocument document= viewer.getDocument();
-           
+               IDocument document = viewer.getDocument();
+
                // prohibit recursion
-//             if (LinkedPositionManager.hasActiveManager(document))
-//                     return;
+               // if (LinkedPositionManager.hasActiveManager(document))
+               // return;
 
                if (!(fContextType instanceof CompilationUnitContextType))
                        return;
-               
-               ((CompilationUnitContextType) fContextType).setContextParameters(document, completionPosition);//mpilationUnit);
-               CompilationUnitContext context= (CompilationUnitContext) fContextType.createContext();
-               int start= context.getStart();
-               int end= context.getEnd();
-               IRegion region= new Region(start, end - start);
+               Point selection = viewer.getSelectedRange();
+               // remember selected text
+               String selectedText = null;
+               if (selection.y != 0) {
+                       try {
+                               selectedText = document.get(selection.x, selection.y);
+                       } catch (BadLocationException e) {
+                       }
+               }
+
+               // ((CompilationUnitContextType)
+               // fContextType).setContextParameters(document, completionPosition,
+               // selection.y); //mpilationUnit);
+               // JavaContext context = (JavaContext) fContextType.createContext();
+               JavaContext context = (JavaContext) fContextType.createContext(
+                               document, completionPosition, selection.y, compilationUnit);
+               context.setVariable("selection", selectedText); //$NON-NLS-1$
+               int start = context.getStart();
+               int end = context.getEnd();
+               IRegion region = new Region(start, end - start);
 
-//             Template[] templates= Templates.getInstance().getTemplates();
-    String identifier = null;
-               for (int i= 0; i != identifiers.length; i++) {
-      identifier = (String) identifiers[i];
-                       if (context.canEvaluate(identifier)) {
-                               fProposals.add(new BuiltInProposal(identifier, context, region, viewer, PHPUiImages.get(PHPUiImages.IMG_BUILTIN))); 
-      }
-    }
+               // Template[] templates= Templates.getInstance().getTemplates();
+               String identifier = null;
+               int maxProposals = PHPeclipsePlugin.MAX_PROPOSALS;
+               PHPElement element = null;
+               for (int i = 0; i != identifiers.size(); i++) {
+                       element = (PHPElement) identifiers.get(i);
+                       if (element instanceof PHPFunction) {
+                               identifier = ((PHPFunction) element).getName();
+                               if (context.canEvaluate(identifier)) {
+                                       if (maxProposals-- < 0) {
+                                               return;
+                                       }
+                                       fProposals.add(new BuiltInProposal(identifier,
+                                                       (PHPFunction) element, context, region, viewer));
+                               }
+                       }
+               }
        }
 
 }
-