Flowpack.NodeTemplates icon indicating copy to clipboard operation
Flowpack.NodeTemplates copied to clipboard

TASK: Improve exception messages -> dont start with `because`

Open mhsdesign opened this issue 2 years ago • 0 comments

Many exceptions are currently thrown like:

                    throw new PropertyIgnoredException(
                        sprintf(
                            'Because property is not declared in NodeType. Got value `%s`.',
                            json_encode($propertyValue)
                        ),
                        1685869035209
                    );

Property "nonDeclaredProperty" in NodeType "Flowpack.NodeTemplates:Content.SomeExceptions" | PropertyIgnoredException(Because property is not declared in NodeType. Got value "hi"., 1685869035209)

This provides not the best output for editors. A fully readable text would be better than starting with Because

mhsdesign avatar Jun 30 '23 17:06 mhsdesign