Fix #1368081: Variable not initialized does not work for array access
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpeclipse / wizards / PHPFileWizard.java
index ea1a3fb..971f3b6 100644 (file)
@@ -1,16 +1,16 @@
 package net.sourceforge.phpeclipse.wizards;
 
 /**********************************************************************
-Copyright (c) 2000, 2002 IBM Corp. and others.
-All rights reserved. This program and the accompanying materials
-are 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
+ Copyright (c) 2000, 2002 IBM Corp. and others.
+ All rights reserved. This program and the accompanying materials
+ are 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
 
-Contributors:
-    IBM Corporation - Initial implementation
-    Klaus Hartlage - www.eclipseproject.de
-**********************************************************************/
+ Contributors:
+ IBM Corporation - Initial implementation
+ www.phpeclipse.de
+ **********************************************************************/
 
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
@@ -47,18 +47,17 @@ import org.eclipse.ui.PlatformUI;
 import org.eclipse.ui.ide.IDE;
 
 /**
- * This wizard creates one file with the extension
- * "php". 
+ * This wizard creates one file with the extension "php".
  */
 public class PHPFileWizard extends Wizard implements INewWizard {
 
   private PHPFileWizardPage page;
+
   private ISelection selection;
 
   // the name of the file to create
-  private String fileName;
+  private String fFileName;
 
-  
   public PHPFileWizard() {
     super();
     setNeedsProgressMonitor(true);
@@ -73,10 +72,8 @@ public class PHPFileWizard extends Wizard implements INewWizard {
   }
 
   /**
-   * This method is called when 'Finish' button is pressed in
-   * the wizard. 
-   * We will create an operation and run it
-   * using wizard as execution context.
+   * This method is called when 'Finish' button is pressed in the wizard. We will create an operation and run it using wizard as
+   * execution context.
    */
   public boolean performFinish() {
     final String containerName = page.getContainerName();
@@ -105,9 +102,8 @@ public class PHPFileWizard extends Wizard implements INewWizard {
   }
 
   /**
-   * The worker method. It will find the container, create the
-   * file if missing or just replace its contents, and open
-   * the editor on the newly created file.
+   * The worker method. It will find the container, create the file if missing or just replace its contents, and open the editor on
+   * the newly created file.
    */
   private void doFinish(String containerName, String fileName, IProgressMonitor monitor) throws CoreException {
     // create a sample file
@@ -124,7 +120,7 @@ public class PHPFileWizard extends Wizard implements INewWizard {
     try {
       InputStream stream;
       if (className == null) {
-        stream = openContentStream();
+        stream = openContentStream(fileName);
       } else {
         stream = openContentStreamClass(className);
       }
@@ -142,7 +138,7 @@ public class PHPFileWizard extends Wizard implements INewWizard {
       public void run() {
         IWorkbenchPage page = PlatformUI.getWorkbench().getActiveWorkbenchWindow().getActivePage();
         try {
-          IDE.openEditor(page,file,true);
+          IDE.openEditor(page, file, true);
         } catch (PartInitException e) {
         }
       }
@@ -152,21 +148,28 @@ public class PHPFileWizard extends Wizard implements INewWizard {
 
   /**
    * Check if the filename is like this anyname.class.php
-   * @param fileName the filename
+   * 
+   * @param fFileName
+   *          the filename
    * @return the anyname or null
    */
   private static final String getClassName(final String fileName) {
     final int lastDot = fileName.lastIndexOf('.');
-    if (lastDot == -1) return null;
-    final int precLastDot = fileName.lastIndexOf('.',lastDot-1);
-    if (precLastDot == -1) return null;
-    if (!fileName.substring(precLastDot+1,lastDot).toUpperCase().equals("CLASS")) return null;
-    return fileName.substring(0,precLastDot);
+    if (lastDot == -1)
+      return null;
+    final int precLastDot = fileName.lastIndexOf('.', lastDot - 1);
+    if (precLastDot == -1)
+      return null;
+    if (!fileName.substring(precLastDot + 1, lastDot).toUpperCase().equals("CLASS"))
+      return null;
+    return fileName.substring(0, precLastDot);
   }
 
   /**
    * We will initialize file contents for a class
-   * @param className the classname
+   * 
+   * @param className
+   *          the classname
    */
   private InputStream openContentStreamClass(final String className) {
     StringBuffer contents = new StringBuffer("<?php\n\n");
@@ -183,37 +186,21 @@ public class PHPFileWizard extends Wizard implements INewWizard {
   /**
    * We will initialize file contents with a sample text.
    */
-  private InputStream openContentStream() {
-       try {
-    Template template= PHPeclipsePlugin.getDefault().getCodeTemplateStore().findTemplate(CodeTemplateContextType.NEWTYPE);
-       if (template == null) {
-               return null;
-       }
-       String lineDelimiter = System.getProperty("line.separator", "\n"); //$NON-NLS-1$ //$NON-NLS-2$
-       CodeTemplateContext context= new CodeTemplateContext(template.getContextTypeId(), null, lineDelimiter);
-//     context.setCompilationUnitVariables(cu);
-
-
-    return new ByteArrayInputStream(StubUtility.evaluateTemplate(context, template).getBytes());
-  } catch (CoreException e) {
-    e.printStackTrace();
-    return null;
-  }
-    
-//    StringBuffer contents = new StringBuffer("<?php\n\n");
-//    contents.append("function f0() {\n\n");
-//    contents.append("}\n\n");
-//    contents.append("function f1() {\n\n");
-//    contents.append("}\n\n");
-//    contents.append("switch($func) {\n");
-//    contents.append("    case \"f1\":\n");
-//    contents.append("    f1();\n");
-//    contents.append("    break;\n\n");
-//    contents.append("    default:\n");
-//    contents.append("    f0();\n");
-//    contents.append("    break;\n\n");
-//    contents.append("}\n\n?>");
-//    return new ByteArrayInputStream(contents.toString().getBytes());
+  private InputStream openContentStream(String fileName) {
+    try {
+      Template template = PHPeclipsePlugin.getDefault().getCodeTemplateStore().findTemplate(CodeTemplateContextType.NEWTYPE);
+      if (template == null) {
+        return null;
+      }
+      String lineDelimiter = System.getProperty("line.separator", "\n"); //$NON-NLS-1$ //$NON-NLS-2$
+      CodeTemplateContext context = new CodeTemplateContext(template.getContextTypeId(), null, lineDelimiter);
+      context.setFileNameVariable(fileName);
+      return new ByteArrayInputStream(StubUtility.evaluateTemplate(context, template).getBytes());
+    } catch (CoreException e) {
+      e.printStackTrace();
+      return null;
+    }
+
   }
 
   private void throwCoreException(String message) throws CoreException {
@@ -222,8 +209,8 @@ public class PHPFileWizard extends Wizard implements INewWizard {
   }
 
   /**
-   * We will accept the selection in the workbench to see if
-   * we can initialize from it.
+   * We will accept the selection in the workbench to see if we can initialize from it.
+   * 
    * @see IWorkbenchWizard#init(IWorkbench, IStructuredSelection)
    */
   public void init(IWorkbench workbench, IStructuredSelection selection) {
@@ -231,10 +218,9 @@ public class PHPFileWizard extends Wizard implements INewWizard {
   }
 
   /**
-   * Sets the name of the file to create
-   * (used to set the class name in the new file)
+   * Sets the name of the file to create (used to set the class name in the new file)
    */
   public void setFileName(String name) {
-    fileName = name;
+    fFileName = name;
   }
 }
\ No newline at end of file