CRM-10712 Make it easy to view case activity attachment

    Details

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

      Description

      Our user finds it difficult to know which case activities have attachments when looking at the manage case screen. I have made this easier for her by adding a 'view attachment' link where an attachment has one activity.

      I had hoped to also add an icon column to indicate where activities have one or more attachments (above only works for exactly one attachment) but although I could add a column to the JSON I couldn't figure out where to change the column headers so will log this in case someone else wants to pick it up & take it further.

      https://github.com/fuzionnz/civicrm/commit/0bb65ff8c28b905148c5a127349cd4f71afdcbf7

        Attachments

          Activity

          [CRM-10712] Make it easy to view case activity attachment
          Eileen McNaughton added a comment -

          I've committed the code as discussed - note that it adds 'no_attachments' to the calling function - this could be changed without affecting anything (it does use that value in an if clause but only in one place directly below where it is assigned)

          Coleman Watts added a comment -

          Went to check this out but cannot load case activities via ajax. Error msg is:
          Database Error Code: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'OUTER JOIN civicrm_option_group og ON og.name="activity_status"
          CRM/Case/BAO/Case.php at line 1064

          Eileen McNaughton added a comment -

          Yeah odd - the end of the line wasn't in the commit

            People

            • Assignee:
              Coleman Watts
              Reporter:
              Eileen McNaughton

              Dates

              • Created:
                Updated:
                Resolved: