From 8a0aa944d61c15845b04aa680844d2ebcaa98379 Mon Sep 17 00:00:00 2001 From: Justin Coyne Date: Fri, 17 Nov 2023 11:26:58 -0600 Subject: [PATCH] Replace try with safe navigation operator Try looks for method defined, but we know the method is defined if the attriute is present. --- app/models/embed/purl/resource_file.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/embed/purl/resource_file.rb b/app/models/embed/purl/resource_file.rb index 46d836d06..56f1889e9 100644 --- a/app/models/embed/purl/resource_file.rb +++ b/app/models/embed/purl/resource_file.rb @@ -77,7 +77,7 @@ def image? # @return [Integer] def size - @file.attributes['size'].try(:value).to_i + @file.attributes['size']&.value.to_i end # unused (9/2016) - candidate for removal?