Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-1423

Voiding partially-paid proforma invoices should cause an error

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 3.3.0-b1
    • Fix Version/s: 3.3.0
    • Component/s: None
    • Labels:
      None

      Description

      When voiding an invoice, the action should only be performed if the invoice does not have payments applied to it. This is how normal invoices currently work. They will display an error when attempting to void them when payments have been applied.

      However, when proforma invoices are voided, no error is shown when partial payments have already been applied to them, and the invoice is successfully voided. Attempting then to unvoid the invoice causes an error that the invoice may not be updated when it has payments applied to it. The same error should be shown when attempting to void the invoice.

        Issue Links

          Activity

          tyson Tyson Phillips (Inactive) created issue -
          tyson Tyson Phillips (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to CORE-1398 [ CORE-1398 ]
          Hide
          tyson Tyson Phillips (Inactive) added a comment -

          Looks like when the invoice is proforma, and it is updated, the rule that validates whether amounts have been applied to it is purposefully ignored. I'm thinking this should probably be updated to not ignore the rule if the new status is 'void'. Thoughts?

          Show
          tyson Tyson Phillips (Inactive) added a comment - Looks like when the invoice is proforma, and it is updated, the rule that validates whether amounts have been applied to it is purposefully ignored. I'm thinking this should probably be updated to not ignore the rule if the new status is 'void'. Thoughts?
          tyson Tyson Phillips (Inactive) made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          tyson Tyson Phillips (Inactive) made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              cody Cody Phillips (Inactive)
              Reporter:
              tyson Tyson Phillips (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                24/Sep/14