Reviewed: https://reviews.mahara.org/6757 Committed: https://git.mahara.org/mahara/mahara/commit/00563fc20dbb7ad7b6e96d52943d11b347861b62 Submitter: Son Nguyen ([email protected]) Branch: master
commit 00563fc20dbb7ad7b6e96d52943d11b347861b62 Author: Aaron Wells <[email protected]> Date: Fri Jul 29 12:03:11 2016 +1200 Bug 1607564: system paths shouldn't be based on a lang string behatnotneeded: IOU a behat test for clamav Change-Id: I43a65b7be7490bbcb13fa8885ca119c130856a98 -- 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 Committed 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
