diff --git a/app/routes/crate/version.js b/app/routes/crate/version.js index 5ad67fd5d0f..a79633978b8 100644 --- a/app/routes/crate/version.js +++ b/app/routes/crate/version.js @@ -45,9 +45,7 @@ export default Route.extend({ .then(versions => { const latestStableVersion = versions.find(version => { // Find the latest version that is stable AND not-yanked. - if (!isUnstableVersion(version.get('num')) && !version.get('yanked')) { - return version; - } + return !isUnstableVersion(version.get('num')) && !version.get('yanked'); }); if (latestStableVersion == null) { @@ -55,13 +53,11 @@ export default Route.extend({ // The fact that "maxVersion" itself cannot be found means that // we have to fall back to the latest one that is unstable.... const latestUnyankedVersion = versions.find(version => { - // Find the latest version that is stable AND not-yanked. - if (!version.get('yanked')) { - return version; - } + // Find the latest version that is not-yanked. + return !version.get('yanked'); }); - if (latestStableVersion == null) { + if (latestUnyankedVersion == null) { // There's not even any unyanked version... params.version_num = maxVersion; } else {