[Resolved] Very basic Php question

[baris22]

very basic php question

Hello,

I have got this code

if($result == '0') {
// 0---
	if(!ctype_alnum($tag)) {
		$info_err_msg = '<div class="info_msg_err">Please make sure that the tags are typed properly (alfa-numeric only).</div>';
	} else {
		mysql_query("INSERT INTO pm_categories SET tag = '".$tag."', name = '".$name."'");
		$info_msg = '<div class="info_msg_ok">Category <b>'.$name.'</b> has been added to the database.</div>';
	}
// 0---
} else {
		$info_msg = '<div class="info_msg_err">The category you submitted is duplicated! The \'TAG\' field must be unique.</div>';
}

}

I do not want to use

if(!ctype_alnum

How can i remove it without breaking the code?

thanks

 

[Barrucadu]

How can i remove it without breaking the code?

I'm not entirely sure what you mean by 'without breaking the code', removing that if statement changes what the code does...

Do you mean this?:

if($result == "0")
    { 
        if(ctype_alnum($tag))
            { 
                mysql_query("INSERT INTO pm_categories SET tag = \"{$tag}\", name = \"{$name}\""); 
                $info_msg = "<div class=\"info_msg_ok\">Category <strong>{$name}</strong> has been added to the database.</div>"; 
            } 
    }
else
    { 
        $info_msg = "<div class=\"info_msg_err\">The category you submitted is duplicated! The 'TAG' field must be unique.</div>"; 
    } 

}

You also had one extra '}' at the end - I removed it.

 

[baris22]

Thank you for your help,

I do not want to use ctype_alnum on the code. Because i want to display "-" in my tag.

I just want to disaible ctype_alnum but i still want the rest.


thanks

I'm not entirely sure what you mean by 'without breaking the code', removing that if statement changes what the code does...

Do you mean this?:

if($result == "0")
    { 
        if(ctype_alnum($tag))
            { 
                mysql_query("INSERT INTO pm_categories SET tag = \"{$tag}\", name = \"{$name}\""); 
                $info_msg = "<div class=\"info_msg_ok\">Category <strong>{$name}</strong> has been added to the database.</div>"; 
            } 
    }
else
    { 
        $info_msg = "<div class=\"info_msg_err\">The category you submitted is duplicated! The 'TAG' field must be unique.</div>"; 
    } 

}

You also had one extra '}' at the end - I removed it.

 

[Barrucadu]

This?

if($result == "0")
    { 
        if(!ctype_alnum($tag))
            {
                //$info_err_msg = '<div class="info_msg_err">Please make sure that the tags are typed properly (alfa-numeric only).</div>';
            }
        else
            {
                mysql_query("INSERT INTO pm_categories SET tag = \"{$tag}\", name = \"{$name}\""); 
                $info_msg = "<div class=\"info_msg_ok\">Category <strong>{$name}</strong> has been added to the database.</div>"; 
            } 
    }
else
    { 
        $info_msg = "<div class=\"info_msg_err\">The category you submitted is duplicated! The 'TAG' field must be unique.</div>"; 
    } 

}

Or this?

if($result == "0")
    { 
        /*if(!ctype_alnum($tag))
            {
                $info_err_msg = '<div class="info_msg_err">Please make sure that the tags are typed properly (alfa-numeric only).</div>';
            }
        else
            {
                mysql_query("INSERT INTO pm_categories SET tag = \"{$tag}\", name = \"{$name}\""); 
                $info_msg = "<div class=\"info_msg_ok\">Category <strong>{$name}</strong> has been added to the database.</div>"; 
            }*/
    }
else
    { 
        $info_msg = "<div class=\"info_msg_err\">The category you submitted is duplicated! The 'TAG' field must be unique.</div>"; 
    } 

}

 

[Joe]

It's presume the following:
(as he probably still wants it to insert into the database.)

if($result == "0")
    { 
                mysql_query("INSERT INTO pm_categories SET tag = \"{$tag}\", name = \"{$name}\""); 
                $info_msg = "<div class=\"info_msg_ok\">Category <strong>{$name}</strong> has been added to the database.</div>"; 
            }
    }
else
    { 
        $info_msg = "<div class=\"info_msg_err\">The category you submitted is duplicated! The 'TAG' field must be unique.</div>"; 
    } 
}

 

[Peter]

I believe he means:-

if($result == '0') {
// 0---
    if(!preg_match('/^[-a-zA-Z0-9]+$/', $tag)) {
        $info_err_msg = '<div class="info_msg_err">Please make sure that the tags are typed properly (alfa-numeric only).</div>';
    } else {
        mysql_query("INSERT INTO pm_categories SET tag = '".$tag."', name = '".$name."'");
        $info_msg = '<div class="info_msg_ok">Category <b>'.$name.'</b> has been added to the database.</div>';
    }
// 0---
} else {
        $info_msg = '<div class="info_msg_err">The category you submitted is duplicated! The \'TAG\' field must be unique.</div>';
}

}

 

[baris22]

Waww, thank you very much.
it was exactly what i wanted.

Thank you

Thank you all for the reply.

I believe he means:-

if($result == '0') {
// 0---
    if(!preg_match('/^[-a-zA-Z0-9]+$/', $tag)) {
        $info_err_msg = '<div class="info_msg_err">Please make sure that the tags are typed properly (alfa-numeric only).</div>';
    } else {
        mysql_query("INSERT INTO pm_categories SET tag = '".$tag."', name = '".$name."'");
        $info_msg = '<div class="info_msg_ok">Category <b>'.$name.'</b> has been added to the database.</div>';
    }
// 0---
} else {
        $info_msg = '<div class="info_msg_err">The category you submitted is duplicated! The \'TAG\' field must be unique.</div>';
}

}