RicLeP / laravel-storyblok-forms

src/MultiInput.php

<?php

namespace Riclep\StoryblokForms;

use Illuminate\Support\Arr;
use Illuminate\Support\Str;

class MultiInput extends Input
{

	/**
	 * Adds extra data to the JSON interpretation of the field
	 *
	 * @return array
	 */
	protected function addToJson(): array
	{
		$json['options'] = $this->options();

$json was never initialized. Although not strictly required by PHP, it is generally a good practice to add $json = array(); before regardless.

		return $json;
	}

	/**
	 * Processes the options entered in Storyblok
	 *
	 * @return \Illuminate\Support\Collection
	 */
	public function options(): \Illuminate\Support\Collection
	{
		return collect(preg_split('/\r\n|\r|\n/', $this->{$this->optionsName}))->filter()->map(function ($option) {

The property optionsName does not exist on Riclep\StoryblokForms\MultiInput. Since you implemented __get, consider adding a @property annotation.

preg_split('/\r\n|\r|\n/', $this->$this->optionsName) of type string[] is incompatible with the type Illuminate\Contracts\Support\Arrayable expected by parameter $value of collect().

			/**
			 * Parses the possible option formats into a named array
			 * name
			 * [*]name
			 * [key]name
			 * [key][*]name
			 * */
			preg_match('/(?:\[(?<value>[\w]+)\])?(?:\[(?<selected>\*)\])?(?<label>.+)/', $option, $settings);

			if ($settings['value'] === '') {
				$settings['value'] = Str::slug($settings['label']);
			}

			if (request()->session()->has('_old_input')) {
				$settings['selected'] = $this->optionIsSelected($settings['selected']);
			} else {
				$settings['selected'] = $settings['selected'] === '*' ? true : false;
			}

			return array_filter($settings, function ($key) {
				return is_string($key);
			}, ARRAY_FILTER_USE_KEY);
		})->values();
	}


	/**
	 * Checks if an option was preselected or in old input
	 *
	 * @param $formInput
	 * @return bool
	 */
	protected function optionIsSelected($formInput): bool
	{
		return request()->old($this->input_name) && (in_array(Str::slug($formInput), Arr::wrap(request()->old($this->input_name))));

The property input_name does not exist on Riclep\StoryblokForms\MultiInput. Since you implemented __get, consider adding a @property annotation.

	}

	/**
	 * Returns the Input’s response after the form has been submitted and validated
	 * All options are returned as an array with their name and a selected boolean
	 * based on the user’s input
	 *
	 * @param $input
	 * @return array
	 */
	public function response($input): array
	{
		$formatted = [
			'label' => $this->label,

The property label does not exist on Riclep\StoryblokForms\MultiInput. Since you implemented __get, consider adding a @property annotation.

			'name' => $this->name,

The property name does not exist on Riclep\StoryblokForms\MultiInput. Since you implemented __get, consider adding a @property annotation.

			'response' => ['selected' => [], 'unselected' => []],
			'type' => $this->type,

The property type does not exist on Riclep\StoryblokForms\MultiInput. Since you implemented __get, consider adding a @property annotation.

		];

		$this->options()->map(function ($formInput) use ($input, &$formatted) {
			if (in_array($formInput['value'], Arr::wrap($input), true)) {
				return $formatted['response']['selected'][$formInput['value']] = $formInput['label'];
			}

			return $formatted['response']['unselected'][$formInput['value']] = $formInput['label'];
		})->toArray();

		return $formatted;
	}
}