From d482264b6af8598ce50677ac6a8f781e219b7d6f Mon Sep 17 00:00:00 2001 From: Jim Hu Date: Sun, 28 Dec 2008 19:30:41 +0000 Subject: fix misc bugs; rewrite some places where warnings squelched with @; require php5.1 or greater --- functions/calendar_functions.php | 4 ++-- functions/date_functions.php | 38 ++----------------------------------- functions/draw_functions.php | 22 ++++++++++----------- functions/init.inc.php | 7 ++++--- functions/parse/end_vevent.php | 13 ++++++------- functions/parse/parse_tzs.php | 4 ++-- functions/parse/recur_functions.php | 3 ++- functions/template.php | 2 +- 8 files changed, 30 insertions(+), 63 deletions(-) (limited to 'functions') diff --git a/functions/calendar_functions.php b/functions/calendar_functions.php index 8f02bd7..ef423b6 100644 --- a/functions/calendar_functions.php +++ b/functions/calendar_functions.php @@ -73,9 +73,9 @@ function availableCalendars($username, $password, $cal_filename, $admin = false) if ($find_all || $phpiCal_config->recursive_path == 'yes' || $phpiCal_config->support_ical == 'yes') { // Open the directory. $dir_handle = opendir($search_path) - or die(error("cal fn 78:".sprintf($lang['l_error_path'], $search_path), implode(',', $cal_filename))); + or die(error(sprintf($lang['l_error_path'], $search_path), implode(',', $cal_filename))); if ($dir_handle === false) - die(error(" cal fn 80:".sprintf($lang['l_error_path'], $search_path), implode(',', $cal_filename))); + die(error(sprintf($lang['l_error_path'], $search_path), implode(',', $cal_filename))); // Add each file in the directory that does not begin with a dot. while (false !== ($file = readdir($dir_handle))) { diff --git a/functions/date_functions.php b/functions/date_functions.php index c03c5ae..fa8cdd9 100644 --- a/functions/date_functions.php +++ b/functions/date_functions.php @@ -1,18 +1,8 @@ 0) return $key; } return $data; -} - -require_once(BASE."functions/is_daylight.php"); -/* function is_daylight($date, $timezone) returns 1 if daylight time, 0 if not - - default is to use the server's date function. This will be off when the timezone's rules are not the same as the server's rules. In php5.2+ there seems to be a better way to do this, but we can't count on users having php5.2+. - - Although we set dt_start and st_start in parse_tzs.php, these are not rrules and I don't know how to use them yet. So we'll do it by brute force for the ones we know about, from: http://www.webexhibits.org/daylightsaving/g.html - - Note that this sends a screwy time value - it's not necessarily UTC unixtime, since the mktime functions that create the time are not using the timezone. - -function is_daylight($time, $timezone){ - global $tz_array; - # default to std time, overwrite if daylight. - $dlst = 0; - switch ($timezone){ - default: - $dlst = date('I', $time); - } - - return $dlst; - -} -*/ -?> \ No newline at end of file +}?> \ No newline at end of file diff --git a/functions/draw_functions.php b/functions/draw_functions.php index a5a98a9..77564c9 100644 --- a/functions/draw_functions.php +++ b/functions/draw_functions.php @@ -1,21 +1,23 @@ gridLength; - preg_match ('/([0-9]{2})([0-9]{2})/', $start, $time); - $sta_h = @$time[1]; - $sta_min = @$time[2]; + if (preg_match ('/([0-9]{2})([0-9]{2})/', $start, $time)){ + $sta_h = $time[1]; + $sta_min = $time[2]; + } $sta_min = sprintf("%02d", floor($sta_min / $gridLength) * $gridLength); if ($sta_min == 60) { $sta_h = sprintf("%02d", ($sta_h + 1)); $sta_min = "00"; } - preg_match ('/([0-9]{2})([0-9]{2})/', $end, $time); - $end_h = @$time[1]; - $end_min = @$time[2]; + if (preg_match ('/([0-9]{2})([0-9]{2})/', $end, $time)){ + $end_h = $time[1]; + $end_min = $time[2]; + } $end_min = sprintf("%02d", floor($end_min / $gridLength) * $gridLength); if ($end_min == 60) { $end_h = sprintf("%02d", ($end_h + 1)); @@ -28,10 +30,8 @@ function drawEventTimes ($start, $end) { $end_h = sprintf("%02d", ($end_h + 1)); $end_min = "00"; } - } - + } $draw_len = ($end_h * 60 + $end_min) - ($sta_h * 60 + $sta_min); - return array ("draw_start" => ($sta_h . $sta_min), "draw_end" => ($end_h . $end_min), "draw_length" => $draw_len); } diff --git a/functions/init.inc.php b/functions/init.inc.php index 8d15778..1b227cc 100644 --- a/functions/init.inc.php +++ b/functions/init.inc.php @@ -11,11 +11,12 @@ include_once(BASE.'error.php'); include_once(BASE.'functions/calendar_functions.php'); include_once(BASE.'functions/userauth_functions.php'); - +# require php 5 +if (phpversion() < '5.1') die (error(sprintf($lang['l_php_version_required'],phpversion()) ) ); // Grab the action (login or logout). +$action = ''; if (isset($_GET['action'])) $action = $_GET['action']; -else if (isset($_POST['action'])) $action = $_POST['action']; -else $action = ''; +else if (isset($_POST['action'])) $action = $_POST['action']; // Login and/or logout. list($username, $password, $invalid_login) = user_login(); diff --git a/functions/parse/end_vevent.php b/functions/parse/end_vevent.php index 9a13bbe..27f7bd3 100644 --- a/functions/parse/end_vevent.php +++ b/functions/parse/end_vevent.php @@ -177,8 +177,9 @@ foreach ($rrule_array as $key => $val) { break; } } +# convert wkst to a 3 char day for strtotime to work +$wkst3char = two2threeCharDays($wkst); if($current_view == 'search') $freq_type = 'none'; - # $recur is the recurrence info that goes into the master array for this VEVENT $recur = @$recur_array[($start_date)][($hour.$minute)][$uid]['recur']; @@ -206,16 +207,16 @@ $start_date_unixtime = strtotime($start_date); $next_range_unixtime = $start_date_unixtime; # conditions where we don't need to iterate over the whole range -# if repeating without limit, and steps are by 1, don't go back before the mArray beginning. +# if repeating without limit, and steps are by 1, don't go back before the mArray beginning. if($count == 1000000 && $interval == 1 && $mArray_begin > $next_range_unixtime) $next_range_unixtime = $mArray_begin; -# if the beginning of our range is less than the start of the item, we may as well set the range to start at start_time +# if the beginning of our range is less than the start of the item, we may as well set the range to start at start_time if ($next_range_unixtime < $start_date_unixtime) $next_range_unixtime = $start_date_unixtime; -# stop at the until limit if set +# stop at the until limit if set if(isset($until) && $end_range_unixtime > $until_unixtime) $end_range_unixtime = $until_unixtime; -# more adjustments +# more adjustments switch ($freq_type){ case 'week': # need to get the first value of $next_range_unixtime onto the right day of the week @@ -231,8 +232,6 @@ if(!isset($rrule_array['FREQ']) && isset($end_date)){ $end_range_unixtime = strtotime($end_date); $count = 1; } -// convert wkst to a 3 char day for strtotime to work -$wkst3char = two2threeCharDays($wkst); /* The while loop below increments $next_range_time by $freq type. For the larger freq types, there is only one $next_range_time per repeat, but the BYXXX rules may write more than one event in that repeat cycle diff --git a/functions/parse/parse_tzs.php b/functions/parse/parse_tzs.php index a135b0f..8b02cb5 100644 --- a/functions/parse/parse_tzs.php +++ b/functions/parse/parse_tzs.php @@ -1,8 +1,8 @@ template","templates/default",$file); } if (!is_file($file)){ - exit(error($lang['l_error_path']." template 1065", $file)); + exit(error($lang['l_error_path'], $file)); } ob_start(); include($file); -- cgit v1.2.3