SitePoint Sponsor |
|
User Tag List
Results 1 to 8 of 8
-
Apr 30, 2007, 14:39 #1
- Join Date
- Aug 2006
- Location
- earth
- Posts
- 923
- Mentioned
- 0 Post(s)
- Tagged
- 0 Thread(s)
Problem with some if/elseif conditionals
Hi,
I'm echoing out $Rversion, which right now is giving me 3.
PHP Code:echo $Rversion;
if($Rversion=="1")
{
$SRupd = "UPDATE sales_reps SET stage='2' status='Unclaimed' WHERE sr_id = '$sr' ";
mysql_query($SRupd);
}
elseif($Rversion=="2")
{
$SRupd = "UPDATE sales_reps SET stage='5' status='Unclaimed' WHERE sr_id = '$sr' ";
mysql_query($SRupd);
}
elseif($Rversion=="3")
{
$SRupd = "UPDATE sales_reps SET stage='6' status='Completed' WHERE sr_id = '$sr' ";
mysql_query($SRupd);
}
What am I doing wrong here? Thanks for any help.No, I REALLY dislike having to use Joomla.
-
Apr 30, 2007, 14:47 #2
- Join Date
- Aug 2004
- Location
- Manchester UK
- Posts
- 13,807
- Mentioned
- 158 Post(s)
- Tagged
- 3 Thread(s)
Personally I would use a switch() statement instead of if else:
PHP Code:
switch($Rversion) {
case 1:
$SRupd = "UPDATE sales_reps SET stage='2' status='Unclaimed' WHERE sr_id = '$sr' ";
break;
case 2:
$SRupd = "UPDATE sales_reps SET stage='5' status='Unclaimed' WHERE sr_id = '$sr' ";
break;
case 3:
$SRupd = "UPDATE sales_reps SET stage='6' status='Completed' WHERE sr_id = '$sr' ";
break;
}
mysql_query($SRupd);
Mike Swiffin - Community Team Advisor
Only a woman can read between the lines of a one word answer.....
-
Apr 30, 2007, 14:52 #3
- Join Date
- Feb 2006
- Posts
- 49
- Mentioned
- 0 Post(s)
- Tagged
- 0 Thread(s)
try this !
try this , must work
PHP Code:echo $Rversion;
if($Rversion=="1")
{
$SRupd = "UPDATE sales_reps SET stage='2' status='Unclaimed' WHERE sr_id = '$sr' ";
mysql_query($SRupd);
}
if($Rversion=="2")
{
$SRupd = "UPDATE sales_reps SET stage='5' status='Unclaimed' WHERE sr_id = '$sr' ";
mysql_query($SRupd);
}
elseif($Rversion=="3")
{
$SRupd = "UPDATE sales_reps SET stage='6' status='Completed' WHERE sr_id = '$sr' ";
mysql_query($SRupd);
}
-
Apr 30, 2007, 14:56 #4
- Join Date
- Aug 2006
- Location
- earth
- Posts
- 923
- Mentioned
- 0 Post(s)
- Tagged
- 0 Thread(s)
spike - Thanks for the suggestion, but it's still not working.
No, I REALLY dislike having to use Joomla.
-
Apr 30, 2007, 14:58 #5
- Join Date
- Aug 2006
- Location
- earth
- Posts
- 923
- Mentioned
- 0 Post(s)
- Tagged
- 0 Thread(s)
-
Apr 30, 2007, 15:03 #6
- Join Date
- Aug 2006
- Location
- earth
- Posts
- 923
- Mentioned
- 0 Post(s)
- Tagged
- 0 Thread(s)
crap, nevermind.
It wasn't my conditionals it was my sql.
See? No commas...
Thanks for the help anyway guys.No, I REALLY dislike having to use Joomla.
-
Apr 30, 2007, 16:15 #7
- Join Date
- Feb 2006
- Posts
- 49
- Mentioned
- 0 Post(s)
- Tagged
- 0 Thread(s)
no problem , ur welcome
-
May 1, 2007, 00:23 #8
- Join Date
- Aug 2004
- Location
- Manchester UK
- Posts
- 13,807
- Mentioned
- 158 Post(s)
- Tagged
- 3 Thread(s)
LOL, would still go for a switch though
Mike Swiffin - Community Team Advisor
Only a woman can read between the lines of a one word answer.....
Bookmarks