mumuki-classroom 8.3.0 → 9.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0f16113532d3735586cd196c1b5a4b51cc04165f02bfbd09a5e7d9abbaa6724a
-  data.tar.gz: adc7f5af977eb56599a76429716b604dffeef6062f899210e890e790d3d6dbdc
+  metadata.gz: 758f9c2cad27a3521fd21607d251b73ba1dccf289d92bb316c3a4b05649e154a
+  data.tar.gz: d7e093105fb0bb1849bf716382d248ab4417a5779d8786d668312b261cbc7010
 SHA512:
-  metadata.gz: fb721faa6ae43fec4d4a1ce9f0e50cfe6b4dc046e86c8953d1ca36d09566e1dbb90aa690f35c9154dab23c0b5b8508872c5bf2c9ae3f3055268babc35798deb9
-  data.tar.gz: efbb8c411390470426c2bac93454155ef0a1b95e893b11e6697480c29d04516190caf2aa09ba9bfac0442af777861d639c91b92a4aa6bb0f5951c69f155674dc
+  metadata.gz: 18e0f05f9cd7b0b7063de7388be63a6bd3f74aed8ca28539c12ee389b9641a93a623ef96461c2156ef61b65a857cfc53fab36e66040183fc3da1f1b25f48408d
+  data.tar.gz: 7e4a6e42039d6866b1aa57d38892aa47b53d437856d9a670008ef139a275b9a379dfb0d93b02843faf02c160aafe1f8fa0335e5872531c9de03c09eb6bfad33c

data/lib/mumuki/classroom/event.rb

@@ -2,3 +2,4 @@ module Mumuki::Classroom::Event
 end
 
 require_relative './event/user_changed'
+require_relative './event/progress_transfer'

data/lib/mumuki/classroom/event/progress_transfer.rb

@@ -0,0 +1,31 @@
+class Mumuki::Classroom::Event::ProgressTransfer
+  attr_reader :body
+
+  def initialize(body)
+    @body = body
+  end
+
+  def execute!
+    transfer_type.new(progress_item, source_organization, destination_organization).execute!
+  end
+
+  def source_organization
+    Organization.locate! body[:from]
+  end
+
+  def destination_organization
+    Organization.locate! body[:to]
+  end
+
+  def progress_item
+    Indicator.find(body[:item_id])
+  end
+
+  def transfer_type
+    self.class.const_get(body[:transfer_type].camelize)
+  end
+end
+
+require_relative 'progress_transfer/base'
+require_relative 'progress_transfer/copy'
+require_relative 'progress_transfer/move'

data/lib/mumuki/classroom/event/progress_transfer/base.rb

@@ -0,0 +1,111 @@
+class Mumuki::Classroom::Event::ProgressTransfer::Base
+  attr_reader :indicator, :source_organization
+
+  def initialize(indicator, source_organization, destination_organization)
+    @indicator = indicator
+    @source_organization = source_organization
+    @destination_organization = destination_organization
+  end
+
+  def execute!
+    raise ActiveRecord::RecordNotFound, "Mumuki::Classroom::Student not found" unless old_student && new_student
+
+    new_student.destroy_progress_for_guide!(indicator.content)
+    destination_organization.switch!
+
+    indicator.assignments.each do |assignment|
+      transfer_sibling_for(assignment)&.update! organization: destination_organization.name,
+                                                course: new_course,
+                                                guide: guide_h
+    end
+
+    transfer_guide_progress!
+    update_student!(old_student)
+    update_student!(new_student)
+  end
+
+  private
+
+  def update_student!(student)
+    student.update_all_stats
+    student.update_last_assignment_for
+  end
+
+  def destination_organization
+    @destination_organization
+  end
+
+  def transfer_guide_progress!
+    transfer_item.update!(guide: guide_h,
+                          student: new_student.dup,
+                          organization: destination_organization.name,
+                          course: new_course)
+  end
+
+  def guide_progress
+    Mumuki::Classroom::GuideProgress.find_by(guide_progress_query)
+  end
+
+  def classroom_sibling_for(assignment)
+    Mumuki::Classroom::Assignment.classroom_sibling_for(assignment, source_organization.name)
+  end
+
+  def new_student
+    @new_student ||= student_for destination_organization.name
+  end
+
+  def old_student
+    @old_student ||= student_for source_organization
+  end
+
+  def student_for(organization)
+    Mumuki::Classroom::Student.last_updated_student_by organization: organization, uid: user.uid
+  end
+
+  def new_course
+    new_student.course
+  end
+
+  def old_course
+    old_student.course
+  end
+
+  def guide_progress_query
+    {
+      organization: source_organization,
+      course: old_course,
+      'guide.slug': guide.slug,
+      'student.uid': user.uid
+    }
+  end
+
+  def user
+    indicator.user
+  end
+
+  def guide
+    indicator.content
+  end
+
+  def guide_parent
+    guide.usage_in_organization
+  end
+
+  def guide_h
+    @guide_h ||= guide.as_json(
+      only: [:slug, :name],
+      include: {
+        language: {
+          only: [:name, :devicon]
+        }
+      }
+    ).merge(
+      parent: {
+        type: guide_parent.class.to_s,
+        name: guide.name,
+        position: guide_parent.number,
+        chapter: guide.chapter.as_json(only: [:id], methods: [:name])
+      }
+    )
+  end
+end

data/lib/mumuki/classroom/event/progress_transfer/copy.rb

@@ -0,0 +1,9 @@
+class Mumuki::Classroom::Event::ProgressTransfer::Copy < Mumuki::Classroom::Event::ProgressTransfer::Base
+  def transfer_sibling_for(assignment)
+    classroom_sibling_for(assignment).dup
+  end
+
+  def transfer_item
+    guide_progress.dup
+  end
+end

data/lib/mumuki/classroom/event/progress_transfer/move.rb

@@ -0,0 +1,9 @@
+class Mumuki::Classroom::Event::ProgressTransfer::Move < Mumuki::Classroom::Event::ProgressTransfer::Base
+  def transfer_sibling_for(assignment)
+    classroom_sibling_for(assignment)
+  end
+
+  def transfer_item
+    guide_progress
+  end
+end

data/lib/mumuki/classroom/models/assignment.rb

@@ -143,6 +143,9 @@ class Mumuki::Classroom::Assignment < Mumuki::Classroom::Document
       stats[:failed] += stats.delete(:errored) || 0
       stats.slice(*empty_stats.keys)
     end
-  end
 
+    def classroom_sibling_for(assignment, organization)
+      find_by(organization: organization, 'student.uid': assignment.user.uid, 'exercise.eid': assignment.exercise.bibliotheca_id)
+    end
+  end
 end

data/lib/mumuki/classroom/models/concerns/with_submission_process.rb

@@ -22,7 +22,7 @@ module WithSubmissionProcess
   end
 
   def find_student_from(json)
-    Mumuki::Classroom::Student.find_by(organization: organization(json), course: course_slug(json), uid: uid(json)).as_json
+    Mumuki::Classroom::Student.find_by(organization: organization(json), course: course_slug(json), uid: uid(json))
   end
 
   def update_student_progress(json)
@@ -99,15 +99,7 @@ module WithSubmissionProcess
   end
 
   def student_from(json)
-    student = json[:student]
-
-    {uid: student[:uid],
-     name: student[:name],
-     email: student[:email],
-     image_url: student[:image_url],
-     social_id: student[:social_id],
-     last_name: student[:last_name],
-     first_name: student[:first_name]}.compact
+    json[:student].as_submission_json
   end
 
   def guide_from(json)
@@ -134,15 +126,6 @@ module WithSubmissionProcess
   end
 
   def submission_from(json)
-    {sid: json[:sid],
-     status: json[:status],
-     result: json[:result],
-     content: json[:content],
-     feedback: json[:feedback],
-     created_at: json[:created_at],
-     test_results: json[:test_results],
-     submissions_count: json[:submissions_count],
-     expectation_results: json[:expectation_results],
-     origin_ip: json[:origin_ip]}.compact
+    Mumuki::Classroom::FailedSubmission.new(json).as_assignment_submission
   end
 end

data/lib/mumuki/classroom/models/failed_submission.rb

@@ -4,7 +4,7 @@ class Mumuki::Classroom::FailedSubmission < Mumuki::Classroom::Document
 
   include Mongoid::Attributes::Dynamic
 
-  field :created_at, type: Time
+  include Mongoid::Timestamps::Created
 
   create_index 'organization': 1, 'submitter.uid': 1
   create_index({'guide.slug': 1, 'exercise.eid': 1}, {name: 'ExBibIdIndex'})
@@ -13,4 +13,7 @@ class Mumuki::Classroom::FailedSubmission < Mumuki::Classroom::Document
   scope :find_by_uid, -> (uid) { where 'submitter.uid': uid }
 
 
+  def as_assignment_submission
+    as_json(only: %i(sid status result content feedback created_at test_results submissions_count expectation_results origin_ip)).compact
+  end
 end

data/lib/mumuki/classroom/models/guide_progress.rb

@@ -52,6 +52,10 @@ class Mumuki::Classroom::GuideProgress < Mumuki::Classroom::Document
     def uid_field
       'student.uid'.to_sym
     end
+
+    def progresses_for(progress_item, student)
+      where(organization: progress_item.organization, course: student.course, slug: progress_item.content.slug, 'student.uid': student.uid)
+    end
   end
 
 end

data/lib/mumuki/classroom/models/student.rb

@@ -16,11 +16,23 @@ class Mumuki::Classroom::Student < Mumuki::Classroom::Document
   end
 
   def destroy_cascade!
-    Mumuki::Classroom::GuideProgress.destroy_all_by!(sub_student_query uid)
-    Mumuki::Classroom::Assignment.destroy_all_by!(sub_student_query uid)
+    destroy_progress!
     destroy!
   end
 
+  def destroy_progress!
+    destroy_progress_for_query!(sub_student_query uid)
+  end
+
+  def destroy_progress_for_guide!(guide)
+    destroy_progress_for_query!(sub_student_query(uid).merge 'guide.slug': guide.slug)
+  end
+
+  def destroy_progress_for_query!(query)
+    Mumuki::Classroom::GuideProgress.destroy_all_by!(query)
+    Mumuki::Classroom::Assignment.destroy_all_by!(query)
+  end
+
   def update_all_stats
     all_stats = Mumuki::Classroom::Assignment.stats_by(sub_student_query uid)
     update_attributes!(stats: all_stats)
@@ -52,6 +64,10 @@ class Mumuki::Classroom::Student < Mumuki::Classroom::Document
     update_attributes!(last_assignment: Mumuki::Classroom::GuideProgress.last_assignment_by(sub_student_query uid))
   end
 
+  def as_submission_json
+    as_json(only: %i(uid name email image_url social_id last_name first_name)).compact
+  end
+
   class << self
     def report(criteria, &block)
       where(criteria).select(&block).as_json(only: [:first_name, :last_name, :email, :created_at, :detached_at])

data/lib/mumuki/classroom/sinatra/teachers.rb

@@ -1,8 +1,18 @@
 class Mumuki::Classroom::App < Sinatra::Application
   Mumukit::Platform.map_organization_routes!(self) do
+    helpers do
+      def list_teachers(matcher)
+        authorize! :headmaster
+        {teachers: Mumuki::Classroom::Teacher.where(matcher).as_json}
+      end
+    end
+
+    get '/teachers' do
+      list_teachers with_organization
+    end
+
     get '/courses/:course/teachers' do
-      authorize! :headmaster
-      {teachers: Mumuki::Classroom::Teacher.where(with_organization_and_course).as_json}
+      list_teachers with_organization_and_course
     end
 
     post '/courses/:course/teachers' do

data/lib/mumuki/classroom/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Classroom
-    VERSION = '8.3.0'
+    VERSION = '9.0.1'
   end
 end

data/lib/tasks/mumuki/progress_transfers.rake

@@ -0,0 +1,17 @@
+namespace :classroom do
+  namespace :progress_transfers do
+    task listen: :environment do
+      Mumukit::Nuntius::Logger.info 'Listening to student progress-transfers'
+
+      Mumukit::Nuntius::Consumer.negligent_start! 'progress-transfers' do |body|
+        begin
+          Mumuki::Classroom::Event::ProgressTransfer.new(body).execute!
+
+          Mumukit::Nuntius::Logger.info "Processing progress transfer #{body[:item_id]}"
+        rescue => e
+          Mumukit::Nuntius::Logger.warn "Mumuki::Classroom::ProgressTransfer failed #{e}. body was: #{body}"
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-classroom
 version: !ruby/object:Gem::Version
-  version: 8.3.0
+  version: 9.0.1
 platform: ruby
 authors:
 - Franco Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-21 00:00:00.000000000 Z
+date: 2021-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 8.3.0
+        version: 9.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 8.3.0
+        version: 9.0.0
 - !ruby/object:Gem::Dependency
   name: mumukit-login
   requirement: !ruby/object:Gem::Requirement
@@ -128,14 +128,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: mongoid
   requirement: !ruby/object:Gem::Requirement
@@ -226,6 +226,10 @@ files:
 - lib/mumuki/classroom/collection.rb
 - lib/mumuki/classroom/engine.rb
 - lib/mumuki/classroom/event.rb
+- lib/mumuki/classroom/event/progress_transfer.rb
+- lib/mumuki/classroom/event/progress_transfer/base.rb
+- lib/mumuki/classroom/event/progress_transfer/copy.rb
+- lib/mumuki/classroom/event/progress_transfer/move.rb
 - lib/mumuki/classroom/event/user_changed.rb
 - lib/mumuki/classroom/locales/en.yml
 - lib/mumuki/classroom/locales/es-CL.yml
@@ -281,6 +285,7 @@ files:
 - lib/mumuki/profile.rb
 - lib/mumuki/views/threads.html.erb
 - lib/tasks/mumuki/messages.rake
+- lib/tasks/mumuki/progress_transfers.rake
 - lib/tasks/mumuki/resubmissions.rake
 - lib/tasks/mumuki/students.rake
 - lib/tasks/mumuki/submissions.rake