locomotivemtl / charcoal-admin

src/Charcoal/Admin/AdminScript.php

<?php

namespace Charcoal\Admin;

// From Pimple
use Pimple\Container;

// From 'league/climate'
use League\CLImate\TerminalObject\Dynamic\Input as LeagueInput;

// From 'charcoal-factory'
use Charcoal\Factory\FactoryInterface;

// From 'charcoal-app'
use Charcoal\App\Script\AbstractScript;

// From 'charcoal-property'
use Charcoal\Property\PropertyInterface;

// From 'charcoal-translator'
use Charcoal\Translator\TranslatorAwareTrait;

// From 'charcoal-admin'
use Charcoal\Admin\Support\BaseUrlTrait;

/**
 *
 */
abstract class AdminScript extends AbstractScript
{
    use BaseUrlTrait;
    use TranslatorAwareTrait;

    /**
     * The model factory.
     *
     * @var FactoryInterface
     */
    private $modelFactory;

    /**
     * @param  Container $container Pimple DI container.
     * @return void
     */
    protected function setDependencies(Container $container)
    {
        parent::setDependencies($container);

        // Satisfies TranslatorAwareTrait dependencies
        $this->setTranslator($container['translator']);

        // Satisfies BaseUrlTrait dependencies
        $this->setBaseUrl($container['base-url']);
        $this->setAdminUrl($container['admin/base-url']);

        // Satisfies AdminScript dependencies
        $this->setModelFactory($container['model/factory']);
    }

    /**
     * @param  PropertyInterface $prop  The property to retrieve input from.
     * @param  string|null       $label Optional. The input label.
     * @return LeagueInput The League's terminal input object.
     */
    protected function propertyToInput(PropertyInterface $prop, $label = null)
    {
        $climate = $this->climate();

        if ($label === null) {
            $label = sprintf(
                'Enter value for "%s":',
                $prop->label()
            );
        }

        if ($prop->type() === 'password') {
            return $this->passwordInput($prop, $label);
        } elseif ($prop->type() === 'boolean') {
            return $this->booleanInput($prop, $label);
        } else {
            $input = $climate->input($label);
            if ($prop->type() === 'text' || $prop->type === 'html') {

Accessing type on the interface Charcoal\Property\PropertyInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

                $input->multiLine();
            }
        }
        return $input;
    }

    /**
     * Get a CLI input from a boolean property.
     *
     * @param  PropertyInterface $prop  The property to retrieve input from.
     * @param  string            $label The input label.
     * @return LeagueInput The League's terminal input object.
     */
    private function booleanInput(PropertyInterface $prop, $label)
    {
        $climate = $this->climate();

        $opts = [
            1 => $prop->trueLabel(),

The method trueLabel() does not exist on Charcoal\Property\PropertyInterface. It seems like you code against a sub-type of Charcoal\Property\PropertyInterface such as Charcoal\Property\BooleanProperty.

            0 => $prop->falseLabel()

The method falseLabel() does not exist on Charcoal\Property\PropertyInterface. It seems like you code against a sub-type of Charcoal\Property\PropertyInterface such as Charcoal\Property\BooleanProperty.

        ];
        $input = $climate->radio(
            $label,
            $opts
        );
        return $input;
    }

    /**
     * Get a CLI password input (hidden) from a password property.
     *
     * @param  PropertyInterface $prop  The property to retrieve input from.
     * @param  string            $label The input label.
     * @return LeagueInput The League's terminal input object.
     */
    private function passwordInput(PropertyInterface $prop, $label)
    {
        unset($prop);

        $climate = $this->climate();

        $input = $climate->password($label);
        return $input;
    }

    /**
     * Set the model factory.
     *
     * @param  FactoryInterface $factory The factory used to create models.
     * @return void
     */
    private function setModelFactory(FactoryInterface $factory)
    {
        $this->modelFactory = $factory;
    }

    /**
     * Retrieve the model factory.
     *
     * @return FactoryInterface
     */
    protected function modelFactory()
    {
        return $this->modelFactory;
    }
}