Reviewed: https://reviews.mahara.org/6831 Committed: https://git.mahara.org/mahara/mahara/commit/6dd7339782eef8267cb3f0e30fff16fe211862ba Submitter: Aaron Wells ([email protected]) Branch: master
commit 6dd7339782eef8267cb3f0e30fff16fe211862ba Author: Robert Lyon <[email protected]> Date: Fri Aug 12 12:29:18 2016 +1200 Bug 1607564: Remove not needed quarantine dirs The old ones based on lang strings behatnotneeded - can't test dataroot Change-Id: I08963fd73f2d2102c9c34535593e8afb50b6104b Signed-off-by: Robert Lyon <[email protected]> -- 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: Fix Released 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 _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp
