vb.org Archive
Page 2 of 3 1 2 3
Show 40 post(s) from this thread on one page

vb.org Archive (https://vborg.vbsupport.ru/index.php)
-   vB3 Programming Discussions (https://vborg.vbsupport.ru/forumdisplay.php?f=15)
-   -   conditional won't work together but work's split up? (https://vborg.vbsupport.ru/showthread.php?t=78157)

sabret00the 03-15-2005 02:41 PM
What the code should do is:
If they are NOT original_poster OR are NOT a_moderator: Error message
(that's simplified cos i left out the check to make sure they have permission to edit their own posts)

Marco van Herwaarden 03-15-2005 02:42 PM
Let's do this again
PHP Code:
if (!$allow_edit and !$allow_edit2) 
is wrong, should be:
PHP Code:
if (!($allow_edit and $allow_edit2)) 
What you want is that both conditions are true, must be the poster and must have permissions to edit. If this combination is not true (as opposed to the 2 seperate conditions being not true) then you want an error.

sabret00the 03-15-2005 02:50 PM
that didn't work so let's simplify the original code
PHP Code:
if (($postinfo['userid'] != $bbuserinfo[userid] AND !$grps_permissions['groupspostedit']) OR (!grps_permissions['groupsmoderater']) 
simplified that would be

PHP Code:
if ($clause_one OR $clause_two) 
so basically i want if either of clause_one or clause_two = true then let the person edit the post, else show a no permission screen.

Marco van Herwaarden 03-15-2005 02:51 PM
Now let's add the moderator part. Let's simplify by making the above into 1 condition, so:
PHP Code:
if ( !(canedit) ) {error} 
Now let's add the moderator with an OR statement:
PHP Code:
if ( !(canedit) OR !(moderator) ) {error} 
Possible combinations:
1. Can NOT edit, is NOT moderator, this will give:
PHP Code:
if ( TRUE OR TRUE) {error} 
Error is shown, this is correct.

2. Can edit, is NOT moderator, this will give:
PHP Code:
if ( FALSE OR TRUE) {error} 
Since you are using an OR statement only 1 of the conditions need to be TRUE. This is the case, so an error. This is not what you want.

3. Can NOT edit, is moderator, this will give:
PHP Code:
if ( TRUE OR FALSE) {error} 
Since you are using an OR statement only 1 of the conditions need to be TRUE. This is the case, so an error. This is not what you want.

Now let's see the same with an AND:
PHP Code:
if ( !(canedit) AND !(moderator) ) {error} 

1. Can NOT edit, is NOT moderator, this will give:
PHP Code:
if ( TRUE AND TRUE) {error} 
Error is shown, this is correct.

2. Can edit, is NOT moderator, this will give:
PHP Code:
if ( FALSE AND TRUE) {error} 
Since you are using an AND statement, both of the conditions need to be TRUE. This is NOT the case, so NO error. This is what you want.

3. Can NOT edit, is moderator, this will give:
PHP Code:
if ( TRUE AND FALSE) {error} 
Since you are using an AND statement, both of the conditions need to be TRUE. This is NOT the case, so NO error. This is what you want.

Quote:
Originally Posted by sabret00the
so basically i want if either of clause_one or clause_two = true then let the person edit the post, else show a no permission screen.
This is not what you are doing: You are showing an error if either of them is true

sabret00the 03-15-2005 02:59 PM
ok so how do i get it to show

PHP Code:
if ((canedit) AND !(moderator)) {no_error} 
then have
PHP Code:
if (!(canedit) AND (moderator)) {no_error} 
but if
PHP Code:
if (!(canedit) OR !(moderator)) {error} 

Marco van Herwaarden 03-15-2005 03:18 PM
Lol now you are confusing me.

What you will want is:
PHP Code:
if (!(canedit) AND !(moderator)) {error} 
Or you can turn the whole game upside down:
PHP Code:
if ((canedit) OR (moderator)) {DO EDIT} 

Jolten 03-15-2005 03:35 PM
Talk about over-thinking things :)

PHP Code:
if ((($postinfo['userid'] != $bbuserinfo[userid]) AND !$grps_permissions['groupspostedit']) AND !$grps_permissions['groupsmoderater']) 
{ 
    print_no_permission(); 
} 
(((user does not equal poster) AND not in allowed to edit group) AND is not moderator)

Marco van Herwaarden 03-15-2005 03:59 PM
Quote:
Originally Posted by Jolten
Talk about over-thinking things :)

PHP Code:
if ((($postinfo['userid'] != $bbuserinfo[userid]) AND !$grps_permissions['groupspostedit']) AND !$grps_permissions['groupsmoderater']) 
{ 
    print_no_permission(); 
} 
(((user does not equal poster) AND not in allowed to edit group) AND is not moderator)
I gree, but he just didn't want to get it ;)

sabret00the 03-15-2005 06:08 PM
win for the overthinking :p your code never worked :(

but

PHP Code:
if (!$grps_permissions['groupsmoderater'])
{
    //they'll only get in here if they're not a mod
    if ($postinfo['userid'] != $bbuserinfo[userid] AND !$grps_permissions['groupspostedit'])
    {
        print_no_permission();
    }
} 
is the working code :)

Marco van Herwaarden 03-15-2005 06:58 PM
So you give up? I am sure one of my codes worked ;)


All times are GMT. The time now is 11:12 PM.
Page 2 of 3 1 2 3
Show 40 post(s) from this thread on one page

Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.

X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.02024 seconds
  • Memory Usage 1,773KB
  • Queries Executed 10 (?)
More Information
Template Usage:
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (21)bbcode_php_printable
  • (2)bbcode_quote_printable
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)pagenav
  • (1)pagenav_curpage
  • (2)pagenav_pagelink
  • (1)post_thanks_navbar_search
  • (1)printthread
  • (10)printthreadbit
  • (1)spacer_close
  • (1)spacer_open 
Phrase Groups Available:
  • global
  • postbit
  • showthread
Included Files:
  • ./printthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/class_bbcode_alt.php
  • ./includes/class_bbcode.php
  • ./includes/functions_bigthree.php 
Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • printthread_start
  • pagenav_page
  • pagenav_complete
  • bbcode_fetch_tags
  • bbcode_create
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • printthread_post
  • printthread_complete