Export release-9011@21680e7b

Author: Hasso Plattner Institut

app/components/course/learner_dashboard/course_progress.rb

@@ -91,7 +91,7 @@ def main_points
       private
 
       def section_completed_items_count(section)
-        return 0 if section['discarded']
+        return 0 if section['discarded'] || section['items'].blank?
 
         section['items'].count {|item| completed_item?(item) && !item['discarded'] }
       end

app/controllers/quiz_submission_controller.rb

@@ -272,7 +272,7 @@ def auth_context
 
   def create_item_presenter!
     Acfs.on the_item, the_section, the_course, the_quiz do |item, section, course, quiz|
-      presenter_class = Module.const_get "#{item.content_type}_item_presenter".camelize
+      presenter_class = ItemPresenter.lookup(item)
       @item_presenter = presenter_class.build item, section, course, current_user, quiz
     end
   end

app/helpers/item_context_helper.rb

@@ -37,7 +37,7 @@ def create_position_presenter!
 
   def create_item_presenter!
     Acfs.on the_item, the_section, the_course do |item, section, course|
-      presenter_class = Module.const_get "#{item.content_type}_item_presenter".camelize
+      presenter_class = ItemPresenter.lookup(item)
       @item_presenter = presenter_class.build item, section, course, current_user, params:
     end
   end

app/presenters/item_presenter.rb

@@ -19,6 +19,18 @@ class ItemPresenter < PrivatePresenter
   def_delegator :@section, :pinboard_closed, :section_pinboard_closed?
   def_delegator :@course, :lang, :lang
 
+  class << self
+    def lookup(item)
+      "#{item.content_type}_item_presenter".camelize.constantize
+    rescue NameError
+      UnsupportedItemPresenter
+    end
+
+    def for(item, course: nil, user: nil)
+      lookup(item).new(item:, course:, user:)
+    end
+  end
+
   def partial_name
     "items/#{content_type}/show_item_#{content_type}"
   end
@@ -44,11 +56,6 @@ def exercise_type
     @item['exercise_type']
   end
 
-  def self.for(item, course: nil, user: nil)
-    class_name = Module.const_get "#{item.content_type}_item_presenter".camelize
-    class_name.new(item:, course:, user:)
-  end
-
   def redirect(controller)
     controller.redirect_to @redirect
   end

app/presenters/unsupported_item_presenter.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require 'addressable'
+
+class UnsupportedItemPresenter < ItemPresenter
+  def self.build(item, section, course, user, **)
+    new(item:, section:, user:, course:)
+  end
+
+  def partial_name
+    'items/show_unsupported_item'
+  end
+end

app/views/items/_show_unsupported_item.html.slim

@@ -0,0 +1,10 @@
+.col-md-12
+  .course-area-main data-lanalytics-context={course_id: item.course_id}.to_json
+    = render 'items/header', item: item
+
+    = render Global::Callout.new(t(:'items.show.unsupported_item_type'), type: :error)
+
+    .mt40
+      = render Course::BottomNavigation.new(course_id: position.course.id,
+        prev_item_id: position.prev_item&.id,
+        next_item_id: position.next_item&.id)

config/initializers/sentry.rb

@@ -32,16 +32,50 @@
   # Do not send full list of gems with each event
   config.send_modules = false
 
-  # Set sampling rates to 1.0 to capture 100% of transactions and
-  # profiles for performance monitoring.
-  config.traces_sample_rate = 1.0
-  config.profiles_sample_rate = 1.0
-
   filter = ActiveSupport::ParameterFilter.new(Rails.application.config.filter_parameters)
   config.before_send = lambda do |event, _hint|
     # use Rails' parameter filter to sanitize the event
     filter.filter(event.to_hash)
   end
+
+  # The default sampling rate for Sentry transactions. Unless other
+  # rules apply below, sample all requests by default.
+  #
+  # Collect profiles are transactions that are sampled.
+  config.traces_sample_rate = ENV.fetch('SENTRY_TRACES_SAMPLE_RATE', '1.0').to_f
+  config.profiles_sample_rate = 1.0
+
+  config.traces_sampler = lambda do |ctx|
+    # If this is the continuation of a trace, just use that decision
+    # (rate controlled by the caller).
+    unless ctx[:parent_sampled].nil?
+      next ctx[:parent_sampled]
+    end
+
+    # `transaction_context` is the transaction object in hash form. Keep
+    # in mind that sampling happens right after the transaction is
+    # initialized for example, at the beginning of the request.
+    transaction_context = ctx[:transaction_context]
+
+    # `transaction_context` helps you sample transactions with more
+    # sophistication. For example, you can provide different sample
+    # rates based on the operation or name.
+    op = transaction_context[:op]
+    transaction_name = transaction_context[:name]
+
+    case op
+      when 'http.server'
+        # For Rails applications, the transaction name would be the
+        # request's path (env["PATH_INFO"]) instead of
+        # "Controller#action".
+        case transaction_name
+          when %r{^/(up|ping|system_info)}
+            return 0.0 # Ignore health check requests
+        end
+    end
+
+    config.traces_sample_rate
+  end
 end
 
 Sentry.set_tags(

config/locales/de.yml

@@ -1997,6 +1997,7 @@ de:
       prev_item: Zurück
       next_item: Weiter
       unsupported_browser: Ihr Browser wird nicht unterstützt. Bitte benutzen Sie einen modernen Browser.
+      unsupported_item_type: Diese Art von Lerninhalt wird nicht mehr unterstützt. Bitte fahren Sie mit dem nächsten fort.
       video:
         preview:
           info: "Dieses Video gehört zum %{platform}-Kurs <b>%{course_title}</b>. Möchten Sie mehr sehen?"

config/locales/en.yml

@@ -2760,6 +2760,7 @@ en:
       prev_item: Previous
       next_item: Next
       unsupported_browser: Your browser is not supported. Please use a modern browser.
+      unsupported_item_type: This item type is no longer supported. Please skip to the next item.
       video:
         preview:
           info: "This video belongs to the %{platform} course <b>%{course_title}</b>. Do you want to see more?"

pyproject.toml

@@ -5,5 +5,5 @@ requires-python = ">=3.12"
 dependencies = [
   "mkdocs-awesome-pages-plugin==2.10.1",
   "mkdocs-git-revision-date-plugin==0.3.2",
-  "mkdocs-material==9.6.7",
+  "mkdocs-material==9.6.9",
 ]