mumuki-domain 9.0.5 → 9.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 46425c74cf597dd17603273a455f7a5956c2f0b80926f81e739f45de0bd4e17e
-  data.tar.gz: 9241fe5525d3c387e726fda5037044c96564d77c802ec4af9e01de086648018e
+  metadata.gz: fc8a965a85ae367436a825f12235e8ce3149baacb9e610d6f2801c97270cf603
+  data.tar.gz: d1f59395e72b047031ff987e47e8a48a596aefd1c57d566480c4c7e33737d058
 SHA512:
-  metadata.gz: f7ff8c266a27514fac9fe776572efa0ec5d742627bda381b42218364853d75bd51c22aa7aae86be0fb0c8692e9b9b43209e96bc8ac6015cebacce0f3cdb869e7
-  data.tar.gz: 4ba53f6362fcbbbb38e9be6fc19e8a90561cfb02e6a0cc2ba82e597ca5cae8fc06d768f2e0ec7ef8255366d241f5558b1a0f2409b34bedc074466792d78f27fa
+  metadata.gz: b9b535002f54d54cfd9487122943bd44e31a6563eda77f672fb3b78347b13bf2cecacc880c7999efa04ed3a028d1352f899764a0fd95cbc4f53f617e9f010904
+  data.tar.gz: dbde9d021212f2fd8f6118df3a9f6585606e317b8b6d14acd0adb1190b834b8fdf63646ba1b2025db204e3424bcefe17c2491aa1c2f63cdeadd4b4b6a7bc798f

data/app/models/application_record.rb

@@ -29,7 +29,7 @@ class ApplicationRecord < ActiveRecord::Base
   def self.serialize_symbolized_hash_array(*keys)
     keys.each do |field|
       serialize field
-      define_method(field) { self[field]&.map { |it| it.symbolize_keys } }
+      define_method(field) { self[field]&.map { |it| it.deep_symbolize_keys } }
     end
   end
 
@@ -134,6 +134,19 @@ class ApplicationRecord < ActiveRecord::Base
     end
   end
 
+  def self.active_between(start_date_field, end_date_field, **options)
+    define_singleton_method(:active) do |actually_filter=true|
+      if actually_filter
+        self.where("(#{start_date_field} IS NULL OR #{start_date_field} < :now) AND (#{end_date_field} IS NULL OR #{end_date_field} > :now)", now: Time.now)
+      else
+        all
+      end
+    end
+
+    aliased_as = options.delete(:aliased_as)
+    singleton_class.send(:alias_method, aliased_as, :active) if aliased_as
+  end
+
   ## Partially implements resource-hash protocol, by
   ## defining `to_resource_h` and helper methods `resource_fields` and `slice_resource_h`
   ## using the given fields

data/app/models/certificate_program.rb

@@ -2,6 +2,8 @@ class CertificateProgram < ApplicationRecord
   belongs_to :organization
   has_many :certificates
 
+  active_between :start_date, :end_date, aliased_as: :ongoing
+
   def friendly
     title
   end
@@ -23,8 +25,8 @@ class CertificateProgram < ApplicationRecord
   }
 </style>
 <!-- You can use interpolations like --
-  <%#= certificate.start_date %>
-  <%#= certificate.end_date %>
+  <%#= certificate.started_at %>
+  <%#= certificate.ended_at %>
   <%#= user.formal_first_name %>
   <%#= user.formal_last_name %>
   <%#= user.formal_full_name %>

data/app/models/chapter.rb

@@ -2,8 +2,8 @@ class Chapter < ApplicationRecord
   include WithStats
   include WithNumber
 
-  include SiblingsNavigation
   include TerminalNavigation
+  include SiblingsNavigation
 
   include FriendlyName
 
@@ -29,8 +29,4 @@ class Chapter < ApplicationRecord
   def structural_parent
     book
   end
-
-  def pending_siblings_for(user)
-    book.pending_chapters(user)
-  end
 end

data/app/models/concerns/navigation/siblings_navigation.rb

@@ -12,10 +12,9 @@ module SiblingsNavigation
     structural_parent.structural_children
   end
 
-  #TODO reestablish this after indicators reliably linked to assignments
-  # def pending_siblings_for(user, organization=Organization.current)
-  #   siblings.reject { |it| it.progress_for(user, organization).completed? }
-  # end
+  def pending_siblings_for(user, organization=Organization.current)
+    siblings.reject { |it| it.progress_for(user, organization).completed? }
+  end
 
   # Names
 

data/app/models/concerns/navigation/terminal_navigation.rb

@@ -18,4 +18,8 @@ module TerminalNavigation
   def siblings
     []
   end
+
+  def next_for(_user)
+    nil
+  end
 end

data/app/models/concerns/with_soft_deletion.rb

@@ -0,0 +1,16 @@
+module WithSoftDeletion
+  extend ActiveSupport::Concern
+
+  included do
+    enum deletion_motive: %i(self_deleted inappropriate_content shares_solution discloses_personal_information)
+    belongs_to :deleted_by, class_name: 'User', optional: true
+  end
+
+  def soft_delete!(motive, deleter)
+    update! deletion_motive: motive, deleted_by: deleter, deleted_at: Time.now
+  end
+
+  def deleted?
+    deleted_at.present?
+  end
+end

data/app/models/discussion.rb

@@ -28,7 +28,7 @@ class Discussion < ApplicationRecord
   delegate :language, to: :item
   delegate :to_discussion_status, to: :status
 
-  MODERATOR_REVIEW_AVERAGE_TIME = 10.minutes
+  MODERATOR_REVIEW_AVERAGE_TIME = 17.minutes
 
   scope :for_user, -> (user) do
     if user.try(:moderator_here?)
@@ -38,6 +38,10 @@ class Discussion < ApplicationRecord
     end
   end
 
+  def visible_messages
+    messages.where.not(deletion_motive: :self_deleted).or(messages.where(deletion_motive: nil))
+  end
+
   def try_solve!
     if opened?
       update! status: reachable_statuses_for(initiator).first
@@ -73,7 +77,7 @@ class Discussion < ApplicationRecord
   end
 
   def last_message_date
-    messages.last&.created_at || created_at
+    visible_messages.last&.created_at || created_at
   end
 
   def friendly
@@ -125,11 +129,11 @@ class Discussion < ApplicationRecord
   end
 
   def has_messages?
-    messages.exists? || description.present?
+    visible_messages.exists? || description.present?
   end
 
   def responses_count
-    messages.where.not(sender: initiator.uid).count
+    visible_messages.where.not(sender: initiator.uid).count
   end
 
   def has_responses?
@@ -186,6 +190,6 @@ class Discussion < ApplicationRecord
   private
 
   def messages_by_updated_at(direction = :desc)
-    messages.reorder(updated_at: direction)
+    messages.where(deletion_motive: nil).reorder(updated_at: direction)
   end
 end

data/app/models/exam_authorization_request.rb

@@ -18,6 +18,17 @@ class ExamAuthorizationRequest < ApplicationRecord
     exam_registration.description
   end
 
+  def icon
+    case status.to_sym
+    when :pending
+      { class: 'info-circle', type: 'info' }
+    when :approved
+      { class: 'check-circle', type: 'success' }
+    when :rejected
+      { class: 'times-circle', type: 'danger' }
+    end
+  end
+
   private
 
   def notify_user!

data/app/models/exam_registration.rb

@@ -5,6 +5,8 @@ class ExamRegistration < ApplicationRecord
   belongs_to :organization
   has_and_belongs_to_many :exams
   has_many :authorization_requests, class_name: 'ExamAuthorizationRequest'
+  has_and_belongs_to_many :registrees, class_name: 'User'
+  has_many :notifications, as: :target
 
   enum authorization_criterion_type: %i(none passed_exercises), _prefix: :authorization_criterion
 
@@ -22,8 +24,20 @@ class ExamRegistration < ApplicationRecord
     authorization_criterion.ensure_valid!
   end
 
-  def start!(users)
-    users.each &method(:notify_user!)
+  def notify_unnotified_registrees!
+    unnotified_registrees.each { |registree| notify_registree! registree }
+  end
+
+  def unnotified_registrees?
+    unnotified_registrees.exists?
+  end
+
+  def register_users!(users)
+    users.each { |user| register! user }
+  end
+
+  def unnotified_registrees
+    registrees.where.not(id: Notification.notified_users_ids_for(self, self.organization))
   end
 
   def process_requests!
@@ -38,9 +52,17 @@ class ExamRegistration < ApplicationRecord
       ExamAuthorizationRequest.new(exam_registration: self, organization: organization)
   end
 
+  def register!(user)
+    registrees << user unless registered?(user)
+  end
+
+  def registered?(user)
+    registrees.include? user
+  end
+
   private
 
-  def notify_user!(user)
-    Notification.create! organization: organization, user: user, target: self
+  def notify_registree!(registree)
+    Notification.create! organization: organization, user: registree, target: self
   end
 end

data/app/models/exercise.rb

@@ -249,18 +249,6 @@ class Exercise < ApplicationRecord
   private
 
   def evaluation_class
-    if manual_evaluation?
-      manual_evaluation_class
-    else
-      automated_evaluation_class
-    end
-  end
-
-  def manual_evaluation_class
-    Mumuki::Domain::Evaluation::Manual
-  end
-
-  def automated_evaluation_class
     Mumuki::Domain::Evaluation::Automated
   end
 

data/app/models/exercise/problem.rb

@@ -33,13 +33,31 @@ class Problem < QueriableChallenge
   end
 
   def evaluation_criteria?
-    manual_evaluation? || expectations? || test.present?
+    manual_evaluation? || automated_evaluation?
+  end
+
+  def mixed_evaluation?
+    manual_evaluation? && automated_evaluation?
+  end
+
+  def automated_evaluation?
+    expectations? || test.present?
   end
 
   def expectations?
     own_expectations.present? || own_custom_expectations.present?
   end
 
+  def evaluation_class
+    if mixed_evaluation?
+      Mumuki::Domain::Evaluation::Mixed
+    elsif manual_evaluation?
+      Mumuki::Domain::Evaluation::Manual
+    else
+      Mumuki::Domain::Evaluation::Automated
+    end
+  end
+
   # Sets the layout. This method accepts input_kids as a synonym of input_primary
   # for historical reasons
   def layout=(layout)

data/app/models/message.rb

@@ -1,4 +1,5 @@
 class Message < ApplicationRecord
+  include WithSoftDeletion
 
   belongs_to :discussion, optional: true
   belongs_to :assignment, foreign_key: :submission_id, primary_key: :submission_id, optional: true
@@ -10,7 +11,6 @@ class Message < ApplicationRecord
   validates_presence_of :submission_id, :unless => :discussion_id?
 
   after_save :update_counters_cache!
-  after_destroy :update_counters_cache!
 
   markdown_on :content
 
@@ -43,7 +43,8 @@ class Message < ApplicationRecord
   end
 
   def to_resource_h
-    as_json(except: [:id, :type, :discussion_id, :approved, :not_actually_a_question],
+    as_json(except: [:id, :type, :discussion_id, :approved, :approved_at, :approved_by_id,
+                     :not_actually_a_question, :deletion_motive, :deleted_at, :deleted_by_id],
             include: {exercise: {only: [:bibliotheca_id]}})
         .merge(organization: Organization.current.name)
   end

data/app/models/notification.rb

@@ -3,6 +3,11 @@ class Notification < ApplicationRecord
   belongs_to :organization
   belongs_to :target, polymorphic: true
 
+
+  scope :notified_users_ids_for, ->(target, organization=Organization.current) do
+    where(target: target, organization: organization).pluck(:user_id)
+  end
+
   def mark_as_read!
     update read: true
   end

data/app/models/organization.rb

@@ -21,6 +21,8 @@ class Organization < ApplicationRecord
   belongs_to :book
   has_many :usages
 
+  has_many :certificate_programs
+
   validates_presence_of :contact_email, :locale
   validates_presence_of :welcome_email_template, if: :greet_new_users?
   validates :name, uniqueness: true,
@@ -31,7 +33,7 @@ class Organization < ApplicationRecord
   after_create :reindex_usages!
   after_update :reindex_usages!, if: lambda { |user| user.saved_change_to_book_id? }
 
-  has_many :guides, through: 'usages', source: 'item', source_type: 'Guide'
+  has_many :guides, -> { where 'usages.parent_item_type' => 'Lesson' }, through: 'usages', source: 'item', source_type: 'Guide'
   has_many :exercises, through: :guides
   has_many :assignments, through: :exercises
   has_many :exams
@@ -108,6 +110,10 @@ class Organization < ApplicationRecord
     name
   end
 
+  def ongoing_certificate_programs?
+    certificate_programs.ongoing.exists?
+  end
+
   # Tells if the given user can
   # ask for help in this organization
   #

data/app/models/user.rb

@@ -211,6 +211,10 @@ class User < ApplicationRecord
     name.split.map(&:first).map(&:capitalize).join(' ')
   end
 
+  def abbreviated_name
+    "#{first_name} #{last_name.first.capitalize + '.' if last_name.present?}".strip
+  end
+
   def progress_at(content, organization)
     Indicator.find_or_initialize_by(user: self, organization: organization, content: content)
   end
@@ -283,11 +287,11 @@ class User < ApplicationRecord
   end
 
   def formal_first_name
-    verified_first_name || first_name
+    verified_first_name.presence || first_name
   end
 
   def formal_last_name
-    verified_last_name || last_name
+    verified_last_name.presence || last_name
   end
 
   def formal_full_name
@@ -302,6 +306,12 @@ class User < ApplicationRecord
     certificates.where(certificate_program: certificate_program).exists?
   end
 
+  def certificate_in(certificate_program, certificate_h)
+    return if certificated_in?(certificate_program)
+    certificate = certificates.create certificate_h.merge(certificate_program: certificate_program)
+    UserMailer.certificate(certificate).deliver_later
+  end
+
   private
 
   def welcome_to_new_organizations!

data/app/models/user_stats.rb

@@ -16,10 +16,10 @@ class UserStats < ApplicationRecord
         exercises: {
             solved_count: organization_exercises
                               .joins(:assignments)
-                              .where(assignments: { top_submission_status: [:passed, :skipped], submitter: user }.merge(date_filter))
+                              .where(assignments: { top_submission_status: [:passed, :skipped], submitter: user, organization: organization }.merge(date_filter))
                               .count,
-            count: organization_exercises.count},
-
+            count: organization_exercises.count
+        },
         messages: messages_in_discussions_count(date_range)
     }
   end

data/db/migrate/20210318191512_add_deletion_fields_to_message.rb

@@ -0,0 +1,7 @@
+class AddDeletionFieldsToMessage < ActiveRecord::Migration[5.1]
+  def change
+    add_column :messages, :deletion_motive, :integer
+    add_column :messages, :deleted_at, :datetime
+    add_reference :messages, :deleted_by, index: true
+  end
+end

data/db/migrate/20210318194559_add_dates_to_certificate_program.rb

@@ -0,0 +1,6 @@
+class AddDatesToCertificateProgram < ActiveRecord::Migration[5.1]
+  def change
+    add_column :certificate_programs, :start_date, :datetime
+    add_column :certificate_programs, :end_date, :datetime
+  end
+end

data/db/migrate/20210318195238_rename_certificate_dates.rb

@@ -0,0 +1,6 @@
+class RenameCertificateDates < ActiveRecord::Migration[5.1]
+  def change
+    rename_column :certificates, :start_date, :started_at
+    rename_column :certificates, :end_date, :ended_at
+  end
+end

data/db/migrate/20210330175706_create_exam_registration_user_join_table.rb

@@ -0,0 +1,8 @@
+class CreateExamRegistrationUserJoinTable < ActiveRecord::Migration[5.1]
+  def change
+    create_join_table :exam_registrations, :users do |t|
+      t.index :user_id
+      t.index :exam_registration_id
+    end
+  end
+end

data/lib/mumuki/domain/evaluation.rb

@@ -3,3 +3,4 @@ end
 
 require_relative './evaluation/manual'
 require_relative './evaluation/automated'
+require_relative './evaluation/mixed'

data/lib/mumuki/domain/evaluation/mixed.rb

@@ -0,0 +1,12 @@
+class Mumuki::Domain::Evaluation::Mixed < Mumuki::Domain::Evaluation::Manual
+  def evaluate!(assignment, submission)
+    evaluation = submission.evaluate! assignment
+    if evaluation[:status].passed?
+      super
+    elsif evaluation[:status].passed_with_warnings?
+      evaluation.merge(status: Mumuki::Domain::Status::Submission::Failed)
+    else
+      evaluation
+    end
+  end
+end

data/lib/mumuki/domain/factories/certificate_factory.rb

@@ -1,7 +1,7 @@
 FactoryBot.define do
   factory :certificate do
-    start_date { 1.month.ago }
-    end_date { 1.minute.ago }
+    started_at { 1.month.ago }
+    ended_at { 1.minute.ago }
     user { build :user, first_name: 'Jane', last_name: 'Doe' }
     certificate_program { build :certificate_program }
   end

data/lib/mumuki/domain/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Domain
-    VERSION = '9.0.5'
+    VERSION = '9.2.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-domain
 version: !ruby/object:Gem::Version
-  version: 9.0.5
+  version: 9.2.0
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-03-19 00:00:00.000000000 Z
+date: 2021-05-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -301,6 +301,7 @@ files:
 - app/models/concerns/with_scoped_queries/page.rb
 - app/models/concerns/with_scoped_queries/sort.rb
 - app/models/concerns/with_slug.rb
+- app/models/concerns/with_soft_deletion.rb
 - app/models/concerns/with_target_audience.rb
 - app/models/concerns/with_terms_acceptance.rb
 - app/models/concerns/with_timed_enablement.rb
@@ -661,12 +662,17 @@ files:
 - db/migrate/20210302181654_add_faqs_to_organizations.rb
 - db/migrate/20210308145910_add_approved_by_and_at_to_message.rb
 - db/migrate/20210310195602_add_delete_account_token_to_user.rb
+- db/migrate/20210318191512_add_deletion_fields_to_message.rb
+- db/migrate/20210318194559_add_dates_to_certificate_program.rb
+- db/migrate/20210318195238_rename_certificate_dates.rb
+- db/migrate/20210330175706_create_exam_registration_user_join_table.rb
 - lib/mumuki/domain.rb
 - lib/mumuki/domain/area.rb
 - lib/mumuki/domain/engine.rb
 - lib/mumuki/domain/evaluation.rb
 - lib/mumuki/domain/evaluation/automated.rb
 - lib/mumuki/domain/evaluation/manual.rb
+- lib/mumuki/domain/evaluation/mixed.rb
 - lib/mumuki/domain/exceptions.rb
 - lib/mumuki/domain/exceptions/blocked_forum_error.rb
 - lib/mumuki/domain/exceptions/disabled_error.rb