activitystreams icon indicating copy to clipboard operation
activitystreams copied to clipboard

Inconsistencies between property domains and type properties

Open cjslep opened this issue 7 years ago • 2 comments

Please Indicate One:

  • [ ] Editorial
  • [ ] Question
  • [X] Feedback
  • [X] Blocking Issue
  • [ ] Non-Blocking Issue

Please Describe the Issue:

(I am splitting my email to the public-socialweb mailing list into separate issues, for background please see those emails).

This is ActivityStream vocabulary specific: https://www.w3.org/TR/activitystreams-vocabulary

The specification is inconsistent with itself:

  • The "altitude" property has domain "Object" in Section 4, but the "Object" class does not list "altitude" as a property in Section 2.
  • The "id" property has domain "Object" and "Link" in Section 4, but the "Object" and "Link" classes do not list "id" as a property in Section 2.
  • The "type" property has domain "Object" and "Link" in Section 4, but the "Object" and "Link" classes do not list "type" as a property in Section 2.
  • The "attributedTo" property has domain "Link" in Section 4, but the "Link" class does not list "attributedTo" as a property in Section 2.

cjslep avatar Jan 01 '18 21:01 cjslep

underlying problem for the first one is probably

  • "altitude" does only appear directly in "Place"
  • this is the same for "latitude", "longitude" but
  • the domain for "altitude" is currently "Object"
  • the domain for "latitude", "longitude" is currently "Place"

so maybe the domain for "altitude" should be "Place"

sebilasse avatar Jan 19 '18 13:01 sebilasse

so maybe the domain for "altitude" should be "Place"

that would make it more consistent. But I think a better consistency would be to have latitude, longitude, and altitude all have domain: Object. Esp considering that the range of location includes Object. Non-Place location values currently can't specify lat/long? IMO that's weird. Also this would be backward-incompatible I believe, whereas restricting altitude Domain would not be.

I disagree that this is a "Blocking Issue", but it is good feedback for an AS2.0.1

gobengo avatar Jan 25 '18 00:01 gobengo

Based on this issue, I've updated the ERRATA document to include the information about altitude and attributedTo.

https://github.com/w3c/activitystreams/blob/master/ERRATA.md

Because id and type are addressed in the Core document, I don't believe the lists of properties require them in Vocabulary.

evanp avatar May 10 '23 16:05 evanp