← Back to team overview

mahara-contributors team mailing list archive

[Bug 1607564] A patch has been submitted for review

 

Patch for "master" branch: https://reviews.mahara.org/6831

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1607564

Title:
  ClamAV quarantine directory should not be a language string

Status in Mahara:
  In Progress

Bug description:
  While looking through Mahara's ClamAV code, I noticed that the name of
  the directory it tries to store infected files in, is controlled by a
  translatable language string:

      $quarantinedir = get_config('dataroot') .
  get_string('quarantinedirname');

  This is not a good idea, because it means that the location of these
  files will vary depending on what language packs are installed and
  what the current user's language setting is! Instead, it should be a
  hard-coded string or a config setting.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1607564/+subscriptions


References