runei Posted October 10, 2008 Share Posted October 10, 2008 Hello. Quick question about the following statement. Is this code fetching id's from the first statement and matching them to the admin id in the second statement where the id of admin in the second statement matches cid in the 'forum_topics' table????? <?php //*Protecting through mss $id = mss($_GET['id']); if($id){ $sql = "SELECT * FROM `forum_topics` WHERE `id`='".$id."'"; $res = mysql_query($sql) or die (mysql_error()); //returns number of rows in the database corresponding to the query if(mysql_num_rows($res) == 0){ echo "This topic does not exists"; }else{ $row = mysql_fetch_assoc($res); $sql2 = "SELECT `admin` FROM `forum_sub_cats` WHERE `id`='".$row['cid']."'"; $res2 = mysql_query($sql2) or die(mysql_error()); $row2 = mysql_fetch_assoc($res2); if($row2['admin'] == 1 && $admin_user_level == 0){ echo "You do not have priveliges for this action"; }else...?> Quote Link to comment https://forums.phpfreaks.com/topic/127862-rowcidneed-clarification/ Share on other sites More sharing options...
Maq Posted October 10, 2008 Share Posted October 10, 2008 Looks like it is but why don't you echo something out to make sure. Quote Link to comment https://forums.phpfreaks.com/topic/127862-rowcidneed-clarification/#findComment-661982 Share on other sites More sharing options...
Stryves Posted October 10, 2008 Share Posted October 10, 2008 1. It's checking forum_topics table for the topic with the ID. If it does exist 2. It checks to see who has admin access to the forum_sub_cats table, as you can assign multiple moderators. I think. Quote Link to comment https://forums.phpfreaks.com/topic/127862-rowcidneed-clarification/#findComment-661985 Share on other sites More sharing options...
budimir Posted October 10, 2008 Share Posted October 10, 2008 Well, this actually isn't making a lot of sence. It's working as you asked... There are simplier ways to do this.. Quote Link to comment https://forums.phpfreaks.com/topic/127862-rowcidneed-clarification/#findComment-661988 Share on other sites More sharing options...
Maq Posted October 10, 2008 Share Posted October 10, 2008 Yes, you should actually join these tables and have 2 condition statements so you only need 1 query. Quote Link to comment https://forums.phpfreaks.com/topic/127862-rowcidneed-clarification/#findComment-661991 Share on other sites More sharing options...
runei Posted October 10, 2008 Author Share Posted October 10, 2008 Im doing a cms online tut and this is part of the code. Its rather confusing and im trying to get my head around what i actually does. Thx for ur replies Quote Link to comment https://forums.phpfreaks.com/topic/127862-rowcidneed-clarification/#findComment-661996 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.