mumuki-domain 9.12.0 → 9.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d20f0f5d9cfe938c4a33f561f9e4df1e5e84ab18296fda644038b87166d631fc
-  data.tar.gz: bf7adb112a37b84a8fb4f8fa7dc74db1c5df29dc0d8763005256e98e381b7951
+  metadata.gz: b312037c43def7204d8591702f2002340539b6f66df217e4bf8b35e5a0392785
+  data.tar.gz: f417d2c02346fcd6e2460c246d38f0a9c4eb1ac8a5162bf72bbe00f983326bec
 SHA512:
-  metadata.gz: fbcc0dcdd8fd5134d463f61f18da8dfc872b7142f221f16ea3d99dac828bdbcf628d47b924559ac40e3224597955ad575720dc318b385297509cc3364b5c9897
-  data.tar.gz: 4a2a358f2b888268d4193b0b9b50873702ac293430c26e5d64576f7c02b185fc4263cbc0b30c02758c611240e9646bb917277ed5ec37951521762db028f0fb52
+  metadata.gz: 5786a0b215fee7332099994a47e13b24d2f8a66647ede57aaa67bdcdd2e77433e471d8dc5f7ef48369bea06fb2636f900ed365d24b3b1549f0cef3c48d316f25
+  data.tar.gz: 282e425ad11463f7c0945b323327f17addc7997e60f191a034b07317907e0156918fcd667786f4905b89682abe9bb2090148fab392aebd9f99c6c0ab1320be2b

data/app/models/application_record.rb

@@ -1,6 +1,8 @@
 class ApplicationRecord < ActiveRecord::Base
   self.abstract_class = true
 
+  include WithPgLock
+
   delegate :whitelist_attributes, to: :class
 
   def self.teaser_on(*args)

data/app/models/assignment.rb

@@ -123,7 +123,7 @@ class Assignment < Progress
   end
 
   def content=(content)
-    if exercise.solvable?
+    if content.present? && exercise.solvable?
       self.solution = exercise.single_choice? ? exercise.choice_index_for(content) : content
     end
   end

data/app/models/concerns/organization/status/base.rb

@@ -0,0 +1,26 @@
+class Organization::Status::Base
+
+  attr_reader :organization
+
+  implements :teacher_access_mode, :student_access_mode, :ex_student_access_mode, :outsider_access_mode, :validate!
+
+  def initialize(organization)
+    @organization = organization
+  end
+
+  def access_mode(user)
+    if user&.teacher_of? organization
+      teacher_access_mode(user)
+    elsif user&.student_of? organization
+      student_access_mode(user)
+    elsif user&.ex_student_of? organization
+      ex_student_access_mode(user)
+    else
+      outsider_access_mode(user)
+    end
+  end
+
+  def validate!(user = nil)
+    validate_enabled! unless user
+  end
+end

data/app/models/concerns/organization/status/disabled.rb

@@ -0,0 +1,27 @@
+class Organization::Status::Disabled < Organization::Status::Base
+
+  def teacher_access_mode(user)
+    OrganizationAccessMode::Full.new user, organization
+  end
+
+  def student_access_mode(user)
+    OrganizationAccessMode::ReadOnly.new user, organization, :faqs, :profile, :exercises, :discussions
+  end
+
+  def ex_student_access_mode(user)
+    OrganizationAccessMode::ReadOnly.new user, organization, :faqs, :profile
+  end
+
+  def outsider_access_mode(user)
+    if organization.public?
+      OrganizationAccessMode::Gone.new user, organization
+    else
+      OrganizationAccessMode::Forbidden.new user, organization
+    end
+  end
+
+  def validate_enabled!
+    raise Mumuki::Domain::DisabledOrganizationError
+  end
+end
+

data/app/models/concerns/organization/status/enabled.rb

@@ -0,0 +1,25 @@
+class Organization::Status::Enabled < Organization::Status::Base
+
+  def teacher_access_mode(user)
+    OrganizationAccessMode::Full.new user, organization
+  end
+
+  def student_access_mode(user)
+    OrganizationAccessMode::Full.new user, organization
+  end
+
+  def ex_student_access_mode(user)
+    OrganizationAccessMode::ReadOnly.new user, organization, :faqs, :profile, :discussions, exercises: :submitted
+  end
+
+  def outsider_access_mode(user)
+    if organization.public?
+      OrganizationAccessMode::Full.new user, organization
+    else
+      OrganizationAccessMode::Forbidden.new user, organization
+    end
+  end
+
+  def validate_enabled!
+  end
+end

data/app/models/concerns/organization/status/in_preparation.rb

@@ -0,0 +1,26 @@
+class Organization::Status::InPreparation < Organization::Status::Base
+
+  def teacher_access_mode(user)
+    OrganizationAccessMode::Full.new user, organization
+  end
+
+  def student_access_mode(user)
+    OrganizationAccessMode::ComingSoon.new user, organization
+  end
+
+  def ex_student_access_mode(user)
+    OrganizationAccessMode::Forbidden.new user, organization
+  end
+
+  def outsider_access_mode(user)
+    if organization.public?
+      OrganizationAccessMode::ComingSoon.new user, organization
+    else
+      OrganizationAccessMode::Forbidden.new user, organization
+    end
+  end
+
+  def validate_enabled!
+    raise Mumuki::Domain::UnpreparedOrganizationError
+  end
+end

data/app/models/concerns/with_assignments.rb

@@ -30,4 +30,10 @@ module WithAssignments
   def assignment_for(user, organization=Organization.current)
     find_assignment_for(user, organization) || user.build_assignment(self, organization)
   end
+
+  def has_progress_for?(user, organization)
+    user.present? && find_assignment_for(user, organization).present?
+  end
+
+
 end

data/app/models/concerns/with_organization_status.rb

@@ -0,0 +1,30 @@
+module WithOrganizationStatus
+
+  delegate :validate_enabled!, :access_mode, to: :status
+
+  def status
+    @status ||= _status
+  end
+
+  def validate_active!
+    status.validate!
+  end
+
+  def validate_active_for!(user)
+    status.validate!(user)
+    access_mode(user).validate_active!
+  end
+
+  private
+
+  def _status
+    if disabled?
+      Organization::Status::Disabled.new self
+    elsif in_preparation?
+      Organization::Status::InPreparation.new self
+    else
+      Organization::Status::Enabled.new self
+    end
+  end
+
+end

data/app/models/concerns/with_progress.rb

@@ -7,6 +7,10 @@ module WithProgress
     progress_for(user, organization).completion_percentage
   end
 
+  def has_progress_for?(user, organization)
+    progress_for(user, organization).persisted?
+  end
+
   def dirty_progresses!
     Indicator.dirty_by_content_change! self
   end

data/app/models/concerns/with_reminders.rb

@@ -1,6 +1,5 @@
 module WithReminders
   extend ActiveSupport::Concern
-  include WithPgLock
 
   def build_reminder
     mailer = UserMailer.new

data/app/models/exam_registration.rb

@@ -1,5 +1,4 @@
 class ExamRegistration < ApplicationRecord
-  include WithPgLock
   include WithTimedEnablement
   include TerminalNavigation
 
@@ -77,6 +76,14 @@ class ExamRegistration < ApplicationRecord
     authorization_criterion.meets_criterion?(user, organization)
   end
 
+  def multiple_options?
+    exams.count > 1
+  end
+
+  def ended?
+    end_time.past?
+  end
+
   private
 
   def notify_registree!(registree)

data/app/models/organization.rb

@@ -6,6 +6,7 @@ class Organization < ApplicationRecord
   include Mumukit::Login::OrganizationHelpers
 
   include WithTargetAudience
+  include WithOrganizationStatus
 
   serialize :profile, Mumuki::Domain::Organization::Profile
   serialize :settings, Mumuki::Domain::Organization::Settings

data/app/models/organization_access_mode/base.rb

@@ -0,0 +1,53 @@
+class OrganizationAccessMode::Base
+  attr_reader :user, :organization
+
+  def initialize(user, organization)
+    @user = user
+    @organization = organization
+  end
+
+  def validate_active!
+  end
+
+  def faqs_here?
+    organization.faqs.present?
+  end
+
+  def submit_solutions_here?
+    false
+  end
+
+  def resolve_discussions_here?
+    false
+  end
+
+  def discuss_here?
+    organization.forum_enabled? && user.discusser_of?(organization) &&
+      user.trusted_as_discusser_in?(organization) && !user.banned_from_forum?
+  end
+
+  def show_discussion_element?
+    false
+  end
+
+  def show_content_element?
+    false
+  end
+
+  def restore_indicators?(_content)
+    false
+  end
+
+  def read_only?
+    false
+  end
+
+  def validate_discuss_here!(_discussion)
+    raise Mumuki::Domain::ForbiddenError
+  end
+
+  def validate_content_here!(content)
+    raise Mumuki::Domain::ForbiddenError unless show_content?(content)
+  end
+end
+

data/app/models/organization_access_mode/coming_soon.rb

@@ -0,0 +1,5 @@
+class OrganizationAccessMode::ComingSoon < OrganizationAccessMode::Forbidden
+  def validate_active!
+    raise Mumuki::Domain::UnpreparedOrganizationError
+  end
+end

data/app/models/organization_access_mode/forbidden.rb

@@ -0,0 +1,22 @@
+class OrganizationAccessMode::Forbidden < OrganizationAccessMode::Base
+  def validate_active!
+    raise Mumuki::Domain::ForbiddenError if organization.private? && user.present?
+  end
+
+  def faqs_here?
+    false
+  end
+
+  def profile_here?
+    false
+  end
+
+  def discuss_here?
+    false
+  end
+
+  def show_content?(_content)
+    false
+  end
+
+end

data/app/models/organization_access_mode/full.rb

@@ -0,0 +1,28 @@
+class OrganizationAccessMode::Full < OrganizationAccessMode::Base
+  def profile_here?
+    true
+  end
+
+  def submit_solutions_here?
+    true
+  end
+
+  def show_discussion_element?
+    true
+  end
+
+  def resolve_discussions_here?
+    discuss_here?
+  end
+
+  def validate_discuss_here!(_discussion)
+  end
+
+  def show_content?(_content)
+    true
+  end
+
+  def show_content_element?
+    true
+  end
+end

data/app/models/organization_access_mode/gone.rb

@@ -0,0 +1,5 @@
+class OrganizationAccessMode::Gone < OrganizationAccessMode::Forbidden
+  def validate_active!
+    raise Mumuki::Domain::DisabledOrganizationError
+  end
+end

data/app/models/organization_access_mode/read_only.rb

@@ -0,0 +1,41 @@
+class OrganizationAccessMode::ReadOnly < OrganizationAccessMode::Base
+  def initialize(user, organization, *global_scopes, **specific_scopes)
+    super user, organization
+    @scopes = global_scopes.map { |scope| [scope, :all] }.to_h.merge specific_scopes
+  end
+
+  def faqs_here?
+    has_scope(:faqs) && super
+  end
+
+  def profile_here?
+    has_scope(:profile)
+  end
+
+  def discuss_here?
+    has_scope(:discussions) && super
+  end
+
+  def validate_discuss_here!(discussion)
+    super(discussion) unless discussion&.initiator == user
+  end
+
+  def show_content?(content)
+    has_scope(:exercises) ||
+      (has_scope(:exercises, :submitted) && content.has_progress_for?(user, organization))
+  end
+
+  def restore_indicators?(book)
+    !book.has_progress_for?(user, organization) && user.has_assignments_in_organization?(organization)
+  end
+
+  def read_only?
+    true
+  end
+
+  private
+
+  def has_scope(key, value = :all)
+    @scopes[key] == value
+  end
+end

data/app/models/organization_access_mode.rb

@@ -0,0 +1,2 @@
+module OrganizationAccessMode
+end

data/app/models/user.rb

@@ -54,7 +54,7 @@ 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
+  organic_on :notifications, :assignments
 
   def last_lesson
     last_guide.try(:lesson)
@@ -64,10 +64,6 @@ 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
@@ -111,6 +107,16 @@ class User < ApplicationRecord
     end
   end
 
+  def restore_organization_progress!(organization)
+    assignments_in_organization(organization).each do |assignment|
+      assignment.tap(&:parent).save!
+    end
+  end
+
+  def has_assignments_in_organization?(organization)
+    assignments_in_organization(organization).exists?
+  end
+
   def accept_invitation!(invitation)
     make_student_of! invitation.course_slug
   end
@@ -201,7 +207,7 @@ class User < ApplicationRecord
   # This is true only when this organization has the forum enabled and the user
   # has the discusser pseudo-permission and the discusser is trusted
   def can_discuss_in?(organization)
-    organization.forum_enabled? && discusser_of?(organization) && trusted_as_discusser_in?(organization) && !banned_from_forum?
+    organization.access_mode(self).discuss_here?
   end
 
   def trusted_as_discusser_in?(organization)
@@ -283,6 +289,10 @@ class User < ApplicationRecord
     }
   end
 
+  def solved_any_exercises?(organization = Organization.current)
+    Assignment.exists?(organization: organization, submitter: self)
+  end
+
   def save_and_notify!
     save!
     notify_permissions_changed!

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

@@ -52,11 +52,6 @@ module Mumuki::Domain::Helpers::Organization
     Mumukit::Platform.application.organic_domain(name)
   end
 
-  def validate_active!
-    raise Mumuki::Domain::DisabledOrganizationError if disabled?
-    raise Mumuki::Domain::UnpreparedOrganizationError if in_preparation?
-  end
-
   ## API Exposure
 
   def to_param

data/lib/mumuki/domain/incognito.rb

@@ -16,6 +16,22 @@ module Mumuki::Domain
       false
     end
 
+    def ex_student_of?(*)
+      false
+    end
+
+    def ex_student_here?
+      false
+    end
+
+    def student_of?(*)
+      false
+    end
+
+    def student_here?
+      false
+    end
+
     def teacher_here?
       false
     end

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

@@ -28,7 +28,7 @@ class Mumuki::Domain::Organization::Settings < Mumukit::Platform::Model
   end
 
   def forum_discussions_minimal_role
-    (@forum_discussions_minimal_role || 'student').to_sym
+    (@forum_discussions_minimal_role || 'ex_student').to_sym
   end
 
   def disabled_from=(disabled_from)

data/lib/mumuki/domain/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mumuki-domain
 version: !ruby/object:Gem::Version
-  version: 9.12.0
+  version: 9.14.1
 platform: ruby
 authors:
 - Franco Leonardo Bulgarelli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-08-12 00:00:00.000000000 Z
+date: 2021-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.9'
+        version: '7.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.9'
+        version: '7.11'
 - !ruby/object:Gem::Dependency
   name: mumukit-assistant
   requirement: !ruby/object:Gem::Requirement
@@ -265,6 +265,10 @@ files:
 - app/models/concerns/navigation/siblings_navigation.rb
 - app/models/concerns/navigation/terminal_navigation.rb
 - app/models/concerns/onomastic.rb
+- app/models/concerns/organization/status/base.rb
+- app/models/concerns/organization/status/disabled.rb
+- app/models/concerns/organization/status/enabled.rb
+- app/models/concerns/organization/status/in_preparation.rb
 - app/models/concerns/submittable/confirmable.rb
 - app/models/concerns/submittable/queriable.rb
 - app/models/concerns/submittable/questionable.rb
@@ -292,6 +296,7 @@ files:
 - app/models/concerns/with_name.rb
 - app/models/concerns/with_notifications.rb
 - app/models/concerns/with_number.rb
+- app/models/concerns/with_organization_status.rb
 - app/models/concerns/with_pg_lock.rb
 - app/models/concerns/with_preferences.rb
 - app/models/concerns/with_profile.rb
@@ -336,6 +341,13 @@ files:
 - app/models/message.rb
 - app/models/notification.rb
 - app/models/organization.rb
+- app/models/organization_access_mode.rb
+- app/models/organization_access_mode/base.rb
+- app/models/organization_access_mode/coming_soon.rb
+- app/models/organization_access_mode/forbidden.rb
+- app/models/organization_access_mode/full.rb
+- app/models/organization_access_mode/gone.rb
+- app/models/organization_access_mode/read_only.rb
 - app/models/preferences.rb
 - app/models/progress.rb
 - app/models/stats.rb