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.
Comment | File | Size | Author |
---|---|---|---|
#7 | fix-notification-for-unpublished-comments-2566117-7.patch | 887 bytes | formatC'vt |
Comments
Comment #2
formatC'vt CreditAttribution: formatC'vt as a volunteer commentedI'm not sure, but we can do not send notifications for unpublished comments
Comment #3
drummLooks good, committed.
Comment #5
drummI'll have to revert this for now, it caused #2578897: Email notifications for issue creation not working.
Comment #7
formatC'vt CreditAttribution: formatC'vt as a volunteer commentedsorry, my bad.
new patch.
Comment #8
drummTested and this looks better now.