basically we can't do a strpos on an empty string. so default to true (because it maybe part of an logical query) but before this is merged up an audit should be done of the other methods.
Coverage remained the same at 100.0% when pulling db67e2ab9988daa4975a2518772cf16ac7c34d28 on c-harris:master into f26d625704ccaec0888a83fbe1376b515f64933a on Algo-Web:master.
basically we can't do a strpos on an empty string. so default to true (because it maybe part of an logical query) but before this is merged up an audit should be done of the other methods.