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.
    • CommentAuthordanisal
    • CommentTimeOct 18th 2008
     # 101
    Will this plugin work with Vanilla 1.1.5?
    • CommentAuthorSubJunk
    • CommentTimeOct 19th 2008
     # 102
    Yes
    • CommentAuthorbwaje
    • CommentTimeOct 21st 2008
     # 103
    I have this in the notifi panel : "There is no submit button", and this extension is not working, when I check "Email me every time (instead of once between each forum visit)" it is not saved.
    • CommentAuthorSubJunk
    • CommentTimeOct 21st 2008
     # 104
    I will upload a new release today that should fix this problem.
    • CommentAuthorIceBone
    • CommentTimeOct 21st 2008
     # 105
    I noticed it doesn't work with the User Info extension. JS incompatibility?
  1.  # 106
    Uploaded version 1.8 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeOct 21st 2008
     # 107
    Yes you're right about that incompatibility, I will have a look at it. Thanks for letting me know.
    • CommentAuthorSubJunk
    • CommentTimeOct 21st 2008
     # 108
    From what I can see it is a conflict between Vanilla's built-in Prototype file and the MooTools extension. I will try to find exactly which part of Prototype is conflicting and if I don't need to use it I can include a custom prototype.js file to circumvent the issue.
  2.  # 109
    Uploaded version 1.8.1 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeOct 22nd 2008
     # 110
    Ok I've tested this release thoroughly so I am confident this is a good one.
    •  
      CommentAuthorspode
    • CommentTimeOct 24th 2008
     # 111
    SubJunk - great extension.

    It would be nice if there was a way of making default that any discussion you post in, or discussion you start you are automatically subscribed and that they can turn this setting off from the account settings?
    • CommentAuthorosid
    • CommentTimeOct 25th 2008
     # 112
    I'm afraid I still get errors

    Notice: Undefined index: NOTIFI_AUTO_ALL in /home/osidco/public_html/OverseasSchoolsInformationDatabase/vanilla/extensions/Notifi/default.php on line 121

    Notice: Undefined index: NOTIFI_ALLOW_ALL in /home/osidco/public_html/OverseasSchoolsInformationDatabase/vanilla/extensions/Notifi/default.php on line 123

    Notice: Undefined index: NOTIFI_ALLOW_CATEGORY in /home/osidco/public_html/OverseasSchoolsInformationDatabase/vanilla/extensions/Notifi/default.php on line 140

    Notice: Undefined index: NOTIFI_ALLOW_DISCUSSION in /home/osidco/public_html/OverseasSchoolsInformationDatabase/vanilla/extensions/Notifi/default.php on line 148

    Notice: Undefined index: NOTIFI_AUTO_ALL in /home/osidco/public_html/OverseasSchoolsInformationDatabase/vanilla/extensions/Notifi/_includes/usersettings.php on line 2

    I am happy to go back to Notify as it worked perfectly but only if the original issue re the email link can be resolved.

    The problem was that the link took people back to the ACTUAL vanilla installation, rather than the Joomla website that the bridge is installed in. I would like to edit the link to just go back to the website. I don't think it is possible to return people to the actual discussion.

    What do you think? Can I put a single link in the email?
    • CommentAuthorSubJunk
    • CommentTimeOct 25th 2008 edited
     # 113
    To osid: it would be easier to help you if you would reply to my messages. Just look up, 2 posts before yours, you will see I asked you some questions days ago.
    I won't provide support for Notify, because that is like opening pandora's box because of all the bugs, all of which have been fixed in Notifi (at least, all the ones that I've been made aware of).

    To spode: that feature does exist already, except it is just for discussions you have started. I will make another option to allow automatic subscription if you post in someone else's discussion because that is a great, useful thing to have.
  3.  # 114
    Any reason why you didn't use SqlBuilder for your queries? It would eliminate all of the table prefix junk. More importantly, it would support database column/table re-mapping -- a critical feature for integration with other apps.
    • CommentAuthorSubJunk
    • CommentTimeOct 27th 2008
     # 115
    It has always been the plan to rewrite a lot of that code. Notifi was based on Notify, which used very non-Vanilla code. I have improved a lot of it and added features and fixed bugs but there are still improvements to be made. Version 2.0 will be a rewrite and will use proper Vanilla syntax throughout.
    • CommentAuthorosid
    • CommentTimeOct 27th 2008
     # 116
    Hello again,

    I'm sorry, I can't see the questions you asked me. I really do appreciate any assistance and do not mean to be a pain. Are refering to this discussion or the 'Notify' one?

    Thanks again
    • CommentAuthorSubJunk
    • CommentTimeOct 28th 2008
     # 117
    Ah that's strange osid. I will post it publically instead of in a whisper, hopefully that will fix it. I said:

    Ok can you go into conf/settings.php and tell me if the NOTIFI fields are there? They will look something like this:$Configuration['NOTIFI_ALLOW_ALL'] = '1';
    $Configuration['NOTIFI_ALLOW_DISCUSSION'] = '1';
    $Configuration['NOTIFI_ALLOW_CATEGORY'] = '1';
    $Configuration['NOTIFI_AUTO_ALL'] = '0';
    $Configuration['NOTIFI_INSTALL_V2_COMPLETE'] = '1';
    If they aren't in there feel free to add them.
    Also I am curious, which version of Vanilla are you using?
    • CommentAuthorosid
    • CommentTimeOct 28th 2008
     # 118
    I found that code as described.

    wrt the version that is tricky. I installed it as a Joomla component but the installation would only install an old version of Vanilla. I then paid a guy to upgrade the installation to 1.1.4 so as far as I am aware that is what is there. How would I check?

    I notice there is now a 1.1.4 joomla version but I hesitate to install it as the layout of my site was edited and I don't want to disturb it.
    • CommentAuthorSubJunk
    • CommentTimeOct 28th 2008
     # 119
    That is really strange osid, because the errors you are seeing are basically just saying "we can't find the notifi configuration variables", but you said they actually are there. Was the Vanilla code edited at all? It is really best to upgrade to Vanilla 1.1.5a if at all possible, because there are security problems with 1.1.4. Hopefully that will also clear up the problems.
    Notifi does work with 1.1.4 but I guess the guy who upgraded it may not have upgraded properly, just a guess
    • CommentAuthorosid
    • CommentTimeOct 28th 2008
     # 120
    This is the code that I can see in conf/settings.php

    $Configuration['NOTIFY_SETUP'] = '1';
    $Configuration['NOTIFY_ALLOW_ALL'] = '1';
    $Configuration['NOTIFY_ALLOW_DISCUSSION'] = '1';
    $Configuration['NOTIFY_ALLOW_CATEGORY'] = '1';
    $Configuration['NOTIFY_VERSION'] = '1.2.0';
    $Configuration['NOTIFY_AUTO_ALL'] = '0';
    $Configuration['NOTIFI_INSTALL_V2_COMPLETE'] = '1';

    In default.php for instance line 121 says

    if ($Context->Session->UserID > 0 && isset($Panel) && $Context->Configuration['NOTIFI_AUTO_ALL'] == 0) {

    line 123 says

    if ($Context->Configuration['NOTIFI_ALLOW_ALL'] == 1 && in_array($Context->SelfUrl, array('comments.php','index.php','categories.php'))) {

    That is all correct?
    • CommentAuthorSubJunk
    • CommentTimeOct 29th 2008
     # 121
    Well in conf/settings.php it should also have the NOTIFI variables, as opposed to the NOTIFY variables. You see how in that list you gave me, only one of those is NOTIFI, the other 6 are for NOTIFY.
    I think the problem is because I uploaded the version of Notifi with the broken installer so it didn't set up properly. So this bug has been fixed already, you just got very unlucky with the timing. Sorry for that.
    So to fix it please copy the following into conf/settings.php:$Configuration['NOTIFI_ALLOW_ALL'] = '1';
    $Configuration['NOTIFI_ALLOW_DISCUSSION'] = '1';
    $Configuration['NOTIFI_ALLOW_CATEGORY'] = '1';
    $Configuration['NOTIFI_AUTO_ALL'] = '0';
    •  
      CommentAuthorfmimoso
    • CommentTimeOct 29th 2008 edited
     # 122
    First of all, thanks for this great extension and for your efforts. :)

    Just FYI, this extension somehow conflicts with Comment Links (as does Notify). The desired behavior in Comments Links is to show a javascript generated menu when clicked. With Notifi enabled, that menu doesn't appear. You might want to address that issue in the next version. :)
    • CommentAuthorSubJunk
    • CommentTimeOct 29th 2008
     # 123
    Thanks for letting me know mate, I will have a look
    • CommentAuthorosid
    • CommentTimeOct 29th 2008
     # 124
    Perfect! I must be blind...

    Now it is working. Thanks very much, I appreciate it :-)

    I can now return to the original issue of the link in the email.

    this is what it says...

    "The following comment was posted in the discussion notification test.
    Post by: admin

    post three

    Click here to view the comment on the forum or copy and paste the following link into your web browser:
    http://www.osid.co.uk/OverseasSchoolsInformationDatabase/vanilla/comments.php?DiscussionID=23&page=1#Item_2

    Kind regards,
    Overseas Schools Information Database "

    The trouble is that the link take the user to the Vanilla installation, not the Joomla site.

    I suspect that the Joomla bridge cannot identify the exact url of the discussion (I suspect this because the url does not change as I look at different discussions).

    What do you suggest?

    Perhaps the easiest thing to do is to simply send a fixed url back in the email (ie the url of the Joomla site - http://www.osid.co.uk/OverseasSchoolsInformationDatabase/component/option,com_vanilla/Itemid,56/ )
    • CommentAuthorSubJunk
    • CommentTimeOct 29th 2008
     # 125
    Yes it seems like the only thing to do is link to the main Joomla page, because the URL doesn't change from page to page as you said. I guess you have your reasons but in general that is very bad practice, people can't give links to friends or bookmark particular discussions in their browser. But that's not why you came here ;)
    Try replacing lines 347 to 382 of default.php with this:<html>
    <body style="background-color:#fff;">
    Hello '.$mName.',<br /><br />
    The following comment was posted in the discussion <strong>'.$discussionName.'</strong>.
    <div style="margin: 20px 0; padding: 10px; background-color: #fef9e9; border: 1px #ffedae solid;">
    <p style="padding: 5px; margin: 0 0 5px 0; background-color: #fff; border: 1px #ccc solid;">
    Post by: <strong>'.$mPosterName.'</strong>
    </p>
    '.$mComment.'<br />
    </div>
    <a href="http://www.osid.co.uk/OverseasSchoolsInformationDatabase/component/option,com_vanilla/Itemid,56/">Click here to view the forum</a> or copy and paste the following link into your web browser:<br />
    http://www.osid.co.uk/OverseasSchoolsInformationDatabase/component/option,com_vanilla/Itemid,56/<br /><br />
    Kind regards,<br />
    '.$DiscussionForm->Context->Configuration['SUPPORT_NAME'].'
    </body>
    </html>';
    }
    // If this is a new discussion
    else {
    $message = '
    <html>
    <body style="background-color:#fff;">
    Hello '.$mName.',<br /><br />
    A new discussion called <strong>'.$discussionName.'</strong> was started, the comment is as follows.
    <div style="margin: 20px 0; padding: 10px; background-color: #fef9e9; border: 1px #ffedae solid;">
    <p style="padding: 5px; margin: 0 0 5px 0; background-color: #fff; border: 1px #ccc solid;">
    Post by: <strong>'.$mPosterName.'</strong>
    </p>
    '.$mComment.'<br />
    </div>
    <a href="http://www.osid.co.uk/OverseasSchoolsInformationDatabase/component/option,com_vanilla/Itemid,56/">Click here to view the forum</a> or copy and paste the following link into your web browser:<br />
    http://www.osid.co.uk/OverseasSchoolsInformationDatabase/component/option,com_vanilla/Itemid,56/<br /><br />
    Kind regards,<br />
    '.$DiscussionForm->Context->Configuration['SUPPORT_NAME'].'
    </body>
    </html>';
    • CommentAuthorosid
    • CommentTimeOct 29th 2008
     # 126
    WOW! Spot on :-)

    I agree that not being able to link directly to a post is not ideal but if I want to run a Joomla site with the SWEETEST forum script what choice do I have?

    Many thanks for your help
    •  
      CommentAuthorspode
    • CommentTimeOct 31st 2008
     # 127
    SubJunk - any chance of making it support BBCode, so that it pre-processes it before mailing out the text to people? Looks a bit crap having [url= blahbal] instead of a link :)
    • CommentAuthorosid
    • CommentTimeOct 31st 2008
     # 128
    The emails I have received using this has both a link and a url to copy/paste.

    The following comment was posted in the discussion notification test.
    Post by: admin

    (image)

    Click here to view the forum or copy and paste the following link into your web browser:
    http://www.osid.co.uk/OverseasSchoolsInformationDatabase/component/option,com_vanilla/Itemid,56/

    Kind regards,


    Is that what you meant?
    • CommentAuthorosid
    • CommentTimeOct 31st 2008
     # 129
    That didn't come out very well...

    The 'Click here to...' is a link

    and the url is not
    • CommentAuthorSubJunk
    • CommentTimeOct 31st 2008
     # 130
    Great idea spode, I'll work that in there for sure with an option to enable/disable it.
    osid, that is the usual format of the email, did you have a question? Maybe it's just late here but I don't know what you're asking.
    • CommentAuthorosid
    • CommentTimeOct 31st 2008
     # 131
    Hi,

    I'm not asking anything. I was doing my best to respond to spode's comment.

    I've obviously got my knickers in a twist...
    • CommentAuthorSubJunk
    • CommentTimeOct 31st 2008
     # 132
    Ah I see :)
    • CommentAuthorIceBone
    • CommentTimeNov 3rd 2008
     # 133
    Also found out it breaks the Collapsible Panel extension. Is there a quick way to fix it and the User Info without having to release a new version?
    • CommentAuthorSubJunk
    • CommentTimeNov 3rd 2008 edited
     # 134
    IceBone, it is actually those extensions that are breaking Notifi, and not the other way around. The problem is that Notifi uses Prototype (which is included with Vanilla) but Collapsible Panel and User Info use a MooTools extension. MooTools and ProtoType are similar in that they both extend JavaScript, making it easier and more functional, but they conflict with eachother (they both try to change JavaScript in some similar ways so it conflicts).
    So it is the MooTools-dependent extensions that break Notifi, not Notifi that breaks them, because Notifi is using Vanilla's own built-in Prototype file :)

    I don't think there is any way to fix the conflict because it is a fundamental incompatibility with the 2 packages, so the only way to fix it would be to make Notifi work with MooTools instead of Prototype which I have no intention of doing unless MooTools is ever included in the core instead of Prototype (I don't know of any plans to do that), or for the other extension's authors to make theirs work with Prototype as they should have done in the first place, in my opinion. I hope that clears it up IceBone.
    • CommentAuthorIceBone
    • CommentTimeNov 4th 2008
     # 135
    Clear as day. I'll go nag in the other two discussions. :P
    • CommentAuthorSubJunk
    • CommentTimeNov 4th 2008
     # 136
    Haha uh oh *ducks punches from those devs*
    •  
      CommentAuthorspode
    • CommentTimeNov 5th 2008
     # 137
    SubJunk - I imagine this goes with the territory. But it would be nice if when doing BBCode rendering, it also added in line breaks for new paragraphs? It would also be nice to have a word count limit. I have people writing entire articles and posting them. I don't want the whole thing going through. Being able to limit it to 150 words would be useful :) If you need/want any help with coding btw - feel free to get in touch.
    • CommentAuthorSubJunk
    • CommentTimeNov 5th 2008
     # 138
    Great ideas again spode. I've already made most of the features suggested by you and other people and will be releasing them along with a mostly rewritten extension, with version 2.0. I will try to work in those things too :)
    • CommentAuthorIceBone
    • CommentTimeNov 6th 2008
     # 139
    Do we have an ETA on that one? :)
    • CommentAuthorSubJunk
    • CommentTimeNov 6th 2008
     # 140
    Naw you will just be disappointed when the ETA probably passes by ;) won't be too long though.
    •  
      CommentAuthorspode
    • CommentTimeNov 9th 2008
     # 141
    I know the feeling SubJunk!
    • CommentAuthorjmcnally
    • CommentTimeNov 20th 2008 edited
     # 142
    I wanted to second the suggestions from Daniel_SJ from September 30th (which weren't answered, btw) that it would be great if the email sent had the commenter listed in the From: field and the subject of the comment in the Subject: field.

    Another complaint our users were making is that the emails sent are HTML-formatted. Is there an easy way to choose text-only messages?

    Thanks for a very useful extension!
    • CommentAuthorSubJunk
    • CommentTimeNov 20th 2008
     # 143
    Well I could make an option to send in HTML format or plaintext. No problem.
    • CommentAuthorjmcnally
    • CommentTimeNov 21st 2008
     # 144
    Thanks, SubJunk. Are you releasing any other versions before 2.0?
    • CommentAuthorSubJunk
    • CommentTimeNov 21st 2008
     # 145
    I wasn't planning on it because I think the current version is stable, but if any bugs are discovered I will release a bugfix ASAP.
  4.  # 146
    Uploaded version 1.8.2 of Notifi.
    • CommentAuthorSubJunk
    • CommentTimeNov 24th 2008
     # 147
    This is just a quick bugfix release; it is always my policy to fix bugs ASAP but don't worry there are still great features coming with version 2.0 soon :)
    • CommentAuthoravital
    • CommentTimeNov 24th 2008
     # 148
    Hi SubJunk.

    Great add-on, thanks! Just wanted to know if you have any plans on releasing a new version with spode's suggestion that you are automatically subscribed to any discussion you post in? If you're busy I'd me happy to figure out how to make that work and send you a patch. Just let me know if you're working on it...

    Avital.
    • CommentAuthoravital
    • CommentTimeNov 24th 2008
     # 149
    Another thing - If a forum doesn't have categories enabled it still shows the option to 'Subscribe to category' on the left panel.

    Again, as before, I'd be happy to change this -- just let me know if you want me to hack at it.
    • CommentAuthorSubJunk
    • CommentTimeNov 24th 2008
     # 150
    Hi avital, yes that feature will be out in version 2.0, most suggestions people have made will be released with that version. Of course if you don't want to wait you can feel free to add whatever features you want, that's what open source is all about :)
    Can you explain what you mean by "doesn't have categories enabled"? I didn't realize that option existed, or is it an extension that allows that to happen?