Kunstmaan / KunstmaanBundlesCMS

src/Kunstmaan/AdminBundle/Twig/GoogleSignInTwigExtension.php

<?php

namespace Kunstmaan\AdminBundle\Twig;

use Twig\Extension\AbstractExtension;
use Twig\TwigFunction;

/**
 * Class GoogleSignInTwigExtension
 *
 * @final since 5.4
 */
class GoogleSignInTwigExtension extends AbstractExtension
{
    private $enabled;

    private $clientId;

    public function __construct($enabled, $clientId)
    {
        $this->enabled = $enabled;
        $this->clientId = $clientId;
    }

    /**
     * Returns a list of functions to add to the existing list.
     *
     * @return array An array of functions

Consider making the return type a bit more specific; maybe use TwigFunction[].

     */
    public function getFunctions()
    {
        return [
            new TwigFunction('google_signin_enabled', [$this, 'isGoogleSignInEnabled']),
            new TwigFunction('google_signin_client_id', [$this, 'getClientId']),
        ];
    }

    public function isGoogleSignInEnabled()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->enabled;
    }

    /**
     * @return mixed
     */
    public function getClientId()
    {
        return $this->clientId;
    }
}