Merge branch 'default-attachment-location' into 'master'

added default attachment location to LO serializer

See merge request !406

app/models/learning_object.rb

@@ -109,6 +109,11 @@ class LearningObject < ApplicationRecord
     nil
   end
 
+  # If a LO has more than one object, this gives the location of the main one
+  def default_attachment_location
+    object_type.try(:name) == ("Vídeo" || "Áudio") ? default_attachment.try(:retrieve_cache_link) : default_attachment.try(:retrieve_url)
+  end
+
   def default_thumbnail
     return thumbnail unless thumbnail.blank?
     return nil if attachments.blank?
@@ -125,8 +130,9 @@ class LearningObject < ApplicationRecord
     default_attachment.retrieve_cache_link
   end
 
+  # Download link with all relevant objects (currently only one)
   def download_link
-    object_type.try(:name) == ("Vídeo" || "Áudio") ? default_attachment.try(:retrieve_cache_link) : default_attachment.try(:retrieve_url)
+    default_attachment_location
   end
 
   ## score methods

app/serializers/learning_object_serializer.rb

@@ -37,6 +37,7 @@ class LearningObjectSerializer < ActiveModel::Serializer
     :thumbnail,
     :object_type,
     :language,
+    :default_attachment_location,
     :default_attachment_id,
     :default_mime_type,
     :score,