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

Resource Class does not map properly #28

Closed
mebrett opened this issue Apr 20, 2016 · 6 comments
Closed

Resource Class does not map properly #28

mebrett opened this issue Apr 20, 2016 · 6 comments
Milestone

Comments

@mebrett
Copy link
Contributor

mebrett commented Apr 20, 2016

I've tried with two different DC class terms (Text and Moving Image) in separate CSVs (one class per CSV). In both instances, all items were successfully created but all were assigned the class "Agent".

Jobs 248 and 251 on the shared install.
MapClass.zip

@patrickmj
Copy link
Contributor

It's a bug that it got mapped to Agent, but the kind of value to use is something from the list of Classes under the vocabulary pages. In the example of the Movies file, that'd be dctype: to match what's in the Classes column under Dublin Core Type on the Vocabularies page.

Not really sure how to point people to that. Maybe explanatory text under each of those buttons for mappings, but that could get ugly, too.

@mebrett
Copy link
Contributor Author

mebrett commented Apr 20, 2016

So, what you're saying is that it should start with dctype? So dctype:Text not Text?

@patrickmj
Copy link
Contributor

Yep! Or, for whatever class, go with whatever is listed under the classes list.

That also reminds me of something I might've forgotten to add in to the original testing issue, but automapping works a similar way. That is, a column in the CSV with heading dcterms:title should get automapped to DC Title.

@mebrett
Copy link
Contributor Author

mebrett commented Apr 20, 2016

Seems to be working now. It will require some explanation in the documentation, but is there room for a short note "include dcterms: prefix" for importing class terms somewhere on the page?

@patrickmj
Copy link
Contributor

Maybe. I'm not entirely sure that on the page is the right place, since if they've already gotten to the page, they've already put together the CSV, and will have to go back anyway. Better logging of issues like that are definitely in order, though.

@patrickmj
Copy link
Contributor

Newest master branch should have better logging for this, and non-matching owner and template name values

@patrickmj patrickmj modified the milestone: May 6 Apr 25, 2016
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

No branches or pull requests

2 participants