mumuki-domain 8.1.3 → 8.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9495d00fe4f85bdc27f5cf67b94e5f09de465a31a795e9e3d56b3f4cd91433e1
-  data.tar.gz: 209256d417ff7c98c309b517cd7076a9743ea00de79124b6bec3e735a49a5238
+  metadata.gz: 5ea9b10953e5031cfcfa680763efb0eac02e63b91219c3dbf71b0d30b5dd3250
+  data.tar.gz: 8fda448216e9d80cb4de41a7f43825882e043895705c66c1c6b9b4bc68c5bfec
 SHA512:
-  metadata.gz: 88ccf91cbccb01f15ec513e232d48832e69a0ec9f5153992e91eab56c44183f10a10dc2a4bcd894509708ab97461b4698ecb73485da8d7dfa8826196837e0c3d
-  data.tar.gz: 80b0c3ec21c9444dedb05c1c1060035cdd0d5f0f731635ae79e4738c0a0d23c6fbf4ac67d48c6bb0ea70a76ee7e825a05a2afaaad95e8d2be54096636e0e2af0
+  metadata.gz: f2f8f341502367fa4a7ba037eb086f0506c19fede8290aba4921430fb22e33cc338faf01bc85d23d9df89fca006dba97705e377f48c3eae789170291644562e5
+  data.tar.gz: e5044aa8475aba10ea9d1140eceefccd9fc36de384cf215007f4f239568aebf9f06583074249af630fff81db9da8d429604fa428db0c509a55c4f9092b25965d

data/app/models/assignment.rb

@@ -23,6 +23,8 @@ class Assignment < Progress
 
   delegate :completed?, :solved?, to: :submission_status
 
+  delegate :content_available_in?, to: :parent
+
   alias_attribute :status, :submission_status
   alias_attribute :attempts_count, :attemps_count
 
@@ -263,6 +265,10 @@ class Assignment < Progress
 
   private
 
+  def duplicates_key
+    { exercise: exercise, submitter: submitter }
+  end
+
   def update_submissions_count!
     self.class.connection.execute(
       "update public.exercises
@@ -278,5 +284,4 @@ class Assignment < Progress
   def update_last_submission!
     submitter.update!(last_submission_date: DateTime.current, last_exercise: exercise)
   end
-
 end

data/app/models/indicator.rb

@@ -66,8 +66,43 @@ class Indicator < Progress
     where(content: content, organization: organization).delete_all
   end
 
+  def _move_to!(organization)
+    move_children_to!(organization)
+    super
+  end
+
+  def _copy_to!(organization)
+    progress_item = super
+    children.each { |it| it._copy_to! organization }
+    progress_item
+  end
+
+  def move_children_to!(organization)
+    children.update_all(organization_id: organization.id)
+
+    indicators.each { |it| it.move_children_to!(organization) }
+  end
+
+  def cascade_delete_children!
+    indicators.each(&:cascade_delete_children!)
+    children.delete_all(:delete_all)
+  end
+
+  def content_available_in?(organization)
+    content.usage_in_organization(organization).present?
+  end
+
+  def delete_duplicates_in!(organization)
+    duplicates_in(organization).each(&:cascade_delete_children!)
+    super
+  end
+
   private
 
+  def duplicates_key
+    { content: content, user: user }
+  end
+
   def children
     indicators.presence || assignments
   end

data/app/models/progress.rb

@@ -12,4 +12,39 @@ class Progress < ApplicationRecord
   def dirty_parent_by_submission!
     parent&.dirty_by_submission!
   end
+
+  def _copy_to!(organization)
+    dup.transfer_to!(organization)
+  end
+
+  def transfer_to!(organization)
+    update! organization: organization, parent: nil
+    self
+  end
+
+  alias_method :_move_to!, :transfer_to!
+
+  %w(copy move).each do |transfer_type|
+    define_method "#{transfer_type}_to!" do |organization|
+      "Mumuki::Domain::ProgressTransfer::#{transfer_type.camelize}".constantize.new(self, organization).execute!
+    end
+  end
+
+  def guide_indicator?
+    is_a?(Indicator) && content_type == 'Guide'
+  end
+
+  def has_duplicates_in?(organization)
+    duplicates_in(organization).present?
+  end
+
+  def delete_duplicates_in!(organization)
+    duplicates_in(organization).delete_all
+  end
+
+  private
+
+  def duplicates_in(organization)
+    self.class.where(duplicates_key.merge(organization: organization)).where.not(id: id)
+  end
 end

data/lib/mumuki/domain.rb

@@ -38,6 +38,7 @@ require_relative './domain/workspace'
 require_relative './domain/helpers'
 require_relative './domain/syncable'
 require_relative './domain/store'
+require_relative './domain/progress_transfer'
 
 class Mumukit::Assistant
   def self.valid?(rules)

data/lib/mumuki/domain/progress_transfer.rb

@@ -0,0 +1,8 @@
+module Mumuki::Domain
+  module ProgressTransfer
+  end
+end
+
+require_relative './progress_transfer/base'
+require_relative './progress_transfer/move'
+require_relative './progress_transfer/copy'

data/lib/mumuki/domain/progress_transfer/base.rb

@@ -0,0 +1,44 @@
+class Mumuki::Domain::ProgressTransfer::Base
+  attr_reader *%i(source_organization destination_organization progress_item transferred_item)
+
+  delegate :user, to: :progress_item
+
+  def initialize(progress_item, destination_organization)
+    @progress_item = progress_item
+    @destination_organization = destination_organization
+  end
+
+  def execute!
+    ActiveRecord::Base.transaction do
+      pre_transfer!
+      transfer!
+      post_transfer!
+    end
+  end
+
+  def pre_transfer!
+    validate_transferrable!
+    @source_organization = progress_item.organization
+    progress_item.delete_duplicates_in!(destination_organization)
+  end
+
+  def transfer!
+    @transferred_item = do_transfer!
+  end
+
+  def post_transfer!
+    transferred_item.dirty_parent_by_submission!
+    notify_transfer!
+    transferred_item
+  end
+
+  def validate_transferrable!
+    raise "Transferred progress' content must be available in destination!" unless progress_item.content_available_in?(destination_organization)
+    raise 'User must be student in destination organization' unless user.student_of?(destination_organization)
+    raise 'Transfer only supported for guide indicators' unless progress_item.guide_indicator?
+  end
+
+  def notify_transfer!
+    Mumukit::Nuntius.notify! 'progress-transfers', { from: source_organization.name, to: destination_organization.name, item_type: transferred_item.class.to_s, item_id: transferred_item.id, transfer_type: transfer_type }
+  end
+end

data/lib/mumuki/domain/progress_transfer/copy.rb

@@ -0,0 +1,9 @@
+class Mumuki::Domain::ProgressTransfer::Copy < Mumuki::Domain::ProgressTransfer::Base
+  def transfer_type
+    :copy
+  end
+
+  def do_transfer!
+    progress_item._copy_to!(destination_organization)
+  end
+end

data/lib/mumuki/domain/progress_transfer/move.rb

@@ -0,0 +1,14 @@
+class Mumuki::Domain::ProgressTransfer::Move < Mumuki::Domain::ProgressTransfer::Base
+  def transfer_type
+    :move
+  end
+
+  def pre_transfer!
+    super
+    progress_item.dirty_parent_by_submission!
+  end
+
+  def do_transfer!
+    progress_item._move_to!(destination_organization)
+  end
+end

data/lib/mumuki/domain/status/submission/manual_evaluation_pending.rb

@@ -10,6 +10,6 @@ module Mumuki::Domain::Status::Submission::ManualEvaluationPending
   end
 
   def self.iconize
-    {class: :info, type: 'clock-o'}
+    {class: :info, type: 'clock'}
   end
 end

data/lib/mumuki/domain/status/submission/submission.rb

@@ -50,4 +50,8 @@ module Mumuki::Domain::Status::Submission
   def exp_given
     0
   end
+
+  def dup
+    self
+  end
 end

data/lib/mumuki/domain/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Domain
-    VERSION = '8.1.3'
+    VERSION = '8.2.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-domain
 version: !ruby/object:Gem::Version
-  version: 8.1.3
+  version: 8.2.0
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-18 00:00:00.000000000 Z
+date: 2021-01-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -704,6 +704,10 @@ files:
 - lib/mumuki/domain/organization/profile.rb
 - lib/mumuki/domain/organization/settings.rb
 - lib/mumuki/domain/organization/theme.rb
+- lib/mumuki/domain/progress_transfer.rb
+- lib/mumuki/domain/progress_transfer/base.rb
+- lib/mumuki/domain/progress_transfer/copy.rb
+- lib/mumuki/domain/progress_transfer/move.rb
 - lib/mumuki/domain/seed.rb
 - lib/mumuki/domain/status.rb
 - lib/mumuki/domain/status/discussion/closed.rb