Not signed in (Sign In)

Categories

Vanilla 1.1.4 is a product of Lussumo. More Information: Documentation, Community Support.

Help keep Vanilla free:
Welcome Guest!
Want to take part in these discussions? If you have an account, sign in now.
If you don't have an account, apply for one now.
    •  
      CommentAuthortechmate
    • CommentTimeJul 5th 2007
     # 101
    I dont know what Im doing wrong, but I just installed this addon and no emails are coming through to me. I checked the mail settings and they are all fine. Even changed them to different accounts just in case. I have it set to notify me for everything. I made a couple test posts but nothing comes through. What should I be looking for?
    Thanks
    • CommentAuthordanielg
    • CommentTimeJul 9th 2007
     # 102
    I'm having the same issue as vzx.

    Notice: Undefined variable: mTitle in /home2/vzx/public_html/temple/extensions/Notify/default.php on line 309
    Notice: Undefined variable: mComment in /home2/vzx/public_html/temple/extensions/Notify/default.php on line 309
    Notice: Undefined variable: mUser in /home2/vzx/public_html/temple/extensions/Notify/default.php on line 309

    Any clues?

    Daniel
    • CommentAuthorappler
    • CommentTimeJul 9th 2007
     # 103
    With forum name in asian characters, the subject of notification mail would be corrupted and not-readable. I've made some modifications to meet the RFC 2047 standard to encode mail subjects correctly.

    1. Notify/default.php:350

    From:

    $e->Subject = $DiscussionForm->Context->Configuration['APPLICATION_TITLE'].' '.$DiscussionForm->Context->GetDefinition('Notification');


    To:

    $e->Subject = mailSubject( $DiscussionForm->Context->Configuration['APPLICATION_TITLE'].' '.$DiscussionForm->Context->GetDefinition('Notification') );


    2. and add the mailSubject() function's code like follows.

    function mailSubject( $str ) {
    $result = '';
    $strArray = explode( ' ', $str );
    for ( $idx=0; $idx<count($strArray); $idx++ ) {
    if ( $idx > 0 ) {
    $blank = ' ';
    } else {
    $blank = '';
    }
    $result = $result . '=?utf-8?b?' . base64_encode( $blank . $strArray[ $idx ] ) . '?=';
    }
    return $result;
    }


    That's it!
    •  
      CommentAuthorfnostro
    • CommentTimeJul 10th 2007
     # 104
    I believe there is an error in the notify setup - Notify adds a number of fields to lum_user, specifically 'SubscribeOwn' & 'Notified', that are defined as "not null". This is causing a fatal, non-recoverable error when a new user record is created, i.e. a new user tries to register. SInce the fields are defined as 'not null' and default values are not supplied, the result is a fatal error.

    So I'm thinking you can either set defaults to 0 and hope that's correct or redefine the fields to allow nulls and again, hope for the best. Either fix will allow users to register again - but I've no idea if Notify will like these solutions. Hutstein, can you comment?
  1.  # 105
    Just installed the plug in NOTIFY. Have read the many other comments and I must say everything seems to be working great for me. EXCEPT...no email. And I see many others have brought up this issue. But many of the answers I see seem to fix other issues, not just the email issue notifying the user that a new post is made.

    Is there anyway just to fix this one issue...so users can receive email notifications of new posts. Just this one issue, I get NO other error messages, nothing gets froze, all seems fine. Just the email issue.

    Please help or suggest.
    •  
      CommentAuthorjimw
    • CommentTimeJul 11th 2007
     # 106
    I use email in the zip2mail add-on successfully. The code uses the mail() function.
    mail($to,$subject,$mail_body,$from);

    The Notify add-on uses uses the send() function.
    $e->Send();
    I don't know the difference between send() and mail(), but perhaps someone can tell me.
  2.  # 107
    The mail function uses sendmail on the server (assuming it's running) the Send function is one of vanillas functions and uses either sendmail or smtp depending on whch is configured in your application settings. Technically Send is the correct way to do it since mail wont work if servers dont have sendmail installed.
    •  
      CommentAuthorjimw
    • CommentTimeJul 11th 2007
     # 108
    So, help me brainstorm and try to figure out the difference between how Notify builds the email structure and how I do it in my add-on. Notify builds the $e object and then fills each component: $e->AddFrom, $e->AddRecipient, $e->Subject and $e->Body. Then it uses $e->Send(). It looks like it has something in the $e->Body called $test which is not in the script. Could this cause the problem?
    $e->Body = str_replace(array("{forum_name}","{topic_url}"),array($DiscussionForm->Context->Configuration['APPLICATION_TITLE'],ConcatenatePath($DiscussionForm->Context->Configuration['BASE_URL'].'comments.php?DiscussionID='.$DiscussionID,'')),$EmailBody).$test;
  3.  # 109
    That $test value does seem to be slightly errant. I wonder if Mark left that in there by accident...

    Can we confirm whether the notify extension works either with or without the SMTP settings?
  4.  # 110
    I have tried to post messages to see if I get email notifications with AND without SMTP settings, and neither seems to work.
    •  
      CommentAuthorWanderer
    • CommentTimeJul 12th 2007
     # 111
    I have this setting: smtp.gmail.com:465

    I get notifications all the time.

    Posted: Friday, 13 July 2007 at 7:26AM

    • CommentAuthordanielg
    • CommentTimeJul 12th 2007
     # 112
    I have managed to get notify working by doing a fresh install and leaving the SMTP settings blank. I now get messages every time. I would assume then that mail() is working for me when send() was not. I have also set automatic notification on in the settings file.

    - Daniel
    • CommentAuthorjaganat
    • CommentTimeJul 26th 2007
     # 113
    We need to make users automatically notified of all forum on signup, but at the same time they can have a link to unsubscribe/subscribe.
    Any clues?
  5.  # 114
    Try the global preferences extension
    • CommentAuthorjaganat
    • CommentTimeJul 26th 2007
     # 115
    which preference name should I switch to "1" ? Can“t find the name
  6.  # 116
    Oh..hmm....the Notify extension doesnt use preferences it uses DB entries so actually that wont work, sorry. I wonder how it's easiest done in this case then... You could run a sql script to change everybody to auto notify but then that'd also wipe peoples preferences if they'd turned it off. Maybe you'd need to attach to a delegate somewhere during user creation and enable it then..

    Any ideas guys?
  7.  # 117
    While evaluating Notify, I discovered Javascript bugs related to prototype not being included. I tracked this down to what looks like a bug in line 429 of the extentions, where the three "AddScript" lines aren't braced in. I added these braces, and this fixed the js error.

    However, the extension still doesn't work for me :-(...no emails.
    • CommentAuthorjprado
    • CommentTimeAug 10th 2007
     # 118
    Nothing seemed to happen after I clicked any of the checkboxes because there seems to be a problem with the ajax request URL. Changing Notify/functions.js:10 to this seems to do the trick.

    var ajax = new Ajax.Request(parent.location.protocol + '//' + document.domain + '/forum/extensions/Notify/ajax.php', {

    There is also an extra statement that needs to be commented out or else the line will disappear after you click the checkbox (because it setting its innerHTML to nothing) just comment it out and it should work.

    Notify/functions.js:17

    //$(Elem).innerHTML = NewText;
  8.  # 119
    I understand that Notify sends an e-mail when a comment is made to a discussion the user has subscribed to. Is there any functionality available that goes the other way? Could a user potentially reply to the e-mail, and have that message tack on a new comment to the forum discussion?
    •  
      CommentAuthorhutstein
    • CommentTimeOct 9th 2007
     # 120
    FYI: I'm working on a complete re-make of the Notify extension!

    so hold on I will soon launch it...
    •  
      CommentAuthorWanderer
    • CommentTimeOct 9th 2007
     # 121
    Great stuff hutstein, I love the way it works now.
    It would be great if there was an option to remove it from the panel and put all the controls on the account page?
    •  
      CommentAuthorhutstein
    • CommentTimeOct 9th 2007
     # 122
    I think I will leave the controls on the panel, but now I added controls directly to the index.php on each discusson after "Last Active xxx" where you can individually see the notification status of each discussion... the same on the categories page...

    And I included a feature that filters all the discussions and shows only discussions you are subscribed to...
    •  
      CommentAuthorWanderer
    • CommentTimeOct 9th 2007
     # 123
    What if one is using Discussions Overview and has hidden all that information from view?
  9.  # 124
    Could you add a 'get email notifications of future comments in this discussion' check box someplace in the 'add your comments' box?

    The checkbox could then default to whatever prior preference the user has set.
    •  
      CommentAuthorhutstein
    • CommentTimeOct 11th 2007
     # 125
    @Wanderer: That's a good point! The other subscribe/unsubscribe on the panel is also working and I think I need to write a "howto implement the notification control to the Discussion Overview"

    @HouseinProgress: a very good feature... I will try to implement it...
    • CommentAuthorhalo_12
    • CommentTimeOct 20th 2007
     # 126
    How's it going hutstein? Really looking forward to a working version of notify: its integral to a forum , really.

    Thanks for working on this.
    • CommentAuthorhalo_12
    • CommentTimeOct 20th 2007
     # 127
    Not sure if this is of any value, but when setting to NOTIFY_ALL = 1, works like a charm.

    When set to 0, the notify table in the DB just doesn't get poulated at all. it's empty. I tried jprado's code tweak to function.php, but made no difference.

    Hope this helps.
    • CommentAuthorChiOne
    • CommentTimeOct 23rd 2007 edited
     # 128
    FULL TEXT AND REPLY FORM
    Want an option to have an html formatted full text of the comment in users email notifications and not only one notification but all comments.

    And the reply link would be nice too and the reply form woulbe grate.(but there are more problems with authentification)
    I have used it when admined my last forum, it was so good when you don't have to enter the forum just to see another smile in a comment.
    i read all forum messages by email in this way.
    •  
      CommentAuthorhutstein
    • CommentTimeOct 23rd 2007
     # 129
    any users who can test the really really pre-release of the new version?

    just send me a mail: mail@hutstein.de
    this version is not for productive use!!!
    • CommentAuthoragentj
    • CommentTimeOct 30th 2007
     # 130
    Just emailed you.
    •  
      CommentAuthorgarvin
    • CommentTimeOct 30th 2007
     # 131
    Emailed you.
    •  
      CommentAuthorDinoboff
    • CommentTimeOct 31st 2007 edited
     # 132
    line 429-432 of default.php of the current version:if (in_array($Context->SelfUrl, array('comments.php','index.php','account.php','categories.php')))
    $Head->AddScript('js/prototype.js');
    $Head->AddScript('js/scriptaculous.js');
    $Head->AddScript('extensions/Notify/functions.js');

    Should be replaced by:if (in_array($Context->SelfUrl, array('comments.php','index.php','account.php','categories.php'))) {
    $Head->AddScript('js/prototype.js');
    $Head->AddScript('js/scriptaculous.js');
    $Head->AddScript('extensions/Notify/functions.js');
    }
    •  
      CommentAuthorWanderer
    • CommentTimeNov 1st 2007
     # 133
    My version is the same as the bottom one, curly bracket and all, so it does not need fixing.

    Posted: Thursday, 1 November 2007 at 6:09PM

    •  
      CommentAuthorDinoboff
    • CommentTimeNov 1st 2007
     # 134
    You might have already fix it or your installation is not up to date.

    The error is in the package from the repository (Notify-2007-05-29.zip). I just checked it again.
    • CommentAuthorhalo_12
    • CommentTimeNov 1st 2007
     # 135
    I was hoping this fix would make notify work- still does not seem to write anything to the LUM_notify table when i subscribe.

    The Ajax doofer seems to run ok, and then the link will change to say "unsubscribe" , but no joy.
    • CommentAuthorJoey
    • CommentTimeNov 7th 2007 edited
     # 136
    Some quick notes about my experience with this extension on my configuration:

    This version 1.2.0 of this Notify extension took quite a bit of work to implement into my configuration of Vanilla 1.1.2

    It did not interact well with JQMedia on my configuration. I now eventually have alot of the features working after disabling JQuery v12-1.1.4 and all other JQ extensions temporarily to enable the Notify extension first then I re-enabled the JQuery v12-1.1.4 and other JQ extensions afterward. I had to modify the default.php for JQmedia 0.5.1 to have JQmedia included in all the pages that had Notify panel links, not just the comments.php and posts.php pages (as is where JQmedia appears to be included by default). For some reason JQmedia and some other JQuery extensions had a bad effect on the Notify extension's javascript includes in the header when displayed in certain various orders even when notify was enabled first through the settings extension control panel. The quickest way I discovered to fix the JQmedia conflict in my configuration was to either disable JQmedia or, as I stated above, just simply included JQmedia on all pages containing the Notify extension "subscribe" and "unsubscribe" links.

    If you are having trouble getting the "subscribe" and "unsubscribe" links on the side panel to respond correctly check into possible extension conflicts especially if you have JQ extensions installed.

    Also keep in mind, the emails I received after subscribing to discussions sometimes came quickly, within several seconds, and other times they took a while to arrive. If you don't receive an email right away and you are pretty confident that you set the extension correctly in your configuration and that you don't have any conflicts check your email again a little later.
    • CommentAuthorhalo_12
    • CommentTimeNov 8th 2007
     # 137
    Thanks joey, will try your suggestions.
    • CommentAuthorJoey
    • CommentTimeNov 8th 2007 edited
     # 138
    So far most of the basic features are working as long as I am subscribed to the entire forum, specific categories or specific discussions via the subscribe/unsubscribe links in the side panel of the index.php, categories.php or comments.php pages or via the "notifcation" section on Vanilla's account page (account.php).

    Because it appears this extensions relies on the whisper function within vanilla, "ENABLE WHISPERS" had to be activated within my configuration from the Vanilla settings page under the "options" header ("application settings")

    However, the subscribe own feature enabled from the "notification" section of the account.php page (Notify me, when comments are posted in my own discussions), which sends an email to the original poster when the discussion they started receives comments, is still not functioning in my configuration.

    The check box(Notify me, when comments are posted in my own discussions) on Vanilla's account.php ("notification" section) inputs the information into the SubscribeOwn field of the User table in the database, however, Notify's extension default.php does not seem to be properly calling, checking and integrating the LUM_User table, SubscribeOwn field when initiating the sending of the emails.

    Seemingly, the emails are being sent BASED ONLY on the information contained within the LUM_Notify table on the database and NOT ALSO BASED on the information within the SubscribeOwn field within the LUM_User table on the databse. So it appears that when subscribing to the entire forum, specific categories or specific discussions via the panel links or notification section of account.php, that information is inputted into the LUM_Notify table and WORKS because that table information is included when checking to send emails but the (Notify me, when comments are posted in my own discussions) data is inputted into the LUM_User table which appears does NOT WORK because the informaton in that field on that table is not included in the information that is checked and emailed to the appropriate users.
    • CommentAuthorJoey
    • CommentTimeNov 8th 2007 edited
     # 139
    I am currently testing some modifications (shared below), that are seemingly working well and consistently with my configurations so far, to get this current version of Notify extension to NOTIFY the original authors who have posted a discussion and who have checked the "Notify me, when comments are posted in my own discussions" option, within the notification control panel from the Vanilla account.php page, but who have not subscribed to the entire forum or entire cateogry

    This modification adds a basic call to the User table to get the information needed to email the original author of comments - since, as I stated above, the information stored to identify and email users in situations where comments have been added to their own discussions is not stored in the Notify table, along with the rest of the subscribe/unsubscribe data, but instead is stored in the User table SubscribeOwn field of the Vanilla database.

    So far, through brief testing, adding this code has not caused repeat emails to be sent over the same comment in cases where the user who is emailed is subscribed to the entire forum, same cateogry, as well as has "Notify me, when comments are posted in my own discussions" set to receive comments added to their discussion.



    Navigating to the Vanilla "conf" folder and adding this line to the settings.php near the other Notify extension info: $Configuration['NOTIFY_ALLOW_SUBSCRIBEOWN'] = '1';

    Then complimented by navigating to Vanilla's extensions folder to the Notify folder, and adding the following code to the defualt.php within the Notify folder - about line248:
    if ($DiscussionForm->Context->Configuration['NOTIFY_ALLOW_SUBSCRIBEOWN'] == 1) {
    $result = mysql_query("SELECT A.UserID,A.Email,A.FirstName,A.LastName,A.SubscribeOwn,B.AuthUserID, B.DiscussionID FROM ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."User AS A, ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."Discussion AS B WHERE B.DiscussionID = '$DiscussionID' AND B.AuthUserID <> '$SelfUser' AND A.UserID <> '$SelfUser' AND B.AuthUserID = A.UserID AND A.Notified = 0 AND A.SubscribeOwn = 1",$DiscussionForm->Context->Database->Connection);
    while ($row = mysql_fetch_row($result))
    if (($Whispered == 1 AND $WhisperUserID == $row[0]) OR ($Whispered == 0))
    array_push($Notifieusers,array($row[0],$row[1],$row[2],$row[3]));
    }



    *** the above code should end just above the following line that is included by default: #Add all users who have subscribed to all, aren't already notified except the posting user

    Hopefully the above modifications will help make this extension more useful and consistent at least temporarily until the next, more stable version is finished.


    In my configuration, I also changed the following line to bypass the 1 email sent limit. As this limitation seems to have been implemented to prevent repeat emails but it appeared that only 1 email would be sent no matter how many comments were posted in different categories (ie users only received emails one time regarding one discussion in one category and never received any other emails informing them about comments posted in other discussions to which they were contributing. Alternately you can very likely just eliminate that mysql_query if you want your users to receive emails everytime a comment is posted to the discussions to which they are subscribing.

    Approximately line:313:

    (Was) mysql_query("UPDATE ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."User SET Notified = 1 WHERE UserID = '$val[0]'");
    (Changed - Notified = 1 to Notified = 0)
    mysql_query("UPDATE ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."User SET Notified = 0 WHERE UserID = '$val[0]'");



    If it helps.
    •  
      CommentAuthorWanderer
    • CommentTimeNov 8th 2007
     # 140
    Joey mate, I think you are wrong about Notify relying on the Whisper function. My Whispers are turned off but Notify works well.

    However, I have subscribed to the entire forum, not just specific discussions so you may be right about this aspect.

    I really wish this add-on would work well, busy people rely on it, they cannot visit every day in case something has been added.
    • CommentAuthorJoey
    • CommentTimeNov 8th 2007
     # 141
    Wanderer: "I really wish this add-on would work well, busy people rely on it, they cannot visit every day in case something has been added."

    Yeah Wanderer, it is definitely a necessary component for conveying up-to-date message board functioning and effectiveness. From sifting through the comments on this extension above it appears that hutstein and some others have been working on it for quite a while and hutstein mentioned above that they may be close to a new version.

    Thanks for the info on your "Enable Whisper" configuration.
    • CommentAuthorJoey
    • CommentTimeNov 8th 2007 edited
     # 142
    ONE quick way to add the Discussion title to the email subject line:

    Open Notify's default.php:
    Hit ctrl+F and search the page for the line (#Remove double inserted users) minus the parentheses:
    paste the code below just under the following default line "$e->HtmlOn = 0;":


    $result = mysql_query("SELECT Name FROM ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."Discussion WHERE DiscussionID = '$DiscussionID'");
    $row = mysql_fetch_row($result);
    $mTitle = $row[0];




    just for reference: the end of the code above should end just above this deafult line "foreach($Notifieusers as $val)
    {"


    THEN REPLACE: $e->Subject = $DiscussionForm->Context->Configuration['APPLICATION_TITLE'].' '.$DiscussionForm->Context->GetDefinition('Notification');

    WITH: $e->Subject = $DiscussionForm->Context->Configuration[''APPLICATION_TITLE''].' "'.$mTitle.'" ';




    In the email_notify.txt in the Notify extension folder you can then ALSO include the title reference (something structured like):

    Hello {name},

    A new comment/discussion was posted within the following topic to which you are subscribed.

    {title}: {topic_url}




    If it helps someone else
    • CommentAuthorBersi
    • CommentTimeNov 21st 2007
     # 143
    What about realisation of other tags that I see in php file (used in mail template) ?
    Like send comment to e-mail or person name...
    • CommentAuthorBersi
    • CommentTimeNov 23rd 2007
     # 144
    Mhhh.. It's plugin too slow! What not use "PreSaveComment" controller here and do not use SQL queryes?
    • CommentAuthorjaz
    • CommentTimeNov 24th 2007
     # 145
    Hi,

    Thanks for a nice extension! However, there is one thing that seems to be wrong.

    Users seem to be notified about discussions in categories to which they do not have access to. This should definitely be checked when sending notifications. I do not want users who do not have access to some restricted categories to get notifications about messages posted into those categories.

    Please correct me if I'm wrong, but I think this is quite a critical fault in this extension.

    BR,
    Johan
    • CommentAuthorjaz
    • CommentTimeNov 24th 2007
     # 146
    Oh well, I already added the missing functionality myself.

    On row 224, in default.php of the Notify-extension, I added CategoryID to the query:
    $result = mysql_query("SELECT WhisperUserID,CategoryID FROM ...

    And then stored the result in a variable and build check lists for blocks:
    $WhisperUserID = $row[0];
    $CategoryID = $row[1];

    // Check for users who have blocked this category (do not notify them)
    $usersWithCategoryBlock = array();
    $result = mysql_query("SELECT UserID FROM ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."CategoryBlock WHERE CategoryID = '$CategoryID'");
    while(1) {
    $row = mysql_fetch_row($result);
    if(!$row)
    break;
    $usersWithCategoryBlock[] = $row[0];
    }

    // Check for roles for which this category is blocked (do not notify them)
    $rolesWithCategoryBlock = array();
    $result = mysql_query("SELECT RoleID FROM ".$DiscussionForm->Context->Configuration['DATABASE_TABLE_PREFIX']."CategoryRoleBlock WHERE CategoryID = '$CategoryID'");
    while(1) {
    $row = mysql_fetch_row($result);
    if(!$row)
    break;
    $rolesWithCategoryBlock[] = $row[0];
    }

    Then, starting from about row 276, the if statements in...
    while ($row = mysql_fetch_row($result))
    if (($Whispered == 1 AND $WhisperUserID == $row[0]) OR ($Whispered == 0))
    array_push($Notifieusers,array($row[0],$row[1],$row[2],$row[3]));

    ...should be surrounded with an additional if-statement:
    while ($row = mysql_fetch_row($result))
    if(!in_array($row[0], $usersWithCategoryBlock) && !in_array($row[4], $rolesWithCategoryBlock)) {
    if (($Whispered == 1 AND $WhisperUserID == $row[0]) OR ($Whispered == 0))
    array_push($Notifieusers,array($row[0],$row[1],$row[2],$row[3]));
    }

    Note, that there four of these if-statements to be completed and also note that some of the $row-variables are called $row2 instead of just $row.

    Ah, I almost forgot. Also add RoleID to the all the queries just before these if-statements:

    $result = mysql_query("SELECT UserID,Email,FirstName, LastName, RoleID FROM ...

    That should do it... Hope this helps to fix the issue.

    BR,
    Johan
    •  
      CommentAuthorDinoboff
    • CommentTimeDec 1st 2007 edited
     # 147
    can someone fix that?
  10.  # 148
    Uploaded version 1.2.1 of Notify.
  11.  # 149
    Added the fix which Dinoboff posted above to prevent this extension from causing faults with the rest of the forum. I have not added any of the potential new features as posted above as i am not aware of them having been fully tested. If someone wants to package them and test them then they are welcome to take over the extension with hutsteins agreement.
    •  
      CommentAuthorWanderer
    • CommentTimeDec 4th 2007
     # 150
    This extension is one that should work reliably.

    In my humble opinion, if it is not deemed worthy enough to be in the core, at least the developer (Hutstein) should be given full cooperation and assistance to make it work and work reliably.

    I only wish my skill set allowed me to help out.