Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-50434

tool_messageinbound: inform_attachment_virus is not defined

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Reopened
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 2.8.6, 2.9
    • Fix Version/s: None
    • Component/s: Libraries
    • Testing Instructions:
      Hide

      The following instructions assume that you have setup messageinbound, and have the forum handler enabled.
      They also assume that you have setup and configured clamd + clamdscan on your system.
      Don't forget to specify the following in your config.php:

      $CFG->pathtoclam = '/usr/local/bin/clamdscan'; // Or wherever it is installed on you system
      $CFG->runclamonupload = true;
      

      1. Create a forum post and tick the 'Mail now' checkbox
      2. Run cron
      3. Check your e-mail
      4. Reply to the message you will have received about your new forum post
      5. Include the EICAR test file in your reply (http://www.eicar.org/86-0-Intended-use.html)
      6. Run cron
      7. Ensure that you were sent a notice about your infection
      8. Reply again, but this time with a non-infected file
      9. Run cron
      10. Ensure that the message was posted this time
      Show
      The following instructions assume that you have setup messageinbound, and have the forum handler enabled. They also assume that you have setup and configured clamd + clamdscan on your system. Don't forget to specify the following in your config.php: $CFG->pathtoclam = '/usr/local/bin/clamdscan'; // Or wherever it is installed on you system $CFG->runclamonupload = true; Create a forum post and tick the 'Mail now' checkbox Run cron Check your e-mail Reply to the message you will have received about your new forum post Include the EICAR test file in your reply ( http://www.eicar.org/86-0-Intended-use.html ) Run cron Ensure that you were sent a notice about your infection Reply again, but this time with a non-infected file Run cron Ensure that the message was posted this time
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull Master Branch:
      MDL-50434-master

      Description

      Was taking a look at this new plugin and found that the tool_messageinbound\manager->inform_attachment_virus is not defined. Here is the line: https://github.com/moodle/moodle/blob/master/admin/tool/messageinbound/classes/manager.php#L657

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              dobedobedoh Andrew Nicols
              Reporter:
              bushido Mark Nielsen
              Peer reviewer:
              Simey Lameze
              Integrator:
              Dan Poltawski
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              Votes:
              1 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated: