sadikoff / i18n-form-bundle

src/Form/EventListener/TranslationsListener.php

<?php

namespace Koff\Bundle\I18nFormBundle\Form\EventListener;

use Koff\Bundle\I18nFormBundle\Form\Manipulator\FormManipulatorInterface;
use Koff\Bundle\I18nFormBundle\Form\Type\AutoFormType;
use Symfony\Component\Form\FormEvent;
use Symfony\Component\Form\FormInterface;

/**
 * Class TranslationsListener.
 *
 * @author David ALLIX
 * @author Sadicov Vladimir <sadikoff@gmail.com>
 */
class TranslationsListener extends KoffI18nListener
{
    /** @var FormManipulatorInterface */
    private $formManipulator;

    public function __construct(FormManipulatorInterface $formManipulator)
    {
        $this->formManipulator = $formManipulator;
    }

    public function preSetData(FormEvent $event)
    {
        $form = $event->getForm();
        $formOptions = $form->getConfig()->getOptions();

        $fieldsOptions = $this->getFieldsOptions($form, $formOptions);
        $translationClass = $this->getTranslationClass($form->getParent());

It seems like $form->getParent() can also be of type null; however, parameter $form of Koff\Bundle\I18nFormBundle\Form\EventListener\TranslationsListener::getTranslationClass() does only seem to accept Symfony\Component\Form\FormInterface, maybe add an additional type check?

        foreach ($formOptions['locales'] as $locale) {
            if (isset($fieldsOptions[$locale])) {
                $form->add($locale, AutoFormType::class, [
                    'data_class' => $translationClass,
                    'required' => \in_array($locale, $formOptions['required_locales'], true),
                    'block_name' => ('field' === $formOptions['theming_granularity']) ? 'locale' : null,
                    'fields' => $fieldsOptions[$locale],
                    'excluded_fields' => $formOptions['excluded_fields'],
                ]);
            }
        }
    }

    private function getTranslationClass(FormInterface $form): string
    {
        do {
            $translatableClass = $form->getConfig()->getDataClass();
        } while ((null === $translatableClass) && $form->getConfig()->getVirtual() && ($form = $form->getParent()));

The method getVirtual() does not exist on Symfony\Component\Form\FormConfigInterface.

        // Knp
        if (method_exists($translatableClass, 'getTranslationEntityClass')) {
            return $translatableClass::getTranslationEntityClass();
        }

        // Gedmo
        if (method_exists($translatableClass, 'getTranslationClass')) {
            return $translatableClass::getTranslationClass();
        }

        return $translatableClass.'Translation';
    }

    public function getFieldsOptions(FormInterface $form, array $formOptions): array
    {
        $fieldsOptions = array_fill_keys($formOptions['locales'], $this->formManipulator->getFieldsConfig($form));

        foreach ($fieldsOptions as $locale => &$field) {
            array_walk($field, function (&$v, $k, $l) {
                if (\array_key_exists('locale_options', $v) && \array_key_exists($l, $v['locale_options'])) {
                    $lo = $v['locale_options'];
                    unset($v['locale_options']);

                    $v = $lo[$l] + $v;
                }
            }, $locale);
        }

        return $fieldsOptions;
    }
}