yify-pop/yify-pop

View on GitHub

Showing 23 of 29 total issues

Avoid deeply nested control flow statements.
Open

               if ( [tmp[0]] == "dn" ) { fileName = tmp[1]; }
Severity: Major
Found in app/helpers/streams.js - About 45 mins to fix

    Avoid deeply nested control flow statements.
    Open

                          if (fileName.substr(i) == '.srt') { // Only unzip the srt file
                            var dir = "public/subtitles/" + yifyResponse.data.movie.title + '/';
                            zip.extractEntryTo(fileName, dir , false, true);
                            fs.renameSync(dir + fileName, dir + lang + '.srt'); // Rename to language.srt
                          }
    Severity: Major
    Found in app/helpers/streams.js - About 45 mins to fix

      Avoid deeply nested control flow statements.
      Open

                      for (var subs in yifySubsResponse.subs) {
                        for (var lang in yifySubsResponse.subs[subs]) {
                          var subUrl = 'http://www.yifysubtitles.com' + _.max(yifySubsResponse.subs[subs][lang], function(s){return s.rating;}).url
                          fetchSub(subUrl, 'public/subtitles/' + lang + '.zip', lang, unzip);
                          // Build the subtitle url
      Severity: Major
      Found in app/helpers/streams.js - About 45 mins to fix
        Severity
        Category
        Status
        Source
        Language