Improvement for CustomizerExtender #23

Author: junichi11

nbproject/genfiles.properties

@@ -1,8 +1,8 @@
-build.xml.data.CRC32=f6605689
+build.xml.data.CRC32=a6abec14
 build.xml.script.CRC32=f6cbff90
 build.xml.stylesheet.CRC32=a56c6a5b@2.62.1
 # This file is used by a NetBeans-based IDE to track changes in generated files such as build-impl.xml.
 # Do not edit this file. You may delete it but then the IDE will never regenerate such files for you.
-nbproject/build-impl.xml.data.CRC32=f6605689
+nbproject/build-impl.xml.data.CRC32=a6abec14
 nbproject/build-impl.xml.script.CRC32=bfa38a5f
 nbproject/build-impl.xml.stylesheet.CRC32=238281d1@2.62.1

nbproject/project.xml

@@ -6,6 +6,15 @@
             <code-name-base>org.netbeans.modules.php.wordpress</code-name-base>
             <standalone/>
             <module-dependencies>
+                <dependency>
+                    <code-name-base>org.netbeans.api.annotations.common</code-name-base>
+                    <build-prerequisite/>
+                    <compile-dependency/>
+                    <run-dependency>
+                        <release-version>1</release-version>
+                        <specification-version>1.21.1</specification-version>
+                    </run-dependency>
+                </dependency>
                 <dependency>
                     <code-name-base>org.netbeans.api.progress</code-name-base>
                     <build-prerequisite/>

src/org/netbeans/modules/php/wordpress/customizer/WordPressCustomizerExtender.java

@@ -46,6 +46,7 @@
 import javax.swing.event.ChangeListener;
 import org.netbeans.modules.php.api.phpmodule.PhpModule;
 import org.netbeans.modules.php.api.util.StringUtils;
+import org.netbeans.modules.php.api.validation.ValidationResult;
 import org.netbeans.modules.php.spi.framework.PhpModuleCustomizerExtender;
 import org.netbeans.modules.php.wordpress.preferences.WordPressPreferences;
 import org.openide.util.HelpCtx;
@@ -60,6 +61,8 @@ public class WordPressCustomizerExtender extends PhpModuleCustomizerExtender {
     private WordPressCustomizerExtenderPanel panel;
     private final PhpModule phpModule;
     private String originalCustomeContentName;
+    private boolean isValid;
+    private String errorMessage;
 
     public WordPressCustomizerExtender(PhpModule phpModule) {
         this.phpModule = phpModule;
@@ -73,10 +76,12 @@ public String getDisplayName() {
 
     @Override
     public void addChangeListener(ChangeListener cl) {
+        getPanel().addChangeListener(cl);
     }
 
     @Override
     public void removeChangeListener(ChangeListener cl) {
+        getPanel().removeChangeListener(cl);
     }
 
     @Override
@@ -91,12 +96,38 @@ public HelpCtx getHelp() {
 
     @Override
     public boolean isValid() {
-        return true;
+        validate();
+        return isValid;
     }
 
     @Override
     public String getErrorMessage() {
-        return null;
+        validate();
+        return errorMessage;
+    }
+
+    private void validate() {
+        String contentName = getPanel().getCustomContentName();
+        ValidationResult result = new WordPressCustomizerValidator()
+                .validateWpContent(phpModule, contentName)
+                .getResult();
+        // error
+        if (result.hasErrors()) {
+            isValid = false;
+            errorMessage = result.getErrors().get(0).getMessage();
+            return;
+        }
+
+        // warning
+        if (result.hasWarnings()) {
+            isValid = false;
+            errorMessage = result.getWarnings().get(0).getMessage();
+            return;
+        }
+
+        // everything ok
+        errorMessage = null;
+        isValid = true;
     }
 
     @Override

src/org/netbeans/modules/php/wordpress/customizer/WordPressCustomizerExtenderPanel.java

@@ -41,17 +41,30 @@
  */
 package org.netbeans.modules.php.wordpress.customizer;
 
+import javax.swing.JPanel;
+import javax.swing.event.ChangeListener;
+import javax.swing.event.DocumentEvent;
+import javax.swing.event.DocumentListener;
+import org.openide.util.ChangeSupport;
+
 /**
  *
  * @author junichi11
  */
-public class WordPressCustomizerExtenderPanel extends javax.swing.JPanel {
+public class WordPressCustomizerExtenderPanel extends JPanel {
+
+    private final ChangeSupport changeSupport = new ChangeSupport(this);
 
     /**
      * Creates new form WordPressCustomizerExtenderPanel
      */
     public WordPressCustomizerExtenderPanel() {
         initComponents();
+        init();
+    }
+
+    private void init() {
+        customContentNameTextField.getDocument().addDocumentListener(new DefaultDocumentListener());
     }
 
     public String getCustomContentName() {
@@ -62,6 +75,18 @@ public void setCustomContentName(String name) {
         customContentNameTextField.setText(name);
     }
 
+    public void addChangeListener(ChangeListener cl) {
+        changeSupport.addChangeListener(cl);
+    }
+
+    public void removeChangeListener(ChangeListener cl) {
+        changeSupport.removeChangeListener(cl);
+    }
+
+    public void fireChange() {
+        changeSupport.fireChange();
+    }
+
     /**
      * This method is called from within the constructor to initialize the form.
      * WARNING: Do NOT modify this code. The content of this method is always
@@ -104,4 +129,29 @@ private void initComponents() {
     private javax.swing.JLabel customContentNameLabel;
     private javax.swing.JTextField customContentNameTextField;
     // End of variables declaration//GEN-END:variables
+
+    private class DefaultDocumentListener implements DocumentListener {
+
+        public DefaultDocumentListener() {
+        }
+
+        @Override
+        public void insertUpdate(DocumentEvent e) {
+            processUpdate();
+        }
+
+        @Override
+        public void removeUpdate(DocumentEvent e) {
+            processUpdate();
+        }
+
+        @Override
+        public void changedUpdate(DocumentEvent e) {
+            processUpdate();
+        }
+
+        private void processUpdate() {
+            fireChange();
+        }
+    }
 }

src/org/netbeans/modules/php/wordpress/customizer/WordPressCustomizerValidator.java

@@ -0,0 +1,90 @@
+/*
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
+ *
+ * Copyright 2013 Oracle and/or its affiliates. All rights reserved.
+ *
+ * Oracle and Java are registered trademarks of Oracle and/or its affiliates.
+ * Other names may be trademarks of their respective owners.
+ *
+ * The contents of this file are subject to the terms of either the GNU
+ * General Public License Version 2 only ("GPL") or the Common
+ * Development and Distribution License("CDDL") (collectively, the
+ * "License"). You may not use this file except in compliance with the
+ * License. You can obtain a copy of the License at
+ * http://www.netbeans.org/cddl-gplv2.html
+ * or nbbuild/licenses/CDDL-GPL-2-CP. See the License for the
+ * specific language governing permissions and limitations under the
+ * License.  When distributing the software, include this License Header
+ * Notice in each file and include the License file at
+ * nbbuild/licenses/CDDL-GPL-2-CP.  Oracle designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Oracle in the GPL Version 2 section of the License file that
+ * accompanied this code. If applicable, add the following below the
+ * License Header, with the fields enclosed by brackets [] replaced by
+ * your own identifying information:
+ * "Portions Copyrighted [year] [name of copyright owner]"
+ *
+ * If you wish your version of this file to be governed by only the CDDL
+ * or only the GPL Version 2, indicate your decision by adding
+ * "[Contributor] elects to include this software in this distribution
+ * under the [CDDL or GPL Version 2] license." If you do not indicate a
+ * single choice of license, a recipient has the option to distribute
+ * your version of this file under either the CDDL, the GPL Version 2 or
+ * to extend the choice of license to its licensees as provided above.
+ * However, if you add GPL Version 2 code and therefore, elected the GPL
+ * Version 2 license, then the option applies only if the new code is
+ * made subject to such option by the copyright holder.
+ *
+ * Contributor(s):
+ *
+ * Portions Copyrighted 2013 Sun Microsystems, Inc.
+ */
+package org.netbeans.modules.php.wordpress.customizer;
+
+import org.netbeans.api.annotations.common.NonNull;
+import org.netbeans.modules.php.api.phpmodule.PhpModule;
+import org.netbeans.modules.php.api.util.StringUtils;
+import org.netbeans.modules.php.api.validation.ValidationResult;
+import org.openide.filesystems.FileObject;
+import org.openide.util.NbBundle;
+
+/**
+ *
+ * @author junichi11
+ */
+public final class WordPressCustomizerValidator {
+
+    private final ValidationResult result = new ValidationResult();
+
+    @NbBundle.Messages({
+        "WordPressCustomizerValidator.wordpress.dir.invalid=Project might be broken...",
+        "WordPressCustomizerValidator.wordpress.content.name.invalid=Existing directory name must be set.",
+        "WordPressCustomizerValidator.wordpress.content.name.contains.shash=The name must not contain slash."
+    })
+    public WordPressCustomizerValidator validateWpContent(@NonNull PhpModule phpModule, String name) {
+        FileObject sourceDirectory = phpModule.getSourceDirectory();
+        if (sourceDirectory == null) {
+            result.addWarning(new ValidationResult.Message("wordpress.dir", Bundle.WordPressCustomizerValidator_wordpress_dir_invalid())); // NOI18N
+            return this;
+        }
+
+        FileObject wpContent = sourceDirectory.getFileObject(name);
+        if (wpContent == null
+                || !wpContent.isFolder()
+                || StringUtils.isEmpty(name)) {
+            result.addWarning(new ValidationResult.Message("wordpress.content.name", Bundle.WordPressCustomizerValidator_wordpress_content_name_invalid())); // NOI18N
+            return this;
+        }
+
+        if (name.contains("/")) { // NOI18N
+            result.addWarning(new ValidationResult.Message("wordpress.content.name.slash", Bundle.WordPressCustomizerValidator_wordpress_content_name_contains_shash())); // NOI18N
+            return this;
+        }
+
+        return this;
+    }
+
+    public ValidationResult getResult() {
+        return result;
+    }
+}