mumuki-laboratory 7.6.2 → 7.7.0

data/app/assets/stylesheets/mumuki_laboratory/application/_modules.scss

@@ -19,6 +19,7 @@
 @import "modules/breadcrumb";
 @import "modules/gs-board";
 @import "modules/kids";
+@import "modules/kindergarten";
 @import "modules/kids_results";
 @import "modules/discussion";
 @import "modules/popover";

data/app/assets/stylesheets/mumuki_laboratory/application/modules/_discussion.scss

@@ -201,10 +201,6 @@ $moderator-star-color: #dd9900;
   }
 }
 
-.discussion-create {
-  cursor: pointer;
-}
-
 .discussion-context {
   margin-top: 40px;
 }
@@ -341,6 +337,30 @@ summary.discussion-summary {
             }
           }
         }
+        .discussion-message-not-actually-a-question {
+          cursor: pointer;
+          text-decoration: none;
+          i {
+            position: relative;
+            &:after {
+              position: absolute;
+              left: 6px;
+              content: ' ';
+              height: 19px;
+              width: 2px;
+              transform: rotate(-45deg);
+              background-color: #aaaaaa;
+            }
+          }
+          &.selected {
+            i {
+              color: $brand-primary;
+              &:after {
+                background-color: $brand-primary;
+              }
+            }
+          }
+        }
         i {
           color: #aaaaaa
         }
@@ -383,4 +403,23 @@ $statuses: (
   }
 }
 
+.no-margin {
+  margin: 0 !important;
+}
+
+.discussion-requires-attention {
+  margin-right: 20px;
+  label {
+    font-weight: normal;
+  }
+}
 
+.discussion-moderator-access {
+  margin-right: 20px;
+  display: flex;
+  flex-direction: column;
+  align-items: center;
+  .moderator-initials {
+    font-size: 14px;
+  }
+}

data/app/assets/stylesheets/mumuki_laboratory/application/modules/_kids.scss

@@ -182,17 +182,17 @@ $kids-speech-tabs-width: 40px;
           cursor: pointer;
         }
       }
-      div.mu-kids-character-speech-bubble-normal {
+      .mu-kids-character-speech-bubble-normal {
         width: 100%;
         height: 100%;
         overflow: hidden;
       }
-      div.mu-kids-character-speech-bubble-failed {
+      .mu-kids-character-speech-bubble-failed {
         width: 100%;
         height: 100%;
         overflow: hidden;
       }
-      ul.mu-kids-character-speech-bubble-tabs {
+      .mu-kids-character-speech-bubble-tabs {
         display: flex;
         flex-direction: column;
         justify-content: center;

data/app/assets/stylesheets/mumuki_laboratory/application/modules/_kindergarten.scss

@@ -0,0 +1,55 @@
+.mu-kids-exercise.mu-kindergarten {
+
+  // layout
+
+  display: flex;
+  flex-flow: column;
+
+  .mu-kids-exercise-workspace {
+    display: flex;
+    flex-flow: row;
+    height: 100%;
+    width: 100%;
+  }
+
+  .mu-kids-single-state,
+  .mu-kids-blocks {
+    width: 50%;
+    height: 100%
+  }
+
+  .mu-kids-blocks {
+    margin-top: 0;
+  }
+
+  // no board numbers,
+  // but a thicker border instead
+
+  table.gbs_board.style-scope.gs-board {
+    padding: 3px;
+  }
+
+  .gbs_lv.gbs_lvl.style-scope.gs-board,
+  .gbs_lv.gbs_lvr.style-scope.gs-board,
+  tr.style-scope.gs-board:first-child,
+  tr.style-scope.gs-board:last-child {
+    display: none
+  }
+
+  // no final state,
+  // initial state fills its area
+
+  .mu-kids-state.mu-state-initial {
+    height: 100%;
+  }
+
+  .mu-kids-state.mu-state-final {
+    display: none;
+  }
+
+  .mu-kids-compass-rose {
+    display: none;
+  }
+}
+
+

data/app/controllers/assets_controller.rb

@@ -2,6 +2,7 @@ class AssetsController < ApplicationController
 
   protect_from_forgery except: [:theme_stylesheet, :extension_javascript]
   skip_before_action :authorize_if_private!
+  skip_before_action :validate_user_profile!
 
   def theme_stylesheet
     render inline: Organization.current.theme_stylesheet.to_s, content_type: 'text/css'

data/app/controllers/concerns/with_authorization.rb

@@ -13,6 +13,10 @@ module WithAuthorization
     authorize! :owner
   end
 
+  def authorize_moderator!
+    authorize! :moderator
+  end
+
   def authorization_slug
     protection_slug || '_/_'
   end

data/app/controllers/discussions_controller.rb

@@ -21,7 +21,12 @@ class DiscussionsController < ApplicationController
     @filtered_discussions = @discussions.scoped_query_by(discussion_filter_params)
   end
 
+  def new
+    @discussion = @debatable.new_discussion_for current_user
+  end
+
   def show
+    @discussion.update_last_moderator_access! current_user
   end
 
   def update

data/app/controllers/discussions_messages_controller.rb

@@ -1,6 +1,9 @@
 class DiscussionsMessagesController < AjaxController
+  include WithAuthorization
+
   before_action :set_discussion!, only: [:create, :destroy]
-  before_action :authorize_user!, only: [:destroy, :approve]
+  before_action :authorize_user!, only: [:destroy]
+  before_action :authorize_moderator!, only: [:question, :approve]
 
   def create
     @discussion.submit_message! message_params, current_user
@@ -17,6 +20,11 @@ class DiscussionsMessagesController < AjaxController
     head :ok
   end
 
+  def question
+    current_message.toggle_not_actually_a_question!
+    head :ok
+  end
+
   private
 
   def set_discussion!

data/app/controllers/exercise_solutions_controller.rb

@@ -22,7 +22,9 @@ class ExerciseSolutionsController < AjaxController
   end
 
   def solution_params
-    params_h = params.require(:solution).permit!.to_h
-    {content: params_h[:content]}
+    {
+      content: params.require(:solution).permit!.to_h[:content],
+      client_result: params[:client_result].try { |it| it.permit(:status, test_results: [:title, :status, :result, :summary]).to_h }
+    }
   end
 end

data/app/helpers/application_helper.rb

@@ -7,10 +7,14 @@ module ApplicationHelper
   end
 
   def profile_picture
-    image_tag(current_user.profile_picture, height: 40, class: 'img-circle', onError: "this.onerror = null; this.src = '#{image_url('user_shape.png')}'")
+    profile_picture_for current_user
   end
 
-  def paginate(object, options={})
+  def profile_picture_for(user, height = 40)
+    image_tag(user.profile_picture, height: height, class: 'img-circle', onError: "this.onerror = null; this.src = '#{image_url('user_shape.png')}'")
+  end
+
+  def paginate(object, options = {})
     "<div class=\"text-center\">#{super(object, {theme: 'twitter-bootstrap-3'}.merge(options))}</div>".html_safe
   end
 
@@ -32,10 +36,10 @@ module ApplicationHelper
     t :chapter_finished_html, chapter: link_to_path_element(chapter) if chapter
   end
 
-  def span_toggle(hidden_text, active_text, active)
+  def span_toggle(hidden_text, active_text, active, **options)
     %Q{
-      <span class="#{'hidden' if active}">#{hidden_text}</span>
-      <span class="#{'hidden' unless active}">#{active_text}</span>
+      <span class="#{'hidden' if active} #{options[:class]}">#{hidden_text}</span>
+      <span class="#{'hidden' unless active} #{options[:class]}">#{active_text}</span>
     }.html_safe
   end
 end

data/app/helpers/discussions_helper.rb

@@ -23,31 +23,31 @@ module DiscussionsHelper
     fixed_fa_icon 'comment', text: text
   end
 
-  def discussions_link(item, path, html_options=nil, organization=Organization.current)
+  def discussions_link(item, path, html_options = nil, organization = Organization.current)
     link_to item, path, html_options if organization.forum_enabled?
   end
 
-  def item_discussion_path(discussion, params={})
+  def item_discussion_path(discussion, params = {})
     polymorphic_path([discussion.item, discussion], params)
   end
 
-  def item_discussions_path(item, params={})
+  def item_discussions_path(item, params = {})
     polymorphic_path([item, :discussions], params)
   end
 
   def solve_discussion_params_for(user)
     if user&.moderator_here?
-      { status: :pending_review, sort: :created_at_asc }
+      {status: :pending_review, sort: :responses_count_asc, requires_moderator_response: true}
     else
-      { status: :opened, sort: :created_at_asc }
+      {status: :opened, sort: :responses_count_desc}
     end
   end
 
   def default_discussions_params
-    { status: :solved, sort: :upvotes_count_desc }
+    {status: :solved, sort: :upvotes_count_desc}
   end
 
-  def user_avatar(user, image_class='')
+  def user_avatar(user, image_class = '')
     image_tag user.profile_picture, height: 40, class: "img-circle #{image_class}"
   end
 
@@ -67,7 +67,7 @@ module DiscussionsHelper
     %Q{
       <span class="discussion-icon fa-stack fa-xs">
         <i class="fa fa-comment-o fa-stack-2x"></i>
-        <i class="fa fa-stack-1x">#{discussion.messages.size}</i>
+        <i class="fa fa-stack-1x">#{discussion.validated_messages_count}</i>
       </span>
     }.html_safe
   end
@@ -96,17 +96,13 @@ module DiscussionsHelper
     %Q{
       <h4>
         <span>#{t(teaser_text)}</span>
-        <a>
-          <span class="discussion-create">
-            #{t(link_text)}
-          </span>
-        </a>
+        #{link_to t(link_text), new_exercise_discussion_path(@debatable, anchor: 'new-discussion-description-container') }
       </h4>
     }.html_safe
   end
 
   def discussion_count_for_status(status, discussions)
-    discussions.scoped_query_by(discussion_filter_params, :status).by_status(status).count
+    discussions.scoped_query_by(discussion_filter_params, excluded_params: [:status], excluded_methods: [:page]).by_status(status).count
   end
 
   def discussions_reset_query_link
@@ -118,7 +114,7 @@ module DiscussionsHelper
   end
 
   def discussions_languages(discussions)
-    discussions.map { |it| it.language.name }.uniq
+    @languages ||= discussions.map { |it| it.language.name }.uniq
   end
 
   def discussion_status_filter_link(status, discussions)
@@ -132,14 +128,14 @@ module DiscussionsHelper
 
   def discussion_status_filter(status, discussions_count)
     %Q{
-      #{discussion_status_fa_icon(status)}
+    #{discussion_status_fa_icon(status)}
       <span>
         #{t("#{status}_count", count: discussions_count)}
       </span>
     }.html_safe
   end
 
-  def discussion_dropdown_filter(label, filters, &block)
+  def discussion_dropdown_filter(label, filters, can_select_all = false, &block)
     if filters.present?
       %Q{
         <div class="dropdown discussions-toolbar-filter">
@@ -147,6 +143,7 @@ module DiscussionsHelper
             #{t label} #{fa_icon :'caret-down', class: 'fa-xs'}
           </a>
           <ul class="dropdown-menu" aria-labelledby="dropdown-#{label}">
+            #{discussion_filter_unselect_item(label, can_select_all)}
             #{discussion_filter_list(label, filters, &block)}
           </ul>
         </div>
@@ -159,7 +156,15 @@ module DiscussionsHelper
   end
 
   def discussion_filter_item(label, filter, &block)
-    content_tag(:li, discussion_filter_link(label, filter, &block), class: "#{'selected' if discussion_filter_selected?(label, filter)}")
+    content_tag(:li, discussion_filter_link(label, filter, &block), class: ('selected' if discussion_filter_selected?(label, filter)))
+  end
+
+  def discussion_filter_unselect_item(label, can_select_all)
+    if can_select_all
+      content_tag(:li,
+                  link_to(t(:all), discussion_filter_params_without_page.except(label)),
+                  class: ('selected' unless discussion_filter_params.include?(label)))
+    end
   end
 
   def discussion_filter_selected?(label, filter)
@@ -167,10 +172,15 @@ module DiscussionsHelper
   end
 
   def discussion_filter_link(label, filter, &block)
-    link_to capture(filter, &block), discussion_filter_params.merge(Hash[label, filter])
+    link_to capture(filter, &block), discussion_filter_params_without_page.merge(Hash[label, filter])
   end
 
   def discussion_info(discussion)
-     "#{t(:time_since, time: time_ago_in_words(discussion.created_at))} · #{t(:message_count, count: discussion.messages.size)}"
+    "#{t(:time_since, time: time_ago_in_words(discussion.created_at))} · #{t(:message_count, count: discussion.messages.size)}"
   end
+
+  def discussion_filter_params_without_page
+    discussion_filter_params.except(:page)
+  end
+
 end

data/app/helpers/exercise_input_helper.rb

@@ -75,7 +75,7 @@ module ExerciseInputHelper
 
   def render_custom_editor(exercise, read_only=false)
     custom_editor_tag = "mu-#{exercise.language}-custom-editor"
-    "<#{custom_editor_tag} class='#{custom_editor_tag}' #{custom_editor_read_only if read_only}> </#{custom_editor_tag}>".html_safe
+    "<#{custom_editor_tag} id='#{custom_editor_tag}' class='#{custom_editor_tag}' #{custom_editor_read_only if read_only}> </#{custom_editor_tag}>".html_safe
   end
 
   def custom_editor_read_only

data/app/helpers/icons_helper.rb

@@ -3,7 +3,7 @@ module IconsHelper
     fa_icon *icon_for(status_like.to_submission_status)
   end
 
-  def fixed_fa_icon(name, options={})
+  def fixed_fa_icon(name, options = {})
     fa_icon name, options.merge(class: 'fa-fw fixed-icon')
   end
 
@@ -47,12 +47,12 @@ module IconsHelper
     iconizable.iconize[:type].to_s
   end
 
-  def label_for_contextualization(contextualization)
+  def label_for_contextualization(contextualization, **options)
     iconized = contextualization.iconize
     %Q{
       <span class="text-#{iconized[:class]} status-label">
         #{fa_icon "#{iconized[:type]}"}
-        <span>#{t contextualization.visible_status}</span>
+        <span class="#{options[:class]}">#{t contextualization.visible_status}</span>
       </span>
     }.html_safe
   end

data/app/views/book_discussions/index.html.erb

@@ -1,9 +1,9 @@
 <%= content_for :breadcrumbs do %>
-  <%= home_breadcrumb %>
+  <%= header_breadcrumbs %> <%= breadcrumb_list_item(t(:discussions), 'last') %>
 <% end %>
 
 <%= content_for :extra_filters do %>
-  <%= discussion_dropdown_filter :language, discussions_languages(@discussions) do |language_filter| %>
+  <%= discussion_dropdown_filter :language, discussions_languages(@discussions), true do |language_filter| %>
     <%= language_filter.capitalize %>
   <% end %>
 <% end %>
@@ -18,6 +18,6 @@
   <% if @discussions.empty? %>
     <h4><%= t :no_questions %></h4>
   <% else %>
-      <%= render partial: 'layouts/discussions' %>
+    <%= render partial: 'layouts/discussions' %>
   <% end %>
 </div>

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

@@ -11,9 +11,14 @@
         <% end %>
         <% if message.authorized? current_user %>
           <span class="actions">
-            <a class="discussion-message-approved <%= 'selected' if message.approved? %>" onclick="mumuki.Forum.discussionMessageToggleApprove('<%= approve_discussion_message_url(@discussion, message) %>', $(this))">
-              <%= fa_icon(:check, class: 'fa-xs') %>
-            </a>
+            <% if current_user.moderator_here? %>
+              <a class="discussion-message-approved <%= 'selected' if message.approved? %>" onclick="mumuki.Forum.discussionMessageToggleApprove('<%= approve_discussion_message_url(@discussion, message) %>', $(this))">
+                <%= fa_icon(:check, class: 'fa-xs') %>
+              </a>
+              <a class="discussion-message-not-actually-a-question <%= 'selected' if message.not_actually_a_question? %>" onclick="mumuki.Forum.discussionMessageToggleNotActuallyAQuestion('<%= question_discussion_message_url(@discussion, message) %>', $(this))">
+                <%= fa_icon('question-circle-o', class: 'fa-xs') %>
+              </a>
+            <% end %>
             <%= link_to fa_icon('trash-o'), discussion_message_path(@discussion, message), method: :delete, data: { confirm: t(:are_you_sure, action: t(:destroy_message)) } %>
           </span>
         <% end %>