WPDFI\Admin\Notice

Complexity

Total Complexity

8

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	initializes()	0	4	1
A	add()	0	7	1
A	display()	0	9	2
A	get_classes()	0	4	1
A	get_type()	0	4	2
A	get_default_type()	0	4	1

<?php
namespace WPDFI\Admin;
/**
 * This class handle all notice stuffs in admin dashboard of this plugin
 *
 * @author Duc Bui Quang <ducbuiquangxd@gmail.com>
 * @since 1.0.0
 */
use WPDFI\Traits\Singleton;

final class Notice 
{
    use Singleton;

    /**
     * @traitDoc
     */
    public function initializes() 
    {   
        //
    }

    /**
     * Type of notice, it can be 'success', 'error', 'warning', 'info'.
     */
    private $type;
        
    /**
     * Message of the notice
     */
    private $message;
    
    /**
     * Add new notice to the admin dashboard.
     *
     * @param string $noticeMessage
     * @param string $noticeType
     * @since 1.0.0
     * @return void
     */
    public function add($noticeMessage, $noticeType = 'info') 
    {
        $this->message = $noticeMessage;
        $this->type = $noticeType;
        
        \add_action( 'admin_notices', [$this, 'display']);

Expected 1 spaces before closing bracket; 0 found

    }
    
    /**
     * Display notice template to the admin dashboard.
     * 
     * @since 1.0.0
     * @return \VA\Templater
     */
    public function display() 
    {
        if(!$this->message) return;

Space after opening control structure is required

It is generally a best practice to always use braces with control structures.

Expected 1 space before "!"; 0 found

Expected 1 space after "!"; 0 found

        
        echo \wpdfi()->templater->render('admin/notice', [

Expected next thing to be a escaping function, not '\'

            'classes'   => $this->get_classes(),
            'message'   => __($this->message, 'wpdfi')

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

Expected next thing to be a escaping function, not '$this'

        ]);
    }
    
    /**
     * Get all the classes of the notice
     * 
     * @since 1.0.0
     * @return string
     */
    private function get_classes() 
    {
        return "notice notice-{$this->get_type()} is-dismissible";
    }
    
    /**
     * Get the type of the notice
     *
     * @since 1.0.0
     * @return string
     */ 
    private function get_type()
    {
        return $this->type ? $this->type : $this->get_default_type();
    }
    
    /**
     * Get default type of the notice
     *
     * @since 1.0.0
     * @return string
     */
    private function get_default_type()
    {
        return 'info';
    }
}