It is a mistake with the diff file.  My original does not have the double explode line.

Ermal Luçi <notifications@github.com> wrote:

>> @@ -335,8 +335,10 @@ function snort_apply_customizations($snortcfg, $if_real) {
>>  		/* Create an array with the full path filenames of the enabled  */
>>  		/* rule category files if we have any.                          */
>>  		if (!empty($snortcfg['rulesets'])) {
>> -			foreach (explode("||", $snortcfg['rulesets']) as $file)
>> -				$enabled_files[] = "{$snortdir}/rules/" . $file;
>> +			foreach (explode("||", $snortcfg['rulesets']) as $file) {
>> +				if (file_exists())
>
>This is probably error no?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/bsdperimeter/pfsense-packages/pull/357/files#r2754584