Closed (fixed)
Project:
Pathologic
Version:
7.x-2.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2012 at 19:32 UTC
Updated:
23 Sep 2012 at 06:33 UTC
This warning is caused by my list of "All base paths for this site" ending with an empty line. I think you should better check on !empty() instead of isset() (which checks on null, not the empty string).
Comments
Comment #1
fietserwinHmmm, It is not an empty line, it is ending with a new line (which should be allowed). So I guess that explode() (correctly) explodes the string "/mypath\r\n" into an array with 2 entries: /mypath (without \r after trim() and the empty string. Anyway, IMO it would be better better to ignore empty strings in the for each loop in _pathologic_filter().
Comment #2
afoster commentedI got this error when I had an empty line in the pathologic settings on my input format. It was installed with features so I think that's how an empty line was saved in there. Removing the empty line stopped the error.
Comment #3
Garrett Albright commentedThanks for the heads up. This was an easy one to fix. I'll try to get a release with the fix out by the end of the day.
Comment #4
Garrett Albright commentedI didn't get it done yesterday, but I just did now. Please give it a download - it should make this bug disappear.