only update landing page info if there is content

datacite · Dec 26, 2018 · bcaa93f · bcaa93f
1 parent c4ac60c
commit bcaa93f
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/app/controllers/dois_controller.rb b/app/controllers/dois_controller.rb
@@ -551,14 +551,11 @@ def safe_params
     end
     p.merge!(version_info: p[:version] || meta["version_info"]) if p.has_key?(:version_info) || meta["version_info"].present?
 
+    # only update landing_page info if something is received via API to not overwrite existing data
+    p.merge!(landing_page: p[:landingPage]) if p[:landingPage].present?
+
     p.merge(
-      regenerate: p[:regenerate] || regenerate,
-      landing_page: p[:landingPage],
-      last_landing_page: p[:lastLandingPage],
-      last_landing_page_status: p[:lastLandingPageStatus],
-      last_landing_page_status_check: p[:lastLandingPageStatusCheck],
-      last_landing_page_status_result: p[:lastLandingPageStatusResult],
-      last_landing_page_content_type: p[:lastLandingPageContentType]
+      regenerate: p[:regenerate] || regenerate
     ).except(
       :confirmDoi, :prefix, :suffix, :publicationYear,
       :rightsList, :relatedIdentifiers, :fundingReferences, :geoLocations,