I don't see any problem in the metadata declaration in the code above. What I had in mind in the previous post is updating your SP metadata on the IDP side.

V.