[Mod] Mail Owner of image when Comment is Posted

[TheOracle]

Well, I do not see ANYWHERE ABOVE that you were talking about the $image_link. You only, as usual, questionned my modification without any specifics. 

[V@no]

So, you are a robot! or a bot? You dont understand if one tells you a clue, and not pointing directly to the issue...very narrow minded...

whatever. Just be quiet, please.

[fgallery]

Why admins do not receive comments for their images?

Only admins? have you tested it with regular member(s) or more then one admin account with VALID email address?

Nevermind V@no I've found the problem.
Now another question to the MOD'S author:
Can I disable notification on comments made by myself?
I mean I don't want to receive my own comments.

[V@no]

I'd suggest you using this code instead:
http://www.4homepages.de/forum/index.php?topic=2705.msg14792#msg14792
then replace

Code: [Select]

if ($user_allowemails == 1) {with:

Code: [Select]

if ($image_row['user_id'] != $user_info['user_id'] && $user_allowemails == 1) {

[fgallery]

I'd suggest you using this code instead:
http://www.4homepages.de/forum/index.php?topic=2705.msg14792#msg14792
then replace

Code: [Select]

if ($user_allowemails == 1) {with:

Code: [Select]

if ($image_row['user_id'] != $user_info['user_id'] && $user_allowemails == 1) {

Wow V@no! It works!
Thank you.

[fgallery]

[EDITED]
All right I've made all corrections specified above & below (till this reply) and the final code was (it works  ):

Code: [Select]

//-----------------------------------------------------
// Start Emailer for comments
//-----------------------------------------------------
   $recipient_name = $image_row['user_name'];
   $recipient_email = $image_row['user_email'];
   $image_url = $script_url."/details.php?".URL_IMAGE_ID."=".$id;

   $sql = "SELECT user_allowemails FROM ".USERS_TABLE." WHERE ".get_user_table_field("", "user_name")." = ".$recipient_name;
   $result = $site_db->query($sql);
   $row = $site_db->fetch_array($result);
   $user_allowemails = ($row['user_allowemails']);

   if ($image_row['user_id'] != $user_info['user_id'] && $user_allowemails == 1) {
   include(ROOT_PATH.'includes/email.php');
   $site_email = new Email();
   $site_email->set_to($recipient_email);
   $site_email->set_from($config['site_email'], $config['site_name']);
   $site_email->set_subject($lang['send_comment_emailsubject']);
   $site_email->register_vars(array(
     "user_name" => stripslashes($user_name),
     "recipient_name" => stripslashes($image_row['user_name']),
     "image_name" => $image_row['image_name'],
     "comment_date" => format_date($config['date_format']." ".$config['time_format'], $current_time),
     "site_name" => $config['site_name'],
     "image_url" => $image_url,
     "comment_text" =>(stripslashes($comment_text)),
     "comment_headline"=>(stripslashes($comment_headline)),
          ));
    $site_email->set_body("comment_email", $config['language_dir']);
    $site_email->send_email();
    }
    }
  }
 unset($row);
 unset($spam_row);
}

Don't forget to add:

Code: [Select]

$lang['send_comment_emailsubject'] = "COMMENT_NOTIFICATION_SUBJECT_HERE";
to /lang/language/main.php

PS.It will also disable sending you your own comments.

Perhaps I'd be useful for someone ...

[TheOracle]

@fgallery:

Looks good. However, these two lines :

$image_url = $script_url."/details.php?".URL_IMAGE_ID."=".$id."";

$sql = "SELECT user_allowemails FROM ".USERS_TABLE." WHERE user_name = '$recipient_name'";

could also simply be stated like this :

$image_url = $script_url."/details.php?".URL_IMAGE_ID."=".$id;

$sql = "SELECT user_allowemails FROM ".USERS_TABLE." WHERE ".get_user_table_field("", "user_name")." = ".$recipient_name;

[fgallery]

@fgallery:

Looks good. However, these two lines :

$image_url = $script_url."/details.php?".URL_IMAGE_ID."=".$id."";

$sql = "SELECT user_allowemails FROM ".USERS_TABLE." WHERE user_name = '$recipient_name'";

could also simply be stated like this :

$image_url = $script_url."/details.php?".URL_IMAGE_ID."=".$id;

$sql = "SELECT user_allowemails FROM ".USERS_TABLE." WHERE ".get_user_table_field("", "user_name")." = ".$recipient_name;

Does it make any difference?
Sorry, I'm not a coder

[TheOracle]

@fgallery:

Well, if you compare both of these quotes, you can see there are more " with your strings.

Example :

URL_IMAGE_ID."=".$id."";

when it could simply be :

URL_IMAGE_ID."=".$id;

since there are no beginning with == > "  < == after the "= (equality) ".

As for :

".get_user_table_field("", "user_name")."

Yes, there is since the user_name field is not an additional user field but a core field from the USERS_TABLE.

[fgallery]

I've upadated my full code, thanks!

PS.I thought your corrections would also fix this problem:
If I write for e.g. testin' in Hedline and/or Comment I'll receive it as testin\'

[TheOracle]

If I write for e.g. testin' in Hedline and/or Comment I'll receive it as testin\'

In your codings, make sure that the related query $HTTP_POST_VARS contains the stripslashes module with it.

If you're not sure where exacly this line might be in the code (and if the code itself is NOT too big) please post it here as I will take a look at it.

[TheOracle]

Update :

I found it. You're probably talking about this line :

"comment_headline"=>$comment_headline,

change it to :

Code: [Select]


"comment_headline"=>(stripslashes($comment_headline)),



[fgallery]

Update :

I found it. You're probably talking about this line :

"comment_headline"=>$comment_headline,

change it to :

Code: [Select]


"comment_headline"=>(stripslashes($comment_headline)),



I guess I should do the same with 'comment_text'.

[TheOracle]

I guess I should do the same with 'comment_text'.

Precisely.

[JensF]

[EDITED]
All right I've made all corrections specified above & below (till this reply) and the final code was (it works  ):

Code: [Select]

//-----------------------------------------------------
// Start Emailer for comments
//-----------------------------------------------------
   $recipient_name = $image_row['user_name'];
   $recipient_email = $image_row['user_email'];
   $image_url = $script_url."/details.php?".URL_IMAGE_ID."=".$id;

   $sql = "SELECT user_allowemails FROM ".USERS_TABLE." WHERE ".get_user_table_field("", "user_name")." = ".$recipient_name;
   $result = $site_db->query($sql);
   $row = $site_db->fetch_array($result);
   $user_allowemails = ($row['user_allowemails']);

   if ($image_row['user_id'] != $user_info['user_id'] && $user_allowemails == 1) {
   include(ROOT_PATH.'includes/email.php');
   $site_email = new Email();
   $site_email->set_to($recipient_email);
   $site_email->set_from($config['site_email'], $config['site_name']);
   $site_email->set_subject($lang['send_comment_emailsubject']);
   $site_email->register_vars(array(
     "user_name" => stripslashes($user_name),
     "recipient_name" => stripslashes($image_row['user_name']),
     "image_name" => $image_row['image_name'],
     "comment_date" => format_date($config['date_format']." ".$config['time_format'], $current_time),
     "site_name" => $config['site_name'],
     "image_url" => $image_url,
     "comment_text" =>(stripslashes($comment_text)),
     "comment_headline"=>(stripslashes($comment_headline)),
          ));
    $site_email->set_body("comment_email", $config['language_dir']);
    $site_email->send_email();
    }
    }
  }
 unset($row);
 unset($spam_row);
}

Don't forget to add:

Code: [Select]

$lang['send_comment_emailsubject'] = "COMMENT_NOTIFICATION_SUBJECT_HERE";
to /lang/language/main.php

PS.It will also disable sending you your own comments.

Perhaps I'd be useful for someone ...

What do this??? Mail at new comments on/off???