CaffGeek/MBACNationals

View on GitHub

Showing 7,504 of 7,504 total issues

Method SendChangeNotification has 31 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    public static void SendChangeNotification(DateTime cutoff, List<String> recipients, string title, string message)
    {
        Syslog.Initialize("logs2.papertrailapp.com", 10236);
        Syslog.Info("SendChangeNotification");
        Syslog.Info($"apikey: {apiKey}");
Severity: Minor
Found in Aggregates/Emailer.cs - About 1 hr to fix

    Method Quit has 31 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

      function Quit($close_on_error=true) {
        $this->error = null; # so there is no confusion
    
        if(!$this->connected()) {
          $this->error = array(
    Severity: Minor
    Found in Web.Admin/2014/wordpress/wp-includes/class-smtp.php - About 1 hr to fix

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

              [Authorize(Roles = "Admin, Host")]
              [HttpPost]
              [OutputCache(NoStore = true, Duration = 0, VaryByParam = "None")]
              public void Delete(string year, string id)
              {
      Severity: Major
      Found in Web.Admin/Controllers/CentresController.cs and 2 other locations - About 1 hr to fix
      Web.Admin/Controllers/HotelsController.cs on lines 83..95
      Web.Admin/Controllers/SponsorsController.cs on lines 63..75

      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 104.

      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 3 locations. Consider refactoring.
      Open

              [Authorize(Roles = "Admin, Host")]
              [HttpPost]
              [OutputCache(NoStore = true, Duration = 0, VaryByParam = "None")]
              public void Delete(string year, string id)
              {
      Severity: Major
      Found in Web.Admin/Controllers/SponsorsController.cs and 2 other locations - About 1 hr to fix
      Web.Admin/Controllers/CentresController.cs on lines 63..75
      Web.Admin/Controllers/HotelsController.cs on lines 83..95

      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 104.

      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 3 locations. Consider refactoring.
      Open

              [Authorize(Roles = "Admin, Host")]
              [HttpPost]
              [OutputCache(NoStore = true, Duration = 0, VaryByParam = "None")]
              public void Delete(string year, string id)
              {
      Severity: Major
      Found in Web.Admin/Controllers/HotelsController.cs and 2 other locations - About 1 hr to fix
      Web.Admin/Controllers/CentresController.cs on lines 63..75
      Web.Admin/Controllers/SponsorsController.cs on lines 63..75

      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 104.

      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 6 locations. Consider refactoring.
      Open

          function get_dc_users()
          {
              global $wpdb;
              // General Housekeeping
              $dcdb = new wpdb(get_option('dcuser'), get_option('dcpass'), get_option('dcname'), get_option('dchost'));
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/dotclear.php and 5 other locations - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 116..126
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 154..164
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 78..94
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 96..113
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 141..151

      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 105.

      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 6 locations. Consider refactoring.
      Open

          function get_txp_cats()
          {
              global $wpdb;
              // General Housekeeping
              $txpdb = new wpdb(get_option('txpuser'), get_option('txppass'), get_option('txpname'), get_option('txphost'));
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/textpattern.php and 5 other locations - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 116..126
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 128..139
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 154..164
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 96..113
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 141..151

      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 105.

      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 6 locations. Consider refactoring.
      Open

          function get_txp_users()
          {
              global $wpdb;
              // General Housekeeping
              $txpdb = new wpdb(get_option('txpuser'), get_option('txppass'), get_option('txpname'), get_option('txphost'));
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/textpattern.php and 5 other locations - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 116..126
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 128..139
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 154..164
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 78..94
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 141..151

      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 105.

      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 6 locations. Consider refactoring.
      Open

          function get_dc_cats()
          {
              global $wpdb;
              // General Housekeeping
              $dcdb = new wpdb(get_option('dcuser'), get_option('dcpass'), get_option('dcname'), get_option('dchost'));
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/dotclear.php and 5 other locations - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 128..139
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 154..164
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 78..94
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 96..113
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 141..151

      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 105.

      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 6 locations. Consider refactoring.
      Open

          function get_txp_comments()
          {
              global $wpdb;
              // General Housekeeping
              $txpdb = new wpdb(get_option('txpuser'), get_option('txppass'), get_option('txpname'), get_option('txphost'));
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/textpattern.php and 5 other locations - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 116..126
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 128..139
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 154..164
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 78..94
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 96..113

      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 105.

      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 6 locations. Consider refactoring.
      Open

          function get_dc_comments()
          {
              global $wpdb;
              // General Housekeeping
              $dcdb = new wpdb(get_option('dcuser'), get_option('dcpass'), get_option('dcname'), get_option('dchost'));
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/dotclear.php and 5 other locations - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 116..126
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 128..139
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 78..94
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 96..113
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 141..151

      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 105.

      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

          function update( $new_instance, $old_instance ) {
              $instance = $old_instance;
              $instance['title'] = strip_tags($new_instance['title']);
              $instance['number'] = (int) $new_instance['number'];
              $this->flush_widget_cache();
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-includes/default-widgets.php and 1 other location - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-includes/default-widgets.php on lines 653..664

      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 105.

      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

          function update( $new_instance, $old_instance ) {
              $instance = $old_instance;
              $instance['title'] = strip_tags($new_instance['title']);
              $instance['number'] = (int) $new_instance['number'];
              $this->flush_widget_cache();
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-includes/default-widgets.php and 1 other location - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-includes/default-widgets.php on lines 564..575

      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 105.

      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

              switch ($step)
              {
                  default:
                  case 0 :
                      $this->greet();
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/textpattern.php and 1 other location - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/dotclear.php on lines 677..703

      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 105.

      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

              switch ($step)
              {
                  default:
                  case 0 :
                      $this->greet();
      Severity: Major
      Found in Web.Admin/2014/wordpress/wp-admin/import/dotclear.php and 1 other location - About 1 hr to fix
      Web.Admin/2014/wordpress/wp-admin/import/textpattern.php on lines 650..676

      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 105.

      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

      function install_themes_featured($page = 1) {
          global $theme_field_defaults;
          $args = array('browse' => 'featured', 'page' => $page, 'fields' => $theme_field_defaults);
          $api = themes_api('query_themes', $args);
          if ( is_wp_error($api) )
      Web.Admin/2014/wordpress/wp-admin/includes/theme-install.php on lines 251..258

      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 105.

      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

      function install_themes_new($page = 1) {
          global $theme_field_defaults;
          $args = array('browse' => 'new', 'page' => $page, 'fields' => $theme_field_defaults);
          $api = themes_api('query_themes', $args);
          if ( is_wp_error($api) )
      Web.Admin/2014/wordpress/wp-admin/includes/theme-install.php on lines 234..241

      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 105.

      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 _readAs has 30 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          function _readAs(file, fn, as, encoding){
              if( api.isBlob(file) && _hasSupportReadAs(as) ){
                  var Reader = new FileReader;
      
                  // Add event listener
      Severity: Minor
      Found in Web.Admin/z-scripts/FileAPI.js - About 1 hr to fix

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

                toPlainData: function (fn){
                    this._to({}, fn, function (file, data, queue){
                        if( file.file ){
                            data.type = file.file;
                        }
        Severity: Minor
        Found in Web.Admin/z-scripts/FileAPI.js - About 1 hr to fix

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

                  function editTeam(team) {
                      var dfd = $q.defer();
                      
                      team.Bowlers = team.Bowlers || [];
          
          
          Severity: Minor
          Found in Web.Admin/AdminApp/Controllers/controller.contingent.js - About 1 hr to fix
            Severity
            Category
            Status
            Source
            Language