media metadata: fix exception in /media_metadata.json?only=media_asset.
Fix NameError exception caused by the MediaMetadataController trying to lookup "MediaMetadatum" as the model name.
This commit is contained in:
@@ -5,6 +5,6 @@ class MediaMetadataController < ApplicationController
|
|||||||
|
|
||||||
def index
|
def index
|
||||||
@media_metadata = authorize MediaMetadata.visible(CurrentUser.user).paginated_search(params, count_pages: true)
|
@media_metadata = authorize MediaMetadata.visible(CurrentUser.user).paginated_search(params, count_pages: true)
|
||||||
respond_with(@media_metadata)
|
respond_with(@media_metadata, model: "MediaMetadata")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -32,4 +32,8 @@ class MediaMetadata < ApplicationRecord
|
|||||||
def frame_delays
|
def frame_delays
|
||||||
metadata["Ugoira:FrameDelays"].to_a
|
metadata["Ugoira:FrameDelays"].to_a
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def self.available_includes
|
||||||
|
[:media_asset]
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -10,6 +10,14 @@ class MediaMetadataControllerTest < ActionDispatch::IntegrationTest
|
|||||||
assert_response :success
|
assert_response :success
|
||||||
end
|
end
|
||||||
|
|
||||||
|
should "work with the only=media_asset param" do
|
||||||
|
metadata = create(:media_metadata)
|
||||||
|
get media_metadata_path(only: "media_asset"), as: :json
|
||||||
|
|
||||||
|
assert_response :success
|
||||||
|
assert_equal(metadata.media_asset.id, response.parsed_body.sole.dig("media_asset", "id"))
|
||||||
|
end
|
||||||
|
|
||||||
context "searching" do
|
context "searching" do
|
||||||
setup do
|
setup do
|
||||||
@jpg = create(:media_metadata, file: "test/files/test.jpg")
|
@jpg = create(:media_metadata, file: "test/files/test.jpg")
|
||||||
|
|||||||
Reference in New Issue
Block a user