mumuki-domain 7.10.0 → 7.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1182f3c80cde421e92bb67ae1a57313bd3a52bbafbb8d0e8cf0a5696dd067ebc
-  data.tar.gz: 6a6ba4ecaf5f0c7d2dd8b10a9900141ae7cccc527c32e6b55484636ac7c1e117
+  metadata.gz: b5cd387b744651b03da4722b106d3c23584a8a610994bf3127fa03cd814e9a64
+  data.tar.gz: 73ecd3f88a806ec887ec68e76554db2468f1b3b0dd9d459197b366bdbd3575fd
 SHA512:
-  metadata.gz: 6e0ab0481ab8cd37417f7d94d18cd73e7f4fb270ff28db3a7ef29f089dde509b34191bf235c1a3945af411c886a3629f3957424a840ff8b6cdbcf71e7fa7d819
-  data.tar.gz: fa2f135124d5d4159e9766e28a71b4fee6ee1a3290c22d37f2230153382e9256e3d15663d6d55af22f7aa9ac65750360b3da10056f0e17f565ca78a8064978a3
+  metadata.gz: d43c7db4b34a5727005cf7f571d7980075fd27fd394942822759bd9e1800626e6880bd1a2bf3085d7332c443d6bdc9602f000e7ee8f3434099c02411a811d8e4
+  data.tar.gz: 24fb849fee899e0636448cedc7c03935931a25902edf5144eb952acb2aabe8d051fb6d6ef9b0c684cde9523b633454e6b6c2c0f24b63c5c54e682f14eef55be2

data/README.md

@@ -1,3 +1,9 @@
+[![Build Status](https://travis-ci.com/mumuki/mumuki-domain.svg?branch=master)](https://travis-ci.com/mumuki/mumuki-domain)
+[![Code Climate](https://codeclimate.com/github/mumuki/mumuki-domain/badges/gpa.svg)](https://codeclimate.com/github/mumuki/mumuki-domain)
+[![Test Coverage](https://codeclimate.com/github/mumuki/mumuki-domain/badges/coverage.svg)](https://codeclimate.com/github/mumuki/mumuki-domain)
+[![Issue Count](https://codeclimate.com/github/mumuki/mumuki-domain/badges/issue_count.svg)](https://codeclimate.com/github/mumuki/mumuki-domain)
+
+
 # Resource Hashes
 
 ## Runner Hashes

data/app/models/application_record.rb

@@ -3,6 +3,16 @@ class ApplicationRecord < ActiveRecord::Base
 
   delegate :whitelist_attributes, to: :class
 
+  def self.teaser_on(*args)
+    args.each do |selector|
+      teaser_selector = "#{selector}_teaser"
+      define_method teaser_selector do
+        send(selector)&.markdown_paragraphs&.first
+      end
+      markdown_on teaser_selector, skip_sanitization: true
+    end
+  end
+
   def self.defaults(&block)
     after_initialize :defaults, if: :new_record?
     define_method :defaults, &block

data/app/models/assignment.rb

@@ -91,7 +91,14 @@ class Assignment < Progress
   end
 
   def notify!
-    Mumukit::Nuntius.notify! 'submissions', to_resource_h unless Organization.silenced?
+    unless Organization.silenced?
+      update_misplaced!(current_notification_contexts.size > 1)
+      Mumukit::Nuntius.notify! 'submissions', to_resource_h
+    end
+  end
+
+  def current_notification_contexts
+    [Organization.current, submitter.current_immersive_context_at(exercise)].uniq
   end
 
   def notify_to_accessible_organizations!
@@ -138,6 +145,10 @@ class Assignment < Progress
     end
   end
 
+  def manual_evaluation_pending!
+    update! submission_status: :manual_evaluation_pending
+  end
+
   def passed!
     update! submission_status: :passed
   end
@@ -169,7 +180,7 @@ class Assignment < Progress
 
   def to_resource_h
     excluded_fields = %i(created_at exercise_id id organization_id parent_id solution submission_id
-                         submission_status submitted_at submitter_id top_submission_status updated_at)
+                         submission_status submitted_at submitter_id top_submission_status updated_at misplaced)
 
     as_json(except: excluded_fields,
               include: {
@@ -238,6 +249,10 @@ class Assignment < Progress
     self.top_submission_status = submission_status unless submission_status.improved_by?(top_submission_status)
   end
 
+  def update_misplaced!(value)
+    update! misplaced: value if value != misplaced?
+  end
+
   private
 
   def update_submissions_count!

data/app/models/avatar.rb

@@ -1,4 +1,6 @@
 class Avatar < ApplicationRecord
+  has_many :users, as: :avatar
+
   include WithTargetAudience
 
   def self.sample_for(user)

data/app/models/concerns/awardee.rb

@@ -0,0 +1,27 @@
+module Awardee
+  def acquired_medals(organization=Organization.current)
+    @acquired_medals ||= medals_for awarded_contents_in(organization)
+  end
+
+  def unacquired_medals(organization=Organization.current)
+    @unacquired_medals ||= medals_for unawarded_contents_in(organization)
+  end
+
+  private
+
+  def medals_for(content)
+    content.map(&:medal)
+  end
+
+  def awarded_contents_in(organization)
+    awardable_contents_in(organization).first
+  end
+
+  def unawarded_contents_in(organization)
+    awardable_contents_in(organization).second
+  end
+
+  def awardable_contents_in(organization)
+    @awardable_contents_in ||= organization.awardable_contents.partition { |c| c.once_completed_for? self, organization }
+  end
+end

data/app/models/concerns/contextualization.rb

@@ -51,7 +51,7 @@ module Contextualization
   end
 
   def single_visible_test_result?
-    test_results.size == 1 && visible_success_output?
+    test_results.single? && visible_success_output?
   end
 
   def first_test_result
@@ -131,8 +131,12 @@ module Contextualization
         .compact
         .merge(
           title: it[:title].affable,
-          result: it[:result].sanitized,
+          result: it[:result],
           status: it[:status])
     end
   end
+
+  def sanitized_affable_test_results
+    affable_test_results.each { |it| it[:result] = it[:result]&.sanitized }
+  end
 end

data/app/models/concerns/with_description.rb

@@ -2,11 +2,8 @@ module WithDescription
   extend ActiveSupport::Concern
 
   included do
-    markdown_on :description, :description_teaser, skip_sanitization: true
+    markdown_on :description, skip_sanitization: true
+    teaser_on :description
     validates_presence_of :description
   end
-
-  def description_teaser
-    description.markdown_paragraphs.first
-  end
 end

data/app/models/concerns/with_medal.rb

@@ -0,0 +1,7 @@
+module WithMedal
+  extend ActiveSupport::Concern
+
+  included do
+    belongs_to :medal, optional: true
+  end
+end

data/app/models/concerns/with_progress.rb

@@ -23,6 +23,10 @@ module WithProgress
     progress_for(user, organization).completed?
   end
 
+  def once_completed_for?(user, organization)
+    progress_for(user, organization).once_completed?
+  end
+
   private
 
   def structural_children_changed?(old_structural_children)

data/app/models/concerns/with_terms_acceptance.rb

@@ -0,0 +1,65 @@
+module WithTermsAcceptance
+  def has_forum_terms_to_accept?
+    !has_accepted_all?(forum_terms)
+  end
+
+  def has_profile_terms_to_accept?
+    !has_accepted_all?(profile_terms)
+  end
+
+  def has_role_terms_to_accept?
+    !has_accepted_all?(role_specific_terms)
+  end
+
+  def role_specific_terms
+    @role_specific_terms ||= Term.role_specific_terms_for(self)
+  end
+
+  def forum_terms
+    @forum_terms ||= Term.forum_related_terms
+  end
+
+  def profile_terms
+    @profile_terms ||= Term.profile_terms_for(self)
+  end
+
+  def accept_profile_terms!
+    accept_terms! profile_terms
+  end
+
+  def accept_forum_terms!
+    accept_terms! forum_terms
+  end
+
+  def has_accepted?(term)
+    term_accepted_at_for(term.scope).try { |it| it > term.updated_at }.present?
+  end
+
+  private
+
+  def unaccepted_terms_in(terms)
+    terms.reject { |term| has_accepted? term}
+  end
+
+  def unaccepted_terms_scopes_in(terms)
+    unaccepted_terms_in(terms).map(&:scope)
+  end
+
+  def has_accepted_all?(terms)
+    unaccepted_terms_in(terms).blank?
+  end
+
+  def term_accepted_at_for(role)
+    send term_acceptance_field_for(role)
+  end
+
+  def term_acceptance_field_for(role)
+    "#{role}_terms_accepted_at"
+  end
+
+  def accept_terms!(terms)
+    update! unaccepted_terms_scopes_in(terms).to_h { |scope| [term_acceptance_field_for(scope), Time.now] }
+  end
+
+end
+

data/app/models/content.rb

@@ -4,10 +4,11 @@ class Content < ApplicationRecord
   include Mumuki::Domain::Syncable
   include WithDescription
   include WithLocale
-  include WithSlug
-  include WithUsages
+  include WithMedal
   include WithName
   include WithProgress
+  include WithSlug
+  include WithUsages
 
   def to_resource_h(*args)
     to_expanded_resource_h(*args).compact

data/app/models/discussion.rb

@@ -4,7 +4,10 @@ class Discussion < ApplicationRecord
   belongs_to :item, polymorphic: true
   has_many :messages, -> { order(:created_at) }, dependent: :destroy
   belongs_to :initiator, class_name: 'User'
+
   belongs_to :last_moderator_access_by, class_name: 'User', optional: true
+  belongs_to :status_updated_by, class_name: 'User', optional: true
+
   belongs_to :exercise, foreign_type: :exercise, foreign_key: 'item_id'
   belongs_to :organization
   has_many :subscriptions
@@ -106,7 +109,11 @@ class Discussion < ApplicationRecord
   end
 
   def update_status!(status, user)
-    update!(status: status) if reachable_status_for?(user, status)
+    if reachable_status_for?(user, status)
+      update! status: status,
+              status_updated_by: user,
+              status_updated_at: Time.now
+    end
   end
 
   def has_messages?

data/app/models/guide.rb

@@ -51,7 +51,10 @@ class Guide < Content
         joins("left join public.assignments assignments
                 on assignments.exercise_id = exercises.id
                 and assignments.submitter_id = #{user.id}
-                and assignments.submission_status = #{Mumuki::Domain::Status::Submission::Passed.to_i}").
+                and assignments.submission_status in (
+                  #{Mumuki::Domain::Status::Submission::Passed.to_i},
+                  #{Mumuki::Domain::Status::Submission::ManualEvaluationPending.to_i}
+                )").
         where('assignments.id is null')
   end
 

data/app/models/indicator.rb

@@ -37,6 +37,7 @@ class Indicator < Progress
   def clean!
     self.dirty_by_submission = false
     self.dirty_by_content_change = false
+    self.once_completed ||= all_children_passed?
   end
 
   def refresh_children_count!
@@ -54,7 +55,15 @@ class Indicator < Progress
 
   def completed?
     rebuild!
-    children_passed_count == children_count
+    all_children_passed?
+  end
+
+  def once_completed?
+    self.once_completed || completed?
+  end
+
+  def self.delete_all_for!(content, organization)
+    where(content: content, organization: organization).delete_all
   end
 
   private
@@ -74,4 +83,8 @@ class Indicator < Progress
   def parent_content
     content.usage_in_organization(organization).structural_parent
   end
+
+  def all_children_passed?
+    children_passed_count == children_count
+  end
 end

data/app/models/medal.rb

@@ -0,0 +1,3 @@
+class Medal < ApplicationRecord
+  has_many :users, as: :avatar
+end

data/app/models/organization.rb

@@ -11,10 +11,12 @@ class Organization < ApplicationRecord
   serialize :settings, Mumuki::Domain::Organization::Settings
   serialize :theme, Mumuki::Domain::Organization::Theme
 
-  markdown_on :description
+  markdown_on :description, :display_description, :page_description
+  teaser_on :display_description
 
   validate :ensure_consistent_public_login
   validate :ensure_valid_activity_range
+  validate :ensure_not_immersive_and_immersible
 
   belongs_to :book
   has_many :usages
@@ -97,11 +99,13 @@ class Organization < ApplicationRecord
   end
 
   def title_suffix
-    central? ? '' : " - #{book.name}"
+    warn "Don't use title_suffix. Use page_name instead"
+    " - #{page_name}"
   end
 
   def site_name
-    central? ? 'mumuki' : name
+    warn "Don't use site_name. Use display_name instead"
+    name
   end
 
   # Tells if the given user can
@@ -130,12 +134,55 @@ class Organization < ApplicationRecord
     update! progressive_display_lookahead: lookahead
   end
 
+  def progressive_display_lookahead=(lookahead)
+    self[:progressive_display_lookahead] = lookahead.to_i.positive? ? lookahead : nil
+  end
+
+  # ==============
+  # Display fields
+  # ==============
+
   def display_name
-    name.gsub(/\W/, ' ').titleize
+    self[:display_name].presence || name.try { |it| it.gsub(/\W/, ' ').titleize }
+  end
+
+  def display_description
+    self[:display_description].presence || I18n.t('defaults.organization.display_description', name: name)
+  end
+
+  # ===========
+  # Page fields
+  # ===========
+
+  # Since an organization has a single book, both concepts may be merged
+  # when describing a site. In such contexts, wins_page?
+  # control whether the book or the organization header fields are
+  # more important
+
+  def page_name
+    wins_page? ? display_name : book.name
+  end
+
+  def page_description
+    wins_page? ? display_description : book.description
+  end
+
+  def all_contents
+    Usage.where(organization: self)
+         .group_by(&:item_type)
+         .flat_map { |item_type, item| item_type.constantize.where(id: item.map(&:item_id)) }
+  end
+
+  def awardable_contents
+    gamification_enabled? ? all_contents.select(&:medal_id) : []
   end
 
   private
 
+  def ensure_not_immersive_and_immersible
+    errors.add(:immersible, :cannot_be_immersive) if immersible? && immersive?
+  end
+
   def ensure_consistent_public_login
     errors.add(:base, :consistent_public_login) if settings.customized_login_methods? && public?
   end

data/app/models/term.rb

@@ -0,0 +1,40 @@
+class Term < ApplicationRecord
+  attribute :locale, :string, default: 'es'
+  markdown_on :content
+
+  GENERAL = %w(legal privacy student)
+  ROLE_SPECIFIC = %w(headmaster janitor teacher moderator)
+  FORUM_RELATED = %w(forum)
+
+  validates :locale, uniqueness: { scope: :scope }
+  validates :content, :scope, :header, presence: true
+
+  def self.terms_for(scope, locale)
+    where(scope: scope, locale: locale)
+  end
+
+  def self.profile_terms_for(user, locale = I18n.locale)
+    general_terms(locale) + role_specific_terms_for(user, locale)
+  end
+
+  def self.role_specific_terms_for(user, locale = I18n.locale)
+    terms_for(current_role_terms_for(user), locale)
+  end
+
+  def self.general_terms(locale = I18n.locale)
+    terms_for(GENERAL, locale)
+  end
+
+  def self.forum_related_terms(locale = I18n.locale)
+    terms_for(FORUM_RELATED, locale)
+  end
+
+  def self.current_role_terms_for(user)
+    return [] unless user.present?
+    (user.any_granted_roles & ROLE_SPECIFIC).to_a
+  end
+
+  def accepted_by?(user)
+    user.term_accepted_at_for(scope).try { |it| it > updated_at }.present?
+  end
+end

data/app/models/topic.rb

@@ -48,7 +48,10 @@ class Topic < Content
       joins("left join public.assignments assignments
                 on assignments.exercise_id = exercises.id
                 and assignments.submitter_id = #{user.id}
-                and assignments.submission_status = #{Mumuki::Domain::Status::Submission::Passed.to_i}").
+                and assignments.submission_status in (
+                  #{Mumuki::Domain::Status::Submission::Passed.to_i},
+                  #{Mumuki::Domain::Status::Submission::ManualEvaluationPending.to_i}
+                )").
       where('assignments.id is null').
       group('public.guides.id', 'lessons.number').map(&:lesson)
   end

data/app/models/usage.rb

@@ -8,6 +8,7 @@ class Usage < ApplicationRecord
 
   before_save :set_slug
   before_destroy :destroy_children_usages!
+  before_destroy :delete_associated_indicators!
 
   def self.destroy_all_where(query)
     where(query).destroy_all
@@ -27,6 +28,10 @@ class Usage < ApplicationRecord
 
   private
 
+  def delete_associated_indicators!
+    Indicator.delete_all_for!(item, organization)
+  end
+
   def set_slug
     self.slug = item.slug
   end

data/app/models/user.rb

@@ -4,13 +4,15 @@ class User < ApplicationRecord
           WithUserNavigation,
           WithReminders,
           WithDiscussionCreation,
+          Awardee,
           Disabling,
+          WithTermsAcceptance,
           Mumuki::Domain::Helpers::User
 
   serialize :permissions, Mumukit::Auth::Permissions
 
-  has_many :assignments, foreign_key: :submitter_id
 
+  has_many :assignments, foreign_key: :submitter_id
   has_many :messages, -> { order(created_at: :desc) }, through: :assignments
 
   has_many :submitted_exercises, through: :assignments, class_name: 'Exercise', source: :exercise
@@ -31,11 +33,12 @@ class User < ApplicationRecord
   has_many :exams, through: :exam_authorizations
 
   enum gender: %i(female male other unspecified)
-  belongs_to :avatar, optional: true
+  belongs_to :avatar, polymorphic: true, optional: true
 
   before_validation :set_uid!
   validates :uid, presence: true
 
+  validates :terms_of_service, acceptance: true
   after_save :welcome_to_new_organizations!, if: :gained_access_to_new_orga?
   after_initialize :init
   PLACEHOLDER_IMAGE_URL = 'user_shape.png'.freeze
@@ -236,6 +239,18 @@ class User < ApplicationRecord
     end
   end
 
+  def current_organic_context
+    Organization.current? ?  Organization.current : main_organization
+  end
+
+  def current_immersive_context_at(exercise)
+    if Organization.current?
+      immersive_organization_at(exercise) || Organization.current
+    else
+      main_organization
+    end
+  end
+
   private
 
   def welcome_to_new_organizations!
@@ -289,12 +304,4 @@ class User < ApplicationRecord
   def self.buried_profile
     (@buried_profile || {}).slice(:first_name, :last_name, :email)
   end
-
-  def current_organic_context
-    if Organization.current?
-      Organization.current
-    else
-      main_organization
-    end
-  end
 end

data/db/migrate/20200828152812_create_medals.rb

@@ -0,0 +1,8 @@
+class CreateMedals < ActiveRecord::Migration[5.1]
+  def change
+    create_table :medals do |t|
+      t.string :image_url
+      t.string :description
+    end
+  end
+end

data/db/migrate/20200828162829_add_medal_to_content.rb

@@ -0,0 +1,7 @@
+class AddMedalToContent < ActiveRecord::Migration[5.1]
+  def change
+    add_reference :books, :medal, index: false
+    add_reference :guides, :medal, index: false
+    add_reference :topics, :medal, index: false
+  end
+end

data/db/migrate/20200915123020_add_polymorphic_avatars_to_users.rb

@@ -0,0 +1,6 @@
+class AddPolymorphicAvatarsToUsers < ActiveRecord::Migration[5.1]
+  def change
+    add_column :users, :avatar_type, :string, default: 'Avatar'
+    add_index :users, [:avatar_type, :avatar_id]
+  end
+end

data/db/migrate/20200915131621_add_once_completed_to_indicators.rb

@@ -0,0 +1,5 @@
+class AddOnceCompletedToIndicators < ActiveRecord::Migration[5.1]
+  def change
+    add_column :indicators, :once_completed, :boolean
+  end
+end

data/db/migrate/20201007143455_add_terms_for_user.rb

@@ -0,0 +1,14 @@
+class AddTermsForUser < ActiveRecord::Migration[5.1]
+  def change
+    add_column :users, :headmaster_terms_accepted_at, :datetime
+    add_column :users, :janitor_terms_accepted_at, :datetime
+    add_column :users, :moderator_terms_accepted_at, :datetime
+    add_column :users, :student_terms_accepted_at, :datetime
+    add_column :users, :teacher_terms_accepted_at, :datetime
+
+    add_column :users, :privacy_terms_accepted_at, :datetime
+    add_column :users, :legal_terms_accepted_at, :datetime
+
+    add_column :users, :forum_terms_accepted_at, :datetime
+  end
+end

data/db/migrate/20201009193949_add_status_updated_fields_to_discussion.rb

@@ -0,0 +1,6 @@
+class AddStatusUpdatedFieldsToDiscussion < ActiveRecord::Migration[5.1]
+  def change
+    add_reference :discussions, :status_updated_by, index: true
+    add_column :discussions, :status_updated_at, :datetime
+  end
+end

data/db/migrate/20201019191036_add_misplaced_flag_to_submission.rb

@@ -0,0 +1,5 @@
+class AddMisplacedFlagToSubmission < ActiveRecord::Migration[5.1]
+  def change
+    add_column :assignments, :misplaced, :boolean
+  end
+end

data/db/migrate/20201026222942_add_display_name_and_description_to_organizations.rb

@@ -0,0 +1,6 @@
+class AddDisplayNameAndDescriptionToOrganizations < ActiveRecord::Migration[5.1]
+  def change
+    add_column :organizations, :display_name, :text
+    add_column :organizations, :display_description, :text
+  end
+end

data/db/migrate/20201026225312_add_organization_wins_page_flag.rb

@@ -0,0 +1,5 @@
+class AddOrganizationWinsPageFlag < ActiveRecord::Migration[5.1]
+  def change
+    add_column :organizations, :wins_page, :boolean
+  end
+end

data/db/migrate/20201027134205_add_immersible_to_organization.rb

@@ -0,0 +1,5 @@
+class AddImmersibleToOrganization < ActiveRecord::Migration[5.1]
+  def change
+    add_column :organizations, :immersible, :boolean
+  end
+end

data/db/migrate/20201027152806_create_terms.rb

@@ -0,0 +1,12 @@
+class CreateTerms < ActiveRecord::Migration[5.1]
+  def change
+    create_table :terms do |t|
+      t.string :locale
+      t.string :scope
+      t.text :content
+      t.text :header
+
+      t.timestamps
+    end
+  end
+end

data/lib/mumuki/domain/extensions/array.rb

@@ -2,6 +2,19 @@ class Array
   def insert_last(element)
     self + [element]
   end
+
+  def single
+    first if single?
+  end
+
+  def single!
+    raise 'There is more than one element' unless single?
+    first
+  end
+
+  def single?
+    size == 1
+  end
 end
 
 class NilClass

data/lib/mumuki/domain/factories.rb

@@ -12,8 +12,10 @@ require_relative './factories/guide_factory'
 require_relative './factories/invitation_factory'
 require_relative './factories/lesson_factory'
 require_relative './factories/login_settings_factory'
+require_relative './factories/medal_factory'
 require_relative './factories/message_factory'
 require_relative './factories/organization_factory'
+require_relative './factories/term_factory'
 require_relative './factories/topic_factory'
-require_relative './factories/user_factory'
 require_relative './factories/usage_factory'
+require_relative './factories/user_factory'

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

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :medal do
+    image_url { Faker::Internet.url }
+  end
+end
+

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

@@ -25,6 +25,7 @@ FactoryBot.define do
 
   factory :test_organization, parent: :public_organization do
     name { 'test' }
+    immersible { true }
     book { create(:book, name: 'test', slug: 'mumuki/mumuki-the-book') }
   end
 

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

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :term do
+    content { Faker::Lorem.sentence(word_count: 2000) }
+    header { Faker::Lorem.sentence(word_count: 5) }
+  end
+end
+

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

@@ -28,6 +28,10 @@ module Mumuki::Domain::Helpers::Organization
     name == 'base'
   end
 
+  def immersed_in?(other)
+    immersible? && other.immersive? && target_audience == other.target_audience
+  end
+
   def switch!
     Mumukit::Platform::Organization.switch! self
   end

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

@@ -15,6 +15,7 @@ module Mumuki::Domain::Helpers::User
            :protect_permissions_assignment!,
            :student_granted_organizations,
            :any_granted_organizations,
+           :any_granted_roles,
            to: :permissions
 
   def platform_class_name
@@ -87,8 +88,24 @@ module Mumuki::Domain::Helpers::User
     student_granted_organizations.first || any_granted_organizations.first
   end
 
+  # Deprecated: use `immersive_organization_at` which
+  # properly looks for a single immersive organization taking
+  # current organization and path into account
   def has_immersive_main_organization?
-    !!main_organization.try(&:immersive?)
+    main_organization.try(&:immersive?).present?
+  end
+
+  def immersive_organization_at(path_item, current = Organization.current)
+    immersive_organizations_at(path_item, current).single
+  end
+
+  def immersive_organizations_at(path_item, current = Organization.current)
+    return [] unless current.immersible?
+
+    usage_filter = path_item ? lambda { |it| path_item.used_in?(it) } : lambda { |_| true }
+    student_granted_organizations
+      .select { |it| current.immersed_in?(it) }
+      .select(&usage_filter)
   end
 
   ## API Exposure

data/lib/mumuki/domain/incognito.rb

@@ -48,6 +48,33 @@ module Mumuki::Domain
       false
     end
 
+    def current_immersive_context_at(_)
+      nil
+    end
+
+    def immersive_organizations_at(_)
+      []
+    end
+
+    def any_granted_roles
+      []
+    end
+
+    # ========
+    # Terms
+    # ========
+
+    # It avoids role terms acceptance redirections
+    def has_role_terms_to_accept?
+      false
+    end
+
+    # It makes terms UI to be shown as if no terms were accepted
+    # It does not force any term to be accepted though
+    def has_accepted?(term)
+      false
+    end
+
     # ========
     # Visiting
     # ========

data/lib/mumuki/domain/locales/activerecord/en.yml

@@ -28,3 +28,5 @@ en:
             base:
               consistent_public_login: 'A public organization can not restrict login methods'
               invalid_activity_range: 'The organization activity range are not valid'
+            immersible:
+              cannot_be_immersive: 'An organization cannot be immersible and immersive at the same time'

data/lib/mumuki/domain/locales/activerecord/es.yml

@@ -45,6 +45,8 @@ es:
             base:
               consistent_public_login: 'Una organización pública no puede restringir los métodos de login'
               invalid_activity_range: 'La fecha de deshabilitación no puede ser anterior a la de inicio del recorrido'
+            immersible:
+              cannot_be_immersive: 'Una organización no puede ser inmersible e inmersiva a la vez'
     models:
       exercise:
         one: Ejercicio

data/lib/mumuki/domain/locales/defaults/en.yml

@@ -0,0 +1,4 @@
+en:
+  defaults:
+    organization:
+      display_description: "In this site you will find programming contents about %{name}"

data/lib/mumuki/domain/locales/defaults/es-CL.yml

@@ -0,0 +1,4 @@
+es-CL:
+  defaults:
+    organization:
+      display_description: "En este sitio encontrarás el contenido sobre programación de %{name}"

data/lib/mumuki/domain/locales/defaults/es.yml

@@ -0,0 +1,4 @@
+es:
+  defaults:
+    organization:
+      display_description: "En este sitio encontrarás el contenido sobre programación de %{name}"

data/lib/mumuki/domain/locales/defaults/pt.yml

@@ -0,0 +1,4 @@
+pt:
+  defaults:
+    organization:
+      display_description: "Neste site você encontrará conteúdos de programação sobre %{name}"

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

@@ -27,8 +27,10 @@ class Mumuki::Domain::Submission::Base
     results
   end
 
-  def dry_run!(assignment, evaluation)
-    evaluation.evaluate! assignment, self
+  # By default evaluation strategy is ignored.
+  # Subclasses may override this behaviour
+  def dry_run!(assignment, _evaluation)
+    evaluate! assignment
   end
 
   def with_client_result(result)
@@ -51,7 +53,11 @@ class Mumuki::Domain::Submission::Base
 
   def save_submission!(assignment)
     assignment.content = content
-    assignment.organization = Organization.current
+    if assignment.organization != Organization.current
+      assignment.dirty_parent_by_submission! if assignment.organization
+      assignment.organization = Organization.current
+      assignment.parent_id = nil
+    end
     assignment.save!
   end
 

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

@@ -1,6 +1,10 @@
 class Mumuki::Domain::Submission::Solution < Mumuki::Domain::Submission::PersistentSubmission
   attr_accessor :content
 
+  def dry_run!(assignment, evaluation)
+    evaluation.evaluate! assignment, self
+  end
+
   def try_evaluate!(assignment)
     assignment
       .run_tests!({client_result: client_result}.compact.merge(content: content))

data/lib/mumuki/domain/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Domain
-    VERSION = '7.10.0'
+    VERSION = '7.12.1'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-domain
 version: !ruby/object:Gem::Version
-  version: 7.10.0
+  version: 7.12.1
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-28 00:00:00.000000000 Z
+date: 2020-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.8'
+        version: '7.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.8'
+        version: '7.9'
 - !ruby/object:Gem::Dependency
   name: mumukit-assistant
   requirement: !ruby/object:Gem::Requirement
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.13'
+        version: '1.18'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.13'
+        version: '1.18'
 - !ruby/object:Gem::Dependency
   name: mumukit-directives
   requirement: !ruby/object:Gem::Requirement
@@ -252,6 +252,7 @@ files:
 - app/models/chapter.rb
 - app/models/complement.rb
 - app/models/concerns/assistable.rb
+- app/models/concerns/awardee.rb
 - app/models/concerns/container.rb
 - app/models/concerns/contextualization.rb
 - app/models/concerns/disabling.rb
@@ -281,6 +282,7 @@ files:
 - app/models/concerns/with_language.rb
 - app/models/concerns/with_layout.rb
 - app/models/concerns/with_locale.rb
+- app/models/concerns/with_medal.rb
 - app/models/concerns/with_messages.rb
 - app/models/concerns/with_name.rb
 - app/models/concerns/with_number.rb
@@ -294,6 +296,7 @@ files:
 - app/models/concerns/with_scoped_queries/sort.rb
 - app/models/concerns/with_slug.rb
 - app/models/concerns/with_target_audience.rb
+- app/models/concerns/with_terms_acceptance.rb
 - app/models/concerns/with_usages.rb
 - app/models/concerns/with_user_navigation.rb
 - app/models/content.rb
@@ -315,11 +318,13 @@ files:
 - app/models/invitation.rb
 - app/models/language.rb
 - app/models/lesson.rb
+- app/models/medal.rb
 - app/models/message.rb
 - app/models/organization.rb
 - app/models/progress.rb
 - app/models/stats.rb
 - app/models/subscription.rb
+- app/models/term.rb
 - app/models/topic.rb
 - app/models/upvote.rb
 - app/models/usage.rb
@@ -620,6 +625,17 @@ files:
 - db/migrate/20200730221001_add_trusted_for_forum_to_user.rb
 - db/migrate/20200731081757_add_last_moderator_access_fields_to_discussion.rb
 - db/migrate/20200804191643_add_incognito_mode_enabled_to_organization.rb
+- db/migrate/20200828152812_create_medals.rb
+- db/migrate/20200828162829_add_medal_to_content.rb
+- db/migrate/20200915123020_add_polymorphic_avatars_to_users.rb
+- db/migrate/20200915131621_add_once_completed_to_indicators.rb
+- db/migrate/20201007143455_add_terms_for_user.rb
+- db/migrate/20201009193949_add_status_updated_fields_to_discussion.rb
+- db/migrate/20201019191036_add_misplaced_flag_to_submission.rb
+- db/migrate/20201026222942_add_display_name_and_description_to_organizations.rb
+- db/migrate/20201026225312_add_organization_wins_page_flag.rb
+- db/migrate/20201027134205_add_immersible_to_organization.rb
+- db/migrate/20201027152806_create_terms.rb
 - lib/mumuki/domain.rb
 - lib/mumuki/domain/area.rb
 - lib/mumuki/domain/engine.rb
@@ -656,8 +672,10 @@ files:
 - lib/mumuki/domain/factories/invitation_factory.rb
 - lib/mumuki/domain/factories/lesson_factory.rb
 - lib/mumuki/domain/factories/login_settings_factory.rb
+- lib/mumuki/domain/factories/medal_factory.rb
 - lib/mumuki/domain/factories/message_factory.rb
 - lib/mumuki/domain/factories/organization_factory.rb
+- lib/mumuki/domain/factories/term_factory.rb
 - lib/mumuki/domain/factories/topic_factory.rb
 - lib/mumuki/domain/factories/usage_factory.rb
 - lib/mumuki/domain/factories/user_factory.rb
@@ -675,6 +693,10 @@ files:
 - lib/mumuki/domain/locales/console_submission/es-CL.yml
 - lib/mumuki/domain/locales/console_submission/es.yml
 - lib/mumuki/domain/locales/console_submission/pt.yml
+- lib/mumuki/domain/locales/defaults/en.yml
+- lib/mumuki/domain/locales/defaults/es-CL.yml
+- lib/mumuki/domain/locales/defaults/es.yml
+- lib/mumuki/domain/locales/defaults/pt.yml
 - lib/mumuki/domain/organization.rb
 - lib/mumuki/domain/organization/profile.rb
 - lib/mumuki/domain/organization/settings.rb