Subversion Repositories SmartDukaan

Last modification

• Rev 37458 2026-08-29 13:22:56
• Author: aman

• Log message:

  LOI payment screenshot: report upload failures instead of reporting the field as missing
  
  Submit validated the hidden #payment-sc-docId, but a file input keeps displaying the
  chosen filename regardless of what happens next - so a failed /document-upload left the
  field looking attached to the operator and missing to the validator ("The following
  fields are required: Payment screenshot" on a form that was in fact filled in).
  
  The failure was silent three ways: doAjaxUploadRequestHandler had no error handler; the
  global ajaxError reporter calls bootbox, which loads from cdnjs and throws if that is
  blocked; and the confirm('Confirm file upload ?') gate returned false with no trace once
  Chrome's "prevent this page from creating additional dialogs" box was ticked.
  
  - common.js: optional error callback on doAjaxUploadRequestHandler (additive, opts out of
  the generic global dialog via suppressGlobalError); uploadErrorMessage() extracts the
  server's real reason from the JSON body or the plain-text duplicate-request body;
  uploadDocument() pre-checks size/type, verifies a document_id actually came back, and
  reports a message naming the file - .HEIC and .webp get specific guidance;
  showAlert() falls back to native alert when bootbox is absent.
  - loi-form.js: drop the confirm() gate, 'input' -> 'change', and clear the file input
  whenever the id does not land so the widget can never claim an unuploaded file. Status
  line under the field (Uploading... / Attached: name / Not attached). Submit now
  distinguishes not-picked, still-uploading and upload-failed. Same treatment for the
  OnboardingDocumentTable and mk_brand-fee-collection-details uploads, and the brand-fee
  dialog names the Attachment field instead of echoing the raw form key.
  - loi-form.vm, pendingForm.vm: accept="..." on both file inputs plus the status span.
  - FileUploadController: reject empty files, report actual size against the actual limit,
  and resolveExtension() falls back to the filename when the browser sends a generic MIME
  type (Android file managers send application/octet-stream for ordinary JPG/PNG/PDF).
  - AppConfig: static resource version 395 -> 396 for the JS change.