save accesses the super-global variable $_POST.

Open

    private function save()
    {
        if (!is_admin()) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

save accesses the super-global variable $_POST.

Open

    private function save()
    {
        if (!is_admin()) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

save accesses the super-global variable $_GET.

Open

    private function save()
    {
        if (!is_admin()) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

save accesses the super-global variable $_POST.

Open

    private function save()
    {
        if (!is_admin()) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

File metaslider.class.php has 337 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

/**
 * Generic Slider super class. Extended by library specific classes.
 *

The class MetaSlider has an overall complexity of 72 which is very high. The configured complexity threshold is 50.

Open

class MetaSlider
{
    public $id         = 0; // slider ID
    public $identifier = 0; // unique identifier
    public $slides     = []; //slides belonging to this slider

• Exclude checks
  • Disable engine
  • Disable check

MetaSlider has 24 functions (exceeds 20 allowed). Consider refactoring.

Open

class MetaSlider
{
    public $id         = 0; // slider ID
    public $identifier = 0; // unique identifier
    public $slides     = []; //slides belonging to this slider

Method get_default_parameters has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function get_default_parameters()
    {
        $params = [
            'type'           => 'flex',
            'random'         => false,

Function get_javascript_parameters has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    private function get_javascript_parameters()
    {
        $options = [];

        // construct an array of all parameters

• Read up
  Read up

Function populate_slides has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    private function populate_slides()
    {
        $slides = [];

        $query = $this->get_slides();

• Read up
  Read up

Function get_setting has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function get_setting($name)
    {
        if (!isset($this->settings[$name])) {
            $defaults = $this->get_default_parameters();

• Read up
  Read up

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '        ' . $this->get_html_after();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '    <script type="text/javascript">';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '<!-- meta slider -->';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '    <div id="' . $this->get_container_id() . '">';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$pairs' which will lead to PHP notices.

Open

                $pairs[] = "{$key}: function($arg) {\n                " . implode("\n                ", $value) . "\n                }";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '    </div>';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $slideshow = implode("\n", $html);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '        ' . $this->get_html();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '</div>';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '<!--// meta slider-->';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '    ' . $this->get_inline_css();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$pairs' which will lead to PHP notices.

Open

                $pairs[] = "{$key}:{$value}";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '        ' . $this->get_inline_javascript();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '<div style="' . $this->get_container_style() . '" class="' . $this->get_container_class() . '">';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$html' which will lead to PHP notices.

Open

        $html[] = '    </script>';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$pairs' which will lead to PHP notices.

Open

        return implode(",\n                ", $pairs);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Missing class import via use statement (line '183', column '22').

Open

        $query = new WP_Query($args);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Avoid using static access to class '\Xmf\Request' in method 'save'.

Open

        if (\Xmf\Request::hasVar('deleteSlide', 'GET')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\Xmf\Request' in method 'save'.

Open

        if (\Xmf\Request::hasVar('settings', 'POST')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\Xmf\Request' in method 'save'.

Open

        if (\Xmf\Request::hasVar('attachment', 'POST')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method get_javascript_parameters uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $pairs[] = "{$key}:{$value}";
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\Xmf\Request' in method 'save'.

Open

        if (\Xmf\Request::hasVar('title', 'POST')) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method get_javascript_parameters uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $options[$param] = '"' . $val . '"';
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method get_setting uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            if (mb_strlen($this->settings[$name]) > 0) {
                return $this->settings[$name];
            }
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method populate_slides uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $slides[] = $return;
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method update_settings uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $new_settings[$checkbox] = 'false';
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class MetaSlider

• Exclude checks
  • Disable engine
  • Disable check

Avoid variables with short names like $id. Configured minimum length is 3.

Open

    public $id         = 0; // slider ID

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $id. Configured minimum length is 3.

Open

    public function __construct($id, $shortcode_settings)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The parameter $new_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $shortcode_settings is not named in camelCase.

Open

    public function __construct($id, $shortcode_settings)
    {
        $this->id         = $id;
        $this->settings   = array_merge($shortcode_settings, $this->get_settings());
        $this->identifier = 'metaslider_' . $this->id;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

Avoid variables with short names like $id. Configured minimum length is 3.

Open

        $id = apply_filters('metaslider_container_id', $container_id, $this->id, $this->settings);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The parameter $slide_id is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething($user_name) {
    }
}

Method name "MetaSlider::enqueue_scripts" is not in camel caps format

Open

    public function enqueue_scripts()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::update_slides" is not in camel caps format

Open

    private function update_slides($data)

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_html_after" is not in camel caps format

Open

    private function get_html_after()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_custom_javascript_after" is not in camel caps format

Open

    private function get_custom_javascript_after()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_container_id" is not in camel caps format

Open

    private function get_container_id()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::delete_slide" is not in camel caps format

Open

    private function delete_slide($slide_id)

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_settings" is not in camel caps format

Open

    private function get_settings()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_slides" is not in camel caps format

Open

    public function get_slides()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_javascript_parameters" is not in camel caps format

Open

    private function get_javascript_parameters()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_default_parameters" is not in camel caps format

Open

    public function get_default_parameters()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_inline_css" is not in camel caps format

Open

    private function get_inline_css()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::update_settings" is not in camel caps format

Open

    public function update_settings($new_settings)

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_inline_javascript" is not in camel caps format

Open

    private function get_inline_javascript()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_custom_javascript_before" is not in camel caps format

Open

    private function get_custom_javascript_before()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_container_class" is not in camel caps format

Open

    private function get_container_class()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_setting" is not in camel caps format

Open

    public function get_setting($name)

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::update_title" is not in camel caps format

Open

    private function update_title($title)

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::render_public_slides" is not in camel caps format

Open

    public function render_public_slides()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::populate_slides" is not in camel caps format

Open

    private function populate_slides()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_container_style" is not in camel caps format

Open

    private function get_container_style()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::get_identifier" is not in camel caps format

Open

    protected function get_identifier()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MetaSlider::render_admin_slides" is not in camel caps format

Open

    public function render_admin_slides()

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 184 characters

Open

        $checkboxes = ['noConflict', 'fullWidth', 'hoverPause', 'links', 'reverse', 'random', 'printCss', 'printJs', 'smoothHeight', 'center', 'smartCrop', 'carouselMode', 'autoPlay'];

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 223 characters

Open

        $script .= "\n            var slider = !window.jQuery ? window.setTimeout(timer_{$this->identifier}, 100) : !jQuery.isReady ? window.setTimeout(timer_{$this->identifier}, 100) : {$this->identifier}(window.jQuery);";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

        if (is_array($settings) && isset($settings['type']) && in_array($settings['type'], ['flex', 'coin', 'nivo', 'responsive'])) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 151 characters

Open

        if (('true' === $this->get_setting('carouselMode') || 'true' === $this->get_setting('fullWidth')) && ('coin' !== $this->get_setting('type'))) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 158 characters

Open

            wp_enqueue_script('metaslider-' . $this->get_setting('type') . '-slider', METASLIDER_ASSETS_URL . $this->js_path, ['jquery'], METASLIDER_VERSION);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 153 characters

Open

            wp_enqueue_style('metaslider-' . $this->get_setting('type') . '-slider', METASLIDER_ASSETS_URL . $this->css_path, false, METASLIDER_VERSION);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 136 characters

Open

                $pairs[] = "{$key}: function($arg) {\n                " . implode("\n                ", $value) . "\n                }";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

            wp_enqueue_style('metaslider-public', METASLIDER_ASSETS_URL . 'metaslider/public.css', false, METASLIDER_VERSION);

• Exclude checks
  • Disable engine
  • Disable check

The variable $custom_js is not named in camelCase.

Open

    private function get_custom_javascript_after()
    {
        $type = $this->get_setting('type');

        $custom_js = apply_filters("metaslider_{$type}_slider_javascript", '', $this->id);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $old_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js is not named in camelCase.

Open

    private function get_custom_javascript_after()
    {
        $type = $this->get_setting('type');

        $custom_js = apply_filters("metaslider_{$type}_slider_javascript", '', $this->id);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $old_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $slide_id is not named in camelCase.

Open

    private function update_slides($data)
    {
        foreach ($data as $slide_id => $fields) {
            do_action("metaslider_save_{$fields['type']}_slide", $slide_id, $this->id, $fields);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js_before is not named in camelCase.

Open

    private function get_inline_javascript()
    {
        $custom_js_before = $this->get_custom_javascript_before();
        $custom_js_after  = $this->get_custom_javascript_after();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js is not named in camelCase.

Open

    private function get_custom_javascript_before()
    {
        $type = $this->get_setting('type');

        $javascript = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_terms is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_terms is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_term is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $container_id is not named in camelCase.

Open

    private function get_container_id()
    {
        $container_id = 'metaslider_container_' . $this->id;

        $id = apply_filters('metaslider_container_id', $container_id, $this->id, $this->settings);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $shortcode_settings is not named in camelCase.

Open

    public function __construct($id, $shortcode_settings)
    {
        $this->id         = $id;
        $this->settings   = array_merge($shortcode_settings, $this->get_settings());
        $this->identifier = 'metaslider_' . $this->id;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_terms is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_term is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js is not named in camelCase.

Open

    private function get_custom_javascript_before()
    {
        $type = $this->get_setting('type');

        $javascript = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $current_term is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js_before is not named in camelCase.

Open

    private function get_inline_javascript()
    {
        $custom_js_before = $this->get_custom_javascript_before();
        $custom_js_after  = $this->get_custom_javascript_after();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js_after is not named in camelCase.

Open

    private function get_inline_javascript()
    {
        $custom_js_before = $this->get_custom_javascript_before();
        $custom_js_after  = $this->get_custom_javascript_after();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $container_id is not named in camelCase.

Open

    private function get_container_id()
    {
        $container_id = 'metaslider_container_' . $this->id;

        $id = apply_filters('metaslider_container_id', $container_id, $this->id, $this->settings);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js_after is not named in camelCase.

Open

    private function get_inline_javascript()
    {
        $custom_js_before = $this->get_custom_javascript_before();
        $custom_js_after  = $this->get_custom_javascript_after();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_terms is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $slide_id is not named in camelCase.

Open

    private function update_slides($data)
    {
        foreach ($data as $slide_id => $fields) {
            do_action("metaslider_save_{$fields['type']}_slide", $slide_id, $this->id, $fields);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js is not named in camelCase.

Open

    private function get_custom_javascript_before()
    {
        $type = $this->get_setting('type');

        $javascript = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $custom_js is not named in camelCase.

Open

    private function get_custom_javascript_after()
    {
        $type = $this->get_setting('type');

        $custom_js = apply_filters("metaslider_{$type}_slider_javascript", '', $this->id);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $slide_id is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $new_terms is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $slide_id is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The method get_setting is not named in camelCase.

Open

    public function get_setting($name)
    {
        if (!isset($this->settings[$name])) {
            $defaults = $this->get_default_parameters();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method populate_slides is not named in camelCase.

Open

    private function populate_slides()
    {
        $slides = [];

        $query = $this->get_slides();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method render_public_slides is not named in camelCase.

Open

    public function render_public_slides()
    {
        $html[] = '<!-- meta slider -->';
        $html[] = '<div style="' . $this->get_container_style() . '" class="' . $this->get_container_class() . '">';
        $html[] = '    ' . $this->get_inline_css();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_html_after is not named in camelCase.

Open

    private function get_html_after()
    {
        $type = $this->get_setting('type');

        $html = apply_filters("metaslider_{$type}_slider_html_after", '', $this->id, $this->settings);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_custom_javascript_after is not named in camelCase.

Open

    private function get_custom_javascript_after()
    {
        $type = $this->get_setting('type');

        $custom_js = apply_filters("metaslider_{$type}_slider_javascript", '', $this->id);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method delete_slide is not named in camelCase.

Open

    private function delete_slide($slide_id)
    {
        // Get the existing terms and only keep the ones we don't want removed
        $new_terms     = [];
        $current_terms = wp_get_object_terms($slide_id, 'ml-slider', ['fields' => 'ids']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method update_slides is not named in camelCase.

Open

    private function update_slides($data)
    {
        foreach ($data as $slide_id => $fields) {
            do_action("metaslider_save_{$fields['type']}_slide", $slide_id, $this->id, $fields);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_settings is not named in camelCase.

Open

    private function get_settings()
    {
        $settings = get_post_meta($this->id, 'ml-slider_settings', true);

        if (is_array($settings) && isset($settings['type']) && in_array($settings['type'], ['flex', 'coin', 'nivo', 'responsive'])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_default_parameters is not named in camelCase.

Open

    public function get_default_parameters()
    {
        $params = [
            'type'           => 'flex',
            'random'         => false,

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_slides is not named in camelCase.

Open

    public function get_slides()
    {
        $args = [
            'force_no_custom_order' => true,
            'orderby'               => 'menu_order',

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_javascript_parameters is not named in camelCase.

Open

    private function get_javascript_parameters()
    {
        $options = [];

        // construct an array of all parameters

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_container_style is not named in camelCase.

Open

    private function get_container_style()
    {
        // default
        $style = "max-width: {$this->get_setting('width')}px;";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_inline_javascript is not named in camelCase.

Open

    private function get_inline_javascript()
    {
        $custom_js_before = $this->get_custom_javascript_before();
        $custom_js_after  = $this->get_custom_javascript_after();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_custom_javascript_before is not named in camelCase.

Open

    private function get_custom_javascript_before()
    {
        $type = $this->get_setting('type');

        $javascript = '';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_container_class is not named in camelCase.

Open

    private function get_container_class()
    {
        $class = "metaslider metaslider-{$this->get_setting('type')} metaslider-{$this->id} ml-slider";

        // apply the css class setting

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_identifier is not named in camelCase.

Open

    protected function get_identifier()
    {
        return $this->identifier;
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method render_admin_slides is not named in camelCase.

Open

    public function render_admin_slides()
    {
        foreach ($this->slides as $slide) {
            echo $slide;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method update_settings is not named in camelCase.

Open

    public function update_settings($new_settings)
    {
        $old_settings = $this->get_settings();

        // convert submitted checkbox values from 'on' or 'off' to boolean values

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_inline_css is not named in camelCase.

Open

    private function get_inline_css()
    {
        $css = apply_filters('metaslider_css', '', $this->settings, $this->id);

        // use this to add the scoped attribute for HTML5 validation (if needed)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method enqueue_scripts is not named in camelCase.

Open

    public function enqueue_scripts()
    {
        if ('true' === $this->get_setting('printJs')) {
            wp_enqueue_script('metaslider-' . $this->get_setting('type') . '-slider', METASLIDER_ASSETS_URL . $this->js_path, ['jquery'], METASLIDER_VERSION);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method update_title is not named in camelCase.

Open

    private function update_title($title)
    {
        $slide = [
            'ID'         => $this->id,
            'post_title' => $title,

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}

The method get_container_id is not named in camelCase.

Open

    private function get_container_id()
    {
        $container_id = 'metaslider_container_' . $this->id;

        $id = apply_filters('metaslider_container_id', $container_id, $this->id, $this->settings);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}