Debugging AcroForm warnings

config/features.yml

@@ -37,6 +37,9 @@ features:
   accredited_representative_portal_skip_itf_check:
     actor_type: user
     description: If enabled, skip existing ITF check in ARP
+  acroform_debug_logs:
+    actor_type: user
+    description: When enabled, AcroForm problematic fields will report in Rails logs
   aedp_vadx:
     actor_type: user
     description: Enables the VADX experimental features in the AEDP application

lib/pdf_fill/filler.rb

@@ -205,13 +205,14 @@ def fill_ancillary_form(form_data, claim_id, form_id, fill_options = {})
     #
     # @return [None]
     #
+
     def fill_form_with_hexapdf(template_path, output_path, hash_data)
+      Rails.logger.info("PdfFill::Filler HexaPDF template: #{template_path}") if Flipper.enabled?(:acroform_debug_logs)
       doc = HexaPDF::Document.open(template_path)
       form = doc.acro_form
       raise 'No AcroForm found in PDF template.' if form.nil?
 
       form.fill(hash_data)
-
       doc.write(output_path)
     end
 

modules/decision_reviews/lib/decision_reviews/pdf_template_stamper.rb

@@ -64,6 +64,10 @@ def build_form_data(data)
     end
 
     def fill_and_write_pdf(form_data, output_path, email_delivery_failure)
+      if Flipper.enabled?(:acroform_debug_logs)
+        Rails.logger.info("DecisionReviews::PdfTemplateStamper #{@template_path}")
+      end
+
       doc = HexaPDF::Document.open(@template_path)
 
       fill_form_fields(doc, form_data, email_delivery_failure)

modules/increase_compensation/lib/increase_compensation/pdf_stamper.rb

@@ -43,7 +43,10 @@ def self.stamp_signature(pdf_path, form_data)
     # @param pdf_path [String] Path to the PDF template
     # @return [Hash, nil] Coordinates hash of the form
     #   `{ x: Float, y: Float, page_number: Integer }` or nil on failure
-    def self.signature_overlay_coordinates(pdf_path = TEMPLATE)
+    def self.signature_overlay_coordinates(pdf_path = TEMPLATE) # rubocop:disable Metrics/MethodLength
+      if Flipper.enabled?(:acroform_debug_logs)
+        Rails.logger.info("IncreaseCompensation::PdfStamper HexaPDF template: #{pdf_path}")
+      end
       doc = HexaPDF::Document.open(pdf_path)
       return unless doc.validate(auto_correct: false)
 
@@ -71,7 +74,7 @@ def self.signature_overlay_coordinates(pdf_path = TEMPLATE)
     end
 
     def self.signature_text_for(form_data)
-      form_data['statementOfTruthSignature'].presence ||
+      form_data['statement_of_truth_signature'].presence ||
         form_data['signature'].presence ||
         veteran_full_name(form_data)
     end

modules/medical_expense_reports/lib/medical_expense_reports/pdf_fill/va21p8416.rb

@@ -150,9 +150,30 @@ def merge_fields(_options = {})
       # @param pdf_path [String] Path to the PDF template
       # @return [Hash, nil] Coordinates hash of the form
       #   `{ x: Float, y: Float, page_number: Integer }` or nil on failure
-      def self.signature_overlay_coordinates(pdf_path)
+      def self.signature_overlay_coordinates(pdf_path) # rubocop:disable Metrics/MethodLength
+        if Flipper.enabled?(:acroform_debug_logs)
+          Rails.logger.info("MedicalExpenseReports::PdfFill::Va21p8416 HexaPDF template: #{pdf_path}")
+        end
+
         doc = HexaPDF::Document.open(pdf_path)
-        field = doc.acro_form&.field_by_name(SIGNATURE_FIELD_NAME)
+        form = doc.acro_form
+        raise 'No AcroForm found in PDF template.' if form.nil?
+
+        if Flipper.enabled?(:acroform_debug_logs)
+          # Find problematic fields
+          problematic_fields = form.each_field.select do |field|
+            field.field_type == :Btn && field[:AP].nil?
+          end
+
+          if problematic_fields.any?
+            Rails.logger.warn("Template #{template_path} has #{problematic_fields.size} fields missing AP")
+            problematic_fields.each do |field|
+              Rails.logger.warn("  - Field: #{field.full_field_name}")
+            end
+          end
+        end
+
+        field = form&.field_by_name(SIGNATURE_FIELD_NAME)
         widget = field&.each_widget&.first
         return unless widget
 

modules/survivors_benefits/lib/survivors_benefits/pdf_fill/va21p534ez.rb

@@ -155,6 +155,10 @@ def self.signature_overlay_coordinates(pdf_path = TEMPLATE)
       end
 
       def self.signature_overlay_coordinates_for(pdf_path)
+        if Flipper.enabled?(:acroform_debug_logs)
+          Rails.logger.info("SurivorsBenefits::PdfFill::Va21p534ez HexaPDF template: #{pdf_path}")
+        end
+
         HexaPDF::Document.open(pdf_path) do |doc|
           field = doc.acro_form&.field_by_name(SIGNATURE_FIELD_NAME)
           widget = field&.each_widget&.first