Refactor class into a separate file

Stops using the half-implemented Singleton pattern for the class.

Author: GaryJones

genesis-js-no-js.php

@@ -10,10 +10,10 @@
  * elements depending if JavaScript is active or not.
  *
  * @package   GenesisJsNoJs
- * @author    Gary Jones, Gamajo Tech <gamajo@gamajo.com>
- * @license   GPL-2.0+
+ * @author    Gary Jones
  * @link      https://github.com/GaryJones/genesis-js-no-js
- * @copyright 2014 Gary Jones, Gamajo Tech
+ * @copyright 2011 Gary Jones, Gamajo Tech
+ * @license   GPL-2.0+
  *
  * @wordpress-plugin
  * Plugin Name:       Genesis js / no-js
@@ -30,76 +30,7 @@
  * GitHub Branch:     master
  */
 
-/**
- * Plugin class for Genesis js / no-js
- *
- * @package GenesisJsNoJs
- */
-class Genesis_Js_No_Js {
-
-	/**
-	 * Holds copy of instance, so other plugins can remove our hooks.
-	 *
-	 * @since 1.0.0
-	 * @link http://core.trac.wordpress.org/attachment/ticket/16149/query-standard-format-posts.php
-	 * @link http://twitter.com/#!/markjaquith/status/66862769030438912
-	 *
-	 * @var Genesis_Js_No_Js
-	 */
-	static $instance;
-
-	/**
-	 * Constructor.
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct() {
-		self::$instance = $this;
-		add_action( 'init', array( &$this, 'init' ) );
-	}
-
-	/**
-	 * Add action and filter.
-	 *
-	 * @since 1.0.0
-	 */
-	public function init() {
-		add_filter( 'body_class', array( $this, 'body_class' ) );
-		add_action( 'genesis_before', array( $this, 'script' ), 1 );
-	}
-
-	/**
-	 * Add 'no-js' class to the body class values.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param array $classes Existing classes
-	 * @return array
-	 */
-	public function body_class( $classes ) {
-		$classes[] = 'no-js';
-		return $classes;
-	}
-
-	/**
-	 * Echo out the script that changes 'no-js' class to 'js'.
-	 *
-	 * @since 1.0.0
-	 */
-	public function script() {
-		?>
-<script type="text/javascript">
-//<![CDATA[
-(function(){
-var c = document.body.className;
-c = c.replace(/no-js/, 'js');
-document.body.className = c;
-})();
-//]]>
-</script>
-		<?php
-	}
-
-}
+require plugin_dir_path( __FILE__ ) . 'includes/class-genesis-js-no-js.php';
 
-new Genesis_Js_No_Js;
+$genesis_js_no_js = new Genesis_Js_No_Js;
+$genesis_js_no_js->run();

includes/class-genesis-js-no-js.php

@@ -0,0 +1,61 @@
+<?php
+/**
+ * Genesis Header Nav
+ *
+ * @package   GenesisJsNoJs
+ * @author    Gary Jones
+ * @link      https://github.com/GaryJones/genesis-header-nav
+ * @copyright 2011 Gary Jones, Gamajo Tech
+ * @license   GPL-2.0+
+ */
+
+/**
+ * Plugin class.
+ *
+ * @package GenesisJsNoJs
+ * @author  Gary Jones
+ */
+class Genesis_Js_No_Js {
+
+	/**
+	 * Add action and filter.
+	 *
+	 * @since 1.0.0
+	 */
+	public function run() {
+		add_filter( 'body_class', array( $this, 'body_class' ) );
+		add_action( 'genesis_before', array( $this, 'script' ), 1 );
+	}
+
+	/**
+	 * Add 'no-js' class to the body class values.
+	 *
+	 * @since 1.0.0
+	 *
+	 * @param array $classes Existing classes
+	 * @return array
+	 */
+	public function body_class( $classes ) {
+		$classes[] = 'no-js';
+		return $classes;
+	}
+
+	/**
+	 * Echo out the script that changes 'no-js' class to 'js'.
+	 *
+	 * @since 1.0.0
+	 */
+	public function script() {
+		?>
+<script type="text/javascript">
+//<![CDATA[
+(function(){
+var c = document.body.className;
+c = c.replace(/no-js/, 'js');
+document.body.className = c;
+})();
+//]]>
+</script>
+		<?php
+	}
+}