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

responsibility 2.0 #52

Closed
stansmith907 opened this issue Aug 23, 2016 · 11 comments
Closed

responsibility 2.0 #52

stansmith907 opened this issue Aug 23, 2016 · 11 comments
Labels
enhancement Improve or modify an existing feature
Milestone

Comments

@stansmith907
Copy link
Contributor

stansmith907 commented Aug 23, 2016

CI_Responsibility was formerly CI_ResponsibleParty. The schema described here is the mdJson 2.0 schema and not part of ISO. It makes use of data maintained as individual and organization contacts saved in the mdJson contacts array.

The new ISO -3 schema centers on “role” rather than the contact.

Changes to the mdJson:

  1. "roleExtent" is a new array of EX_Extent objects added to define the extent for the role. ISO allows for full geographic, vertical, and temporal extents but I have limited mdJson to the elements of "description" and "temporalExtent [ ]", ("timeInstant", "timePeriod"). However, "geographicExtent [ ] and verticalExtent [ ] could be included with no changes to the software, it's ready to receive these.
  2. “party” is a new array added to identify all the individual and/or organization contacts associated with the role.
    • “partyType” is a new enumerated type added (individual or organization) to identify the type of party being described.
    • “organizationMembers” is a new array of individual contactId(s) identifying individuals that have an association with an organization contact. Valid only for party with partyType = “organization”.
{
   "role": "CI_RoleCode",
   "roleExtent": [
      {
         "see": "extent 2.0"
      }
   ],
   "party": [
      {
         "contactId": "",
         "organizationMembers": [
            "individual contact ID"
         ]
      }
   ]
}

no ISO example

@stansmith907 stansmith907 added the enhancement Improve or modify an existing feature label Aug 23, 2016
This was referenced Aug 31, 2016
@stansmith907 stansmith907 added this to the 2.0 milestone Sep 1, 2016
@jlblcc
Copy link
Member

jlblcc commented Sep 27, 2016

Remove partyType.

@jlblcc
Copy link
Member

jlblcc commented Sep 27, 2016

roleExtent should be an array.

@stansmith907
Copy link
Contributor Author

Replace start and end date with temporalExtent.

@stansmith907
Copy link
Contributor Author

partyType has been removed.

@stansmith907
Copy link
Contributor Author

I had limited the role extent to one object since I did not allow the expression of a full EX_Extent. The full EX_Extent would permit descriptive, graphic, vertical, as well as temporal extents to be defined. But technically this is an array in ISO. I have made the change per your request but have not opened the mdJson for other than temporal extents.

@jlblcc
Copy link
Member

jlblcc commented Oct 14, 2016

Why don't we just use timePeriod instead of roleExtent? The use of extent is confusing. In fact, I would suggest dropping the use of extent anywhere we're restricting the type of extent - we don't need to wrap them in an additional layer. Just add the geographic, vertical, or temporal extent properties directly to the object in question.

@jlblcc
Copy link
Member

jlblcc commented Oct 14, 2016

Just add the geographic, vertical, or temporal extent properties directly to the object in question.

Unless we're allowing all three.

@stansmith907
Copy link
Contributor Author

Fair enough. If we are going to call it timePeriod it may as well be the full timePeriod. I changed roleExtent to timePeriod.

@stansmith907 stansmith907 mentioned this issue Oct 14, 2016
This was referenced Oct 22, 2016
@jlblcc
Copy link
Member

jlblcc commented Jan 19, 2017

"timePeriod" should be an array, right? Example shows it an an object. Also see here.

jlblcc added a commit that referenced this issue Jan 19, 2017
@stansmith907
Copy link
Contributor Author

stansmith907 commented Jan 19, 2017

Yep, right you are. I missed it since "responsibility" with "extent" is a new construct in -1. So, I had not tested yet. I'll patch it now. If fact, I think I'll make it a "temporalExtent" array.

jlblcc added a commit that referenced this issue Jan 21, 2017
@stansmith907
Copy link
Contributor Author

I changed "timePeriod" out for the full "extent". Not only does this match the ISO 19115-1 standard, it was getting confusing limiting extent type to "timePeriod" when there were different use cases for responsibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve or modify an existing feature
Projects
None yet
Development

No branches or pull requests

2 participants