I've managed to figure out the reason for this and wish to inform the project maintainer's and programmers that there's a small issue with the jGrowl Notification.

There's something thats changed in drupal6.x version of jGrowl library. Please do update the library with the drupal 7 version's library. It seems to work now.

Comments

julien’s picture

julien commented

Can you be more descriptive? like your versions for example, because it has seems to work for me with the version included.

There's something thats changed in drupal6.x version of jGrowl library. Please do update the library with the drupal 7 version's library.

I'm confused. I don't understand where you see a difference, as they are both on 6.x-1.x-dev and 7.x-1.x-dev, a version of 1.2.5

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Active
julien’s picture

julien commented

Made a diff on the jgrowl libraries files on both version and i can see no difference.

evvolutions’s picture

evvolutions commented

I found that the codes for javascript file in nodejs.notify.js in nodejs-6.x-1.x-dev as below.

(function ($) {

Drupal.Nodejs.callbacks.nodejsNotify = {
  callback: function (message) {
    $.jGrowl(message.data.body, {header: message.data.subject, life:(Drupal.settings.nodejs_notify.notification_time*1000)});
  }
};

})(jQuery);

and in the same file in version nodejs-7.x-1.0-beta6 as shown below.


(function ($) {

Drupal.Nodejs.callbacks.nodejsNotify = {  
  callback: function (message) {
    var notifyTime = Drupal.settings.nodejs_notify.notification_time;
    if (notifyTime > 0) {
      $.jGrowl(message.data.body, {header: message.data.subject, life:(notifyTime * 1000)});
    }
    else {
      $.jGrowl(message.data.body, {header: message.data.subject, sticky:true});
    }
  }
};

})(jQuery);

// vi:ai:expandtab:sw=2 ts=2

I noticed this change and replaced all the javascript files of (library and nodejs.notify.js) in nodejs_notify module and jGrowl notification worked.

julien’s picture

julien commented

Thank you, the nodejs_notify.js file have been updated and commited here: http://drupalcode.org/project/nodejs.git/commit/d38ef11

evvolutions’s picture

evvolutions commented
Status: Active » Closed (fixed)