talho/openphin

View on GitHub

Showing 3,486 of 3,486 total issues

Function initData has 45 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    initData : function(newStore, newColModel) {
        var me = this;
        
        if (me.ds) {
            var oldStore = me.ds;
Severity: Minor
Found in app/assets/javascripts/ext/src/widgets/grid/GridView.js - About 1 hr to fix

    Function ButtonToggleMgr has 45 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

    Ext.ButtonToggleMgr = function(){
       var groups = {};
    
       function toggleGroup(btn, state){
           if(state){
    Severity: Minor
    Found in app/assets/javascripts/ext/src/widgets/Button.js - About 1 hr to fix

      Function renderRows has 45 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          renderRows : function(startRow, endRow) {
              var grid          = this.grid,
                  rows          = grid.extractData(),
                  rowCount      = rows.length,
                  templates     = this.templates,
      Severity: Minor
      Found in app/assets/javascripts/ext/src/widgets/grid/PivotGridView.js - About 1 hr to fix

        Function insertHtml has 45 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

                insertHtml : function(where, el, html){
                    var hash = {},
                        hashVal,
                        setStart,
                        range,
        Severity: Minor
        Found in app/assets/javascripts/ext/src/ext-core/src/core/DomHelper.js - About 1 hr to fix

          Function initComponent has 45 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

            initComponent: function () {
              
              this.store.each(function (record) { 
                var test = record;
              });
          Severity: Minor
          Found in app/assets/javascripts/ext_extensions/FlashGraph.js - About 1 hr to fix

            Function functor has 45 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

                functor = dominoes.functor = dataHolder( function( _id , func ) {
                
                    var parts = /^([^$()]+)(?:\(([|SOF+]*)\))?$/.exec( _id );
                    
                    if ( parts ) {
            Severity: Minor
            Found in app/assets/javascripts/lib/dominoes-1.0-rc2.js - About 1 hr to fix

              Function parseChain has 45 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

              function parseChain( chain ) {
                  
                  chain = chain.split( R_DELIM );
                  
                  var i = 0,
              Severity: Minor
              Found in app/assets/javascripts/lib/dominoes-1.0-rc2.js - About 1 hr to fix

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

                            if(s.height && s.height != 'auto'){
                                h = parseFloat(s.height);
                                if(me.isBorderBox()){
                                   h -= me.getFrameWidth('tb');
                                }
                Severity: Major
                Found in app/assets/javascripts/ext/src/core/Element.style-more.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/core/Element.style-more.js on lines 288..293

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                        if ( !event.which && ((event.charCode || event.charCode === 0) ? event.charCode : event.keyCode) )
                            event.which = event.charCode || event.keyCode;
                Severity: Major
                Found in app/assets/javascripts/jquery-tooltip/lib/jquery.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/jquery.js on lines 2746..2747

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                            if(s.width && s.width != 'auto'){
                                w = parseFloat(s.width);
                                if(me.isBorderBox()){
                                   w -= me.getFrameWidth('lr');
                                }
                Severity: Major
                Found in app/assets/javascripts/ext/src/core/Element.style-more.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/core/Element.style-more.js on lines 295..300

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                    destroy : function(){
                        Ext.dd.DropTarget.superclass.destroy.call(this);
                        if(this.containerScroll){
                            Ext.dd.ScrollManager.unregister(this.el);
                        }
                Severity: Major
                Found in app/assets/javascripts/ext/src/dd/DropTarget.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/dd/DragZone.js on lines 135..140

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                        ct.slideOut('t', {
                            callback : function(){
                               this.animating = false;
                               Ext.callback(callback);
                            },
                Severity: Major
                Found in app/assets/javascripts/ext/src/widgets/tree/TreeNodeUI.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/widgets/tree/TreeNodeUI.js on lines 374..381

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                    setCls : function(cls){
                        var old = this.attributes.cls;
                        this.attributes.cls = cls;
                        if(this.rendered){
                            this.ui.onClsChange(this, cls, old);
                Severity: Major
                Found in app/assets/javascripts/ext/src/widgets/tree/TreeNode.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/widgets/tree/TreeNode.js on lines 285..291

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                    destroy : function(){
                        Ext.dd.DragZone.superclass.destroy.call(this);
                        if(this.containerScroll){
                            Ext.dd.ScrollManager.unregister(this.el);
                        }
                Severity: Major
                Found in app/assets/javascripts/ext/src/dd/DragZone.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/dd/DropTarget.js on lines 116..121

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                    setIconCls : function(cls){
                        var old = this.attributes.iconCls;
                        this.attributes.iconCls = cls;
                        if(this.rendered){
                            this.ui.onIconClsChange(this, cls, old);
                Severity: Major
                Found in app/assets/javascripts/ext/src/widgets/tree/TreeNode.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/widgets/tree/TreeNode.js on lines 334..340

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                        ct.slideIn('t', {
                           callback : function(){
                               this.animating = false;
                               Ext.callback(callback);
                            },
                Severity: Major
                Found in app/assets/javascripts/ext/src/widgets/tree/TreeNodeUI.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/ext/src/widgets/tree/TreeNodeUI.js on lines 408..415

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

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

                        if ( !event.which && ((event.charCode || event.charCode === 0) ? event.charCode : event.keyCode) )
                            event.which = event.charCode || event.keyCode;
                Severity: Major
                Found in app/assets/javascripts/jquery.js and 1 other location - About 1 hr to fix
                app/assets/javascripts/jquery-tooltip/lib/jquery.js on lines 2111..2112

                Duplicated Code

                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                Tuning

                This issue has a mass of 71.

                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                Refactorings

                Further Reading

                Function click has 44 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                            click: function(column, grid, rowIndex, e) {
                              if(this.currentInvitationId != store.getAt(rowIndex).data['id']) {
                                this.currentInvitationId = store.getAt(rowIndex).data['id'];
                                header.setText(store.getAt(rowIndex).data['name']);
                                newstore = new Ext.data.JsonStore({
                Severity: Minor
                Found in app/assets/javascripts/invitations/Invitations.js - About 1 hr to fix

                  Function initComponent has 44 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                    initComponent: function(){
                      var tpl = new Ext.XTemplate(
                        '<tpl for=".">',
                          '<div class="org-row" style="border:1px solid black;line-height:30px;padding:5px;margin:5px 0px">',
                            'Organization - {organization}, User - {name}, Email - <a href="mailto:{email}">{email}</a>',

                    Function appendChild has 44 lines of code (exceeds 25 allowed). Consider refactoring.
                    Open

                        appendChild : function(node){
                            var multi = false;
                            if(Ext.isArray(node)){
                                multi = node;
                            }else if(arguments.length > 1){
                    Severity: Minor
                    Found in app/assets/javascripts/ext/src/data/Tree.js - About 1 hr to fix
                      Severity
                      Category
                      Status
                      Source
                      Language