Not signed in (Sign In)

Categories

Vanilla 1.1.5 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.
    • CommentAuthorgarysmith
    • CommentTimeSep 15th 2008
     # 51
    SubJunk
    Downloaded 1.6.2 and it is now working as advertised.
    Thanks for all the work on this extension.
    • CommentAuthorchitoso
    • CommentTimeSep 15th 2008
     # 52
    Hi guys!, I'd like to remove the "Suscribe to Forum" and maybe even "Suscribe to category" options, since I think it maybe too much overhead and mails going out from the server.
    Which is the best way to approach to this?

    Thanks!
    • CommentAuthoriamspammer
    • CommentTimeSep 15th 2008
     # 53
    thanks.

    Updated and works like charm.

    I found another bug (depends on users). I have noticed that, notifications are also sent if users edit their post. This morning, a notification was sent 4 times because user made 3 edits after adding it.

    Users were all over me asking why they are getting repetitive emails. I then compared the emails (forwarded by them) and found the edits. Users gave few suggestions:

    - Do not send notification on edit; just new discussion and new comment
    - Make a user option (OFF by default) to get UPDATE notifications in case of edit

    In any case, I am not sure how we can check if this is an EDIT or new discussion/comment. Any idea how that can be detected? May be the easy option is to turn it off at the moment.

    Thanks again for responding and providing the fix so quickly. Much appreciated
    • CommentAuthorSubJunk
    • CommentTimeSep 15th 2008 edited
     # 54
    To garysmith:
    Thanks for the feedback mate, I'm glad it's working now :)

    To chitoso:
    Thanks for reminding me, I need to make a UI for this. But for now it will work if you download your conf/settings.php file (important: make sure you are editing the version from the server, it is a file that has changed since you installed Vanilla and you need the current version or you will lose settings) and you will see these three lines:
    $Configuration['NOTIFI_ALLOW_ALL'] = '1';
    $Configuration['NOTIFI_ALLOW_DISCUSSION'] = '1';
    $Configuration['NOTIFI_ALLOW_CATEGORY'] = '1';
    Change the 0s to 1s where applicable. So if you change "ALL" to 0, it removes the ability to subscribe to the whole forum, and "CATEGORY" the ability to subscribe to whole categories. So you want it to look like this:
    $Configuration['NOTIFI_ALLOW_ALL'] = '0';
    $Configuration['NOTIFI_ALLOW_DISCUSSION'] = '1';
    $Configuration['NOTIFI_ALLOW_CATEGORY'] = '0';
    Sorry for the messy way of doing it.

    To iamspammer:
    Yes that is a good point, it isn't a bug but it is something to think about. I will have a think and a look at the code and get back to you on that one mate.
    • CommentAuthormiquel
    • CommentTimeSep 16th 2008
     # 55
    SubJunk, take a look at the SetList extension, it will help you ;)
    • CommentAuthorSubJunk
    • CommentTimeSep 16th 2008
     # 56
    Looks great miquel, I'll check it out, thanks :)
  1.  # 57
    Uploaded version 1.7 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeSep 16th 2008
     # 58
    To miquel:
    That extension looks cool but I didn't use it this time, instead I just copied code from another extension and modified it, always works like a charm ;) Maybe I will try SetList in the future.

    To iamspammer:
    It no longer sends emails for edits, just new posts as requested. Good idea.

    To chitoso:
    With version 1.7 you can change those settings with checkboxes instead of editing the code ;) Sorry for the inconvenience, I really don't know how I forgot to do that before now :)
    • CommentAuthoriamspammer
    • CommentTimeSep 16th 2008
     # 59
    You responded as quick as always. Updated and worked like charm.

    Thanks. Much Much appreciated.
    • CommentAuthorSubJunk
    • CommentTimeSep 18th 2008
     # 60
    Happy to help mate :)
    Now my main priorities with this extension are to make it faster loading and compatible with Low-Cal Vanilla. I started looking closely at Low-Cal Vanilla today and I will keep doing so to find out what's getting mucked up. I've also emailed the author of that extension in the hopes that 2 heads will be better than 1 :)
  2.  # 61
    Version number changed from 1.7 to 1.7.1.
  3.  # 62
    Version number changed from 1.7.1 to 1.7.
  4.  # 63
    Uploaded version 1.7.1 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeSep 18th 2008 edited
     # 64
    Notifi is finally compatible with Low-Cal Vanilla! Be sure you download the new versions of both the extensions or you will get an error (of course there is no error if you don't already have Low-Cal Vanilla, don't worry it isn't a dependency).
    • CommentAuthorpalmdoc
    • CommentTimeSep 19th 2008
     # 65
    Hi SubJunk. I have installed 1.7.1 and notice an error message after activating the extension:


    Notice: Undefined index: LOWCALVANILLA_TOOLS_PATH in Hostpath\httpdocs\forum\extensions\Notifi\_includes\install.php on line 76
    • CommentAuthorSubJunk
    • CommentTimeSep 19th 2008
     # 66
    Can you please tell me if you have Low-Cal Vanilla installed, or if you have ever had it installed, and which version?
    •  
      CommentAuthorDinoboff
    • CommentTimeSep 19th 2008
     # 67
    oops... you should use: if (!empty($Configuration['LOWCALVANILLA_TOOLS_PATH'])) { to check if LowCal is installed
    • CommentAuthorpalmdoc
    • CommentTimeSep 19th 2008
     # 68
    No I don't have Low-Cal Vanilla installed or ever had it installed. The extensions I have installed are:

    # Community Moderation 1.0
    # FCKeditor 2.5.4
    # Forum Statistics 0.4.5
    # Global Preference Setting 0.1
    # Html Formatter 2.4
    # Janine 1.1.1
    # MembersList 1.2
    # Notifi 1.7.1 (temp deactivated)
    # Preview Post 2.5.2
    # Set List 1.0.1
    # Stuff Displayer 2.1
    # reCAPTCHA 1.10.0
    • CommentAuthorpalmdoc
    • CommentTimeSep 19th 2008
     # 69
    @ Dinoboff - thanks I changed the line as you suggested to check if LowCal is installed and it did the trick
  5.  # 70
    Uploaded version 1.7.2 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeSep 19th 2008
     # 71
    Ah dino beat me to it ;) anyway that is exactly the same thing I did in this update. Sorry for the trouble palmdoc I didn't do enough testing before release.
    Dino you might want to edit this post with the updated code too.
    • CommentAuthorpalmdoc
    • CommentTimeSep 19th 2008
     # 72
    Wow that was fast ;) Anyway great work Subjunk!
  6.  # 73
    Uploaded version 1.7.3 of Notifi.
    • CommentAuthorpalmdoc
    • CommentTimeSep 23rd 2008
     # 74
    Funny I am still getting the occasional error.

    Error Message
    An error occurred while sending the email.

    Affected Elements
    Email.ServerParse();

    The error occurred on or near: Couldn't get mail server response codes.

    after the latest Notifi update.

    I have not made any SMTP server changes but it seems to work only on and off.
    Any clues?
    • CommentAuthorSubJunk
    • CommentTimeSep 24th 2008
     # 75
    Thanks for letting me know. Can you please tell me exactly what you do to get that error? It will be much easier for me to fix it if I can reproduce it
    • CommentAuthorpalmdoc
    • CommentTimeSep 24th 2008 edited
     # 76
    I'm not quite sure. After the updating with the latest default.php file, it seemed to work. Later on I got the message again so I disabled the extension. Then I enabled it again and it seemed to be OK for a while but it then gave the same error again when posting comments in threads. I don't know how to reproduce it. I don't know if it is sensitive to server time outs perhaps? But I don't have any issues with email from my main Wordpress blog anyway.
    It's in http://palmdoc.net/forum if you want to check it out
    At the moment I enabled the extension but I am getting the error
    • CommentAuthorSubJunk
    • CommentTimeSep 24th 2008
     # 77
    Well instead of taking a lot of space up on this discussion why don't you email me, my email address is in Notifi/default.php. I signed up at your forum but it appears you disabled Notifi again so I can't test it. Anyway I'll wait for your email.
  7.  # 78
    Many thanks for the work done on Notifi. Version 1.7.3 is working very well for me.
    Would it be possible to configure it to just post the first e-mail of each discussion? This would enable people to go to the forum on the web if they were interested in the topic and would not block up everyone's e-mail too much with the answers. I am working with a group who are used to an e-mail forum and getting them to accept Vanilla has been an uphill struggle. Our old forum (MailMan) works by e-mailing the question and the answers are sent only to the person who asked the question. This person then has the task of summarising all the answers and sending the summary to the forum. If we could use Vanilla in this way the summary would be redundant.
    • CommentAuthorSubJunk
    • CommentTimeSep 24th 2008
     # 79
    To Tony:
    Yes that is very possible to make and it's a good idea. I was hoping someone would make a new feature suggestion soon. I don't have much free time at the moment in between work and other personal projects, but when I get a chance I will make that feature.
  8.  # 80
    Thank you very much SubJunk. I look forward to that and when completed I will owe you a few beers or something!
    •  
      CommentAuthorWanderer
    • CommentTimeSep 25th 2008
     # 81
    Just thought you should all know that this extension does not work with Vanilla 1.1.2

    No errors are generated but no emails either :-(
    • CommentAuthorSubJunk
    • CommentTimeSep 26th 2008 edited
     # 82
    On the main page for Notifi it has always said "Requires Vanilla 1.1.4", why would it say that if it wasn't true?
    •  
      CommentAuthorWanderer
    • CommentTimeSep 26th 2008
     # 83
    Alright already so I missed that little snippet, never said it wasn't true, in fact my statement verified it!

    Posted: Friday, 26 September 2008 at 8:59PM

    • CommentAuthorSubJunk
    • CommentTimeSep 26th 2008
     # 84
    :)
    • CommentAuthorDaniel_SJ
    • CommentTimeSep 30th 2008
     # 85
    A few ideas and wishes if you're doing a rewrite:

    1. Shift focus from notification only when there is something new to subscribing by e-mail (but with the option to only get a notification when there is something new if the user so wishes).
    2. Send the mail with the correct headers, so it's sent as a reply to the actual thread - thus allowing for threaded e-mail notification.
    3. Send the e-mail with the from-address not as the system normal, but as the user who wrote the posts (if that e-mail is public).

    Would also love to see a different type of mail for editing comments, moving threads etc.

    Love the extension!

    Best,
    Daniel
    • CommentAuthortate
    • CommentTimeOct 6th 2008
     # 86
    I seem to running into an error. Whenever I add a new comment to a discussion, with Notifi activated, I get the error below. I ran into a similar error with other emails that Vanilla sent (such as lost password), but I added 'date_default_timezone_set('Europe/London');' to the /library/Framework/Framework.Class.Email.php file and it fixed it. I tried a similar approach with Notifi's Framework.Class.Email.php but it didn't fix the issue. Any ideas?

    Notice: Undefined variable: rolesAry in /home2/ourbigea/public_html/dev/forum/extensions/Notifi/default.php on line 296
    A fatal, non-recoverable error has occurred
    Technical information (for support personel):

    Error Message
    An error occurred while sending the email.
    Affected Elements
    Email.Send();

    The error occurred on or near: date() [function.date]: It is not safe to rely on the system's timezone settings. Please use the date.timezone setting, the TZ environment variable or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected 'America/Denver' for 'MDT/-6.0/DST' instead

    For additional support documentation, visit the Lussumo Documentation website at: lussumo.com/docs


    Thanks!
    tate
    • CommentAuthorSubJunk
    • CommentTimeOct 6th 2008
     # 87
    Are you using Notifi 1.7.3 and Vanilla 1.1.5a, or are you using older versions? The reason I ask is that if you use the latest version of both, the Framework.Class.Email.php file in the Notifi folder will be skipped, it isn't needed anymore with Vanilla 1.1.5+.
    • CommentAuthortate
    • CommentTimeOct 9th 2008
     # 88
    Hi SubJunk,

    Yes, I am using both Notifi 1.7.3 and Vanilla 1.1.5a. I tried removing the Framework.Class.Email.php from the Notifi folder and I got an include error.
    • CommentAuthorSubJunk
    • CommentTimeOct 9th 2008 edited
     # 89
    Hmmm...
    Well let's do an experiment.
    Open extensions/Notifi/default.php and find lines 273 to 279, they should be this: // Include the custom email class
    if (!isset($Configuration['DEFAULT_EMAIL_MIME_TYPE'])) {
    include_once('_includes/Framework.Class.Email.php');
    }
    else {
    $Configuration['DEFAULT_EMAIL_MIME_TYPE'] = 'text/html';
    }
    Replace them with just this: $Configuration['DEFAULT_EMAIL_MIME_TYPE'] = 'text/html';and tell me what happens, and if it does work, if the emails look normal or like a bunch of code.
    • CommentAuthortate
    • CommentTimeOct 12th 2008
     # 90
    Hi SubJunk,

    I switched out the code in default.php and I am getting the same error (below). No email was sent either.

    Notice: Undefined variable: rolesAry in /home2/ourbigea/public_html/dev/forum/extensions/Notifi/default.php on line 290

    Notice: Undefined variable: php_errormsg in /home2/ourbigea/public_html/dev/forum/library/Framework/Framework.Class.Email.php on line 165
    A fatal, non-recoverable error has occurred
    Technical information (for support personel):

    Error Message
    An error occurred while sending the email.
    Affected Elements
    Email.Send();

    For additional support documentation, visit the Lussumo Documentation website at: lussumo.com/docs

    Thanks!
    tate
    • CommentAuthorSubJunk
    • CommentTimeOct 12th 2008
     # 91
    Ok I'll whisper you from now on until we find a solution
    • CommentAuthortekhammer
    • CommentTimeOct 16th 2008
     # 92
    just tried the latest version and not getting any luck....

    when I click to subscribe to a discussion I get:

    [Thu Oct 16 18:26:35 2008] [error] [client 00.00.00.00] PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in ****/extensions/Notifi/ajax.php on line 13, referer: http://bbs.****/comments.php?DiscussionID=46&page=1
    [Thu Oct 16 18:26:35 2008] [error] [client 00.00.00.00] PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in ****/extensions/Notifi/ajax.php on line 15, referer: http://bbs.****/comments.php?DiscussionID=46&page=1
    [Thu Oct 16 18:26:35 2008] [error] [client 00.00.00.00] PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in ****/extensions/Notifi/ajax.php on line 18, referer: http://bbs.****/comments.php?DiscussionID=46&page=1
    [Thu Oct 16 18:26:35 2008] [error] [client 00.00.00.00] PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in ****/extensions/Notifi/ajax.php on line 22, referer: http://bbs.****/comments.php?DiscussionID=46&page=1


    using:

    PHP 5.1.6 (cli) (built: Jul 16 2008 19:53:00)

    if I tick the "email me every time" in my account settings, in my error_log, I get the above errors again, plus:

    [Thu Oct 16 18:41:44 2008] [error] [client 00.00.00.00] PHP Notice: Undefined index: NOTIFI_ALLOW_KEEPEMAILING in ****/extensions/Notifi/default.php on line 78, referer: http://bbs.****/account.php?u=1&PostBackAction=Notification
    • CommentAuthortekhammer
    • CommentTimeOct 16th 2008
     # 93
    sorry, to elaborate, discussions are being subscribed to, but no email is being sent out - with and without explicitly setting the smtp server in the main settings.

    there are no logs that it even attempted to send out the email.
    • CommentAuthorreedloden
    • CommentTimeOct 16th 2008 edited
     # 94
    I have Vanilla 1.1.5a install with Notifi 1.7.3, and Notifi doesn't seem to be working at all.

    Here's the PHP warnings from the apache error log:

    PHP Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /var/www/html/forum/extensions/Notifi/default.php on line 209, referer: https://localhost/forum/comments.php?DiscussionID=542&page=1
    PHP Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /var/www/html/forum/extensions/Notifi/default.php on line 221, referer: https://localhost/forum/comments.php?DiscussionID=542&page=1
    PHP Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /var/www/html/forum/extensions/Notifi/default.php on line 231, referer: https://localhost/forum/comments.php?DiscussionID=542&page=1

    Ideas?


    Updated:
    Running this MySQL statement fixes that error: ALTER TABLE `LUM_User` ADD `KeepEmailing` TINYINT( 1 ) DEFAULT 0;

    Apparently _install.php doesn't do that by default? :(
    • CommentAuthorreedloden
    • CommentTimeOct 16th 2008
     # 95
    After fixing the MySQL error, I ran into this error:

    PHP Notice: Undefined variable: rolesAry in /var/www/html/forum/extensions/Notifi/default.php on line 296, referer: https://localhost/forum/comments.php?DiscussionID=542
    PHP Notice: Undefined variable: rolesAry in /var/www/html/forum/extensions/Notifi/default.php on line 296, referer: https://localhost/forum/comments.php?DiscussionID=542
    PHP Notice: Undefined variable: rolesAry in /var/www/html/forum/extensions/Notifi/default.php on line 296, referer: https://localhost/forum/comments.php?DiscussionID=542

    I fixed it by adding:
    $rolesAry = array();

    before this block of code:
    while ($row = mysql_fetch_row($results)) {
    $rolesAry[] = $row;
    }
    • CommentAuthorSubJunk
    • CommentTimeOct 16th 2008 edited
     # 96
    tekhammer's issue was fixed, was just an oversight on his end.

    Still working on tate's problem, hopefully we can fix that soon.

    reedloden, yes declaring the array was overlooked in the last release. It has been in the code for a while but I had been waiting to do some more things before releasing another version. I guess it causes problems on some PHP installs though so I will release the update.
    As for the KeepEmailing column, yes the logic in the installer is all screwed up. It has always been a problem and I need to fix it. It's hard because the original author of the extension Notify, which Notifi is based on, used this bad method of installing and upgrading (no offense to him, I am sure he is a good programmer, we all make bad judgements) so it is hard for me to keep upgradeability from old versions while fixing the installation code. I tried once with version 1.5 and it broke the extension haha, so I need to take the time to really fix it fully.
    So, I will release a bugfix now that fixes some PHP errors and warnings and the installer problems (hopefully). The new features are still coming, but of course bugfixes are more important :)
    I will be watching this discussion closely to catch any bugs caused by the installer changes. It should be ok though.
  9.  # 97
    Uploaded version 1.7.4 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeOct 16th 2008
     # 98
    For some reason I wasn't able to upload a zip file so I had to use a rar file. Sorry for the problems that causes but I couldn't find a way to get around it.
    •  
      CommentAuthorRaize
    • CommentTimeOct 17th 2008 edited
     # 99
    Can this extension be setup so that Notification is on by default for the person who starts the discussion? For example, they get an email as soon as someone replies (the next email should not be sent until after the original poster has replied)

    I had a previous forum setup like this and it was great because it reminds new members to come back and check out the replies they recieved, instead of having them forget that they posted there
    • CommentAuthorSubJunk
    • CommentTimeOct 17th 2008
     # 100
    Yes it can, I put that option on the Account tab. In that tab, click Notification on the left, then check "Automatically subscribe me to discussions I have started". It should be selected by default.