regarding the prepareForPipeline function

[chick84]

Hello, regarding the prepareForPipeline function, there is only one $properties parameter, I think you also need to enter $payload! in order for prepareForPipeline to handle more complex business logic, what do you think?

    public function execute(mixed $value, CreationContext $creationContext): array
    {
        $properties = null;

        foreach ($this->normalizers as $normalizer) {
            $properties = $normalizer->normalize($value);

            if ($properties !== null) {
                break;
            }
        }

        if ($properties === null) {
            throw CannotCreateData::noNormalizerFound($this->dataClass->name, $value);
        }
        
         //status quo
        $properties = ($this->dataClass->name)::prepareForPipeline($properties);

        //my opinion
       $properties = ($this->dataClass->name)::prepareForPipeline($properties, $value);
    }

    public static function prepareForPipeline(array $properties, mixed $value): array
    {
          //Processing certain operations based on value
          
          return $properties;
    }

[chick84]

My solution:

class BaseResponse extends Resource
{
    public static function pipeline(): DataPipeline
    {
        return parent::pipeline()->firstThrough(static::beforeFillPipe());
    }

    public static function beforeFillPipe(): DataPipe
    {
        return new class() implements DataPipe {
            public function handle(mixed $payload, DataClass $class, array $properties, CreationContext $creationContext): array
            {
                //return
                return ($class->name)::beforeFill($properties, $payload);
            }
        };
    }

    public static function beforeFill(array $properties, mixed $payload): array
    {
        return $properties;
    }
}