search

uklibraries / metadata_editor

Metadata editor satisfying the KDL metadata guidelines
0 stars 0 forks source link

Add spacial coverage #51

Closed rrmait0 closed 11 years ago

cokernel commented 11 years ago

@rrmait0, I've merged master into this branch and fixed some typos (one of which I made during the merge).

This isn't ready to be merged into master, though. I am unable to add spatial coverage fields to a record from the record edit page.

rrmait0 commented 11 years ago

@cokernel Yeah, it's not fully complete since I'm not sure how to do that part. It's similar to source where one should be able to look up the information in a table and add the information if it does not exist.

cokernel commented 11 years ago

Well, this pull request is fine as far as it goes. I can merge this into master and just keep #51 open. But I recommend finishing #51 before picking up a new feature.

I think you will want to use find_or_create_by_name or so for creating new spatial coverage fields.

I would also recommend having SpatialCoverages be owned by Repositories (to be assigned automatically to the Repository containing the Record) so that different repositories can have their own sets of coordinates.

rrmait0 commented 11 years ago

Some examples of find_or_create_by_name would be helpful. I did find this one:

http://stackoverflow.com/questions/11875546/find-or-create-by-name-not-working-when-trying-to-associate-tasks-with-lists-on

I will check the Ruby Doc as well.