From d8a849677b7c049f1336b5546d7c4f414b63e8c3 Mon Sep 17 00:00:00 2001
From: ernilambar <nilambar@outlook.com>
Date: Fri, 20 Jan 2017 12:05:18 +0545
Subject: [PATCH] Fix sniffer issues in admin files

---
 inc/admin.php      | 23 +++++++++++++++++++----
 inc/helpers.php    | 25 ++++++++++++++++++++++++-
 ns-theme-check.php |  9 +++++++--
 3 files changed, 50 insertions(+), 7 deletions(-)

diff --git a/inc/admin.php b/inc/admin.php
index bcdc2e5..546ed87 100644
--- a/inc/admin.php
+++ b/inc/admin.php
@@ -1,13 +1,28 @@
 <?php
+/**
+ * Admin functions
+ *
+ * @package NS_Theme_Check
+ */
 
-add_action( 'admin_menu', 'ns_theme_check_admin_menu' );
-
+/**
+ * Register admin menu.
+ *
+ * @since 0.1.0
+ */
 function ns_theme_check_admin_menu() {
 
-	add_theme_page( __( 'NS Theme Check', 'ns-theme-check' ), __( 'NS Theme Check', 'ns-theme-check' ), 'manage_options', 'ns-theme-check', 'ns_theme_check_render_admin_page', 'dashicons-tickets' );
+	add_theme_page( esc_html__( 'NS Theme Check', 'ns-theme-check' ), esc_html__( 'NS Theme Check', 'ns-theme-check' ), 'manage_options', 'ns-theme-check', 'ns_theme_check_render_admin_page' );
 
 }
 
+add_action( 'admin_menu', 'ns_theme_check_admin_menu' );
+
+/**
+ * Callback for admin page.
+ *
+ * @since 0.1.0
+ */
 function ns_theme_check_render_admin_page() {
 	?>
 	<div class="wrap">
@@ -16,6 +31,6 @@ function ns_theme_check_render_admin_page() {
 		<?php ns_theme_check_render_form(); ?>
 		<hr />
 		<?php ns_theme_check_render_output(); ?>
-	</div><!-- .wrap -->
+	</div>
 	<?php
 }
diff --git a/inc/helpers.php b/inc/helpers.php
index 9c087e2..0bcba2a 100644
--- a/inc/helpers.php
+++ b/inc/helpers.php
@@ -1,5 +1,15 @@
 <?php
-
+/**
+ * Helper functions
+ *
+ * @package NS_Theme_Check
+ */
+
+/**
+ * Render form.
+ *
+ * @since 0.1.0
+ */
 function ns_theme_check_render_form() {
 	$all_themes = wp_get_themes();
 	$themes = array();
@@ -41,6 +51,11 @@ function ns_theme_check_render_form() {
 	<?php
 }
 
+/**
+ * Render sniff results.
+ *
+ * @since 0.1.0
+ */
 function ns_theme_check_render_output() {
 
 	// Bail if empty.
@@ -66,6 +81,14 @@ function ns_theme_check_render_output() {
 
 }
 
+/**
+ * Perform sniff check.
+ *
+ * @since 0.1.0
+ *
+ * @param string $theme Theme slug.
+ * @param array  $args Arguments.
+ */
 function ns_theme_check_do_sniff( $theme, $args = array() ) {
 
 	require_once NS_THEME_CHECK_DIR . '/vendor/autoload.php';
diff --git a/ns-theme-check.php b/ns-theme-check.php
index 97f51c3..7b946f0 100644
--- a/ns-theme-check.php
+++ b/ns-theme-check.php
@@ -1,14 +1,16 @@
 <?php
 /**
  * Plugin Name:       NS Theme Check
- * Plugin URI:
+ * Plugin URI:        https://github.com/ernilambar/ns-theme-check
  * Description:       Theme Check using sniffs.
- * Version:           0.1.0
+ * Version:           0.1.1
  * Author:            Nilambar Sharma
  * Author URI:        http://nilambar.net
  * Text Domain:       ns-theme-check
  * License:           GPL-2.0+
  * License URI:       http://www.gnu.org/licenses/gpl-2.0.txt
+ *
+ * @package NS_Theme_Check
  */
 
 // If this file is called directly, abort.
@@ -20,5 +22,8 @@
 define( 'NS_THEME_CHECK_DIR', rtrim( plugin_dir_path( __FILE__ ), '/' ) );
 define( 'NS_THEME_CHECK_URL', rtrim( plugin_dir_url( __FILE__ ), '/' ) );
 
+// Load helpers.
 require_once NS_THEME_CHECK_DIR . '/inc/helpers.php';
+
+// Load admin.
 require_once NS_THEME_CHECK_DIR . '/inc/admin.php';