mumuki-domain 9.8.1 → 9.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 27f2a30ca00c41bffa5a3673fe92328ee1b0565d27a8e3e2ac34e40e791ac2ca
-  data.tar.gz: be141181663a286b037540f0a7b553a190d20b254429eff0e9f64263f60bbfc4
+  metadata.gz: d20f0f5d9cfe938c4a33f561f9e4df1e5e84ab18296fda644038b87166d631fc
+  data.tar.gz: bf7adb112a37b84a8fb4f8fa7dc74db1c5df29dc0d8763005256e98e381b7951
 SHA512:
-  metadata.gz: 9c5bb7a56c9fb93812146a45e21c74e438036812243c3566f01baf1916d0fce49c0be5a3153c0c973122d45e4b5556eddc64602f794fd2c838669470208b6032
-  data.tar.gz: 2eb10bc2416e79bb20e05bf4eea458ea834aa68fa5e14af7d4e45043e3c49827961de4d35eaf7a2b0fb34fb4360b2654a85fada4056bd7ba3f22f32c4aaf7703
+  metadata.gz: fbcc0dcdd8fd5134d463f61f18da8dfc872b7142f221f16ea3d99dac828bdbcf628d47b924559ac40e3224597955ad575720dc318b385297509cc3364b5c9897
+  data.tar.gz: 4a2a358f2b888268d4193b0b9b50873702ac293430c26e5d64576f7c02b185fc4263cbc0b30c02758c611240e9646bb917277ed5ec37951521762db028f0fb52

data/app/models/application_record.rb

@@ -152,7 +152,7 @@ class ApplicationRecord < ActiveRecord::Base
   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)
+        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.current)
       else
         all
       end

data/app/models/assignment.rb

@@ -7,11 +7,7 @@ class Assignment < Progress
 
   belongs_to :exercise
   has_one :guide, through: :exercise
-  has_many :messages,
-           -> { where.not(submission_id: nil).order(date: :desc) },
-           foreign_key: :submission_id,
-           primary_key: :submission_id,
-           dependent: :destroy
+  has_many :messages, -> { order(date: :desc) }, dependent: :destroy
 
   belongs_to :organization
   belongs_to :submitter, class_name: 'User'
@@ -28,13 +24,13 @@ class Assignment < Progress
   alias_attribute :status, :submission_status
   alias_attribute :attempts_count, :attemps_count
 
-  scope :by_exercise_ids, -> (exercise_ids) {
+  scope :by_exercise_ids, -> (exercise_ids) do
     where(exercise_id: exercise_ids) if exercise_ids
-  }
+  end
 
-  scope :by_usernames, -> (usernames) {
+  scope :by_usernames, -> (usernames) do
     joins(:submitter).where('users.name' => usernames) if usernames
-  }
+  end
 
   defaults do
     self.query_results = []
@@ -86,11 +82,10 @@ class Assignment < Progress
     end
   end
 
-  def persist_submission!(submission)
+  def save_submission!(submission)
     transaction do
-      messages.destroy_all if submission_id.present?
       update! submission_id: submission.id
-      update! submitted_at: DateTime.current
+      update! submitted_at: Time.current
       update_submissions_count!
       update_last_submission!
     end
@@ -286,6 +281,6 @@ class Assignment < Progress
   end
 
   def update_last_submission!
-    submitter.update!(last_submission_date: DateTime.current, last_exercise: exercise)
+    submitter.update!(last_submission_date: Time.current, last_exercise: exercise)
   end
 end

data/app/models/concerns/onomastic.rb

@@ -0,0 +1,37 @@
+module Onomastic
+  extend ActiveSupport::Concern
+
+  included do
+    alias_method :name, :full_name
+  end
+
+  def formal_first_name
+    verified_first_name.presence || first_name
+  end
+
+  def formal_last_name
+    verified_last_name.presence || last_name
+  end
+
+  def has_verified_full_name?
+    verified_first_name? && verified_last_name?
+  end
+
+  def formal_full_name
+    join_names formal_first_name, formal_last_name
+  end
+
+  def full_name
+    join_names first_name, last_name
+  end
+
+  def verified_full_name
+    join_names verified_first_name, verified_last_name
+  end
+
+  private
+
+  def join_names(first, last)
+    "#{first} #{last}".strip
+  end
+end

data/app/models/concerns/with_discussion_creation/subscription.rb

@@ -8,7 +8,7 @@ module WithDiscussionCreation::Subscription
   end
 
   def subscriptions_in_organization
-    subscriptions.joins(:discussion).where(discussion: discussions_in_organization)
+    subscriptions.where(discussion: Organization.current.discussions)
   end
 
   def subscribed_to?(discussion)

data/app/models/concerns/with_messages.rb

@@ -9,7 +9,7 @@ module WithMessages
   end
 
   def build_message(body)
-    messages.build({date: DateTime.current}.merge(body))
+    messages.build({date: Time.current, submission_id: submission_id}.merge(body))
   end
 
   def has_messages?

data/app/models/concerns/with_notifications.rb

@@ -7,8 +7,7 @@ module WithNotifications
 
   def unread_notifications
     # TODO: message and discussion should trigger a notification instead of being one
-    # include message notifications once they are properly implemented
-    all = notifications.where(read: false) + unread_discussions
+    all = notifications_in_organization.where(read: false) + unread_messages + unread_discussions
     all.sort_by(&:created_at).reverse
   end
 

data/app/models/concerns/with_reminders.rb

@@ -10,8 +10,8 @@ module WithReminders
   end
 
   def remind!
-    build_reminder.deliver_now
-    update! last_reminded_date: Time.now
+    build_reminder.post!
+    update! last_reminded_date: Time.current
   end
 
   def should_remind?

data/app/models/concerns/with_responsible_moderator.rb

@@ -34,7 +34,7 @@ module WithResponsibleModerator
   private
 
   def responsible!(moderator)
-    update! responsible_moderator_at: Time.now, responsible_moderator_by: moderator
+    update! responsible_moderator_at: Time.current, responsible_moderator_by: moderator
   end
 
   def no_responsible!

data/app/models/concerns/with_soft_deletion.rb

@@ -7,7 +7,7 @@ module WithSoftDeletion
   end
 
   def soft_delete!(motive, deleter)
-    update! deletion_motive: motive, deleted_by: deleter, deleted_at: Time.now
+    update! deletion_motive: motive, deleted_by: deleter, deleted_at: Time.current
   end
 
   def deleted?

data/app/models/concerns/with_terms_acceptance.rb

@@ -58,8 +58,7 @@ module WithTermsAcceptance
   end
 
   def accept_terms!(terms)
-    update! unaccepted_terms_scopes_in(terms).to_h { |scope| [term_acceptance_field_for(scope), Time.now] }
+    update! unaccepted_terms_scopes_in(terms).to_h { |scope| [term_acceptance_field_for(scope), Time.current] }
   end
 
 end
-

data/app/models/concerns/with_timed_enablement.rb

@@ -2,7 +2,7 @@ module WithTimedEnablement
   extend ActiveSupport::Concern
 
   def enabled?
-    enabled_range.cover? DateTime.current
+    enabled_range.cover? Time.current
   end
 
   def enabled_range

data/app/models/course.rb

@@ -14,7 +14,7 @@ class Course < ApplicationRecord
   resource_fields :slug, :shifts, :code, :days, :period, :description, :period_start, :period_end
 
   def current_invitation
-    invitations.where('expiration_date > ?', Time.now).first
+    invitations.where('expiration_date > ?', Time.current).first
   end
 
   def import_from_resource_h!(resource_h)
@@ -49,7 +49,7 @@ class Course < ApplicationRecord
     period =~ /^(\d{4})?/
     year = $1.to_i
 
-    return nil unless year.between? 2014, (DateTime.now.year + 1)
+    return nil unless year.between? 2014, (Time.current.year + 1)
 
     self.period_start = DateTime.new(year).beginning_of_year
     self.period_end = DateTime.new(year).end_of_year

data/app/models/discussion.rb

@@ -21,6 +21,7 @@ class Discussion < ApplicationRecord
   scope :no_responsible_moderator, -> { where('responsible_moderator_at < ?', Time.now - MODERATOR_MAX_RESPONSIBLE_TIME)
                                           .or(where(responsible_moderator_at: nil)) }
   scope :pending_review, -> { where(status: :pending_review) }
+  scope :unread_first, -> { includes(:subscriptions).reorder('subscriptions.read', created_at: :desc) }
 
   after_create :subscribe_initiator!
 
@@ -42,7 +43,7 @@ class Discussion < ApplicationRecord
   end
 
   def visible_messages
-    messages.where.not(deletion_motive: :self_deleted).or(messages.where(deletion_motive: nil))
+    messages.visible
   end
 
   def try_solve!
@@ -54,7 +55,7 @@ class Discussion < ApplicationRecord
   def navigable_content_in(_)
     nil
   end
-  
+
   def target
     self
   end
@@ -95,15 +96,15 @@ class Discussion < ApplicationRecord
     upvotes.find_by(user: user)
   end
 
-  def unread_subscriptions(user)
-    subscriptions.where.not(user: user).map(&:unread!)
+  def mark_subscriptions_as_unread!(user)
+    subscriptions.where.not(user: user).each(&:unread!)
   end
 
   def submit_message!(message, user)
     message.merge!(sender: user.uid)
     messages.create(message)
     user.subscribe_to! self
-    unread_subscriptions(user)
+    mark_subscriptions_as_unread!(user)
     no_responsible! if responsible?(user)
   end
 
@@ -128,7 +129,7 @@ class Discussion < ApplicationRecord
     if reachable_status_for?(user, status)
       update! status: status,
               status_updated_by: user,
-              status_updated_at: Time.now
+              status_updated_at: Time.current
 
       no_responsible! if responsible?(user)
     end
@@ -182,6 +183,11 @@ class Discussion < ApplicationRecord
     klazz.constantize.find(debatable_id)
   end
 
+  # TODO remove this once discussions generate notifications
+  def subject
+    'discussion'
+  end
+
   private
 
   def messages_by_updated_at(direction = :desc)

data/app/models/exam.rb

@@ -32,7 +32,7 @@ class Exam < ApplicationRecord
   end
 
   def enabled_for?(user)
-    enabled_range_for(user).cover? DateTime.current
+    enabled_range_for(user).cover? Time.current
   end
 
   def in_progress_for?(user)
@@ -179,8 +179,8 @@ class Exam < ApplicationRecord
     exam[:organization_id] = Organization.current.id
     exam[:course_id] = Course.locate!(exam[:course].to_s).id
     exam[:users] = User.where(uid: exam[:uids])
-    exam[:start_time] = exam[:start_time].to_time
-    exam[:end_time] = exam[:end_time].to_time
+    exam[:start_time] = exam[:start_time].in_time_zone
+    exam[:end_time] = exam[:end_time].in_time_zone
     exam[:classroom_id] = exam[:eid] if exam[:eid].present?
   end
 

data/app/models/exam_authorization.rb

@@ -4,7 +4,7 @@ class ExamAuthorization < ApplicationRecord
   belongs_to :exam
 
   def start!
-    update!(started: true, started_at: Time.now) unless started?
+    update!(started: true, started_at: Time.current) unless started?
   end
 
 end

data/app/models/exam_authorization_request.rb

@@ -32,6 +32,6 @@ class ExamAuthorizationRequest < ApplicationRecord
   private
 
   def notify_user!
-    Notification.create! organization: organization, user: user, target: self if saved_change_to_status?
+    Notification.create_and_notify_via_email! organization: organization, user: user, target: self, subject: 'exam_authorization_request_updated' if saved_change_to_status?
   end
 end

data/app/models/exam_registration.rb

@@ -17,7 +17,7 @@ class ExamRegistration < ApplicationRecord
 
   alias_attribute :name, :description
 
-  scope :should_process, -> { where(processed: false).where(arel_table[:end_time].lt(Time.now)) }
+  scope :should_process, -> { where(processed: false).where(arel_table[:end_time].lt(Time.current)) }
 
   def authorization_criterion
     @authorization_criterion ||= ExamRegistration::AuthorizationCriterion.parse(authorization_criterion_type, authorization_criterion_value)
@@ -73,9 +73,13 @@ class ExamRegistration < ApplicationRecord
     registrees.include? user
   end
 
+  def meets_criterion?(user)
+    authorization_criterion.meets_criterion?(user, organization)
+  end
+
   private
 
   def notify_registree!(registree)
-    Notification.create! organization: organization, user: registree, target: self
+    Notification.create_and_notify_via_email! organization: organization, user: registree, subject: :exam_registration, target: self
   end
 end

data/app/models/exam_registration/authorization_criterion.rb

@@ -34,6 +34,10 @@ class ExamRegistration::AuthorizationCriterion
   rescue
     raise "Invalid criterion type #{type}"
   end
+
+  def meets_authorization_criteria?(authorization_request)
+    meets_criterion? authorization_request.user, authorization_request.organization
+  end
 end
 
 class ExamRegistration::AuthorizationCriterion::None < ExamRegistration::AuthorizationCriterion
@@ -45,7 +49,7 @@ class ExamRegistration::AuthorizationCriterion::None < ExamRegistration::Authori
     !value
   end
 
-  def meets_authorization_criteria?(_authorization_request)
+  def meets_criterion?(_user, _organization)
     true
   end
 end
@@ -55,7 +59,7 @@ class ExamRegistration::AuthorizationCriterion::PassedExercises < ExamRegistrati
     value.positive?
   end
 
-  def meets_authorization_criteria?(authorization_request)
-    authorization_request.user.passed_submissions_count_in(authorization_request.organization) >= value
+  def meets_criterion?(user, organization)
+    user.passed_submissions_count_in(organization) >= value
   end
 end

data/app/models/exercise.rb

@@ -168,14 +168,6 @@ class Exercise < ApplicationRecord
     Exercise.find(id)
   end
 
-  def messages_path_for(user)
-    "api/guides/#{guide.slug}/#{bibliotheca_id}/student/#{URI.escape user.uid}/messages?language=#{language}"
-  end
-
-  def messages_url_for(user)
-    Mumukit::Platform.classroom_api.organic_url_for(Organization.current, messages_path_for(user))
-  end
-
   def description_context
     splitted_description.first.markdownified
   end

data/app/models/exercise/playground.rb

@@ -3,8 +3,9 @@ class Playground < QueriableChallenge
 
   name_model_as Exercise
 
-  def setup_query_assignment!(assignment)
+  def save_query_submission!(assignment, submission)
     assignment.running!
+    assignment.save_submission! submission
   end
 
   def save_query_results!(assignment)

data/app/models/exercise/problem.rb

@@ -9,7 +9,7 @@ class Problem < QueriableChallenge
 
   name_model_as Exercise
 
-  def setup_query_assignment!(assignment)
+  def save_query_submission!(_assignment, _submission)
   end
 
   def save_query_results!(assignment)

data/app/models/message.rb

@@ -2,18 +2,48 @@ class Message < ApplicationRecord
   include WithSoftDeletion
 
   belongs_to :discussion, optional: true
-  belongs_to :assignment, foreign_key: :submission_id, primary_key: :submission_id, optional: true
+  belongs_to :assignment, optional: true
   belongs_to :approved_by, class_name: 'User', optional: true
 
   has_one :exercise, through: :assignment
 
   validates_presence_of :content, :sender
-  validates_presence_of :submission_id, :unless => :discussion_id?
+  validate :ensure_contextualized
 
   after_save :update_counters_cache!
 
   markdown_on :content
 
+  # Visible messages are those that can be publicly seen
+  # in forums. non-direct messages are never visible.
+  scope :visible, -> () do
+    where.not(deletion_motive: :self_deleted)
+      .or(where(deletion_motive: nil))
+      .where(assignment_id: nil)
+  end
+
+  def contextualization
+    direct? ? assignment : discussion
+  end
+
+  def contextualized?
+    assignment_id.present? ^ discussion_id.present?
+  end
+
+  # Whether this message is stale, that is, it
+  # targets a submission that is not the latest one.
+  #
+  # Only direct messages may become stale.
+  def stale?
+    direct? && assignment.submission_id != submission_id
+  end
+
+  # Whether this message is direct, that is, whether it comes from rise-hand feature.
+  # Forum messages are non-direct.
+  def direct?
+    submission_id.present?
+  end
+
   def notify!
     Mumukit::Nuntius.notify! 'student-messages', to_resource_h unless Organization.silenced?
   end
@@ -85,33 +115,34 @@ class Message < ApplicationRecord
     self
   end
 
-  def self.parse_json(json)
-    message = json.delete 'message'
-    json
-        .except('uid', 'exercise_id')
-        .merge(message)
-  end
-
   def self.read_all!
     update_all read: true
   end
 
-  def self.import_from_resource_h!(json)
-    message_data = parse_json json
-    Organization.find_by!(name: message_data.delete('organization')).switch!
-
-    if message_data['submission_id'].present?
-      Assignment.find_by(submission_id: message_data.delete('submission_id'))&.receive_answer! message_data
+  def self.import_from_resource_h!(resource_h)
+    if resource_h['submission_id'].present?
+      assignment = Assignment.find_by(submission_id: resource_h['submission_id'])
+      assignment&.receive_answer! sender: resource_h['message']['sender'],
+                                  content: resource_h['message']['content']
     end
   end
 
+  # TODO remove this once messages generate notifications
+  def subject
+    'message'
+  end
+
   private
 
   def approve!(user)
-    update! approved: true, approved_at: Time.now, approved_by: user
+    update! approved: true, approved_at: Time.current, approved_by: user
   end
 
   def disapprove!
     update! approved: false, approved_at: nil, approved_by: nil
   end
+
+  def ensure_contextualized
+    errors.add(:base, :not_properly_contextualized) unless contextualized?
+  end
 end

data/app/models/notification.rb

@@ -1,8 +1,12 @@
 class Notification < ApplicationRecord
   belongs_to :user
   belongs_to :organization
-  belongs_to :target, polymorphic: true
+  belongs_to :target, polymorphic: true, optional: true
 
+  enum subject: %i(
+    custom
+    exam_authorization_request_updated
+    exam_registration)
 
   scope :notified_users_ids_for, ->(target, organization=Organization.current) do
     where(target: target, organization: organization).pluck(:user_id)
@@ -11,4 +15,12 @@ class Notification < ApplicationRecord
   def mark_as_read!
     update read: true
   end
+
+  def self.create_and_notify_via_email!(args)
+    create!(args).tap(&:notify_via_email!)
+  end
+
+  def notify_via_email!
+    user.notify_via_email! self
+  end
 end

data/app/models/organization.rb

@@ -23,7 +23,7 @@ class Organization < ApplicationRecord
 
   has_many :certificate_programs
 
-  validates_presence_of :contact_email, :locale
+  validates_presence_of :contact_email, :locale, :time_zone
   validates_presence_of :welcome_email_template, if: :greet_new_users?
   validates :name, uniqueness: true,
                    presence: true,
@@ -148,6 +148,10 @@ class Organization < ApplicationRecord
     [default_date, in_preparation_until&.to_date].compact.max
   end
 
+  def discussions
+    book.discussions_in_organization(self)
+  end
+
   # ==============
   # Display fields
   # ==============

data/app/models/user.rb

@@ -9,10 +9,11 @@ class User < ApplicationRecord
           Disabling,
           WithTermsAcceptance,
           WithPreferences,
+          Onomastic,
           Mumuki::Domain::Helpers::User
 
   serialize :permissions, Mumukit::Auth::Permissions
-
+  serialize :ignored_notifications, Array
 
   has_many :notifications
   has_many :assignments, foreign_key: :submitter_id
@@ -53,6 +54,8 @@ class User < ApplicationRecord
   resource_fields :uid, :social_id, :email, :permissions, :verified_first_name, :verified_last_name, *profile_fields
   with_temporary_token :delete_account_token
 
+  organic_on :notifications
+
   def last_lesson
     last_guide.try(:lesson)
   end
@@ -61,6 +64,10 @@ class User < ApplicationRecord
     messages.where('assignments.organization': organization)
   end
 
+  def notifications_in_organization(organization = Organization.current)
+    notifications.where(organization: organization)
+  end
+
   def passed_submissions_count_in(organization)
     assignments.where(top_submission_status: Mumuki::Domain::Status::Submission::Passed.to_i, organization: organization).count
   end
@@ -251,7 +258,7 @@ class User < ApplicationRecord
 
   def age
     if birthdate.present?
-      @age ||= Time.now.round_years_since(birthdate.to_time)
+      @age ||= Time.current.round_years_since(birthdate.in_time_zone)
     end
   end
 
@@ -288,18 +295,6 @@ class User < ApplicationRecord
     end
   end
 
-  def formal_first_name
-    verified_first_name.presence || first_name
-  end
-
-  def formal_last_name
-    verified_last_name.presence || last_name
-  end
-
-  def formal_full_name
-    "#{formal_first_name} #{formal_last_name}"
-  end
-
   def certificates_in_organization(organization = Organization.current)
     certificates.where certificate_program: CertificateProgram.where(organization: organization)
   end
@@ -311,7 +306,7 @@ class User < ApplicationRecord
   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
+    UserMailer.certificate(certificate).post!
   end
 
   def clear_progress_for!(organization)
@@ -326,11 +321,19 @@ class User < ApplicationRecord
     user_stats.where(location).delete_all
   end
 
+  def notify_via_email!(notification)
+    UserMailer.notification(notification).post! unless ignores_notification? notification
+  end
+
+  def ignores_notification?(notification)
+    ignored_notifications.include? notification.subject
+  end
+
   private
 
   def welcome_to_new_organizations!
     new_accessible_organizations.each do |organization|
-      UserMailer.welcome_email(self, organization).deliver_now rescue nil if organization.greet_new_users?
+      UserMailer.welcome_email(self, organization).post! rescue nil if organization.greet_new_users?
     end
   end
 

data/app/models/user_stats.rb

@@ -11,7 +11,7 @@ class UserStats < ApplicationRecord
   end
 
   def activity(date_range = nil)
-    date_filter = { updated_at: date_range }.compact
+    date_filter = { submitted_at: date_range }.compact
     {
         exercises: {
             solved_count: organization_exercises

data/db/migrate/20210707143002_add_assignment_id_to_message.rb

@@ -0,0 +1,5 @@
+class AddAssignmentIdToMessage < ActiveRecord::Migration[5.1]
+  def change
+    add_reference :messages, :assignment, index: true
+  end
+end

data/db/migrate/20210719145706_add_new_fields_to_notifications.rb

@@ -0,0 +1,8 @@
+class AddNewFieldsToNotifications < ActiveRecord::Migration[5.1]
+  def change
+    add_column :notifications, :subject, :integer
+    add_column :notifications, :custom_title, :text
+    add_column :notifications, :custom_content_plain_text, :text
+    add_column :notifications, :custom_content_html, :text
+  end
+end

data/db/migrate/20210803175124_add_ignored_notifications_to_users.rb

@@ -0,0 +1,5 @@
+class AddIgnoredNotificationsToUsers < ActiveRecord::Migration[5.1]
+  def change
+    add_column :users, :ignored_notifications, :text
+  end
+end

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

@@ -28,12 +28,20 @@ FactoryBot.define do
   end
 
   factory :exercise_base do
+    transient do
+      indexed { false }
+    end
+
     language { guide ? guide.language : create(:language) }
     sequence(:bibliotheca_id) { |n| n }
     sequence(:number) { |n| n }
 
     locale { :en }
     guide
+
+    after(:build) do |exercise, evaluator|
+      exercise.guide = create(:indexed_guide) if evaluator.indexed
+    end
   end
 
   factory :challenge, parent: :exercise_base do
@@ -73,7 +81,7 @@ FactoryBot.define do
   factory :exercise, parent: :problem
 
   factory :indexed_exercise, parent: :exercise do
-    guide { create(:indexed_guide) }
+    indexed { true }
   end
 
   factory :x_equal_5_exercise, parent: :exercise do

data/lib/mumuki/domain/helpers/user.rb

@@ -61,12 +61,6 @@ module Mumuki::Domain::Helpers::User
 
   ## Profile
 
-  def full_name
-    "#{first_name} #{last_name}".strip
-  end
-
-  alias_method :name, :full_name
-
   def profile_completed?
     self.class.profile_fields.map { |it| self[it] }.all? &:present?
   end

data/lib/mumuki/domain/organization/profile.rb

@@ -41,7 +41,7 @@ class Mumuki::Domain::Organization::Profile < Mumukit::Platform::Model
   end
 
   def time_zone
-    @time_zone || 'Buenos Aires'
+    @time_zone
   end
 
 end

data/lib/mumuki/domain/organization/settings.rb

@@ -32,11 +32,11 @@ class Mumuki::Domain::Organization::Settings < Mumukit::Platform::Model
   end
 
   def disabled_from=(disabled_from)
-    @disabled_from = disabled_from&.to_time
+    @disabled_from = disabled_from&.in_time_zone
   end
 
   def in_preparation_until=(in_preparation_until)
-    @in_preparation_until = in_preparation_until&.to_time
+    @in_preparation_until = in_preparation_until&.in_time_zone
   end
 
   def disabled?

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

@@ -2,6 +2,6 @@ class Mumuki::Domain::Submission::PersistentSubmission < Mumuki::Domain::Submiss
   def save_submission!(assignment)
     assignment.running!
     super
-    assignment.persist_submission! self
+    assignment.save_submission! self
   end
 end

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

@@ -6,7 +6,7 @@ class Mumuki::Domain::Submission::Query < Mumuki::Domain::Submission::ConsoleSub
   end
 
   def save_submission!(assignment)
-    assignment.exercise.setup_query_assignment!(assignment)
+    assignment.exercise.save_query_submission!(assignment, self)
     super
   end
 

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

@@ -5,7 +5,6 @@ class Mumuki::Domain::Submission::Question < Mumuki::Domain::Submission::Base
   end
 
   def save_submission!(assignment)
-    assignment.persist_submission! self
   end
 
   def try_evaluate!(*)

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

@@ -8,6 +8,7 @@ class Mumuki::Domain::Submission::Try < Mumuki::Domain::Submission::ConsoleSubmi
   def save_submission!(assignment)
     assignment.query_results = [] if cookie.blank?
     assignment.queries = cookie.insert_last(query)
+    assignment.save_submission! self
     assignment.save!
   end
 

data/lib/mumuki/domain/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Domain
-    VERSION = '9.8.1'
+    VERSION = '9.12.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-domain
 version: !ruby/object:Gem::Version
-  version: 9.8.1
+  version: 9.12.0
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-06 00:00:00.000000000 Z
+date: 2021-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -72,14 +72,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '1.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '1.11'
 - !ruby/object:Gem::Dependency
   name: mumukit-core
   requirement: !ruby/object:Gem::Requirement
@@ -264,6 +264,7 @@ files:
 - app/models/concerns/navigation/parent_navigation.rb
 - app/models/concerns/navigation/siblings_navigation.rb
 - app/models/concerns/navigation/terminal_navigation.rb
+- app/models/concerns/onomastic.rb
 - app/models/concerns/submittable/confirmable.rb
 - app/models/concerns/submittable/queriable.rb
 - app/models/concerns/submittable/questionable.rb
@@ -670,6 +671,9 @@ files:
 - db/migrate/20210330175706_create_exam_registration_user_join_table.rb
 - db/migrate/20210512200453_add_processed_flag_to_exam_registration.rb
 - db/migrate/20210518100153_rename_last_moderator_access.rb
+- db/migrate/20210707143002_add_assignment_id_to_message.rb
+- db/migrate/20210719145706_add_new_fields_to_notifications.rb
+- db/migrate/20210803175124_add_ignored_notifications_to_users.rb
 - lib/mumuki/domain.rb
 - lib/mumuki/domain/area.rb
 - lib/mumuki/domain/engine.rb