CRM-19660 Financial Transaction stores wrong transaction date if the contribution is cancelled

          Details

          • Type: Bug
          • Status: Done/Fixed
          • Priority: Minor
          • Resolution: Fixed/Completed
          • Affects Version/s: 4.7.14
          • Fix Version/s: 4.7.15
          • Component/s: CiviContribute
          • Labels:
            None
          • Versioning Impact:
            Minor (add functionality in backwards-compatible manner)
          • Documentation Required?:
            None
          • Funding Source:
            Core Team Funds
          • Verified?:
            No

            Description

            Steps to replicate:

            1. Create a contribution with status completed.

            2. Edit contribution with status cancelled and cancelled date as any other date than transaction date above.

            3. Financial transaction trnx_date does NOT reflect the trxn_date as cancelled date.

              Attachments

                Activity

                [CRM-19660] Financial Transaction stores wrong transaction date if the contribution is cancelled
                Jitendra Purohit added a comment -

                Submitted a PR - https://github.com/civicrm/civicrm-core/pull/9405

                Yashodha Chaku added a comment -

                Pradeep Nayak Joe Murray Can you guys confirm it's a bug?

                Pradeep Nayak added a comment -

                Joe Murray as I recall you had said few days back that civicrm_financial_trxn.trxn_date should be always now().

                Eileen McNaughton added a comment -

                No - it can't always be 'now' because people don't always enter data 'as it happens' - it would need to reflect the entered date.

                I'm curious as to how this one hit the core team's plate - is it a sponsored bug - or just something that got noticed while QA-ing something else?

                Yashodha Chaku added a comment -

                Eileen McNaughton I happened to come across this while I was checking something else. It looked buggy hence filed an issue so that we keep track.

                Eileen McNaughton added a comment -

                Thanks Yashi

                  People

                  • Assignee:
                    Jitendra Purohit
                    Reporter:
                    Yashodha Chaku

                    Dates

                    • Created:
                      Updated:
                      Resolved: