Here is a patch for my proposed fix. This was done on version 19.04.0_RELEASE as that's the version we will be deploying next. This is my first patch attempt... hopefully I've done everything correctly, it was a pretty simple fix.
** Patch added: "Bug_1831414_patch.diff" https://bugs.launchpad.net/mahara/+bug/1831414/+attachment/5268857/+files/Bug_1831414_patch.diff -- 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/1831414 Title: Skin import gets virus scanned when virus checking is disabled Status in Mahara: Triaged Bug description: Mahara version: 18.04.1 Importing a skin still gets virus checked if ClamAV path is configured and virus checking is disabled. Looking at the code it seems like it should check before lines 265 and 327 of htdocs/skin/import.php. To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1831414/+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
