# HG changeset patch
# User Waqas Hussain <waqas20@gmail.com>
# Date 1311542500 -18000
# Node ID d54dbbfda3bb4f473c00c22ee9fc2f65cd25c7ce
# Parent  fec9bc52aa422a4242a09d50d917990095b131ab
mod_muc_log_http: Simplified calculating day count for a month.

diff -r fec9bc52aa42 -r d54dbbfda3bb mod_muc_log_http/muc_log_http/mod_muc_log_http.lua
--- a/mod_muc_log_http/muc_log_http/mod_muc_log_http.lua	Mon Jul 25 02:21:40 2011 +0500
+++ b/mod_muc_log_http/muc_log_http/mod_muc_log_http.lua	Mon Jul 25 02:21:40 2011 +0500
@@ -121,35 +121,19 @@
 end
 
 -- Calendar stuff
-local function getDaysForMonth(month, year)
-	local daysCount = 30;
-	local leapyear = false;
-
-	if year%4 == 0 and year%100 == 0 then
-		if year%400 == 0 then
-			leapyear = true;
-		else
-			leapyear = false; -- turn of the century but not a leapyear
-		end
-	elseif year%4 == 0 then
-		leapyear = true;
+local function get_days_for_month(month, year)
+	if month == 2 then
+		local is_leap_year = (year % 4 == 0 and year % 100 ~= 0) or year % 400 == 0;
+		return is_leap_year and 29 or 28;
+	elseif (month < 8 and month%2 == 1) or (month >= 8 and month%2 == 0) then
+		return 31;
 	end
-
-	if month == 2 and leapyear then
-		daysCount = 29;
-	elseif month == 2 and not leapyear then
-		daysCount = 28;
-	elseif  month < 8 and month%2 == 1 or
-		month >= 8 and month%2 == 0
-	then
-		daysCount = 31;
-	end
-	return daysCount;
+	return 30;
 end
 
 local function createMonth(month, year, dayCallback)
 	local htmlStr = html.month.header;
-	local days = getDaysForMonth(month, year);
+	local days = get_days_for_month(month, year);
 	local time = os_time{year=year, month=month, day=1};
 	local dow = tostring(os_date("%a", time))
 	local title = tostring(os_date("%B", time));