Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-17750

Move BAO_FInancialTrxn to under CRM_Financial

    Details

    • Type: Improvement
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Won't Fix
    • Affects Version/s: 4.7
    • Fix Version/s: Unscheduled
    • Component/s: None
    • Labels:
      None
    • Versioning Impact:
      Patch (backwards-compatible bug fixes)
    • Documentation Required?:
      None
    • Funding Source:
      Needs Funding

      Description

      We really need to change

      CRM_Core_BAO_FinancialTrxn
      to
      CRM_Financial_BAO_FInancialTrxn

      currently

      CRM_Core_BAO_FinancialTrxn extends CRM_Financial_DAO_FinancialTrxn

      The impact of this is confusion + possibly future cleanup hassle + the probability the BAO is currently being bypassed by the api.

      There are only 40 refs so it's an easy find & replace but if payment code is still being merged it could require conflict resolution

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              eileen Eileen McNaughton
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: