mumuki-classroom 8.5.0 → 9.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2689b5e9d65f6442b877791edebb807db6f4e8658d21ff2167e1149f9f4f776d
-  data.tar.gz: 3b433d35e0f31680f54e2f99291f0e67be60c8cba56abc5848353cc185701a62
+  metadata.gz: 4a46d4bfcc9b123f3585dace449a4878c5aab314061e52d2e9017e873494e994
+  data.tar.gz: 7ebaaa89a846bcf12ce895de21e51dd9040700236e9248fdad6c4a5798dc5c50
 SHA512:
-  metadata.gz: 1a7798738314f76157ee18dbd3f114149e68098a7a2121fa40dc85a72d78e0a6806b9b3dc749d4a858f6ed848029ffa6b0507e67d483cc253c6f826c8c80140f
-  data.tar.gz: 8d1dfe3a69bda6beaed3464ab31bfec591b7c36b2ecd155d1436e34f6af92278e2e5579216793838710e9e4e6d3f162b10a3fd8f2d697f612c0f9eb5835b30f6
+  metadata.gz: 6b64c14a1d833934f6e55ad40522a401e54bae94549fcae8f8ff5fb3f1b5233ebcfc2493bea5ec901bf613341bedc45601c496d64465c21e9cd1522b7438ddd2
+  data.tar.gz: 04e82d680eac93fda5c027accdb5a9e71d4bfe6bce83caa10b08894bab9ad25f47aebd378618c0f19570b5372ee399cf5e0c1f9d2bb339bab383a88c8938b1b9

data/config/initializers/inflections.rb

@@ -0,0 +1,3 @@
+ActiveSupport::Inflector.inflections(:en) do |inflect|
+  inflect.acronym 'FAQs'
+end

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/reporting.rb

@@ -17,8 +17,8 @@ module Reporting
     main_pipeline << {'$match': query}
     main_pipeline.concat searching.pipeline
     main_pipeline.concat sorting.pipeline
-    main_pipeline << {'$project': projection}
     main_pipeline << {'$sort': sorting.order_by(ordering)}
+    main_pipeline << {'$project': projection}
   end
 
 end

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

@@ -2,7 +2,7 @@ module Sorting
 
   def self.aggregate(collection, query, paginated_params, query_params)
     reporting_pipeline = Reporting.build_pipeline(collection, query, paginated_params, query_params, projection)
-    query = collection.collection.aggregate(pipeline paginated_params, reporting_pipeline).first
+    query = collection.collection.aggregate(pipeline(paginated_params, reporting_pipeline), allow_disk_use: true).first # Must allow disk use for sorting large collections by non-index
     query_results(query)
   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.5.0'
+    VERSION = '9.1.0'
   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.5.0
+  version: 9.1.0
 platform: ruby
 authors:
 - Franco Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-04 00:00:00.000000000 Z
+date: 2021-04-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 8.5.0
+        version: 9.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 8.5.0
+        version: 9.1.1
 - !ruby/object:Gem::Dependency
   name: mumukit-login
   requirement: !ruby/object:Gem::Requirement
@@ -222,10 +222,15 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- config/initializers/inflections.rb
 - lib/mumuki/classroom.rb
 - 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 +286,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