Wordpress - php suggestion

Comments

• Nomad Member

  March 2017

  Something like this?
• jetchirag Member

  March 2017

  Simply use is_user_logged_in() with if statement
• quick Member

  March 2017

  @jetchirag said:
  Simply use is_user_logged_in() with if statement

  so this could do the job?

  if(!is_user_logged_in()){
     <?php include('adcode.php'); ?>
  }
  
• jetchirag Member

  March 2017

  Give it a try
• jetchirag Member

  March 2017

  Also remove php tags inside if statement
• quick Member

  March 2017 edited March 2017

  @jetchirag said:
  Also remove php tags inside if statement

  no chance, I see the code snippet in the header area

  btw I am adding this code into the header.php of my theme. I think I should do it elsewhere
• K4Y5 Member

  March 2017

  @quick said:

  @jetchirag said:
  Also remove php tags inside if statement

  no chance, I see the code snippet in the header area

  btw I am adding this code into the header.php of my theme. I think I should do it elsewhere

  Theme functions file.
• JustAMacUser Member

  March 2017 edited March 2017

  In the theme's functions.php:

  add_action( 'wp_head', 'adcode' );
  function adcode() {
      if ( ! is_user_logged_in() )
          include( 'adcode.php' );
  }
  

  As mentioned above, you really should put more logic-related stuff in the theme's functions.php file and leave the template files for just printing content to the screen, basic loops, etc.

  Thanked by 4raindog308 WSS K4Y5 quick
• K4Y5 Member

  March 2017

  @JustAMacUser said:
  In the theme's functions.php:

  add_action( 'wp_head', 'adcode' );
  function adcode() {
      if ( ! is_user_logged_in() )
          include( 'adcode.php' );
  }
  

  As mentioned above, you really should put more logic-related stuff in the theme's functions.php file and leave the template files for just printing content to the screen, basic loops, etc.

  Exactly!

  I was just too lazy to type that up on a mobile at 5am :P