Correct validation

Signed-off-by: Kirtan Gajjar <kirtangajjar95@gmail.com>

Author: kirtangajjar

src/Cron_Command.php

@@ -89,7 +89,7 @@ public function add( $args, $assoc_args ) {
 			}
 		}
 
-		$this->semi_colon_check( $command );
+		$this->validate_command( $command );
 		$command = $this->add_sh_c_wrapper( $command );
 
 		EE::db()->insert(
@@ -171,7 +171,7 @@ public function update( $args, $assoc_args ) {
 			$data_to_update['sitename'] = $site;
 		}
 		if ( $command ) {
-			$this->semi_colon_check( $command );
+			$this->validate_command( $command );
 			$command                   = $this->add_sh_c_wrapper( $command );
 			$data_to_update['command'] = $command;
 		}
@@ -332,12 +332,18 @@ private function site_php_container( $site ) {
 		return str_replace( '.', '', $site ) . '_php_1';
 	}
 
-	private function semi_colon_check( $command ) {
-		// Semicolons in commands do not work for now due to limitation of INI style config ofelia uses
+	/**
+	 * Ensures given command will not create problem with INI syntax
+	 */
+	private function validate_command( $command ) {
+		// Semicolons and Hash(#) in commands do not work for now due to limitation of INI style config ofelia uses
 		// See https://github.com/EasyEngine/cron-command/issues/4
 		if ( strpos( $command, ';' ) !== false ) {
 			EE::error( 'Command chaining using `;` - semi-colon is not supported currently. You can either use `&&` or `||` or creating a second cron job for the chained command.' );
 		}
+		if ( strpos( $command, '#' ) !== false ) {
+			EE::error( 'EasyEngine does not support commands with #' );
+		}
 	}
 
 	private function add_sh_c_wrapper( $command ) {