Noosfero/noosfero

File chat.js has 856 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/* XMPP/Jabber Noosfero's client

XMPP Core:
http://xmpp.org/rfcs/rfc3920.html

Function on_roster has 42 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    on_roster: function (iq) {
      log('receiving roster');
      var profiles = [];
      var contacts_to_insert = {};
      var groups_to_insert = [];

Function parse has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    parse: function (stanza) {
      var result = {};
      if (Strophe.isTagEqual(stanza, 'presence')) {
        result.from = $(stanza).attr('from');
        result.type = $(stanza).attr('type');

Function connect has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    connect: function() {
      if (("Notification" in window) && Notification.permission !== "granted" && Notification.permission !== "denied") {
        Notification.requestPermission(function (permission) {
          if (!('permission' in Notification)) {
            Notification.permission = permission;

Function on_connect has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    on_connect: function (status) {
      switch (status) {
        case Strophe.Status.CONNECTING:
          log('connecting...');
        break;

Function create_conversation_tab has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  function create_conversation_tab(title, jid_id) {
    var conversation_id = Jabber.conversation_prefix + jid_id;
    var conversation = $('#' + conversation_id);
    if (conversation.length > 0) {
      return conversation;

Function success has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        success: function(data){
          $(data.friends).each(function(index, friend){
            var jid = friend.jid;
            profiles.push(getIdentifier(jid));
            var name = friend.name;

Function show_message has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    show_message: function (jid, name, body, who, identifier, time, offset) {
      if(!offset) offset = 0;
      if (body) {
        body = Jabber.render_body_message(body);
        var jid_id = Jabber.jid_to_id(jid);

Function on_presence has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    on_presence: function (presence) {
      presence = Jabber.parse(presence);
      if (presence.type != 'error') {
        if (presence.is_from_room) {
          log('receiving room presence from ' + presence.from + ' as ' + presence.show);

Function insert_or_update_user has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

    insert_or_update_user: function (list, item, jid, name, presence, template, type, remove_on_offline) {

Function show_message has 7 arguments (exceeds 4 allowed). Consider refactoring.

Open

    show_message: function (jid, name, body, who, identifier, time, offset) {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  jQuery.expr[':'].Contains = function(a,i,m){
    return (a.textContent || a.innerText || "").toUpperCase().indexOf(m[3].toUpperCase())>=0;
  };

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      presence = presence || ($(item).length > 0 ? $(item).parent('li').attr('class') : 'offline');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      presence = presence || ($(item).length > 0 ? $(item).parent('li').attr('class') : 'offline');

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

      $("#chat-window .tab a[href='#"+ Jabber.conversation_prefix + jid_id +"']")
      .removeClass()
      .addClass('icon-menu-' + presence + '-11');

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

      $("#chat-window .tab a[href='#"+ Jabber.conversation_prefix + jid_id +"']")
      .removeClass()
      .addClass('icon-menu-' + presence + '-11');

• Read up
  Read up