From 4ab232823cecbbfce482017987097f2c2dce9754 Mon Sep 17 00:00:00 2001 From: foglabs Date: Thu, 23 Feb 2023 10:14:33 -0500 Subject: [PATCH] 'conditional language for shared segments based on media type' --- app/controllers/catalog_controller.rb | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/app/controllers/catalog_controller.rb b/app/controllers/catalog_controller.rb index f8f1b6b574..18711a5af6 100644 --- a/app/controllers/catalog_controller.rb +++ b/app/controllers/catalog_controller.rb @@ -255,9 +255,13 @@ def show if redirect_to_proxy_start_time?(@pbcore, params) redirect_to catalog_path(params["id"], proxy_start_time: @pbcore.proxy_start_time) and return - elsif params["start"] && params["end"] + elsif params["start"] && params["end"] && @pbcore.media_type != "other" + # media type: 'other' records can exist, but they shouldnt have media, so no segmenter + + media_type_verb = @pbcore.media_type == "Moving Image" ? "view" : "listen to" + # proxy start time takes precendence, this is for 'share a segment' - @clip_message = %(This is a segment of a longer program (#{ seconds_to_hms(params['start']) }-#{ seconds_to_hms(params['end']) }). Click here to view full-length item.) + @clip_message = %(This is a segment of a longer program (#{ seconds_to_hms(params['start']) }-#{ seconds_to_hms(params['end']) }). Click here to #{ media_type_verb } full-length item.) end end