Don't try to load profile on an unsaved (without a pid) datastream about rubydora HOT 5 CLOSED

I'm thinking of changing line 30 to:

elsif dsid && profile.has_key?(profile_name.to_s)

Are you okay with that @cbeer?

from rubydora.

Um, sure. I'm not sure I understand why you'd start with a datastream without a dsid, but fixing that case sounds fine to me.

from rubydora.

cbeer: it's an uncommon case, but ActiveFedora has the capability of doing auto-generated dsids. To use that you do this:

file_ds2 = ActiveFedora::Datastream.new(newthing.inner_object, nil, dsLabel: 'Minivan Plays', altIDs: 'default', controlGroup: 'M', content: file)
newthing.add_datastream(file_ds2)
=> "DS1"

from rubydora.

Maybe we could just shift that into the AF::Datastream initializer so that if you try to do it will a nil dsid, it'll just auto-assign one.

from rubydora.

@cbeer was right. I fixed this in ActiveFedora instead.

from rubydora.