mnylc / islandora_multi_importer

This is a flexible, twig based, all cmodel, tabular data to islandora Object importer with optional ZeroMQ processing
GNU General Public License v3.0
16 stars 15 forks source link

RELS-EXT update process #65

Open Natkeeran opened 6 years ago

Natkeeran commented 6 years ago

With this PR merged, multi_importer offers three ways to update (including a "don't update" option) RELS-EXT. In all three cases, it discards duplicate predicates except for membership predicates . (This maybe because Diego tried to help me out with this issue).

In RELS-EXT, there can be valid duplicate predicates such as below:

<islandora:isManageableByUser>user1</islandora:isManageableByUser>
<islandora:isManageableByUser>user2</islandora:isManageableByUser>
<islandora:isManageableByUser>user3</islandora:isManageableByUser>

Possible solutions:

Natkeeran commented 6 years ago

A cleaner and simpler solution would be to only remove fully duplicate statements (predicate and value).

DiegoPino commented 6 years ago

But would leave issues like sequence number, page number, etc around that would interfere with a lot viewers and SP (imagine moving pages from one book to another). Also would leave other dependencies. The real solution is sadly not easy. Its to (once for all) make a compendium of needed-basic properties. And deal with the not defined there as you suggest here. I will work on this.