Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

Most relevant candidate jobs for refactoring

./create_derivatives_job.rb    https://github.com/samvera/hyrax/issues/4191

./content_depositor_change_event_job.rb  https://github.com/samvera/hyrax/issues/4194

./import_url_job.rb    https://github.com/samvera/hyrax/issues/4195 

./ingest_local_file_job.rb  https://github.com/samvera/hyrax/issues/4196 

./attach_files_to_work_job.rb   https://github.com/samvera/hyrax/issues/4197 

./visibility_copy_job.rb   https://github.com/samvera/hyrax/issues/4198 

./content_event_job.rb           https://github.com/samvera/hyrax/issues/4193

./inherit_permissions_job.rb   https://github.com/samvera/hyrax/issues/4200 

./attach_files_to_work_with_ordered_members_job.rb   https://github.com/samvera/hyrax/issues/4201 

./content_restored_version_event_job.rb   https://github.com/samvera/hyrax/issues/4202 

Event jobs

Events may or may not operate on an instance of a repository object; if they do, they should be covered by ContentEventJob:

./content_delete_event_job.rb   Doesn't involve an actual object instance, just an object ID
./content_deposit_event_job.rb  Should be covered by refactor of content_event_job
./content_update_event_job.rb   Should be covered by refactor of content_event_job

...


Do these files make sense to refactor for AF or Valkyrie?

./createfile_work_job.rb
./batch_create_job.rb
./ingest_job.rb

The following jobs potentially call perform with an ActiveFedora object and have not yet been refactored to accept a Valkyrie resource:

[jobs]$ grep -rL valk .                   
./import_export_job.rb  
./stream_notifications_job.rb
./content_depositor_change_event_job.rb
./import_url_job.rb
./ingest_local_file_job.rb
./attach_files_to_work_job.rb
./event_job.rb
./visibility_copy_job.rb
./file_set_attached_event_job.rb
./fixity_check_job.rb
./inherit_permissions_job.rb
./attach_files_to_work_with_ordered_membersset_attached_event_job.rb Should be covered by refactor of content_event_job

Refactor?

These jobs involve creating works - do these make sense to refactor?

./create_work_job.rb
./user_edit_profile_eventbatch_create_job.rb
./content_restored_version_event_job.rb
./content_new_version_event_job.rb
./create_derivativesingest_job.rb