UIB : Modifying for static analyzer 34/18234/1
authorjeonghwan kim <jeonghwan0613.kim@samsung.com>
Wed, 19 Mar 2014 06:48:21 +0000 (15:48 +0900)
committerjeonghwan kim <jeonghwan0613.kim@samsung.com>
Wed, 19 Mar 2014 06:48:21 +0000 (15:48 +0900)
Change-Id: I1abf143d8b903fb7a1ef34f718164a64d7aa99fd
Signed-off-by: jeonghwan kim <jeonghwan0613.kim@samsung.com>
16 files changed:
org.tizen.webuibuilder/src/org/tizen/webuibuilder/gef/dnd/model/ResourceTemplate.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/gef/dnd/model/SnippetTemplate.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/gef/dnd/model/WidgetTemplate.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/pagetemplate/loader/TemplateMetadataLoader.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/pagetemplate/ui/wizards/NewPageWizard.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/pagetemplate/ui/wizards/NewPageWizardFirstPage.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/pagetemplate/ui/wizards/TemplateComposite.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/pagetemplate/ui/wizards/TemplateGroupComposite.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/snippet/loader/DescriptorLoader.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/snippet/loader/MetadataLoader.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/snippet/ui/wizards/ExportWizardFirstPage.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/ui/editor/ComponentsCreator.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/ui/views/resources/ResourcesPage.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/ui/views/resources/model/Resource.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/utility/ImageUtils.java
org.tizen.webuibuilder/src/org/tizen/webuibuilder/utility/PageManager.java

index 3189bef..6d8a852 100644 (file)
@@ -29,8 +29,8 @@ import org.tizen.webuibuilder.ui.views.resources.model.Resource;
 
 
 public class ResourceTemplate extends AbstractTemplate {
-    private Resource resource;
-    private String path;
+    private Resource resource = null;
+    private String path = null;
 
     public ResourceTemplate(Resource resource) {
         this.resource = resource;
index cb77626..c1ea055 100644 (file)
@@ -31,13 +31,15 @@ import org.tizen.webuibuilder.snippet.model.ISnippet;
 
 
 public class SnippetTemplate extends AbstractTemplate {
-    private ISnippet snippet;
+    private ISnippet snippet = null;
+    private DescriptorManager descriptorManager = null;
 
     public SnippetTemplate(ISnippet snippet, DescriptorManager descriptorManager) {
         this.snippet = snippet;
+        this.descriptorManager = descriptorManager;
         setType(SNIPPET);
         SnippetCreationFactory factory =
-                new SnippetCreationFactory(this.snippet, descriptorManager);
+                new SnippetCreationFactory();
         setFactory(factory);
     }
 
@@ -46,14 +48,6 @@ public class SnippetTemplate extends AbstractTemplate {
     }
 
     private class SnippetCreationFactory implements CreationFactory {
-        private ISnippet snippet;
-        private DescriptorManager descriptorManager;
-
-        public SnippetCreationFactory(ISnippet snippet, DescriptorManager descriptorManager) {
-            this.snippet = snippet;
-            this.descriptorManager = descriptorManager;
-        }
-
         /*
          * (non-Javadoc)
          * 
@@ -74,7 +68,7 @@ public class SnippetTemplate extends AbstractTemplate {
          */
         @Override
         public Object getObjectType() {
-            return SNIPPET;
+            return getType();
         }
     }
 }
index be52adb..edc9f4e 100644 (file)
@@ -25,10 +25,8 @@ package org.tizen.webuibuilder.gef.dnd.model;
 
 import java.util.List;
 
-import org.eclipse.core.resources.IProject;
 import org.eclipse.gef.requests.CreationFactory;
 import org.tizen.webuibuilder.BuilderConstants;
-import org.tizen.webuibuilder.animator.model.part.AnimatorPagePart;
 import org.tizen.webuibuilder.model.Part;
 import org.tizen.webuibuilder.model.app.AppManager;
 import org.tizen.webuibuilder.model.descriptors.DescriptorManager;
@@ -40,11 +38,15 @@ import org.tizen.webuibuilder.model.descriptors.WidgetPartDescriptor;
 
 
 public class WidgetTemplate extends AbstractTemplate {
-       
-    public WidgetTemplate(Object ownerRef, String descriptorId, AppManager appManager) {
+    private String descriptorId = null;
+    private AppManager appManager = null;
+
+    public WidgetTemplate(String descriptorId, AppManager appManager) {
+        this.descriptorId = descriptorId;
+        this.appManager = appManager;
+
         setType(WIDGET);
-        PartCreationFactory factory =
-                new PartCreationFactory(ownerRef, descriptorId, appManager);
+        PartCreationFactory factory = new PartCreationFactory();
         setFactory(factory);
     }
 
@@ -52,26 +54,6 @@ public class WidgetTemplate extends AbstractTemplate {
      * A factory used to create a new {@link Part}.
      */
     public class PartCreationFactory implements CreationFactory {
-
-        private Object ownerRef;
-        private String descriptorId;
-       private AppManager appManager = null;           
-
-        /**
-         * Constructor.
-         * 
-         * @param descriptorId
-         *            a descriptor id of a {@link Part}
-         * @param project
-         *            an {@link IProject}
-         */
-        public PartCreationFactory(Object ownerRef, String descriptorId,
-                AppManager appManager) {
-            this.ownerRef = ownerRef;
-            this.descriptorId = descriptorId;
-            this.appManager = appManager;
-        }
-
         /**
          * Gets a {@link DescriptorManager}
          * 
@@ -96,26 +78,15 @@ public class WidgetTemplate extends AbstractTemplate {
             }
 
             PartDescriptor desc = manager.getPartDescriptor(descriptorId);
-            // IProject project = manager.getProject();
             if (desc != null) {
                 if (desc instanceof PagePartDescriptor) {
-                       part = appManager.getPartFactory().createPart("tizen.page");
-//                    part = new Part(ownerRef, (PagePartDescriptor) desc);
-//                    part.setPartId(part.getUniquePartId());
-//                    ((PagePart)part).setTransition("none");
-//                    part.setPagePart(true);
-                    // part.setProject(project);
-                    // part.setDescriptorManager(descriptorManager);
+                    part = appManager.getPartFactory().createPart("tizen.page");
                     setDefaultContent(part, desc);
                 } else if (desc instanceof WidgetPartDescriptor) {
-                       //part = appManager.getPartFactory().createPart("tizen.widget");
-                       part = appManager.getPartFactory().createPart(desc.getDescriptorId());
-//                    part = new WidgetPart(ownerRef, (WidgetPartDescriptor) desc);
-//                    part.setPartId(part.getUniquePartId());
-                    // part.setProject(project);
-                    // part.setDescriptorManager(descriptorManager);
+                    part = appManager.getPartFactory().createPart(desc.getDescriptorId());
                     setDefaultContent(part, desc);
-                } // else : return null;
+                } else {
+                }
             }
 
             return part;
@@ -128,22 +99,7 @@ public class WidgetTemplate extends AbstractTemplate {
          */
         @Override
         public Object getObjectType() {
-            // DescriptorManager manager = getDescriptorManager();
-            // if (manager == null) {
-            // return null;
-            // }
-            //
-            // PartDescriptor desc = manager.getPartDescriptor(descriptorId);
-            //
-            // if (desc == null) {
-            // return null;
-            // } else if (desc instanceof PagePartDescriptor) {
-            // return PagePart.class;
-            // } else if (desc instanceof WidgetPartDescriptor) {
-            // return WidgetPart.class;
-            // } // else : return null;
-            //
-            return WIDGET;
+            return getType();
         }
 
         /**
index 4c9c89d..29da603 100644 (file)
@@ -37,8 +37,6 @@ import org.tizen.webuibuilder.snippet.model.jaxb.Metadata;
 
 
 public class TemplateMetadataLoader extends MetadataLoader {
-    public Metadata metadata;
-
     public TemplateMetadataLoader() {
         super();
     }
index a9a6ebd..050a57a 100644 (file)
@@ -37,10 +37,10 @@ import org.tizen.webuibuilder.utility.ResourceManager;
 
 
 public class NewPageWizard extends Wizard implements INewWizard {
-    private AppManager appManager;
-    private ISelection fSelection;
-    private NewPageWizardFirstPage fFirstPage;
-    private NewPageWizardSecondPage fSecondPage;
+    private AppManager appManager = null;
+    private ISelection fSelection = null;
+//    private NewPageWizardFirstPage fFirstPage = null;
+    private NewPageWizardSecondPage fSecondPage = null;
 
     /*
      * (non-Javadoc)
index 24b708a..3ad6272 100644 (file)
@@ -137,7 +137,7 @@ public class NewPageWizardFirstPage extends WizardPage implements ModifyListener
         }
     }
 
-    private class ProjectViewerLabelProvider implements ILabelProvider {
+    private static  class ProjectViewerLabelProvider implements ILabelProvider {
         /*
          * (non-Javadoc)
          * 
index cf7ae84..518e327 100644 (file)
@@ -174,7 +174,7 @@ public class TemplateComposite extends Composite implements ISelectionProvider,
         }
     }
 
-    private class TemplateViewerLabelProvider implements ILabelProvider {
+    private static class TemplateViewerLabelProvider implements ILabelProvider {
         /*
          * (non-Javadoc)
          * 
index 4b95411..3fd63f7 100644 (file)
@@ -117,29 +117,63 @@ public class TemplateGroupComposite extends Composite implements SelectionListen
         fTemplateGroupViewer.removeSelectionChangedListener(this);
     }
 
-    private class TemplateGroupViewerLabelProvider implements ILabelProvider {
+    private static class TemplateGroupViewerLabelProvider implements ILabelProvider {
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.IBaseLabelProvider#addListener(org.eclipse.jface.viewers.
+         * ILabelProviderListener)
+         */
         @Override
         public void addListener(ILabelProviderListener listener) {
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.IBaseLabelProvider#dispose()
+         */
         @Override
         public void dispose() {
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.IBaseLabelProvider#isLabelProperty(java.lang.Object,
+         * java.lang.String)
+         */
         @Override
         public boolean isLabelProperty(Object element, String property) {
             return false;
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see
+         * org.eclipse.jface.viewers.IBaseLabelProvider#removeListener(org.eclipse.jface.viewers
+         * .ILabelProviderListener)
+         */
         @Override
         public void removeListener(ILabelProviderListener listener) {
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.ILabelProvider#getImage(java.lang.Object)
+         */
         @Override
         public Image getImage(Object element) {
             return null;
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.ILabelProvider#getText(java.lang.Object)
+         */
         @Override
         public String getText(Object element) {
             if (!(element instanceof ITemplateGroup)) {
@@ -150,15 +184,32 @@ public class TemplateGroupComposite extends Composite implements SelectionListen
         }
     }
 
-    private class TemplateGroupViewerContentProvider implements IStructuredContentProvider {
+    private static class TemplateGroupViewerContentProvider implements IStructuredContentProvider {
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.IContentProvider#dispose()
+         */
         @Override
         public void dispose() {
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see
+         * org.eclipse.jface.viewers.IContentProvider#inputChanged(org.eclipse.jface.viewers.Viewer,
+         * java.lang.Object, java.lang.Object)
+         */
         @Override
         public void inputChanged(Viewer viewer, Object oldInput, Object newInput) {
         }
 
+        /*
+         * (non-Javadoc)
+         * 
+         * @see org.eclipse.jface.viewers.IStructuredContentProvider#getElements(java.lang.Object)
+         */
         @Override
         public Object[] getElements(Object inputElement) {
             if (!(inputElement instanceof ITemplateManager)) {
index 5d840ce..4e9dd97 100644 (file)
@@ -65,118 +65,6 @@ public class DescriptorLoader implements IDescriptorLoader {
         return descriptor;
     }
 
-    // public static Part convert(String text, DescriptorManager descriptorManager) {
-    // SAXReader reader = new SAXReader();
-    // Document doc = null;
-    // try {
-    // doc = reader.read(new StringBufferInputStream(text));
-    // } catch (DocumentException e) {
-    // return null;
-    // }
-    // Element rootElement = doc.getRootElement();
-    // List<CssSelector> selectors = null;
-    // for (Object child : rootElement.elements("tizen.css")) {
-    // Element childElement = (Element) child;
-    // selectors = PageReader_ver_2.parseStyle((Element) childElement);
-    // }
-    //
-    // Element partElement = rootElement.element("tizen.part");
-    // List<Element> elements = partElement.elements();
-    // Part part = parse(elements.get(0), descriptorManager, selectors);
-    // return part;
-    // }
-    //
-    // private static Part parse(Element element, DescriptorManager descriptorManager,
-    // List<CssSelector> selectors) {
-    // Part part = null;
-    // PartDescriptor descriptor = descriptorManager.getPartDescriptor(element.getName());
-    // if (descriptor == null) {
-    // Logger.error(BuilderConstants.LOG_DESC_CANNOT_BE_FOUND_ERROR);
-    // } else {
-    // if (descriptor instanceof WidgetPartDescriptor) {
-    // part = new WidgetPart(null, (WidgetPartDescriptor) descriptor);
-    // } else if (descriptor instanceof PagePartDescriptor) {
-    // part = new PagePart(null, (PagePartDescriptor) descriptor);
-    // } else {
-    // // cannot be here
-    // assert (false);
-    // }
-    //
-    // // Sets events and properties.
-    // Attribute attr = null;
-    // for (Object attrObj : element.attributes()) {
-    // attr = (Attribute) attrObj;
-    // if (BuilderConstants.ATTRIBUTE_EVENT_PREFIX.equals((attr.getName()).substring(0,
-    // BuilderConstants.ATTRIBUTE_EVENT_PREFIX.length()))) {
-    // part.setEventValue(attr.getName(), attr.getValue());
-    // } else {
-    // part.setPropertyValueWhenPageRead(attr.getName(), attr.getValue());
-    // }
-    // }
-    // String text = element.getText();
-    // if (!text.isEmpty()) {
-    // PropertyDescriptor prop = null;
-    // for (Object attrObj : descriptor.getPropertyDescriptors()) {
-    // prop = (PropertyDescriptor) attrObj;
-    // if (prop.getPropertyType().equals(TypeDescriptor.ID_OF_HTMLSTRING)) {
-    // part.setPropertyValueWhenPageRead(prop.getPropertyName(), text);
-    // break;
-    // }
-    // }
-    // }
-    //
-    // // Sets selectors.
-    // int size = selectors.size();
-    // for (int i = 0; i < size; i++) {
-    // CssSelector selector = selectors.get(i);
-    // if (part.getIdPropertyValue().equals(selector.getPartId())) {
-    // part.addSelector(selector.getSelectorName(), selector);
-    // }
-    // }
-    //
-    // // Adds children.
-    // Element child = null;
-    // Part childpart = null;
-    // for (Object childObj : element.elements()) {
-    // child = (Element) childObj;
-    // childpart = parse(child, descriptorManager, selectors);
-    // part.addChildWhenPageRead(childpart);
-    // }
-    // }
-    //
-    // return part;
-    // }
-    //
-    // public static String convert(Part part) {
-    // Document document = DocumentHelper.createDocument();
-    //
-    // Element rootElement = DocumentHelper.createElement("snippet");
-    //
-    // Element cssElement = DocumentHelper.createElement("tizen.css");
-    //
-    // Element partElement = DocumentHelper.createElement("tizen.part");
-    // partElement.add(PageWriter_ver_2.writePart(part, false));
-    //
-    // rootElement.add(cssElement);
-    // rootElement.add(partElement);
-    //
-    // document.add(rootElement);
-    //
-    // StringWriter stringWriter = new StringWriter();
-    // OutputFormat prettyFormat = OutputFormat.createPrettyPrint();
-    // prettyFormat.setIndentSize(INDENT_SIZE);
-    //
-    // XMLWriter xmlwriter = new XMLWriter(stringWriter, prettyFormat);
-    // try {
-    // xmlwriter.write(document);
-    // xmlwriter.close();
-    // } catch (IOException e) {
-    // e.printStackTrace();
-    // }
-    //
-    // return stringWriter.getBuffer().toString();
-    // }
-
     /*
      * (non-Javadoc)
      * 
index 1b58b86..d6e91ac 100644 (file)
@@ -34,7 +34,7 @@ import org.tizen.webuibuilder.snippet.model.jaxb.Metadata;
 
 
 public class MetadataLoader implements IMetadataLoader {
-    public Metadata metadata;
+    public Metadata metadata = null;
 
     public MetadataLoader() {
         super();
index 9949972..3e25494 100644 (file)
@@ -170,7 +170,7 @@ public class ExportWizardFirstPage extends WizardPage implements ISelectionChang
         removeListeners();
     }
 
-    private class SnippetViewerContentsProvider implements IStructuredContentProvider {
+    private static class SnippetViewerContentsProvider implements IStructuredContentProvider {
         /*
          * (non-Javadoc)
          * 
@@ -206,7 +206,7 @@ public class ExportWizardFirstPage extends WizardPage implements ISelectionChang
         }
     }
 
-    private class SnippetViewerLabelProvider implements ILabelProvider {
+    private static class SnippetViewerLabelProvider implements ILabelProvider {
         /*
          * (non-Javadoc)
          * 
@@ -274,7 +274,7 @@ public class ExportWizardFirstPage extends WizardPage implements ISelectionChang
         }
     }
 
-    private class SnippetViewerComparator extends ViewerComparator {
+    private static class SnippetViewerComparator extends ViewerComparator {
         /*
          * (non-Javadoc)
          * 
index af68635..6eb6c05 100644 (file)
@@ -97,7 +97,7 @@ public class ComponentsCreator {
                     ImageDescriptor smallIcon =
                             ResourceManager.getImageDescriptor(BuilderConstants.ICON_DIR,
                                                                desc.getSmallIcon());
-                    WidgetTemplate template = new WidgetTemplate(ownerRef, id, appManager);
+                    WidgetTemplate template = new WidgetTemplate(id, appManager);
                     CombinedTemplateCreationEntry entry =
                             new CombinedTemplateCreationEntry(name, description, template,
                                                               template.getFactory(), smallIcon,
index 83c7d1d..cdb7297 100644 (file)
@@ -81,7 +81,6 @@ import org.eclipse.ui.ISharedImages;
 import org.eclipse.ui.PlatformUI;
 import org.eclipse.ui.actions.ActionFactory;
 import org.eclipse.ui.part.Page;
-import org.tizen.common.util.Assert;
 import org.tizen.common.util.EFSUtil;
 import org.tizen.webuibuilder.gef.dnd.model.ResourceTemplate;
 import org.tizen.webuibuilder.ui.SelectionProvider;
@@ -386,7 +385,10 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
     }
 
     public void setInput(IProject project) {
-        Assert.notNull(project);
+        if (project == null) {
+            throw new IllegalArgumentException();
+        }
+
         IResource resource = project.findMember(ResourcesViewMessages.RESOURCES_FOLDER);
         if (resource == null) {
             // TODO :
@@ -403,8 +405,11 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
         setInput(folder);
     }
 
-    public void setInput(IFolder folder) {
-        Assert.notNull(folder);
+    private void setInput(IFolder folder) {
+        if (folder == null) {
+            throw new IllegalArgumentException();
+        }
+
         project = folder.getProject();
         currentFolder = folder;
         resources = getResources(currentFolder);
@@ -413,17 +418,18 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
         setSelection((IResource) null);
     }
 
-    private Image resize(Image image, int width, int height) {
-        return ImageUtils.resize(image, width, height);
-    }
-
+    /**
+     * IResource[] --> List<Resources>
+     * 
+     * @param folder
+     * @return
+     */
     public List<Resource> getResources(IFolder folder) {
         IResource[] members = null;
         try {
             members = folder.members();
         } catch (CoreException e) {
-            e.printStackTrace();
-            return null;
+            throw new IllegalArgumentException();
         }
 
         List<Resource> resources = new LinkedList<Resource>();
@@ -448,8 +454,14 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
      * @return {@link PaletteRoot}
      */
     private PaletteRoot getPaletteRoot(IFolder folder, List<Resource> resources) {
-        Assert.notNull(folder);
-        Assert.notNull(resources);
+        if (folder == null) {
+            throw new IllegalArgumentException();
+        }
+
+        if (resources == null) {
+            throw new IllegalArgumentException();
+        }
+
         // initialize PaletteRoot
         PaletteRoot root = new PaletteRoot();
         // initialize PaletteDrawer(group)
@@ -459,7 +471,7 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
         for (Resource resource : resources) {
             String name = resource.getName();
             Image image = resource.getImage();
-            image = resize(image, ITEM_WIDTH, ITEM_HEIGHT);
+            image = ImageUtils.resize(image, ITEM_WIDTH, ITEM_HEIGHT);
             ImageDescriptor imageDescriptor = ImageDescriptor.createFromImage(image);
             ResourceTemplate template = new ResourceTemplate(resource);
             CombinedTemplateCreationEntry entry =
@@ -736,8 +748,14 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
     }
 
     private List<Resource> getRegExpFilteredResources(List<Resource> resources, String exp) {
-        Assert.notNull(resources);
-        Assert.notNull(exp);
+        if (resources == null) {
+            throw new IllegalArgumentException();
+        }
+
+        if (exp == null) {
+            throw new IllegalArgumentException();
+        }
+
         if (exp.equals("")) {
             return resources;
         }
@@ -753,8 +771,14 @@ public class ResourcesPage extends Page implements PalettePage, ModifyListener,
     }
 
     private List<Resource> getExtensionFilteredResources(List<Resource> resources, String exp) {
-        Assert.notNull(resources);
-        Assert.notNull(exp);
+        if (resources == null) {
+            throw new IllegalArgumentException();
+        }
+
+        if (exp == null) {
+            throw new IllegalArgumentException();
+        }
+
         if (exp.equals("*.*")) {
             return resources;
         }
index 5540dca..fc0c536 100644 (file)
@@ -43,12 +43,12 @@ public class Resource {
     public static int SOUND = 3;
     public static int MOVIE = 4;
 
-    private IResource resource;
-    private int type;
-    private String name;
-    private String description;
-    private Image image;
-    private String path;
+    private IResource resource = null;
+    private int type = INVALID;
+    private String name = null;
+    private String description = null;
+    private Image image = null;
+    private String path = null;
 
     private Resource() {
     };
index 1c452fa..38bdf04 100644 (file)
@@ -69,23 +69,22 @@ public class ImageUtils {
      * @param maxHeight
      * @return
      */
-    public static Image resizeBestSize(Image img, int maxWidth, int maxHeight) {
+    public static Image resizeBestSize(Image image, int maxWidth, int maxHeight) {
         // Check for null
-        if (img == null)
-            return null;
+        if (image == null)
+            throw new IllegalArgumentException();
 
         // Calculate best size
         Point newSize =
-                getBestSize(img.getBounds().width, img.getBounds().height, maxWidth, maxHeight);
+                getBestSize(image.getBounds().width, image.getBounds().height, maxWidth, maxHeight);
 
         // Resize image
-        return ImageUtils.resize(img, newSize.x, newSize.y);
+        return ImageUtils.resize(image, newSize.x, newSize.y);
     }
 
     public static ImageData resize(ImageData imageData, int width, int height, boolean antiAliasing) {
-
         if (imageData == null)
-            return null;
+            throw new IllegalArgumentException();
 
         if (imageData.width == width && imageData.height == height)
             return imageData;
@@ -118,9 +117,8 @@ public class ImageUtils {
      * @return
      */
     public static Image resize(Image image, int width, int height) {
-
         if (image == null)
-            return null;
+            throw new IllegalArgumentException();
 
         final Image scaled = new Image(Display.getDefault(), width, height);
         GC gc = new GC(scaled);
@@ -142,7 +140,7 @@ public class ImageUtils {
 
     public static byte[] saveImage(Image image, int format) {
         if (image == null)
-            return null;
+            throw new IllegalArgumentException();
 
         ImageLoader il = new ImageLoader();
         il.data = new ImageData[] { image.getImageData() };
index d53d1ca..83752e4 100644 (file)
@@ -318,13 +318,15 @@ public class PageManager {
         WebUIBuilderBuilder.build(res);
 
         IFile jsFile = getJsFile(res);
-        if (jsFile == null) {
+        if (jsFile != null) {
+            ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_JS, jsFile);
         }
-        ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_JS, jsFile);
+        
         IFile cssFile = getCssFile(res);
-        if (cssFile == null) {
+        if (cssFile != null) {
+            ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_CSS, cssFile);
         }
-        ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_CSS, cssFile);
+
     }
 
     public static void remove(IResource res, PageData pageData) throws CoreException, IOException {
@@ -407,14 +409,15 @@ public class PageManager {
         AnimatorFileWriter.save(appManager, pageData.getRootPart());
 
         IFile jsFile = getJsFile(dst);
-        if (jsFile == null) {
+        if (jsFile != null) {
+            ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_JS, jsFile);
         }
-        ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_JS, jsFile);
 
         IFile cssFile = getCssFile(dst);
-        if (cssFile == null) {
+        if (cssFile != null) {
+            ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_CSS, cssFile);
         }
-        ResourceManager.copy(BuilderMessages.NEWTEMPLATEFILE_CSS, cssFile);
+
     }
 
     public static void move(IResource src, PageData srcData, IResource dst) throws CoreException,