You are correct.  I noticed it after I had sent the reply, but since Ermal fixed it anyway, I did not send a follow up.

Sorry about that,
Bill


Erik Fonnesbeck <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())
>
>I think you are misreading the diff.  There is only one explode that I see.  I'm fairly sure he's talking about line 339 with the "file_exists()" with no parameters.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/bsdperimeter/pfsense-packages/pull/357/files#r2765759