Change "Watch from Here" to "Access from Here"

[caseyedavis12]

User Story

As a user, when I do a search of the AAPB and the keywords appear in a transcript, I should see a link that says "Access from Here" rather than "Watch from Here" since not all content in the AAPB is TV.

[mrharpo]

@caseyedavis12 It looks like the code already does this, at least a little:
From #1667 (comment)

Have the text read "Watch from here" or "Listen from here"

And in snippet_helper.rb, it renders:

<button type="button" class="btn btn-default snippet-link">#{media_type == 'Moving Image' ? "Watch" : "Listen"} from here</button>

[caseyedavis12]

Hmmm... here is an example of a radio series where the button says "Watch from Here"

https://americanarchive.org/catalog?f%5Bspecial_collections%5D%5B%5D=backstory&q=aids&sort=asset_date+asc&utf8=%E2%9C%93&f[access_types][]=online

[mrharpo]

Very strange!

Here is that record you found:
https://americanarchive.org/catalog/cpb-aacip-532-3r0pr7nx4w

But the .pbcore says:

<instantiationMediaType>Sound</instantiationMediaType>

[mrharpo]

It looks like the media_type is always set to "Moving Image", and does not come from the type of media:

AAPB2/app/controllers/snippets_controller.rb

Lines 36 to 46 in 0a3882b

	if transcript_file.file_type == TranscriptFile::JSON_FILE && !transcript_file.file_content.empty?
	ts = TimecodeSnippet.new(this_id, terms_array, transcript_file.plaintext, JSON.parse(transcript_file.file_content)["parts"])
	# fix media type
	snippet_view_string = transcript_snippet(ts.snippet, "Moving Image", ts.url_at_timecode) if ts.snippet
	elsif transcript_file.file_type == TranscriptFile::TEXT_FILE
	ts = Snippet.new(this_id, terms_array, transcript_file.plaintext)
	# fix it
	snippet_view_string = transcript_snippet(ts.snippet, "Moving Image") if ts.snippet
	end

[foglabs]

@mrharpo you're right, it'll always get set to 'Moving Image' - I think this is an oversight from my last refactor 😬

if you wanted to maintain the old behavior for this, you would need to get the xml from the SolrDocument, use it to instantiate a PBCorePresenter, and then pass in your_pbcoreprsenter_instance.media_type instead of "Moving Image"

I think casey's change to 'Access' is better here, because there's actually a lot of processing and loading that happens during PBCorePresenter.new(xml), and this snippet code will potentially be run once for each record in every page of a catalog search.