User without "Post comments" & "Skip comment approval" permissions have this error:

Notice: Undefined offset: 167 in _project_issue_email_notify() (line 74 of ...modules/project_issue/includes/mail.inc).
Notice: Trying to get property of non-object in _project_issue_email_notify() (line 75 of ...modules/project_issue/includes/mail.inc).
Notice: Undefined index: comment_node_article in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_page in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_project in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_project_issue in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_project_release in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Trying to get property of non-object in _project_issue_email_notify() (line 82 of ...modules/project_issue/includes/mail.inc).
Warning: array_flip(): Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->load() (line 175 of ...drupal/includes/entity.inc).
Warning: array_flip(): Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->cacheGet() (line 388 of ...drupal/includes/entity.inc).
Notice: Trying to get property of non-object in _project_issue_email_notify() (line 107 of ...modules/project_issue/includes/mail.inc).
Notice: Undefined offset: 167 in _project_issue_mail() (line 274 of ...modules/project_issue/includes/mail.inc).
Notice: Trying to get property of non-object in _project_issue_mail() (line 275 of ...modules/project_issue/includes/mail.inc).
Notice: Undefined index: comment_node_article in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_page in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_project in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_project_issue in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Undefined index: comment_node_project_release in nodechanges_get_field_name() (line 965 of ...modules/nodechanges/nodechanges.module).
Notice: Trying to get property of non-object in project_issue_get_comment_thread_id() (line 1969 of ...modules/project_issue/project_issue.module).
Notice: Trying to get property of non-object in project_issue_email_get_comment_header() (line 395 of ...modules/project_issue/includes/mail.inc).
Warning: date_timezone_set() expects parameter 1 to be DateTime, boolean given in format_date() (line 2040 of ...drupal/includes/common.inc).
Warning: date_format() expects parameter 1 to be DateTimeInterface, boolean given in format_date() (line 2050 of ...drupal/includes/common.inc).
Notice: Trying to get property of non-object in project_issue_email_get_comment_header() (line 396 of ...modules/project_issue/includes/mail.inc).
Notice: Trying to get property of non-object in project_issue_email_get_comment_header() (line 397 of ...modules/project_issue/includes/mail.inc).
Notice: Trying to get property of non-object in project_issue_email_get_comment_header() (line 397 of ...modules/project_issue/includes/mail.inc).
Unable to send e-mail. Contact the site administrator if the problem persists.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

formatC'vt created an issue. See original summary.

formatC'vt’s picture

Status: Active » Needs review
FileSize
1.68 KB

I'm not sure, but we can do not send notifications for unpublished comments

drumm’s picture

Status: Needs review » Fixed

Looks good, committed.

  • drumm committed 7ecb999 on 7.x-2.x authored by formatC'vt
    Issue #2566117 by formatC'vt: E-mail notification for unpublished...
drumm’s picture

Status: Fixed » Needs work

I'll have to revert this for now, it caused #2578897: Email notifications for issue creation not working.

  • drumm committed 2318fc1 on 7.x-2.x
    Revert last commit for #2566117
    
formatC'vt’s picture

Status: Needs work » Needs review
FileSize
887 bytes

sorry, my bad.
new patch.

drumm’s picture

Status: Needs review » Fixed

Tested and this looks better now.

  • drumm committed d217c79 on 7.x-2.x authored by formatC'vt
    Issue #2566117 by formatC'vt: E-mail notification for unpublished...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.