Tan-007 / myslideshow

includes/class-myslideshow.php

Spacing +16 added lines, -16 removed lines

@@ -67,7 +67,7 @@ 
 	 * @since    1.0.0
 	 */
 	public function __construct() {
-		if ( defined( 'MYSLIDESHOW_VERSION' ) ) {
+		if (defined('MYSLIDESHOW_VERSION')) {
 			$this->version = MYSLIDESHOW_VERSION;
 		} else {
 			$this->version = '1.0.0';
@@ -103,29 +103,29 @@ 
 		 * The class responsible for orchestrating the actions and filters of the
 		 * core plugin.
 		 */
-		require_once plugin_dir_path( dirname( __FILE__ ) ) . 'includes/class-myslideshow-loader.php';
+		require_once plugin_dir_path(dirname(__FILE__)) . 'includes/class-myslideshow-loader.php';
 
 		/**
 		 * The class responsible for defining internationalization functionality
 		 * of the plugin.
 		 */
-		require_once plugin_dir_path( dirname( __FILE__ ) ) . 'includes/class-myslideshow-i18n.php';
+		require_once plugin_dir_path(dirname(__FILE__)) . 'includes/class-myslideshow-i18n.php';
 
 		/**
 		 * The class responsible for defining all actions that occur in the admin area.
 		 */
-		require_once plugin_dir_path( dirname( __FILE__ ) ) . 'admin/class-myslideshow-admin.php';
+		require_once plugin_dir_path(dirname(__FILE__)) . 'admin/class-myslideshow-admin.php';
 
 		/**
 		 * The class responsible for defining all actions that occur in the public-facing
 		 * side of the site.
 		 */
-		require_once plugin_dir_path( dirname( __FILE__ ) ) . 'public/class-myslideshow-public.php';
+		require_once plugin_dir_path(dirname(__FILE__)) . 'public/class-myslideshow-public.php';
 
 		/**
 		 * this file contains all admin-side rendering
 		 */
-		require_once( plugin_dir_path( dirname( __FILE__ ) ) . 'admin/partials/myslideshow-admin-display.php' );
+		require_once(plugin_dir_path(dirname(__FILE__)) . 'admin/partials/myslideshow-admin-display.php');
 
 
 		$this->loader = new Myslideshow_Loader();
@@ -145,7 +145,7 @@ 
 
 		$plugin_i18n = new Myslideshow_i18n();
 
-		$this->loader->add_action( 'plugins_loaded', $plugin_i18n, 'load_plugin_textdomain' );
+		$this->loader->add_action('plugins_loaded', $plugin_i18n, 'load_plugin_textdomain');
 
 	}
 
@@ -158,19 +158,19 @@ 
 	 */
 	private function define_admin_hooks() {
 
-		$plugin_admin = new Myslideshow_Admin( $this->get_plugin_name(), $this->get_version() );
+		$plugin_admin = new Myslideshow_Admin($this->get_plugin_name(), $this->get_version());
 
-		$this->loader->add_action( 'admin_enqueue_scripts', $plugin_admin, 'enqueue_styles' );
-		$this->loader->add_action( 'admin_enqueue_scripts', $plugin_admin, 'enqueue_scripts' );
+		$this->loader->add_action('admin_enqueue_scripts', $plugin_admin, 'enqueue_styles');
+		$this->loader->add_action('admin_enqueue_scripts', $plugin_admin, 'enqueue_scripts');
 
 		// Hook for slideshow settings page
-		$this->loader->add_action( 'admin_menu', $plugin_admin, 'mss_settings_page' );
+		$this->loader->add_action('admin_menu', $plugin_admin, 'mss_settings_page');
 
 		// Hook for registering settings
-		$this->loader->add_action( 'admin_init', $plugin_admin, 'mss_register_settings' );
+		$this->loader->add_action('admin_init', $plugin_admin, 'mss_register_settings');
 
 		 // Hook for registering shortcode
-		 $this->loader->add_action( 'init', $plugin_admin, 'mss_register_shortcodes' );
+		 $this->loader->add_action('init', $plugin_admin, 'mss_register_shortcodes');
 
 	}
 
@@ -183,10 +183,10 @@ 
 	 */
 	private function define_public_hooks() {
 
-		$plugin_public = new Myslideshow_Public( $this->get_plugin_name(), $this->get_version() );
+		$plugin_public = new Myslideshow_Public($this->get_plugin_name(), $this->get_version());
 
-		$this->loader->add_action( 'wp_enqueue_scripts', $plugin_public, 'enqueue_styles' );
-		$this->loader->add_action( 'wp_enqueue_scripts', $plugin_public, 'enqueue_scripts' );
+		$this->loader->add_action('wp_enqueue_scripts', $plugin_public, 'enqueue_styles');
+		$this->loader->add_action('wp_enqueue_scripts', $plugin_public, 'enqueue_scripts');
 
 	}
 

includes/class-myslideshow-i18n.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 		load_plugin_textdomain(
 			'myslideshow',
 			false,
-			dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
+			dirname(dirname(plugin_basename(__FILE__))) . '/languages/'
 		);
 
 	}

includes/class-myslideshow-loader.php

Spacing +9 added lines, -9 removed lines

@@ -63,8 +63,8 @@ 
 	 * @param    int                  $priority         Optional. The priority at which the function should be fired. Default is 10.
 	 * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
 	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		$this->actions = $this->add( $this->actions, $hook, $component, $callback, $priority, $accepted_args );
+	public function add_action($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		$this->actions = $this->add($this->actions, $hook, $component, $callback, $priority, $accepted_args);
 	}
 
 	/**
@@ -77,8 +77,8 @@ 
 	 * @param    int                  $priority         Optional. The priority at which the function should be fired. Default is 10.
 	 * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
 	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		$this->filters = $this->add( $this->filters, $hook, $component, $callback, $priority, $accepted_args );
+	public function add_filter($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		$this->filters = $this->add($this->filters, $hook, $component, $callback, $priority, $accepted_args);
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @param    int                  $accepted_args    The number of arguments that should be passed to the $callback.
 	 * @return   array                                  The collection of actions and filters registered with WordPress.
 	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+	private function add($hooks, $hook, $component, $callback, $priority, $accepted_args) {
 
 		$hooks[] = array(
 			'hook'          => $hook,
@@ -116,12 +116,12 @@ 
 	 */
 	public function run() {
 
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->filters as $hook) {
+			add_filter($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->actions as $hook) {
+			add_action($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
 	}

myslideshow.php

Indentation +3 added lines, -3 removed lines

@@ -41,13 +41,13 @@
  * Global options
  */
 $defaults = array(
-    'image_urls' => '',
-    'images_number' => '0',
+	'image_urls' => '',
+	'images_number' => '0',
 );
 $slides_options = get_option( 'mss_settings', $defaults );
 
 if ( $slides_options == '' ) {
-    update_option( 'mss_settings', $defaults );
+	update_option( 'mss_settings', $defaults );
 }
 
 /**

Spacing +10 added lines, -10 removed lines

@@ -26,7 +26,7 @@ 
  */
 
 // If this file is called directly, abort.
-if ( ! defined( 'WPINC' ) ) {
+if ( ! defined('WPINC')) {
 	die;
 }
 
@@ -35,7 +35,7 @@ 
  * Start at version 1.0.0 and use SemVer - https://semver.org
  * Rename this for your plugin and update it as you release new versions.
  */
-define( 'MYSLIDESHOW_VERSION', '1.0.1' );
+define('MYSLIDESHOW_VERSION', '1.0.1');
 
 /**
  * Global options
@@ -44,10 +44,10 @@ 
     'image_urls' => '',
     'images_number' => '0',
 );
-$slides_options = get_option( 'mss_settings', $defaults );
+$slides_options = get_option('mss_settings', $defaults);
 
-if ( $slides_options == '' ) {
-    update_option( 'mss_settings', $defaults );
+if ($slides_options == '') {
+    update_option('mss_settings', $defaults);
 }
 
 /**
@@ -55,7 +55,7 @@ 
  * This action is documented in includes/class-myslideshow-activator.php
  */
 function activate_myslideshow() {
-	require_once plugin_dir_path( __FILE__ ) . 'includes/class-myslideshow-activator.php';
+	require_once plugin_dir_path(__FILE__) . 'includes/class-myslideshow-activator.php';
 	Myslideshow_Activator::activate();
 }
 
@@ -64,18 +64,18 @@ 
  * This action is documented in includes/class-myslideshow-deactivator.php
  */
 function deactivate_myslideshow() {
-	require_once plugin_dir_path( __FILE__ ) . 'includes/class-myslideshow-deactivator.php';
+	require_once plugin_dir_path(__FILE__) . 'includes/class-myslideshow-deactivator.php';
 	Myslideshow_Deactivator::deactivate();
 }
 
-register_activation_hook( __FILE__, 'activate_myslideshow' );
-register_deactivation_hook( __FILE__, 'deactivate_myslideshow' );
+register_activation_hook(__FILE__, 'activate_myslideshow');
+register_deactivation_hook(__FILE__, 'deactivate_myslideshow');
 
 /**
  * The core plugin class that is used to define internationalization,
  * admin-specific hooks, and public-facing site hooks.
  */
-require plugin_dir_path( __FILE__ ) . 'includes/class-myslideshow.php';
+require plugin_dir_path(__FILE__) . 'includes/class-myslideshow.php';
 
 /**
  * Begins execution of the plugin.

uninstall.php

Spacing +1 added lines, -1 removed lines

@@ -26,6 +26,6 @@
  */
 
 // If uninstall not called from WordPress, then exit.
-if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
+if ( ! defined('WP_UNINSTALL_PLUGIN')) {
 	exit;
 }

public/class-myslideshow-public.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
 	 * @param      string    $plugin_name       The name of the plugin.
 	 * @param      string    $version    The version of this plugin.
 	 */
-	public function __construct( $plugin_name, $version ) {
+	public function __construct($plugin_name, $version) {
 
 		$this->plugin_name = $plugin_name;
 		$this->version = $version;
@@ -73,7 +73,7 @@ 
 		 * class.
 		 */
 
-		wp_enqueue_style( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'css/myslideshow-public.css', array(), $this->version, 'all' );
+		wp_enqueue_style($this->plugin_name, plugin_dir_url(__FILE__) . 'css/myslideshow-public.css', array(), $this->version, 'all');
 
 	}
 
@@ -96,7 +96,7 @@ 
 		 * class.
 		 */
 
-		wp_enqueue_script( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'js/myslideshow-public.js', array( 'jquery' ), $this->version, false );
+		wp_enqueue_script($this->plugin_name, plugin_dir_url(__FILE__) . 'js/myslideshow-public.js', array('jquery'), $this->version, false);
 
 	}
 

admin/class-myslideshow-admin.php

Indentation +84 added lines, -84 removed lines

@@ -102,94 +102,94 @@
 	}
 
 	/********* CUSTOM FUNCTIONS START HERE ***********/
-	    /**
-     * Function for our settings page
-     * 
-     * @since    1.0.1
-     */
-    public function mss_settings_page() {
-
-        /**
-         * for adding the settings page under "Settings" menu
-         * add_options_page( $page_title, $menu_title, $capability, $menu_slug, $function );
-        */
-
-        /**
-         * for adding the page under root menu
-         * add_menu_page( string $page_title, string $menu_title, string $capability,
-         *  string $menu_slug, callable $function = '', string $icon_url = '', int $position = null )
-         */
-
-        //'book_render_settings_page' function in : .../admin/partials/rahi_wpbook-admin-display.php   
-        add_menu_page( 'My Slide Show Settings', 'Slideshow Settings', 'manage_options', 'slideshow-settings', 'mss_render_settings_page',
-            'dashicons-chart-pie', '59' );
+		/**
+		 * Function for our settings page
+		 * 
+		 * @since    1.0.1
+		 */
+	public function mss_settings_page() {
+
+		/**
+		 * for adding the settings page under "Settings" menu
+		 * add_options_page( $page_title, $menu_title, $capability, $menu_slug, $function );
+		 */
+
+		/**
+		 * for adding the page under root menu
+		 * add_menu_page( string $page_title, string $menu_title, string $capability,
+		 *  string $menu_slug, callable $function = '', string $icon_url = '', int $position = null )
+		 */
+
+		//'book_render_settings_page' function in : .../admin/partials/rahi_wpbook-admin-display.php   
+		add_menu_page( 'My Slide Show Settings', 'Slideshow Settings', 'manage_options', 'slideshow-settings', 'mss_render_settings_page',
+			'dashicons-chart-pie', '59' );
 	}
 	
 	/**
-     * Function for registering settings
-     * 
-     * @since    1.0.1
-     */
-    function mss_register_settings() {
-        register_setting( 'mss-settings-group', 'mss_settings' );
-    }
+	 * Function for registering settings
+	 * 
+	 * @since    1.0.1
+	 */
+	function mss_register_settings() {
+		register_setting( 'mss-settings-group', 'mss_settings' );
+	}
 
 	/**
-     * Function for adding short code
-     * 
-     * @since    1.0.2
-     */
-    public function mss_add_shortcode(  ) {
-
-        // $atts = shortcode_atts(
-        //     array( 
-        //         'id'          => '',
-        //         'author_name' => '',
-        //         'year'        => '',
-        //         'category'    => '',
-        //         'tag'         => '',
-        //         'publisher'   => '',
-        //     ), $atts
-        // );
-
-        // I messed it up here
-        // $args = array(
-        //     'post_type'      => 'book',
-        //     'post_status'    => 'publish',
-        //     'posts_per_page' => 999999,
-        //     'order'          => 'ASC',
-        //     'author'         => $atts[ 'author_name' ],
-        // );
-
-        // if ( $atts[ 'category' ] != '' ) {
-        //     $args['tax_query'] = array(
-        //         array(
-        //             'taxonomy' => 'book_category',
-        //             'terms'    => array( $atts[ 'category' ] ),
-        //             'field'    => 'slug',
-        //             'operator' => 'IN',
-        //         ),
-        //     );
-        // }
-
-        // if ( $atts[ 'tag' ] != '' ) {
-        //     $args[ 'tax_query' ] = array(
-        //         array(
-        //             'taxonomy' => 'book_tag',
-        //             'terms'    => array( $atts[ 'tag' ] ),
-        //             'field'    => 'slug',
-        //             'operator' => 'AND',
-        //         ),
-        //     );
-        // }
-
-        // function in : .../admin/partials/myslideshow-admin-display.php        
-        return render_mss_shortcode(  );
-
-    }
-
-    function mss_register_shortcodes() {
-        add_shortcode( 'slideshow', array( $this, 'mss_add_shortcode' ) );
-    }
+	 * Function for adding short code
+	 * 
+	 * @since    1.0.2
+	 */
+	public function mss_add_shortcode(  ) {
+
+		// $atts = shortcode_atts(
+		//     array( 
+		//         'id'          => '',
+		//         'author_name' => '',
+		//         'year'        => '',
+		//         'category'    => '',
+		//         'tag'         => '',
+		//         'publisher'   => '',
+		//     ), $atts
+		// );
+
+		// I messed it up here
+		// $args = array(
+		//     'post_type'      => 'book',
+		//     'post_status'    => 'publish',
+		//     'posts_per_page' => 999999,
+		//     'order'          => 'ASC',
+		//     'author'         => $atts[ 'author_name' ],
+		// );
+
+		// if ( $atts[ 'category' ] != '' ) {
+		//     $args['tax_query'] = array(
+		//         array(
+		//             'taxonomy' => 'book_category',
+		//             'terms'    => array( $atts[ 'category' ] ),
+		//             'field'    => 'slug',
+		//             'operator' => 'IN',
+		//         ),
+		//     );
+		// }
+
+		// if ( $atts[ 'tag' ] != '' ) {
+		//     $args[ 'tax_query' ] = array(
+		//         array(
+		//             'taxonomy' => 'book_tag',
+		//             'terms'    => array( $atts[ 'tag' ] ),
+		//             'field'    => 'slug',
+		//             'operator' => 'AND',
+		//         ),
+		//     );
+		// }
+
+		// function in : .../admin/partials/myslideshow-admin-display.php        
+		return render_mss_shortcode(  );
+
+	}
+
+	function mss_register_shortcodes() {
+		add_shortcode( 'slideshow', array( $this, 'mss_add_shortcode' ) );
+	}
 
 }

Spacing +7 added lines, -7 removed lines

@@ -47,7 +47,7 @@ 
 	 * @param      string    $plugin_name       The name of this plugin.
 	 * @param      string    $version    The version of this plugin.
 	 */
-	public function __construct( $plugin_name, $version ) {
+	public function __construct($plugin_name, $version) {
 
 		
 		$this->plugin_name = $plugin_name;
@@ -74,7 +74,7 @@ 
 		 * class.
 		 */
 
-		wp_enqueue_style( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'css/myslideshow-admin.css', array(), $this->version, 'all' );
+		wp_enqueue_style($this->plugin_name, plugin_dir_url(__FILE__) . 'css/myslideshow-admin.css', array(), $this->version, 'all');
 
 	}
 
@@ -97,7 +97,7 @@ 
 		 * class.
 		 */
 
-		wp_enqueue_script( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'js/myslideshow-admin.js', array( 'jquery' ), $this->version, false );
+		wp_enqueue_script($this->plugin_name, plugin_dir_url(__FILE__) . 'js/myslideshow-admin.js', array('jquery'), $this->version, false);
 
 	}
 
@@ -121,8 +121,8 @@ 
          */
 
         //'book_render_settings_page' function in : .../admin/partials/rahi_wpbook-admin-display.php   
-        add_menu_page( 'My Slide Show Settings', 'Slideshow Settings', 'manage_options', 'slideshow-settings', 'mss_render_settings_page',
-            'dashicons-chart-pie', '59' );
+        add_menu_page('My Slide Show Settings', 'Slideshow Settings', 'manage_options', 'slideshow-settings', 'mss_render_settings_page',
+            'dashicons-chart-pie', '59');
 	}
 	
 	/**
@@ -131,7 +131,7 @@ 
      * @since    1.0.1
      */
     function mss_register_settings() {
-        register_setting( 'mss-settings-group', 'mss_settings' );
+        register_setting('mss-settings-group', 'mss_settings');
     }
 
 	/**
@@ -189,7 +189,7 @@ 
     }
 
     function mss_register_shortcodes() {
-        add_shortcode( 'slideshow', array( $this, 'mss_add_shortcode' ) );
+        add_shortcode('slideshow', array($this, 'mss_add_shortcode'));
     }
 
 }

admin/partials/myslideshow-admin-display.php

Indentation +36 added lines, -36 removed lines

@@ -19,23 +19,23 @@ 
  */
 function mss_render_settings_page() {
 
-    global $slides_options;
-    $number_of_images = $slides_options[ 'images_number' ];
-    $urls = explode( ";", $slides_options[ 'image_urls' ] );
-
-    if(function_exists( 'wp_enqueue_media' )){
-        wp_enqueue_media();
-    }else{
-        wp_enqueue_style('thickbox');
-        wp_enqueue_script('media-upload');
-        wp_enqueue_script('thickbox');
-    }
-
-    ?>
+	global $slides_options;
+	$number_of_images = $slides_options[ 'images_number' ];
+	$urls = explode( ";", $slides_options[ 'image_urls' ] );
+
+	if(function_exists( 'wp_enqueue_media' )){
+		wp_enqueue_media();
+	}else{
+		wp_enqueue_style('thickbox');
+		wp_enqueue_script('media-upload');
+		wp_enqueue_script('thickbox');
+	}
+
+	?>
         <form id="settings_form" method="post" action="options.php">
     <?php
-        settings_fields( 'mss-settings-group' );
-    ?>
+		settings_fields( 'mss-settings-group' );
+	?>
             <div class="settings-wrapper">
                 
                 <div class="header">
@@ -49,18 +49,18 @@ 
                     </div>
 
                     <?php 
-                    // render images from the db
-                    for ( $i = 0; $i < $number_of_images; $i++ ) {
+					// render images from the db
+					for ( $i = 0; $i < $number_of_images; $i++ ) {
 
-                        echo '<div class="image sortable" style="background-image: url(' . $urls[ $i ] . ');">
+						echo '<div class="image sortable" style="background-image: url(' . $urls[ $i ] . ');">
                             <div class="delete-image">
                                 <span class="dashicons dashicons-trash"></span>
                             </div>
                         </div>';
 
-                    }
+					}
                     
-                    ?>
+					?>
                 </div>
 
                 <div class="footer">
@@ -80,27 +80,27 @@ 
  */
 function render_mss_shortcode() {
 
-    // get slide show options
-    global $slides_options;
-    $html = '';
+	// get slide show options
+	global $slides_options;
+	$html = '';
     
-    $number_of_images = $slides_options[ 'images_number' ];
+	$number_of_images = $slides_options[ 'images_number' ];
 
-    // check if there are any images added in the slideshow or not
-    if ( $number_of_images == 0 ) {
-        $html .= '<h5> No images found. You can add images from \'Slideshow Settings\' menu in the admin panel. </h5>';
-        return $html;
-    }
+	// check if there are any images added in the slideshow or not
+	if ( $number_of_images == 0 ) {
+		$html .= '<h5> No images found. You can add images from \'Slideshow Settings\' menu in the admin panel. </h5>';
+		return $html;
+	}
 
-    $image_urls = explode( ";", $slides_options[ 'image_urls' ] );
-    $html .= '<div id="slides-container">';
+	$image_urls = explode( ";", $slides_options[ 'image_urls' ] );
+	$html .= '<div id="slides-container">';
 
-    for ( $i = 0; $i < $number_of_images; $i++ ) {
-        $html .= '<img class="slide" src="'. $image_urls[ $i ] . '"/>';
-    }
+	for ( $i = 0; $i < $number_of_images; $i++ ) {
+		$html .= '<img class="slide" src="'. $image_urls[ $i ] . '"/>';
+	}
 
-    $html .= '</div>';
+	$html .= '</div>';
 
-    return $html;
+	return $html;
 }
 ?>

Spacing +12 added lines, -12 removed lines

@@ -20,12 +20,12 @@ 
 function mss_render_settings_page() {
 
     global $slides_options;
-    $number_of_images = $slides_options[ 'images_number' ];
-    $urls = explode( ";", $slides_options[ 'image_urls' ] );
+    $number_of_images = $slides_options['images_number'];
+    $urls = explode(";", $slides_options['image_urls']);
 
-    if(function_exists( 'wp_enqueue_media' )){
+    if (function_exists('wp_enqueue_media')) {
         wp_enqueue_media();
-    }else{
+    } else {
         wp_enqueue_style('thickbox');
         wp_enqueue_script('media-upload');
         wp_enqueue_script('thickbox');
@@ -34,7 +34,7 @@ 
     ?>
         <form id="settings_form" method="post" action="options.php">
     <?php
-        settings_fields( 'mss-settings-group' );
+        settings_fields('mss-settings-group');
     ?>
             <div class="settings-wrapper">
                 
@@ -50,9 +50,9 @@ 
 
                     <?php 
                     // render images from the db
-                    for ( $i = 0; $i < $number_of_images; $i++ ) {
+                    for ($i = 0; $i < $number_of_images; $i++) {
 
-                        echo '<div class="image sortable" style="background-image: url(' . $urls[ $i ] . ');">
+                        echo '<div class="image sortable" style="background-image: url(' . $urls[$i] . ');">
                             <div class="delete-image">
                                 <span class="dashicons dashicons-trash"></span>
                             </div>
@@ -84,19 +84,19 @@ 
     global $slides_options;
     $html = '';
     
-    $number_of_images = $slides_options[ 'images_number' ];
+    $number_of_images = $slides_options['images_number'];
 
     // check if there are any images added in the slideshow or not
-    if ( $number_of_images == 0 ) {
+    if ($number_of_images == 0) {
         $html .= '<h5> No images found. You can add images from \'Slideshow Settings\' menu in the admin panel. </h5>';
         return $html;
     }
 
-    $image_urls = explode( ";", $slides_options[ 'image_urls' ] );
+    $image_urls = explode(";", $slides_options['image_urls']);
     $html .= '<div id="slides-container">';
 
-    for ( $i = 0; $i < $number_of_images; $i++ ) {
-        $html .= '<img class="slide" src="'. $image_urls[ $i ] . '"/>';
+    for ($i = 0; $i < $number_of_images; $i++) {
+        $html .= '<img class="slide" src="' . $image_urls[$i] . '"/>';
     }
 
     $html .= '</div>';

Braces +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
     if(function_exists( 'wp_enqueue_media' )){
         wp_enqueue_media();
-    }else{
+    } else{
         wp_enqueue_style('thickbox');
         wp_enqueue_script('media-upload');
         wp_enqueue_script('thickbox');