mumuki-laboratory 9.14.1 → 9.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc6cd361b66c4bf555f6c76814b671fb3dd477554858bf597a069524bad986de
-  data.tar.gz: 456af32a6deea453adc9b107351cb7fdf08f7f2139127850bc7b169813f002b7
+  metadata.gz: '0617268be6df9b74ff993247e4876c84492970ce49b6967db2aabb9ede077b5b'
+  data.tar.gz: 7c8387588bafd36583f3304f081401d91824fe8db75f023edefd4518d3c4e3f9
 SHA512:
-  metadata.gz: f96e1aa34781e87da4108b64c4bec13ef5dd071755250eb9ce3a1cd484808f2f9939df6d95fd87c2cd401990cc39df6462569ba65f5dddb78c2888efa9e89126
-  data.tar.gz: 9c4a276ad0b94d856d26cfc8761167de630ee6cb75abedaadec6802b1f6a01c4cabb3c9270069e1c6d6a5a4bc4fe39545f0bcb834b8b1beefbe940f71dfcd56c
+  metadata.gz: 9aa1867aaa19195882d1eab24537353237354417dff78f134fce70790475d005b7e42fc7212a5420e41a6aa2d051ab6e107e8ea824363782406ffc98a435dc28
+  data.tar.gz: 05127705df408c4f031c4dd9c9b03a8685034fcc68b97ddd27d70aaaa87a85598c5dafebb7000fede11b3c8f09b9324b711a07d4362ca8518538d2912d118e6c

data/app/assets/javascripts/mumuki_laboratory/application/codemirror-builder.js

@@ -27,7 +27,11 @@
       this.$textarea = $(textarea);
     }
 
-    setupEditor() {
+    setupEditor(readonly = false) {
+      return readonly ? this._setupReadOnlyEditor() : this._setupCommonEditor();
+    }
+
+    _setupCommonEditor() {
       this.editor = this.createEditor({
         lineNumbers: true,
         extraKeys: {
@@ -61,7 +65,7 @@
       return this;
     }
 
-    setupReadOnlyEditor() {
+    _setupReadOnlyEditor() {
       this.editor = this.createEditor({
         readOnly: true,
         cursorBlinkRate: -1, //Hides the cursor

data/app/assets/javascripts/mumuki_laboratory/application/codemirror.js

@@ -6,10 +6,11 @@ mumuki.page.editors = [];
 (() => {
   function createCodeMirrors() {
     return $(".editor").map(function (index, textarea) {
-      var $textarea = $("#solution_content");
+      const $textarea = $("#solution_content");
+      const readonly = $textarea.data('readonly');
 
       return new mumuki.editor.CodeMirrorBuilder(textarea)
-        .setupEditor()
+        .setupEditor(readonly)
         .setupMinLines($textarea.data('lines'))
         .setupLanguage()
         .build();

data/app/assets/javascripts/mumuki_laboratory/application/console.js

@@ -87,7 +87,7 @@
     }
     get content() {
       var firstEditor = mumuki.page.editors[0];
-      if (firstEditor && $("#include_solution").prop("checked"))
+      if (firstEditor && this.includeSolution())
         return firstEditor.getValue();
       else
         return '';
@@ -142,6 +142,12 @@
     get _requestData() {
       return {content: this.content, query: this.line, cookie: this.cookie};
     }
+    includeSolution() {
+      return !this._includeSolutionCheckbox || this._includeSolutionCheckbox.checked;
+    }
+    get _includeSolutionCheckbox() {
+      return $("#include_solution")[0];
+    }
   }
 
 

data/app/assets/javascripts/mumuki_laboratory/application/csrf-token.js

@@ -1,12 +1,12 @@
 mumuki.CsrfToken =  (() => {
   class CsrfToken {
-    constructor() {
-      this.value = $('meta[name="csrf-token"]').attr('content');
+    get token() {
+      return $('meta[name="csrf-token"]').attr('content');
     }
+
     newRequest(data) {
-      var self = this;
-      data.beforeSend = function (xhr) {
-        xhr.setRequestHeader('X-CSRF-Token', self.value);
+      data.beforeSend = (xhr) => {
+        xhr.setRequestHeader('X-CSRF-Token', this.token);
       };
       return data;
     }

data/app/assets/javascripts/mumuki_laboratory/application/discussions.js

@@ -19,25 +19,18 @@ mumuki.load(() => {
       .build();
   }
 
-  function createReadOnlyEditors() {
-    return $(".read-only-editor").map(function (index, textarea) {
-      var $textarea = $("#solution_content");
-
-      return new mumuki.editor.CodeMirrorBuilder(textarea)
-        .setupReadOnlyEditor()
-        .setupMinLines($textarea.data('lines'))
-        .setupLanguage()
-        .build();
-    });
-  }
-
-  createReadOnlyEditors();
   let editor = createNewMessageEditor();
 
   var Forum = {
     toggleButton: function (elements) {
       elements.toggleClass('d-none');
     },
+    disableButton: function (elements) {
+      elements.attr('disabled', true);
+    },
+    reenableButton: function (elements) {
+      elements.attr('disabled', false);
+    },
     token: new mumuki.CsrfToken(),
     tokenRequest: function (data) {
       return $.ajax(Forum.token.newRequest(data));
@@ -56,6 +49,7 @@ mumuki.load(() => {
       Forum.discussionPostAndToggle(url, $upvoteButtons);
     },
     discussionResponsible: function (url) {
+      Forum.disableButton($responsibleButton);
       Forum.discussionPostToggleAndRenderToast(url, $responsibleButton);
       $('.responsible-moderator-badge').toggleClass('d-none');
     },
@@ -66,6 +60,7 @@ mumuki.load(() => {
       Forum.discussionPost(url)
         .done(function (response) {
           Forum.toggleButton(elem);
+          Forum.reenableButton(elem);
           mumuki.toast.addToast(response);
         })
         .fail(function (response) {
@@ -75,6 +70,7 @@ mumuki.load(() => {
     discussionMessageToggleApprove: function (url, elem) {
       Forum.discussionPost(url).done(function () {
         elem.toggleClass("selected");
+        elem.attr('data-bs-original-title', '');
       });
     },
     discussionMessageToggleNotActuallyAQuestion: function (url, elem) {

data/app/assets/javascripts/mumuki_laboratory/application/exercise.js

@@ -1,3 +1,31 @@
+(() => {
+  function shouldSkipChangesCheck() {
+    return mumuki.isKidsExercise() || mumuki.exercise.isReadOnly;
+  }
+
+  function solutionChangedSinceLastSubmission() {
+    return  mumuki.exercise.id &&
+            mumuki.SubmissionsStore.getLastSubmissionAndResult(mumuki.exercise.id) &&
+            !mumuki.SubmissionsStore.getSubmissionResultFor(mumuki.exercise.id, mumuki.editors.getSubmission());
+  }
+
+  function shouldWarnOfChanges() {
+    return !shouldSkipChangesCheck() && solutionChangedSinceLastSubmission();
+  }
+
+  window.addEventListener("beforeunload", (event) => {
+    if (shouldWarnOfChanges()) {
+      event.returnValue = 'unsaved_progress';
+    } else {
+      delete event['returnValue'];
+    }
+  });
+
+  window.addEventListener("turbolinks:before-visit", (event) => {
+    if (shouldWarnOfChanges() && !confirm(mumuki.I18n.t('unsaved_progress'))) event.preventDefault();
+  });
+})();
+
 /**
  * @typedef {"input_right" | "input_bottom" | "input_primary" | "input_kindergarten"} Layout
  * @typedef {{id: number, layout: Layout, settings: any}} Exercise
@@ -46,6 +74,13 @@ mumuki.exercise = {
     return this._current;
   },
 
+  /**
+   * @type {Boolean?}
+   */
+  get isReadOnly() {
+    return $('#mu-exercise-read-only').val() === 'true';
+  },
+
   /**
    * Set global current exercise information
    */

data/app/assets/javascripts/mumuki_laboratory/application/i18n.js

@@ -9,6 +9,7 @@ mumuki.I18n = (() => {
       passed_with_warnings: () => "Tu solución funcionó, pero hay cosas que mejorar",
       pending: () => "Pendiente",
       skipped: () => "Venís aprendiendo muy bien, por lo que aprobaste este ejercicio",
+      unsaved_progress: () => "Tu solución tiene cambios sin guardar, ¿Querés salir de todos modos?",
     },
     'es-CL': {
       aborted: () => "Ups, no pudimos evaluar tu solución",
@@ -18,6 +19,7 @@ mumuki.I18n = (() => {
       passed_with_warnings: () => "Tu solución funcionó, pero hay cosas que mejorar",
       pending: () => "Pendiente",
       skipped: () => "Vienes aprendiendo muy bien, por lo que aprobaste este ejercicio",
+      unsaved_progress: () => "Tu solución tiene cambios sin guardar, ¿Quieres salir de todos modos?",
     },
     'en': {
       aborted: () => "Oops, we couldn't evaluate your solution",
@@ -27,6 +29,7 @@ mumuki.I18n = (() => {
       passed_with_warnings: () => "It worked, but you can do better",
       pending: () => "Pending",
       skipped: () => "You are doing very well, so you've passed this exercise",
+      unsaved_progress: () => "Your solution has unsaved changes, leave anyways?",
     },
     'pt': {
       aborted: () => "Opa, não pudemos avaliar sua solução",
@@ -36,6 +39,7 @@ mumuki.I18n = (() => {
       passed_with_warnings: () => "Sua solução funcionou, mas há coisas para melhorar",
       pending: () => "Pendente",
       skipped: () => "Você está aprendendo muito bem e passou neste exercício",
+      unsaved_progress: () => "Sua solução tem alterações não salvas. Deseja sair mesmo assim?",
     }
   }
 

data/app/controllers/ajax_controller.rb

@@ -9,6 +9,6 @@ class AjaxController < ApplicationController
   end
 
   def validate_organization_enabled!
-    Organization.current.validate_enabled!
+    Organization.current.validate_enabled! unless current_user&.teacher_here?
   end
 end

data/app/controllers/application_controller.rb

@@ -19,11 +19,11 @@ class ApplicationController < ActionController::Base
   before_action :set_time_zone!
 
   before_action :ensure_user_enabled!, if: :current_user?
-
   before_action :redirect_to_proper_context!, if: :immersive_context_wrong?
-  before_action :validate_active_organization!
 
   before_action :authorize_if_private!
+  before_action :validate_active_organization!
+
   before_action :validate_user_profile!, if: :current_user?
   before_action :validate_accepted_role_terms!, if: :current_user?
   before_action :ensure_restore_progress!, if: :current_user?

data/app/controllers/exam_authorization_requests_controller.rb

@@ -1,36 +1,37 @@
 class ExamAuthorizationRequestsController < ApplicationController
 
-  before_action :verify_registration_opened!, on: [:create, :update]
+  before_action :set_registration!
+  before_action :set_exam!
+  before_action :verify_registration_opened!
 
   def create
-    authorization_request = ExamAuthorizationRequest.find_or_create_by! create_authorization_request_params do |it|
-      it.assign_attributes authorization_request_params
-    end
-    current_user.read_notification! authorization_request.exam_registration
+    authorization_request = @registration.request_authorization! current_user, @exam
+    current_user.read_notification! @registration
     flash.notice = I18n.t :exam_authorization_request_created
     redirect_to root_path
   end
 
   def update
-    ExamAuthorizationRequest.update params[:id], authorization_request_params
+    @registration.update_authorization_request_by_id! params[:id], @exam
     flash.notice = I18n.t :exam_authorization_request_saved
     redirect_to root_path
   end
 
   private
 
-  def create_authorization_request_params
-    authorization_request_params.slice :exam_registration_id, :user, :organization
+  def authorization_request_params
+    params.require(:exam_authorization_request).permit(:exam_id, :exam_registration_id)
   end
 
-  def authorization_request_params
-    params
-        .require(:exam_authorization_request).permit(:exam_id, :exam_registration_id)
-        .merge(user: current_user, organization: Organization.current)
+  def set_registration!
+    @registration = Organization.current.exam_registrations.find(authorization_request_params[:exam_registration_id])
+  end
+
+  def set_exam!
+    @exam = @registration.exams.find(authorization_request_params[:exam_id])
   end
 
   def verify_registration_opened!
-    exam_registration = ExamRegistration.find(authorization_request_params[:exam_registration_id])
-    raise Mumuki::Domain::GoneError if exam_registration.ended?
+    raise Mumuki::Domain::GoneError if @registration.ended?
   end
 end

data/app/controllers/exam_registrations_controller.rb

@@ -1,6 +1,6 @@
 class ExamRegistrationsController < ApplicationController
   def show
-    @registration = ExamRegistration.find(params[:id])
+    @registration = Organization.current.exam_registrations.find(params[:id])
     @authorization_request = @registration.authorization_request_for(current_user)
   end
 end

data/app/helpers/discussions_helper.rb

@@ -184,7 +184,10 @@ module DiscussionsHelper
   end
 
   def discussion_info(discussion)
-    "#{t(:time_since, time: time_ago_in_words(discussion.created_at))} · #{t(:reply_count, count: discussion.visible_messages.size)}"
+    <<~HTML.html_safe
+      <span>#{friendly_time(discussion.created_at, :time_since)}</span>
+      <span> · #{t(:reply_count, count: discussion.visible_messages.size)}</span>
+    HTML
   end
 
   def discussion_filter_params_without_page

data/app/helpers/editor_helper.rb

@@ -7,8 +7,8 @@ module EditorHelper
   end
 
   def read_only_editor(content, language, options = {})
-    editor_options = editor_defaults(language, options, 'read-only-editor')
-    text_area_tag :solution_content, content, editor_options
+    editor_options = editor_defaults(language, options.deep_merge(data: { readonly: true }), 'editor')
+    text_area_tag 'solution[content]', content, editor_options
   end
 
   def spell_checked_editor(name, options = {})

data/app/helpers/exams_helper.rb

@@ -0,0 +1,33 @@
+module ExamsHelper
+  def exam_information_for(user, exam)
+    %Q{
+      #{course_information_for(user, exam)}
+      #{date_information_for(exam)}
+      #{duration_information_for(exam)}
+    }.html_safe
+  end
+
+  private
+
+  def course_information_for(user, exam)
+    if user.teacher_here?
+      "#{fa_icon('graduation-cap', class: 'fa-fw',
+                 text: "<strong>#{t :course}:</strong> #{exam.course.canonical_code}".html_safe)}
+      <br>"
+    end
+  end
+
+  def date_information_for(exam)
+    "#{fa_icon('calendar-alt', class: 'fa-fw',
+               text: "<strong>#{t :date_and_time}:</strong> #{local_time_without_time_zone(exam.start_time)} - #{local_time(exam.end_time)}".html_safe)}
+    <br>"
+  end
+
+  def duration_information_for(exam)
+    if exam.duration?
+      "#{fa_icon(:stopwatch, class: 'fa-fw',
+                 text: "<strong>#{t :available_time}:</strong> #{t :time_in_minutes, time: exam.duration}".html_safe)}
+      <br>"
+    end
+  end
+end

data/app/helpers/exercise_input_helper.rb

@@ -35,10 +35,6 @@ module ExerciseInputHelper
     should_render_exercise_tabs?(exercise) { exercise.has_messages_for? user }
   end
 
-  def should_render_read_only_exercise_tabs?(discussion)
-    should_render_exercise_tabs?(discussion.exercise) { discussion.has_submission? }
-  end
-
   def should_render_message_input?(exercise, organization = Organization.current)
     exercise.is_a?(Problem) && !exercise.hidden? && organization.raise_hand_enabled?
   end

data/app/helpers/menu_bar_helper.rb

@@ -38,12 +38,13 @@ module MenuBarHelper
     li_tag menu_item('sign-out-alt', :sign_out, logout_path(origin: url_for, organization: Organization.current))
   end
 
-  def menu_item(icon, name, url, css_class = nil, **translation_params)
-    menu_text_item(icon, t(name, translation_params), url, css_class)
+  def menu_item(icon, name, url, css_class = nil, translation_params = {}, options = {})
+    menu_text_item(icon, t(name, translation_params), url, css_class, **options)
   end
 
-  def menu_text_item(icon, text, url, css_class = nil)
-    link_to fixed_fa_icon(icon, text: text), url, role: 'menuitem', tabindex: '-1', class: "dropdown-item #{css_class}"
+  def menu_text_item(icon, text, url, css_class = nil, **options)
+    link_options = { role: 'menuitem', tabindex: '-1', class: "dropdown-item #{css_class}" }.merge(options)
+    link_to fixed_fa_icon(icon, text: text), url, link_options
   end
 
   def any_menu_bar_links?

data/app/helpers/notifications_helper.rb

@@ -4,7 +4,7 @@ module NotificationsHelper
   end
 
   def notification_preview_item(icon, name, url, **translation_params)
-    menu_item icon, name, url, 'mu-notification-preview', **translation_params
+    menu_item icon, name, url, 'mu-notification-preview', translation_params
   end
 
   def notification_text_preview_item(icon, text, url)

data/app/helpers/time_helper.rb

@@ -0,0 +1,12 @@
+module TimeHelper
+  def friendly_time(time, t_key = nil)
+    friendly_time = time_ago_in_words time
+    friendly_time_t = t_key ? t(t_key, time: friendly_time) : friendly_time
+
+    <<~HTML.html_safe
+      <time title="#{time}">
+        #{friendly_time_t}
+      </time>
+    HTML
+  end
+end

data/app/helpers/time_zone_helper.rb

@@ -1,5 +1,9 @@
 module TimeZoneHelper
   def local_time(time, time_zone = Time.zone.name)
-    "#{l(time.in_time_zone(time_zone), format: :long)} (#{time_zone})"
+    "#{local_time_without_time_zone(time, time_zone)} (#{time_zone})"
+  end
+
+  def local_time_without_time_zone(time, time_zone = Time.zone.name)
+    l(time.in_time_zone(time_zone), format: :long)
   end
 end

data/app/helpers/user_discussions_helper.rb

@@ -1,6 +1,6 @@
 module UserDiscussionsHelper
   def user_discussions_table_title(discussion, user, last_read)
-    %Q{
+    <<~HTML.html_safe
       <tr></tr>
       <thead>
         <tr>
@@ -9,30 +9,30 @@ module UserDiscussionsHelper
           </td>
         </tr>
       </thead>
-    }.html_safe
+    HTML
   end
 
   def user_discussions_table_header
-    %Q{
+    <<~HTML.html_safe
       <tr class="fw-bold">
         <td></td>
         <td>#{t(:exercise)}</td>
         <td>#{t(:discussion_created_by)}</td>
         <td>#{t(:last_message)}</td>
       </tr>
-    }.html_safe
+    HTML
   end
 
   def user_discussions_table_item(discussion, user)
-    %Q{
+    <<~HTML.html_safe
       <tr>
         <td class="text-center">
           #{icon_for_read(discussion.read_by?(user))}
         </td>
         <td>#{link_to discussion.item.name, item_discussion_path(discussion)}</td>
         <td>#{discussion_user_name discussion.initiator}</td>
-        <td>#{t(:time_since, time: time_ago_in_words(discussion.last_message_date))}</td>
+        <td>#{friendly_time(discussion.last_message_date, :time_since)}</td>
       </tr>
-    }.html_safe
+    HTML
   end
 end

data/app/views/book/show.html.erb

@@ -45,8 +45,10 @@
     <h2><%= t(:exams) %></h2>
     <% @exams.each_with_index do |it, index| %>
       <div class="chapter">
-        <h3> <%= index + 1 %>. <%= link_to_path_element it, mode: :plain %></h3>
-
+        <h3><%= link_to_path_element it, mode: :plain %></h3>
+        <div class="fs-7">
+          <%= exam_information_for(current_user, it) %>
+        </div>
         <div class="text-box">
           <%= it.guide.description_teaser_html %>
         </div>

data/app/views/discussions/_actions.html.erb

@@ -0,0 +1 @@
+<%= render partial: 'discussions/basic_actions' %>

data/app/views/discussions/_basic_actions.html.erb

@@ -0,0 +1,20 @@
+<% if @discussion.can_toggle_responsible?(current_user) %>
+  <div class="discussion-responsible me-1">
+    <%= btn_toggle responsible_icon, not_responsible_icon, @discussion.any_responsible?, class: 'btn-sm',
+                   onclick: "mumuki.Forum.discussionResponsible('#{responsible_discussion_url(@discussion)}')" %>
+  </div>
+<% end %>
+
+<% if @discussion.subscribable? %>
+  <div class="discussion-subscription me-1">
+    <%= btn_toggle subscription_icon, unsubscription_icon, current_user.subscribed_to?(@discussion), class: 'btn-sm',
+                   onclick: "mumuki.Forum.discussionSubscription('#{subscription_discussion_url(@discussion)}')" %>
+  </div>
+<% end %>
+
+<% if @discussion.solved? %>
+  <div class="discussion-upvote">
+    <%= btn_toggle upvote_icon, undo_upvote_icon, current_user.upvoted?(@discussion), class: 'btn-sm',
+                   onclick: "mumuki.Forum.discussionUpvote('#{upvote_discussion_url(@discussion)}')" %>
+  </div>
+<% end %>

data/app/views/discussions/_description_message.html.erb

@@ -4,7 +4,7 @@
       <div class="discussion-message-bubble-title">
         <%= linked_discussion_user_name(discussion.initiator) %>
         <span class="message-date">
-          <%= t(:time_since, time: time_ago_in_words(discussion.created_at)) %>
+          <%= friendly_time(discussion.created_at, :time_since) %>
         </span>
       </div>
     </div>

data/app/views/discussions/_message.html.erb

@@ -7,12 +7,16 @@
           <span class="moderator-badge"><%= t(:moderation) %></span>
         <% end %>
         <span class="message-date">
-          <%= t(:time_since, time: time_ago_in_words(message.created_at)) %>
+          <%= friendly_time(message.created_at, :time_since) %>
         </span>
         <span class="actions">
           <% if message.authorized?(current_user) && !message.deleted? %>
             <% if current_user&.moderator_here? %>
               <a class="discussion-message-approved <%= 'selected' if message.approved? %>"
+                 <% if message.approved? && current_user&.forum_supervisor_here? %>
+                   data-bs-toggle="tooltip" data-bs-placement="left"
+                   title="<%= t :approved_by, validator: message.approved_by.name, date: local_time(message.approved_at) %>"
+                 <% end %>
                  onclick="mumuki.Forum.discussionMessageToggleApprove('<%= approve_discussion_message_url(@discussion, message) %>', $(this))">
                 <%= fa_icon(:check, class: 'fa-lg') %>
               </a>
@@ -47,7 +51,7 @@
           <% if current_user&.moderator_here? %>
             <hr>
             <%= t :deleted_by, deleter: message.deleted_by.name %>
-            <%= t(:time_since, time: time_ago_in_words(message.deleted_at)) %>.
+            <%= friendly_time(message.deleted_at, :time_since) %>
             <a href='<%= "#deletedMessage#{message.id}" %>' data-bs-toggle="collapse">
               <%= t :show_message %>
             </a>

data/app/views/discussions/new.html.erb

@@ -22,12 +22,18 @@
         <div class="container-fluid">
           <div class="row">
             <div class="discussion-new-message-content">
-              <%= f.editor :description, '', {id: 'discussion-new-message', class: 'form-control', placeholder: t(:discussion_description_placeholder)} %>
+              <%= spell_checked_editor 'discussion[description]',
+                                       { id: 'discussion-new-message', placeholder: t(:discussion_description_placeholder) } %>
             </div>
           </div>
         </div>
       </div>
     </div>
+    <% if faqs_enabled_here? %>
+      <div class="fs-7">
+        <%= fa_icon('exclamation-triangle', text: t(:only_forum_questions_on_forum, link: link_to_faqs).html_safe) %>
+      </div>
+    <% end %>
     <%= f.submit t(:publish_discussion), class: 'btn btn-complementary w-100 discussion-new-message-button' %>
   <% end %>
 <% end %>

data/app/views/discussions/show.html.erb

@@ -15,24 +15,7 @@
     <h3 class="flex-grow-1 me-3"><%= t :messages %></h3>
     <% if current_user && @discussion.persisted? && current_access_mode.show_discussion_element? %>
       <span class="d-flex">
-        <% if @discussion.can_toggle_responsible?(current_user) %>
-          <div class="discussion-responsible me-1">
-            <%= btn_toggle responsible_icon, not_responsible_icon, @discussion.any_responsible?, class: 'btn-sm',
-                           onclick: "mumuki.Forum.discussionResponsible('#{responsible_discussion_url(@discussion)}')" %>
-          </div>
-        <% end %>
-        <% if @discussion.subscribable? %>
-          <div class="discussion-subscription me-1">
-            <%= btn_toggle subscription_icon, unsubscription_icon, current_user.subscribed_to?(@discussion), class: 'btn-sm',
-                           onclick: "mumuki.Forum.discussionSubscription('#{subscription_discussion_url(@discussion)}')" %>
-          </div>
-        <% end %>
-        <% if @discussion.solved? %>
-          <div class="discussion-upvote">
-            <%= btn_toggle upvote_icon, undo_upvote_icon, current_user.upvoted?(@discussion), class: 'btn-sm',
-                           onclick: "mumuki.Forum.discussionUpvote('#{upvote_discussion_url(@discussion)}')" %>
-          </div>
-        <% end %>
+        <%= render partial: 'discussions/actions' %>
       </span>
     <% end %>
   </div>

data/app/views/exam_registrations/show.html.erb

@@ -25,7 +25,7 @@
         <%= f.hidden_field :exam_registration_id, value: @registration.id %>
         <div class="mb-4">
           <%= f.label :exam_id, t(:exam_registration_choose_exam) %>
-          <% @registration.exams.each do |exam| %>
+          <% @registration.available_exams.each do |exam| %>
             <div class="form-check">
               <%= f.radio_button(:exam_id, exam.id, id: exam.id, required: true, class: 'form-check-input mu-read-only-input',
                                  checked: @authorization_request.exam_id == exam.id) %>