importer tries, fails to create FileSets before Works are persisted (tested in 5.5.1, 7.0.0)

[aploshay]

Behavior occurs running this branch of essi, with bulkrax 5.5.0:

• on the first run of an importer, all the FileSet imports fail with error:

NoMethodError - undefined method `permissions' for nil:NilClass

which occurs because a nil value is returned for work by find_record called in create_file_set, since apparently the work hasn't been persisted yet. All the work imports do, however, succeed.

• on the second run of that same importer (running with "Update metadata and replace files"), all the FileSet records do import, because now the parent works are persisted and found.

I've also tested bulkrax 7.0.0 -- the same issue occurs on the first run, but the second run now also fails to import FileSet records.

I'm surprised that it's possible for the Work, FileSet creation sequencing to get flipped around somehow -- any notions of what could be causing this?

[bkiahstroud]

1. Which parser are you using? CSV, etc.
2. If CSV, how are the files being imported? Does each FileSet have its own row, or are the files listed in a column on the work's row?

[aploshay]

1. CSV
2. Each FileSet in its own row -- Method 2

[kirkkwang]

Hi @aploshay, I added this commit which worked for me, however, I'm seeing different results in other applications which could be a configuration issue. Do you mind give main a try?

[bkiahstroud]

This ticket might be related to this other one:

• ActiveFedora x Bulkrax v8.0.0 bug: Unable to import FileSet row  #953

[aploshay]

Thanks, @kirkkwang! I ran into unrelated problems trying to upgrade from 5.5.1 to main, so I switched tracks to cherry-picking your change in that commit back into 5.5.1 and it's working.

[kirkkwang]

That's great to hear @aploshay ! Do you think we can close this issue now?

[aploshay]

@kirkkwang I believe so, thanks