Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Actors/Statuses with multiple types #7305

Merged
merged 4 commits into from
May 2, 2018

Conversation

ekiru
Copy link
Contributor

@ekiru ekiru commented May 2, 2018

This would address #7193:

It allows processing type properties with multiple values where one of the values is a recognized/supported type in several places by:

  • replacing checks that an ActivityStreams Object's type equals a specific value with calls to the preexisting equals_or_includes? helper from JsonLdHelper and
  • replacing checks that an Object's type is among a set of values with calls to a new equals_or_includes_any? helper method.

I've added a spec testing that the ResolveAccountService handles Actors whose type property is ["as:Person", "vcard:individual"] as if they were just a Person. I can add more specs testing other places if that seems necessary.

@Gargron Gargron merged commit dc786c0 into mastodon:master May 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants