You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@community.apache.org by se...@apache.org on 2015/10/29 00:34:07 UTC

svn commit: r1711164 - /comdev/reporter.apache.org/trunk/site/render_proposed.js

Author: sebb
Date: Wed Oct 28 23:34:07 2015
New Revision: 1711164

URL: http://svn.apache.org/viewvc?rev=1711164&view=rev
Log:
Tidy up

Modified:
    comdev/reporter.apache.org/trunk/site/render_proposed.js

Modified: comdev/reporter.apache.org/trunk/site/render_proposed.js
URL: http://svn.apache.org/viewvc/comdev/reporter.apache.org/trunk/site/render_proposed.js?rev=1711164&r1=1711163&r2=1711164&view=diff
==============================================================================
--- comdev/reporter.apache.org/trunk/site/render_proposed.js (original)
+++ comdev/reporter.apache.org/trunk/site/render_proposed.js Wed Oct 28 23:34:07 2015
@@ -84,10 +84,7 @@ function everyMonth(s) {
 	if (s.indexOf('Next month') == 0) {
 		return true
 	}
-	if (s == 'Every month') {
-		return true
-	}
-	return false
+	return s == 'Every month'
 }
 
 var m = ['January', 'February', 'March', 'April', 'May', 'June', 'July', 'August', 'September', 'October', 'November', 'December']
@@ -234,7 +231,6 @@ function PMCchanges(json, pmc, after) {
         } else {
             changes.innerHTML += "<h5>Changes within the last 3 months:</h5>"
         }
-        var l = 0; // number of recent additions found
 
         // pre-flight check
         var c = 0; // total number of pmc members
@@ -258,12 +254,11 @@ function PMCchanges(json, pmc, after) {
                 npn = entry[0];  // full name
             }
             if (entry[1] > afterTime) {
-                l++;
                 changes.innerHTML += "&rarr; " + entry[0] + " was added to the PMC on " + new Date(entry[1] * 1000).toDateString() + "<br>";
                 addLine(pmc, (npmc > 1 ? "   " : "") + " - " + entry[0] + " was added to the PMC on " + new Date(entry[1] * 1000).toDateString())
             }
         }
-        if (l == 0) {
+        if (npmc == 0) {
             addLine(pmc, " - No new PMC members added in the last 3 months")
             changes.innerHTML += "&rarr; <font color='red'><b>No new PMC members in the last 3 months.</b></font><br>";
         }