sunnysideup / silverstripe-cms-dark-theme

src/Control/LeftAndMainDarkThemeToggle.php

<?php

namespace Sunnysideup\CMSDarkTheme\Control;

use SilverStripe\Control\Controller;
use SilverStripe\Control\HTTPRequest;
use SilverStripe\Control\HTTPResponse;
use Sunnysideup\CMSDarkTheme\Api\LeftAndMainDarkThemeApi;

/**
 * Class \Sunnysideup\CMSDarkTheme\Control\LeftAndMainDarkThemeToggle
 *
 */
class LeftAndMainDarkThemeToggle extends Controller
{
    private static $url_segment = 'admin/displaymode';

    private static $url_rule = '/$Action/$ID/$OtherID';

    // private static $ignore_menuitem = false;

    private static $allowed_actions = [
        'switch',
        'setlightmode',
        'setdarkmode',
    ];

    /**
     * set mode and then.
     *
     * @param HTTPRequest $request
     *
     * @return HTTPResponse
     */
    public function index($request)
    {
        return $this->switch($request);
    }

    /**
     * set mode and then.
     *
     * @param HTTPRequest $request
     *
     * @return HTTPResponse
     */
    public function switch($request)

The parameter $request is not used and could be removed.

    {
        $owner = $this->getOwner();

The method getOwner() does not exist on Sunnysideup\CMSDarkTheme\Control\LeftAndMainDarkThemeToggle. Since you implemented __call, consider adding a @method annotation.

        if (LeftAndMainDarkThemeApi::is_dark_mode()) {
            LeftAndMainDarkThemeApi::set_display_mode('Light');
        } else {
            LeftAndMainDarkThemeApi::set_display_mode('Dark');
        }

        return $owner->redirect('/admin/myprofile#Root_Cms');
    }

    public function setlightmode($request): HTTPResponse

The parameter $request is not used and could be removed.

    {
        $owner = $this->getOwner();
        LeftAndMainDarkThemeApi::set_display_mode('Light');

        return $owner->redirect('admin/myprofile#Root_Cms');
    }

    public function setdarkmode($request): HTTPResponse

The parameter $request is not used and could be removed.

    {
        $owner = $this->getOwner();
        LeftAndMainDarkThemeApi::set_display_mode('Dark');

        return $owner->redirect('admin/myprofile#Root_Cms');
    }
}