[RESOLVED] function run on 2 strings?

[jasondavis]

My function below returns
$spam = 0; or $spam = 1;

After I run it on my subject and bulletin string if bulletin has bad text it marks as 1 correctly but if bulletin comes as 0 then subject could be set as 1 but then bulletin turns it back into 0

How can I make this work for both where the result of 1 will not return the wrong value, like if either string has the term vote then I need to return $spam=1

<?
function bulletinspam($str){
if (preg_match("/(votes|vote|v o t e s|v o t e|rate|rates|rating|r a t e)/i", $str)) {
        $spam = 1;
    } else {
        $spam = 0;	
	}
    return $spam;
} 

$spam=bulletinspam($subject);
$spam=bulletinspam($bulletin);

?>

[jasondavis]

think I got it now

if($spam1=='1' || $spam2=='1'){
				$spam = 1;
				} else {
				$spam = 0;

[laserlight]

I suggest relying explicitly on the boolean type instead of 1 and 0.

<?php

function bulletinspam($str) {
    $pattern = '/(votes|vote|v o t e s|v o t e|rate|rates|rating|r a t e)/i';
    return preg_match($pattern, $str) > 0;
}

$has_spam = bulletinspam($subject) || bulletinspam($bulletin);

?>

[jasondavis]

Ill try that, im having trouble on the display now

SELECT auto_id, subject, submit_date, user_id, color, spam
FROM friend_bulletin
WHERE user_id = '1'
OR user_id
IN ( 345, 564, 644, 45645, 434 )
AND spam =1
ORDER BY submit_date DESC
LIMIT 0 , 5

returns bulletin post with a spam value of 0 and 1 but im not sure why

[laserlight]

Sounds like you want something like:

SELECT auto_id, subject, submit_date, user_id, color, spam
    FROM friend_bulletin
    WHERE user_id IN ( 1, 345, 564, 644, 45645, 434 )
        AND spam=1
    ORDER BY submit_date DESC
    LIMIT 0, 5

[jasondavis]

thanks