diff --git a/pom.xml b/pom.xml
index d5db706..3b890b7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -4,7 +4,7 @@
 
     <groupId>de.markiewb.netbeans.plugins</groupId>
     <artifactId>AdditionalHints</artifactId>
-    <version>1.1.0</version>
+    <version>1.2.0-SNAPSHOT</version>
     <packaging>nbm</packaging>
 
     <name>Additional Java hints</name>
@@ -13,14 +13,12 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <endorsed.dir>${project.build.directory}/endorsed</endorsed.dir>
         <netbeans.version>RELEASE721</netbeans.version>
-        <netbeans.run.params.ide />
-        <netbeans.run.params>${netbeans.run.params.ide}</netbeans.run.params>
     </properties>
     <organization>
         <name>Benno Markiewicz (benno.markiewicz@googlemail.com)</name>
     </organization>
     <url>https://github.com/markiewb/nb-additional-hints</url>
-    
+
     <repositories>
         <!--
         Repository hosting NetBeans modules, especially APIs.
@@ -120,7 +118,7 @@
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>nbm-maven-plugin</artifactId>
-                <version>3.11</version>
+                <version>3.11.1</version>
                 <extensions>true</extensions>
                 <configuration>
                     <!-- keep this id, else the update from existing versions will fail
@@ -220,12 +218,18 @@
 &lt;li&gt;" Convert to assertTrue/assertFalse" (since 1.1)&lt;/li&gt;
 &lt;li&gt;"Support transformation to BigDecimal constants" (since 1.1)&lt;/li&gt;
 &lt;li&gt;"Remove "public abstract" modifiers from method declarations within interfaces" (since 1.1)&lt;/li&gt;
+&lt;li&gt;"Change modifiers" (since 1.2)&lt;/li&gt;
 &lt;/ul&gt;
 
 &lt;h2&gt;Example:&lt;/h2&gt;
 &lt;img src="https://raw.github.com/markiewb/nb-additional-hints/master/doc/screenshot-1.1.0.png"/&gt;
 
 &lt;h2&gt;Updates&lt;/h2&gt;
+&lt;h3&gt;1.2.0:&lt;/h3&gt;
+&lt;ul&gt;
+&lt;li&gt;[&lt;a href="https://github.com/markiewb/nb-additional-hints/pull/22"&gt;New Hint&lt;/a&gt;]: Change the modifier of a class/method/field to public/package protected/protected/private  (by &lt;a href="https://github.com/rasa-silva"&gt;rasa-silva&lt;/a&gt;)&lt;/li&gt;
+&lt;/ul&gt;
+
 &lt;h3&gt;1.1.0:&lt;/h3&gt;
 &lt;ul&gt;
 &lt;li&gt;[&lt;a href="https://github.com/markiewb/nb-additional-hints/issues/11"&gt;New Hint&lt;/a&gt;]: Support transformation to BigDecimal constants&lt;/li&gt;
diff --git a/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePackageProtected.java b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePackageProtected.java
new file mode 100644
index 0000000..24d0582
--- /dev/null
+++ b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePackageProtected.java
@@ -0,0 +1,99 @@
+/*
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
+ *
+ * Copyright 1997-2007 Sun Microsystems, Inc. All rights reserved.
+ *
+ * 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.  Sun designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Sun 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]"
+ *
+ * Contributor(s):
+ *
+ * The Original Software is NetBeans. The Initial Developer of the Original
+ * Software is Sun Microsystems, Inc. Portions Copyright 1997-2007 Sun
+ * Microsystems, Inc. All Rights Reserved.
+ *
+ * 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.
+ * 
+ * "Portions Copyrighted 2013 rasa-silva"
+ * "Portions Copyrighted 2013 Benno Markiewicz"
+ */
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import com.sun.source.tree.ModifiersTree;
+import com.sun.source.tree.Tree;
+import com.sun.source.util.TreePath;
+import java.util.EnumSet;
+import javax.lang.model.element.Element;
+import javax.lang.model.element.Modifier;
+import org.netbeans.spi.editor.hints.ErrorDescription;
+import org.netbeans.spi.editor.hints.Fix;
+import org.netbeans.spi.editor.hints.Severity;
+import org.netbeans.spi.java.hints.ErrorDescriptionFactory;
+import org.netbeans.spi.java.hints.Hint;
+import org.netbeans.spi.java.hints.HintContext;
+import org.netbeans.spi.java.hints.TriggerTreeKind;
+import org.netbeans.spi.java.hints.support.FixFactory;
+import org.openide.util.NbBundle;
+
+/**
+ * Turns a class, method of field public.
+ * @author https://github.com/rasa-silva
+ */
+@NbBundle.Messages({
+    "ERR_MakePackageProtected=Make Package Protected",
+    "DN_MakePackageProtected=Make Package Protected",
+    "DESC_MakePackageProtected=Makes a class, method or field package protected.<p>Provided by <a href=\"https://github.com/markiewb/nb-additional-hints\">nb-additional-hints</a> plugin</p>"})
+public class MakePackageProtected {
+
+    private static final EnumSet<Modifier> oppositeModifiers = EnumSet.of(Modifier.PRIVATE, Modifier.PUBLIC, Modifier.PROTECTED);
+
+    @Hint(displayName = "#DN_MakePackageProtected", description = "#DESC_MakePackageProtected", category = "suggestions",
+            hintKind = Hint.Kind.INSPECTION, severity = Severity.HINT)
+    @TriggerTreeKind({Tree.Kind.CLASS, Tree.Kind.METHOD, Tree.Kind.VARIABLE})
+    public static ErrorDescription convert(HintContext ctx) {
+
+        TreePath path = ctx.getPath();
+        Element element = ctx.getInfo().getTrees().getElement(path);
+
+        if (element == null) {
+            return null;
+        }
+
+        ModifiersTree modifiers = ModifierUtils.getModifiersTree(path, element);
+
+        if (modifiers == null || (!modifiers.getFlags().contains(Modifier.PROTECTED)
+                && !modifiers.getFlags().contains(Modifier.PRIVATE)
+                && !modifiers.getFlags().contains(Modifier.PUBLIC))) {
+            return null;
+        }
+        final EnumSet<Modifier> toAdd = EnumSet.noneOf(Modifier.class);
+        final EnumSet<Modifier> toRemove = oppositeModifiers;
+
+        Fix fix = FixFactory.changeModifiersFix(ctx.getInfo(), new TreePath(path, modifiers), toAdd, toRemove, Bundle.ERR_MakePackageProtected());
+        return ErrorDescriptionFactory.forName(ctx, path, Bundle.ERR_MakePackageProtected(), fix);
+    }
+}
diff --git a/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePrivate.java b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePrivate.java
new file mode 100644
index 0000000..21d9fd3
--- /dev/null
+++ b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePrivate.java
@@ -0,0 +1,102 @@
+/*
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
+ *
+ * Copyright 1997-2007 Sun Microsystems, Inc. All rights reserved.
+ *
+ * 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.  Sun designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Sun 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]"
+ *
+ * Contributor(s):
+ *
+ * The Original Software is NetBeans. The Initial Developer of the Original
+ * Software is Sun Microsystems, Inc. Portions Copyright 1997-2007 Sun
+ * Microsystems, Inc. All Rights Reserved.
+ *
+ * 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.
+ * 
+ * "Portions Copyrighted 2013 rasa-silva"
+ * "Portions Copyrighted 2013 Benno Markiewicz"
+ */
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import com.sun.source.tree.ModifiersTree;
+import com.sun.source.tree.Tree;
+import com.sun.source.util.TreePath;
+import java.util.EnumSet;
+import javax.lang.model.element.Element;
+import javax.lang.model.element.Modifier;
+import org.netbeans.spi.editor.hints.ErrorDescription;
+import org.netbeans.spi.editor.hints.Fix;
+import org.netbeans.spi.editor.hints.Severity;
+import org.netbeans.spi.java.hints.ErrorDescriptionFactory;
+import org.netbeans.spi.java.hints.Hint;
+import org.netbeans.spi.java.hints.HintContext;
+import org.netbeans.spi.java.hints.TriggerTreeKind;
+import org.netbeans.spi.java.hints.support.FixFactory;
+import org.openide.util.NbBundle;
+
+/**
+ * 
+ * @author https://github.com/rasa-silva
+ */
+@NbBundle.Messages({
+    "ERR_MakePrivate=Make Private",
+    "DN_MakePrivate=Make Private",
+    "DESC_MakePrivate=Makes a class, method or field private.<p>Provided by <a href=\"https://github.com/markiewb/nb-additional-hints\">nb-additional-hints</a> plugin</p>"})
+public class MakePrivate {
+
+    private static final EnumSet<Modifier> oppositeModifiers = EnumSet.of(Modifier.PUBLIC, Modifier.PROTECTED);
+
+    @Hint(displayName = "#DN_MakePrivate", description = "#DESC_MakePrivate", category = "suggestions",
+            hintKind = Hint.Kind.INSPECTION, severity = Severity.HINT)
+    @TriggerTreeKind({Tree.Kind.CLASS, Tree.Kind.METHOD, Tree.Kind.VARIABLE})
+    public static ErrorDescription convert(HintContext ctx) {
+
+        TreePath path = ctx.getPath();
+        Element element = ctx.getInfo().getTrees().getElement(path);
+
+        if (element == null) {
+            return null;
+        }
+
+        //XXX top level classes cannot be made private
+        if (ModifierUtils.isTopLevelClass(element)) {
+            return null;
+        }
+
+        ModifiersTree modifiers = ModifierUtils.getModifiersTree(path, element);
+
+        if (modifiers == null || modifiers.getFlags().contains(Modifier.PRIVATE)) {
+            return null;
+        }
+        final EnumSet<Modifier> toAdd = EnumSet.of(Modifier.PRIVATE);
+        final EnumSet<Modifier> toRemove = oppositeModifiers;
+
+        Fix fix = FixFactory.changeModifiersFix(ctx.getInfo(), new TreePath(path, modifiers), toAdd, toRemove, Bundle.ERR_MakePrivate());
+        return ErrorDescriptionFactory.forName(ctx, path, Bundle.ERR_MakePrivate(), fix);
+    }
+}
diff --git a/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakeProtected.java b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakeProtected.java
new file mode 100644
index 0000000..b717add
--- /dev/null
+++ b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakeProtected.java
@@ -0,0 +1,102 @@
+/*
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
+ *
+ * Copyright 1997-2007 Sun Microsystems, Inc. All rights reserved.
+ *
+ * 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.  Sun designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Sun 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]"
+ *
+ * Contributor(s):
+ *
+ * The Original Software is NetBeans. The Initial Developer of the Original
+ * Software is Sun Microsystems, Inc. Portions Copyright 1997-2007 Sun
+ * Microsystems, Inc. All Rights Reserved.
+ *
+ * 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.
+ * 
+ * "Portions Copyrighted 2013 rasa-silva"
+ * "Portions Copyrighted 2013 Benno Markiewicz"
+ */
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import com.sun.source.tree.ModifiersTree;
+import com.sun.source.tree.Tree;
+import com.sun.source.util.TreePath;
+import java.util.EnumSet;
+import javax.lang.model.element.Element;
+import javax.lang.model.element.Modifier;
+import org.netbeans.spi.editor.hints.ErrorDescription;
+import org.netbeans.spi.editor.hints.Fix;
+import org.netbeans.spi.editor.hints.Severity;
+import org.netbeans.spi.java.hints.ErrorDescriptionFactory;
+import org.netbeans.spi.java.hints.Hint;
+import org.netbeans.spi.java.hints.HintContext;
+import org.netbeans.spi.java.hints.TriggerTreeKind;
+import org.netbeans.spi.java.hints.support.FixFactory;
+import org.openide.util.NbBundle;
+
+/**
+ * Turns a class, method of field public.
+ * @author https://github.com/rasa-silva
+ */
+@NbBundle.Messages({
+    "ERR_MakeProtected=Make Protected",
+    "DN_MakeProtected=Make Protected",
+    "DESC_MakeProtected=Makes a class, method or field protected.<p>Provided by <a href=\"https://github.com/markiewb/nb-additional-hints\">nb-additional-hints</a> plugin</p>"})
+public class MakeProtected {
+
+    private static final EnumSet<Modifier> oppositeModifiers = EnumSet.of(Modifier.PRIVATE, Modifier.PUBLIC);
+
+    @Hint(displayName = "#DN_MakeProtected", description = "#DESC_MakeProtected", category = "suggestions",
+            hintKind = Hint.Kind.INSPECTION, severity = Severity.HINT)
+    @TriggerTreeKind({Tree.Kind.CLASS, Tree.Kind.METHOD, Tree.Kind.VARIABLE})
+    public static ErrorDescription convert(HintContext ctx) {
+
+        TreePath path = ctx.getPath();
+        Element element = ctx.getInfo().getTrees().getElement(path);
+
+        if (element == null) {
+            return null;
+        }
+
+        //XXX top level classes cannot be made protected
+        if (ModifierUtils.isTopLevelClass(element)) {
+            return null;
+        }
+
+        ModifiersTree modifiers = ModifierUtils.getModifiersTree(path, element);
+
+        if (modifiers == null || modifiers.getFlags().contains(Modifier.PROTECTED)) {
+            return null;
+        }
+        final EnumSet<Modifier> toAdd = EnumSet.of(Modifier.PROTECTED);
+        final EnumSet<Modifier> toRemove = oppositeModifiers;
+
+        Fix fix = FixFactory.changeModifiersFix(ctx.getInfo(), new TreePath(path, modifiers), toAdd, toRemove, Bundle.ERR_MakeProtected());
+        return ErrorDescriptionFactory.forName(ctx, path, Bundle.ERR_MakeProtected(), fix);
+    }
+}
diff --git a/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePublic.java b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePublic.java
new file mode 100644
index 0000000..8fb8a6e
--- /dev/null
+++ b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePublic.java
@@ -0,0 +1,97 @@
+/*
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
+ *
+ * Copyright 1997-2007 Sun Microsystems, Inc. All rights reserved.
+ *
+ * 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.  Sun designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Sun 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]"
+ *
+ * Contributor(s):
+ *
+ * The Original Software is NetBeans. The Initial Developer of the Original
+ * Software is Sun Microsystems, Inc. Portions Copyright 1997-2007 Sun
+ * Microsystems, Inc. All Rights Reserved.
+ *
+ * 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.
+ * 
+ * "Portions Copyrighted 2013 rasa-silva"
+ * "Portions Copyrighted 2013 Benno Markiewicz"
+ */
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import com.sun.source.tree.ModifiersTree;
+import com.sun.source.tree.Tree;
+import com.sun.source.util.TreePath;
+import java.util.EnumSet;
+import javax.lang.model.element.Element;
+import javax.lang.model.element.Modifier;
+import org.netbeans.spi.editor.hints.ErrorDescription;
+import org.netbeans.spi.editor.hints.Fix;
+import org.netbeans.spi.editor.hints.Severity;
+import org.netbeans.spi.java.hints.ErrorDescriptionFactory;
+import org.netbeans.spi.java.hints.Hint;
+import org.netbeans.spi.java.hints.HintContext;
+import org.netbeans.spi.java.hints.TriggerTreeKind;
+import org.netbeans.spi.java.hints.support.FixFactory;
+import org.openide.util.NbBundle;
+
+/**
+ * Turns a class, method of field public.
+ * @author https://github.com/rasa-silva
+ */
+@NbBundle.Messages({
+    "ERR_MakePublic=Make Public",
+    "DN_MakePublic=Make Public",
+    "DESC_MakePublic=Makes a class, method or field public.<p>Provided by <a href=\"https://github.com/markiewb/nb-additional-hints\">nb-additional-hints</a> plugin</p>"})
+public class MakePublic {
+
+    private static final EnumSet<Modifier> oppositeModifiers = EnumSet.of(Modifier.PRIVATE, Modifier.PROTECTED);
+
+    @Hint(displayName = "#DN_MakePublic", description = "#DESC_MakePublic", category = "suggestions",
+            hintKind = Hint.Kind.INSPECTION, severity = Severity.HINT)
+    @TriggerTreeKind({Tree.Kind.CLASS, Tree.Kind.METHOD, Tree.Kind.VARIABLE})
+    public static ErrorDescription convert(HintContext ctx) {
+
+        TreePath path = ctx.getPath();
+        Element element = ctx.getInfo().getTrees().getElement(path);
+
+        if (element == null) {
+            return null;
+        }
+
+        ModifiersTree modifiers = ModifierUtils.getModifiersTree(path, element);
+
+        if (modifiers == null || modifiers.getFlags().contains(Modifier.PUBLIC)) {
+            return null;
+        }
+        final EnumSet<Modifier> toAdd = EnumSet.of(Modifier.PUBLIC);
+        final EnumSet<Modifier> toRemove = oppositeModifiers;
+
+        Fix fix = FixFactory.changeModifiersFix(ctx.getInfo(), new TreePath(path, modifiers), toAdd, toRemove, Bundle.ERR_MakePublic());
+        return ErrorDescriptionFactory.forName(ctx, path, Bundle.ERR_MakePublic(), fix);
+    }
+}
diff --git a/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/ModifierUtils.java b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/ModifierUtils.java
new file mode 100644
index 0000000..b50dd91
--- /dev/null
+++ b/src/main/java/de/markiewb/netbeans/plugins/hints/modifiers/ModifierUtils.java
@@ -0,0 +1,96 @@
+/*
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
+ *
+ * Copyright 1997-2007 Sun Microsystems, Inc. All rights reserved.
+ *
+ * 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.  Sun designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Sun 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]"
+ *
+ * Contributor(s):
+ *
+ * The Original Software is NetBeans. The Initial Developer of the Original
+ * Software is Sun Microsystems, Inc. Portions Copyright 1997-2007 Sun
+ * Microsystems, Inc. All Rights Reserved.
+ *
+ * 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.
+ * 
+ * "Portions Copyrighted 2013 rasa-silva"
+ * "Portions Copyrighted 2013 Benno Markiewicz"
+ */
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import com.sun.source.tree.ClassTree;
+import com.sun.source.tree.MethodTree;
+import com.sun.source.tree.ModifiersTree;
+import com.sun.source.tree.Tree;
+import com.sun.source.tree.VariableTree;
+import com.sun.source.util.TreePath;
+import javax.lang.model.element.Element;
+import javax.lang.model.element.ElementKind;
+import javax.lang.model.element.NestingKind;
+import javax.lang.model.element.TypeElement;
+
+/**
+ * 
+ * @author https://github.com/rasa-silva
+ */
+class ModifierUtils {
+
+    static boolean isTopLevelClass(Element element) {
+
+        if (element == null) {
+            return false;
+        }
+
+        if (element.getKind() == ElementKind.CLASS) {
+            TypeElement clazz = (TypeElement) element;
+            if (clazz.getNestingKind() == NestingKind.TOP_LEVEL) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    static ModifiersTree getModifiersTree(TreePath path, Element element) {
+        ModifiersTree modifiers = null;
+        Tree leaf = path.getLeaf();
+        switch (element.getKind()) {
+            case FIELD:
+                modifiers = ((VariableTree) leaf).getModifiers();
+                break;
+            case CLASS:
+                modifiers = ((ClassTree) leaf).getModifiers();
+                break;
+            case METHOD:
+                modifiers = ((MethodTree) leaf).getModifiers();
+                break;
+        }
+
+        return modifiers;
+    }
+}
diff --git a/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePackageProtectedTest.java b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePackageProtectedTest.java
new file mode 100644
index 0000000..5ecda43
--- /dev/null
+++ b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePackageProtectedTest.java
@@ -0,0 +1,198 @@
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import org.junit.Test;
+import org.netbeans.modules.java.hints.test.api.HintTest;
+
+public class MakePackageProtectedTest {
+
+    @Test
+    public void testFixWorking_Private_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private String s = null;\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:19-2:20:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix().assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Private_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private class Inner {}\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:18-2:23:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Private_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:24-2:28:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * 1:23 modifier private not allowed here
+     *
+     * @throws Exception
+     */
+    @Test(expected = AssertionError.class)
+    public void testFixWorking_Private_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; private class Test {}")
+                .run(MakePackageProtected.class);
+    }
+
+    @Test
+    public void testFixWorking_Protected_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected String s = null;\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:21-2:22:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix().assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Protected_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected class Inner {}\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:20-2:25:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix().assertCompilable().
+                assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Protected_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:26-2:30:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * 1:25 modifier protected not allowed here
+     *
+     * @throws Exception
+     */
+    @Test(expected = AssertionError.class)
+    public void testFixWorking_Protected_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; protected class Test {}")
+                .run(MakePackageProtected.class);
+    }
+
+    @Test
+    public void testFixWorking_Public_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public String s = null;\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:18-2:19:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Public_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public class Inner {}\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:17-2:22:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Public_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePackageProtected.class)
+                .findWarning("2:23-2:27:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Public_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; public class Test {}")
+                .run(MakePackageProtected.class)
+                .findWarning("0:27-0:31:hint:" + Bundle.ERR_MakePackageProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test; class Test {}");
+    }
+}
diff --git a/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePrivateTest.java b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePrivateTest.java
new file mode 100644
index 0000000..5bf1658
--- /dev/null
+++ b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePrivateTest.java
@@ -0,0 +1,185 @@
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import org.junit.Test;
+import org.netbeans.modules.java.hints.test.api.HintTest;
+
+public class MakePrivateTest {
+
+    @Test
+    public void testFixWorking_PackageProtected_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    String s = null;\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:11-2:12:hint:" + Bundle.ERR_MakePrivate())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    private String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    class Inner {}\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:10-2:15:hint:" + Bundle.ERR_MakePrivate())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    private class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:16-2:20:hint:" + Bundle.ERR_MakePrivate())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    private static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * top level classes cannot be made private
+     * @throws Exception 
+     */
+    @Test
+    public void testFixWorking_PackageProtected_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; class Test {}")
+                .run(MakePrivate.class)
+                .assertNotContainsWarnings("0:20-0:24:hint:" + Bundle.ERR_MakePrivate());
+    }
+
+    @Test
+    public void testFixWorking_Protected_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected String s = null;\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:21-2:22:hint:" + Bundle.ERR_MakePrivate());
+    }
+
+    @Test
+    public void testFixWorking_Protected_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected class Inner {}\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:20-2:25:hint:" + Bundle.ERR_MakePrivate());
+    }
+
+    @Test
+    public void testFixWorking_Protected_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:26-2:30:hint:" + Bundle.ERR_MakePrivate());
+    }
+
+    /**
+     * 1:25 modifier protected not allowed here
+     *
+     * @throws Exception
+     */
+    @Test(expected = AssertionError.class)
+    public void testFixWorking_Protected_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; protected class Test {}")
+                .run(MakePrivate.class);
+    }
+
+    @Test
+    public void testFixWorking_Public_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public String s = null;\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:18-2:19:hint:" + Bundle.ERR_MakePrivate())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    private String s = null;\n"
+                        + "}\n")
+                ;
+    }
+
+    @Test
+    public void testFixWorking_Public_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public class Inner {}\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:17-2:22:hint:" + Bundle.ERR_MakePrivate())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    private class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Public_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePrivate.class)
+                .findWarning("2:16-2:20:hint:" + Bundle.ERR_MakePrivate())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    private static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                ;
+    }
+
+    /**
+     * top level classes cannot be made private
+     * @throws Exception 
+     */
+    @Test
+    public void testFixWorking_Public_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; public class Test {}")
+                .run(MakePrivate.class)
+                .assertNotContainsWarnings("0:27-1:31:hint:" + Bundle.ERR_MakePrivate());
+    }
+}
diff --git a/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakeProtectedTest.java b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakeProtectedTest.java
new file mode 100644
index 0000000..d088dd5
--- /dev/null
+++ b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakeProtectedTest.java
@@ -0,0 +1,200 @@
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import org.junit.Test;
+import org.netbeans.modules.java.hints.test.api.HintTest;
+
+public class MakeProtectedTest {
+
+    @Test
+    public void testFixWorking_PackageProtected_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    String s = null;\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:11-2:12:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    class Inner {}\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:10-2:15:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:16-2:20:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * top level classes cannot be made protected
+     * @throws Exception 
+     */
+    @Test
+    public void testFixWorking_PackageProtected_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; class Test {}")
+                .run(MakeProtected.class)
+                .assertNotContainsWarnings("0:20-0:24:hint:" + Bundle.ERR_MakeProtected());
+    }
+
+    @Test
+    public void testFixWorking_Private_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private String s = null;\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:19-2:20:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Private_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private class Inner {}\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:18-2:23:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Private_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:24-2:28:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * 1:23 modifier private not allowed here
+     *
+     * @throws Exception
+     */
+    @Test(expected = AssertionError.class)
+    public void testFixWorking_Private_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; private class Test {}")
+                .run(MakeProtected.class);
+    }
+
+    @Test
+    public void testFixWorking_Public_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public String s = null;\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:18-2:19:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected String s = null;\n"
+                        + "}\n")
+                ;
+    }
+
+    @Test
+    public void testFixWorking_Public_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    public class Inner {}\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:17-2:22:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Public_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakeProtected.class)
+                .findWarning("2:16-2:20:hint:" + Bundle.ERR_MakeProtected())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                ;
+    }
+
+    @Test
+    public void testFixWorking_Public_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; public class Test {}")
+                .run(MakeProtected.class)
+                .assertNotContainsWarnings("0:20-1:24:hint:" + Bundle.ERR_MakeProtected());
+    }
+}
diff --git a/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePublicTest.java b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePublicTest.java
new file mode 100644
index 0000000..da39bdd
--- /dev/null
+++ b/src/test/java/de/markiewb/netbeans/plugins/hints/modifiers/MakePublicTest.java
@@ -0,0 +1,202 @@
+package de.markiewb.netbeans.plugins.hints.modifiers;
+
+import org.junit.Test;
+import org.netbeans.modules.java.hints.test.api.HintTest;
+
+public class MakePublicTest {
+
+    @Test
+    public void testFixWorking_PackageProtected_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    String s = null;\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:11-2:12:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    class Inner {}\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:10-2:15:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:16-2:20:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_PackageProtected_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; class Test {}")
+                .run(MakePublic.class)
+                .findWarning("0:20-0:24:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test; public class Test {}");
+    }
+
+    @Test
+    public void testFixWorking_Private_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private String s = null;\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:19-2:20:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Private_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private class Inner {}\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:18-2:23:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Private_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    private static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:24-2:28:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * 1:23 modifier private not allowed here
+     *
+     * @throws Exception
+     */
+    @Test(expected = AssertionError.class)
+    public void testFixWorking_Private_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; private class Test {}")
+                .run(MakePublic.class);
+    }
+
+    @Test
+    public void testFixWorking_Protected_Field() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected String s = null;\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:21-2:22:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public String s = null;\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Protected_Inner_Class() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected class Inner {}\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:20-2:25:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public class Inner {}\n"
+                        + "}\n");
+    }
+
+    @Test
+    public void testFixWorking_Protected_Method() throws Exception {
+        HintTest.create()
+                .input("package test;\n"
+                        + "public class Test {\n"
+                        + "    protected static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n")
+                .run(MakePublic.class)
+                .findWarning("2:26-2:30:hint:" + Bundle.ERR_MakePublic())
+                .applyFix()
+                .assertCompilable()
+                .assertOutput("package test;\n"
+                        + "public class Test {\n"
+                        + "    public static void main(String[] args) {\n"
+                        + "    }\n"
+                        + "}\n");
+    }
+
+    /**
+     * 1:25 modifier protected not allowed here
+     *
+     * @throws Exception
+     */
+    @Test(expected = AssertionError.class)
+    public void testFixWorking_Protected_TopLevel_Class() throws Exception {
+        HintTest.create()
+                .input("package test; protected class Test {}")
+                .run(MakePublic.class);
+    }
+
+}
diff --git a/src/test/java/example/ChangeModifier.java b/src/test/java/example/ChangeModifier.java
new file mode 100644
index 0000000..9393023
--- /dev/null
+++ b/src/test/java/example/ChangeModifier.java
@@ -0,0 +1,21 @@
+package example;
+
+public class ChangeModifier {
+
+    public String s = null;
+
+    public void mainPublic() {
+    }
+
+    void mainDefault() {
+    }
+
+    protected void mainProtected() {
+    }
+
+    private void mainPrivate() {
+    }
+
+    public class InnerClass {
+    }
+}