Notice: Undefined index: name in _comment_notify_mailalert() (line 417 of /home/landi578/domains/lundiak.in.ua/public_html/sites/all/modules/comment_notify/comment_notify.module).

It was just after these steps:
1) install module
2) Enable
3) Post a comment (with checked "Notify me ...")
4) Submit a comment and vualya :)

After trying to add once again, with different settings - THE SAME

CommentFileSizeAuthor
#2 821600_general_cleanup.patch4.73 KBgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Priority: Critical » Normal
Issue tags: -comments, -notify +E_ALL

The code in question is:

  if (function_exists('locale')) {
    $languages = locale_language_list();
    $languages = $languages['name'];
  }

Cleaning up tags and resetting priority. I don't think this drastically affects the functionality of the module.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: Undefined Index in MailAlert error » Undefined Index in MailAlert error and general cleanups
Status: Active » Needs review
StatusFileSize
new 821600_general_cleanup.patch4.73 KB

The $languages were never being used...which is weird.

So, I just removed them :)

I also cleaned a few other things in that general area.

parasolx’s picture

parasolx commented
Status: Needs review » Reviewed & tested by the community

#2 patch solve my problem.. no more error.. and email testing, received.

so i mark as reviewed and tested by community

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Assigned: Unassigned » greggles
Status: Reviewed & tested by the community » Fixed

Now committed - thanks parasolx for the review!

Status: Fixed » Closed (fixed)
Issue tags: -E_ALL

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

Issue tags: +E_ALL

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.