Skip to content

Commit

Permalink
Allow {control_number} in classification URIs
Browse files Browse the repository at this point in the history
As documented `{control_number}` in an URI template is allowed, also for classification entries.
  • Loading branch information
nichtich authored Oct 4, 2017
1 parent fd56336 commit cdf977f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion mc2skos/record.py
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@ def generate_uris(self):
self.scheme_uris.append(uri)

# Record URI
self.uri = self.scheme.get_uri(collection='class', object=self.notation)
self.uri = self.scheme.get_uri(collection='class', object=self.notation, control_number=self.control_number)

def parse(self, options):

Expand Down

1 comment on commit cdf977f

@nichtich
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not enough because parent_notation is still used to link to a broader concept but 153 does not contain the control_number. If URIs are based on control_number, links to broader concepts should only come from 5XX fields (see #31).

Please sign in to comment.