metadata_presenter 2.19.4 → 2.19.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 287793ad62c89ad266e3c05508365cf24bcd5269d4c08e28f9a631aa50f5762a
-  data.tar.gz: 885cc9f6bc43d4d1bd86ba1fb376d1994905000e4f14bfe7173076db7d531633
+  metadata.gz: f0d20b5161033659492619628a0c1f26c1a69926193dbea87353a1b3a2a0a984
+  data.tar.gz: 15940e4f5bbf2656108d89a589535635d31bd2f8f813855b628ee67e4ef9ddf1
 SHA512:
-  metadata.gz: c3ab0c526f54ea3d22925baa1da1b4260deceb51bbd720695635b55ff3629116f0a93479fc8118e83bef39cc8ccd52bb718a1128fa573ad851247597e1c04821
-  data.tar.gz: '024489fcbd77662977c8684f2949fcd1b4bc5126e7933feba40a6acd689f12a63de3ad112a8eb92a00113c77979c0fcba90f6c65f05e6e6a1eda9d607f02546b'
+  metadata.gz: 9b3db24698b1637328ba375d2676c903331cab7fcdc1789faeedc4b626b41f685f2a50225ea48f04c378d72be288928322eb24cccff816085dd66784c1d6377b
+  data.tar.gz: 47f23df2526d20da3066044b4224faee03113375c4654bce63154935b50e8df94cd588196b841a034641f591447577da8d9246dbbf2a1cd6ddb46f5c4fabba18

data/app/controllers/metadata_presenter/answers_controller.rb

@@ -22,6 +22,14 @@ module MetadataPresenter
       end
     end
 
+    def update_count_matching_filenames(original_filename, user_data)
+      extname = File.extname(original_filename)
+      basename = File.basename(original_filename, extname)
+      filename_regex = /^#{Regexp.quote(basename)}(?>-\((\d)\))?#{Regexp.quote(extname)}/
+
+      user_data.select { |_k, v| v.instance_of?(Hash) && v['original_filename'] =~ filename_regex }.count
+    end
+
     private
 
     def page
@@ -72,6 +80,12 @@ module MetadataPresenter
       @page_answers.page.upload_components.each do |component|
         answer = user_data[component.id]
 
+        original_filename = answer.nil? ? @page_answers.send(component.id)['original_filename'] : answer['original_filename']
+
+        if original_filename.present?
+          @page_answers.count = update_count_matching_filenames(original_filename, user_data)
+        end
+
         @page_answers.uploaded_files.push(uploaded_file(answer, component))
       end
     end

data/app/models/metadata_presenter/offline_upload_adapter.rb

@@ -1,7 +1,7 @@
 module MetadataPresenter
   class OfflineUploadAdapter
     include ActiveModel::Model
-    attr_accessor :session, :file_details, :allowed_file_types
+    attr_accessor :session, :file_details, :allowed_file_types, :count
 
     def call
       {}

data/app/models/metadata_presenter/page_answers.rb

@@ -4,6 +4,7 @@ module MetadataPresenter
     include ActiveModel::Validations
     include ActionView::Helpers
     attr_reader :page, :answers, :uploaded_files, :autocomplete_items
+    attr_accessor :count
 
     def initialize(page, answers, autocomplete_items = nil)
       @page = page
@@ -28,7 +29,7 @@ module MetadataPresenter
       if component && component.type == 'date'
         date_answer(component.id)
       elsif component && component.type == 'upload'
-        upload_answer(component.id)
+        upload_answer(component.id, count)
       elsif component && component.type == 'checkboxes'
         answers[method_name.to_s].to_a
       else
@@ -36,15 +37,13 @@ module MetadataPresenter
       end
     end
 
-    def upload_answer(component_id)
+    def upload_answer(component_id, _count)
       file_details = answers[component_id.to_s]
 
       return {} unless file_details
 
       if file_details.is_a?(Hash) || file_details.is_a?(ActionController::Parameters)
-
         file_details.merge('original_filename' => sanitize(filename(file_details['original_filename'])))
-
       else
         {
           'original_filename' => sanitize(filename(file_details.original_filename)),
@@ -75,7 +74,16 @@ module MetadataPresenter
     def filename(path)
       return sanitize(path) if path.nil?
 
-      sanitize(path).gsub(/&gt;/, '').gsub(/&lt;/, '').delete('>"[]{}*?:|]/<').delete('\\')
+      filename = sanitize(path).gsub(/&gt;/, '').gsub(/&lt;/, '').delete('>"[]{}*?:|]/<').delete('\\')
+
+      if count.present? && count.positive?
+        extname = File.extname(filename)
+        basename = File.basename(filename, extname)
+
+        filename = "#{basename}-(#{count})#{extname}"
+      end
+
+      filename
     end
   end
 end

data/lib/metadata_presenter/version.rb

@@ -1,3 +1,3 @@
 module MetadataPresenter
-  VERSION = '2.19.4'.freeze
+  VERSION = '2.19.5'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: metadata_presenter
 version: !ruby/object:Gem::Version
-  version: 2.19.4
+  version: 2.19.5
 platform: ruby
 authors:
 - MoJ Forms
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-05-18 00:00:00.000000000 Z
+date: 2023-05-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: govuk_design_system_formbuilder
@@ -216,16 +216,16 @@ dependencies:
   name: site_prism
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement