inject_into_file 'config/initializers/sufia.rb', after: "Sufia.config do |config|\n" do
" # Injected via `rails g sufia:work #{class_name}`\n" \
" config.register_curation_concern :#{file_name}\n"
end
The important part there is that each work that gets generated is inserted above the others at the top of the config block, changing what primary_work_type returns (via .first).
I'm fairly confident that a user who generates a new type after years of stable operation would be surprised to find features behaving differently and code that suggests the new type is now "primary".
IMHO, we should get rid of primary_work_type altogether.
The work_generator does:
The important part there is that each work that gets generated is inserted above the others at the top of the
config
block, changing whatprimary_work_type
returns (via.first
).I'm fairly confident that a user who generates a new type after years of stable operation would be surprised to find features behaving differently and code that suggests the new type is now "primary".
IMHO, we should get rid of
primary_work_type
altogether.