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

Code simplification - remove unused params from signatures

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.7
    • Fix Version/s: 4.7
    • Component/s: None
    • Labels:
      None
    • Documentation Required?:
      None
    • Funding Source:
      Contributed Code

      Description

      CRM_Core_BAO_CustomField::postProcess
      &
      CRM/Core/BAO/CustomValueTable:;postProcess
      both take the value $customFields & then completely ignore it!

      Let's get this out of the function signature & simplify

      https://github.com/civicrm/civicrm-core/pull/5804/files

      Would be good if someone could check I haven't missed something. NB drupal 6 & 7 also need civiengage fix WRT this function - after merge

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: