Validation Error for Non-Admin Users Publishing Content with Umbraco Commerce Price Property in Safari

[mdp-jf]

Which component is this issue related to?

Umbraco Commerce (Core)

Which Umbraco Commerce version are you using? (Please write the exact version, example: 10.1.0)

15.3.4

Bug summary

Non-admin users encounter a validation error when attempting to publish content containing required Umbraco Commerce price properties, but only when using Safari browser. Admin users and other browsers work correctly.

Specifics

Error Details
{
"type": "Error",
"title": "Validation failed",
"status": 400,
"detail": "One or more properties did not pass validation",
"operationStatus": "PropertyValidationError",
"errors": {
"$.values[?(@.alias == 'price' && @.culture == null && @.segment == null)].Value": [
"#validation_invalidNull"
]
}
}

Steps to reproduce

Steps to Reproduce

• Install Umbraco 15.4.1 with Umbraco Commerce 15.3.4
• Create a new store
• Create a document type that uses said store
• Create a document type that is a child of the first document type with a required field that uses the price property from Umbraco Commerce
• Create content using this document types
• Use Safari browser
• Log in as a non-admin user (without admin rights)
• Create new content as a non-admin
• Attempt to publish the content

Expected result / actual result

Expected Behavior

• Non-admin users should be able to publish content with Commerce price properties regardless of browser
• Behavior should be consistent across all supported browsers

Actual Behavior

• Non-admin users receive validation error when publishing in Safari
• Error message indicates price property validation failure with "#validation_invalidNull"
• Same content publishes successfully for admin users in Safari
• Same content publishes successfully for non-admin users in other browsers

Dependencies

Environment

• Umbraco Version: 15.4.1
• Umbraco Commerce: Installed 15.3.4
• Affected Browser: Safari (all versions tested)
• Working Browsers: Chrome, Firefox, Edge
• User Permissions: Issue affects non-admin users only

---

This item has been added to our backlog AB#53359

[mattbrailsford]

Thanks for reporting this, I've added it to our sprint backlog to review. In the meantime it sounds like a valid workaround would be to a) use a different browser or b) don't make the price field mandatory.

[mattbrailsford]

Hmm, I can't seem to replicate your exact issue.

I do think I am seeing an issue, in that the event handlers on the price input don't look to be firing correctly and so that updated price value isn't getting written back to the model to persist. This could cause the error you are seeing as if it's a blank product page, then you'll be sending a null value and so that would fail the mandatory requirement.

The thing I can't replicate though is that I'm finding this consistent across any member and so signing out and back in as an admin doesn't seem to change anything. I'll investigate the issue further and see if I can see why they event handlers aren't firing and then we'll have to see if that resolves the issue for you or not.

[mattbrailsford]

Ok so I think I've resolved this in Umbraco Commerce for now. It appears there is an issue in Safari that you can't have an onInput and an onChange event handler registered on an input as the onChange event won't fire. Internally the price input field does this and requires that both work. We've implemented a workaround for now which seems to work.

I've also raised this on the UUI github repo as I think it would be a better place to handle this and wouldn't require UC to have to implement workarounds in multiple places.

Fix will be in an upcoming 16.1.0 release