Fixed issue with PATCH incorrectly replacing entire complex attr

[jasonfagerberg-toast]

Fix for: #725

[jasonfagerberg-toast]

I think one issue here is if a complex attribute is within a complex attribute, i.e. this request

{
  "schemas": [
    "urn:ietf:params:scim:api:messages:2.0:PatchOp"
  ],
  "Operations": [
    {
      "op": "replace",

      "value": {
        "name": {
          "givenName": "New Given Name"
         }
      }
    }
  ]
}

I think the behavior will still be that name is replaced entirely but ill write a test to see

[jasonfagerberg-toast]

I think the behavior will still be that name is replaced entirely but ill write a test to see

Added a test and its passing so nice work everyone, take off early for the day

[bdemers]

Great find @jasonfagerberg-toast!

I left a quick nit, but this change looks good, I'll make sure the CI build gets kicked off!