The uri_field data type should use the uri data type for its value, not string.

CommentFileSizeAuthor
#1 uri-field-2039961-1.patch628 bytesklausi

Comments

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Needs review
StatusFileSize
new uri-field-2039961-1.patch628 bytes

I guess we don't need to write a test for this?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

the uri type will have validations assigned, I guess that's why you noticed this?

If so, then we will get implicit test coverage when we test validation of feeds?

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Reviewed & tested by the community

ad #2 - indeed, that should work.

The fix is good, so let's move on with this.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed b64ada1 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.