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

Move CRM_Core_Session::setStatus calls out of CRM_Core_Tag_BAO layer

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Minor
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.4.0
    • Fix Version/s: 4.4.4
    • Component/s: CiviCRM API, Core CiviCRM
    • Labels:
      None

      Description

      When the Tag_BAO layer calls setStatus it removes the developer's control over status messages. Instead, the burden should always be placed on the calling context to set the appropriate status, if necessary.

      I ran into this when our AJAX call that used the civicrm_api to delete a tag was generating status messages on the next page (because the api in turn used the BAO layer). By the time the user loads another page, these message are no longer relevant and can become confusing.

        Attachments

          Activity

            People

            • Assignee:
              colemanw Coleman Watts
              Reporter:
              shadesofgraylin Graylin Kim
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: