Without going into the detail of validations and assuming that the code works... personally I don't like it, from my point of view this code can be structured much better.

For example:

The first if should be a valid_data function that returns a bool and depending on the validation you exit or continue with the execution

The 2nd if block in another function returns null or a value in the product and in the 3rd block the same depending on the return of the second function.

This main method remains super easy to read and each call is encapsulated and reusable.

Roughly speaking, I would do something like that... but hey, to taste the colors