$start = getIntFromRequest('start');
$end = getIntFromRequest('end');
-if (!$start || !$end) $z =& $report->getMonthStartArr();
+if (!$start || !$end) {
+ $z =& $report->getMonthStartArr();
+}
if (!$start) {
$start = $z[0];
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if ($start == $end) {
$error_msg .= _('Start and end dates must be different');
$start = getIntFromRequest('start');
$end = getIntFromRequest('end');
-if (!$start || !$end) $z =& $report->getMonthStartArr();
+if (!$start || !$end) {
+ $z =& $report->getMonthStartArr();
+}
if (!$start) {
$start = $z[0];
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if ($start == $end) {
$error_msg .= _('Start and end dates must be different');
$start = getIntFromRequest('start');
$end = getIntFromRequest('end');
-if (!$start || !$end) $z =& $report->getMonthStartArr();
+if (!$start || !$end) {
+ $z =& $report->getMonthStartArr();
+}
if (!$start) {
$start = $z[0];
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if ($start == $end) {
$error_msg .= _('Start and end dates must be different');
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if ($start == $end) {
$error_msg .= _('Start and end dates must be different');
$start = getIntFromRequest('start');
$end = getIntFromRequest('end');
-if (!$start || !$end) $z =& $report->getMonthStartArr();
+if (!$start || !$end) {
+ $z =& $report->getMonthStartArr();
+}
if (!$start) {
$start = $z[0];
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if ($start == $end) {
$error_msg .= _('Start and end dates must be different');
$end = getIntFromRequest('end');
$tstat = getStringFromRequest('tstat');
-if (!$start || !$end) $z =& $report->getWeekStartArr();
+if (!$start || !$end) {
+ $z =& $report->getWeekStartArr();
+}
+
if (!$start) {
$start = $z[0];
}
+
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if (!$tstat) {
$tstat='1';
$dev_id = getIntFromRequest('dev_id');
$type = getStringFromRequest('type');
-if (!$start || !$end) $z =& $report->getMonthStartArr();
+if (!$start || !$end) {
+ $z =& $report->getMonthStartArr();
+}
if (!$start) {
$start = $z[0];
if (!$end) {
$end = $z[count($z)-1];
}
-if ($end < $start) list($start, $end) = array($end, $start);
+if ($end < $start) {
+ list($start, $end) = array($end, $start);
+}
if ($typ != 'r' && $start == $end) {
$error_msg .= _('Start and end dates must be different');
}
}
- if (!$scm_changed)
+ if (!$scm_changed) {
// Don't call scm plugin update if their form wasn't displayed
// to avoid processing an apparently empty form and reset configuration
plugin_hook("scm_admin_update", $hook_params);
+ }
} elseif (getStringFromRequest('scmhook_submit')) {
$hook_params = array();
$hook_params['group_id'] = $group_id;
$data = array();
while ($row = db_fetch_array($res)) {
$data[$row[0]] = $row[1];
- if (!$firstDateInDB)
+ if (!$firstDateInDB) {
$firstDateInDB = $row[0];
+ }
}
$start = $g->getStartDate();
foreach ($headers as $header) {
if (preg_match('/^Content-Type:\s*(([^;]*)(\s*;\s*charset=(.*))?)/i', $header, $matches)) {
$content_type = $matches[2];
- if (isset($matches[4])) $charset = $matches[4];
+ if (isset($matches[4])) {
+ $charset = $matches[4];
+ }
// we'll validate content-type or transcode body below
} elseif (preg_match('/^Transfer-Encoding: chunked/', $header)) {
// curl already de-chuncked the body
$res = $renderer->searchQuery->getData($renderer->searchQuery->getRowsPerPage(),$renderer->searchQuery->getOffset());
if (count($res) > 0) {
- if ($renderer->searchQuery->getRowsTotalCount() >= $renderer->searchQuery->getRowsPerPage())
+ if ($renderer->searchQuery->getRowsTotalCount() >= $renderer->searchQuery->getRowsPerPage()) {
$result .= html_e('em', array(), sprintf(_('Note: only the first %d results for this category are displayed.'), $renderer->searchQuery->getRowsPerPage()));
+ }
$result .= $HTML->listTabletop($renderer->tableHeaders);
$result .= $renderer->getRows();
$result .= $HTML->listTableBottom();
$return .= '<fieldset>'."\n";
$return .= '<legend>';
$return .= '<input type="checkbox" id="'.$key.'_checkall" name="'.$key.'_checkall"';
- if (getStringFromRequest($key.'_checkall'))
+ if (getStringFromRequest($key.'_checkall')) {
$return .= ' checked="checked"';
+ }
$return .= ' class="checkall" /><label for="'.$key.'_checkall">'.$group_subsection_names[$key].'</label>';
$return .= "\n";
$return .= '</legend>';
if (!is_array($section)) {
$return .= ' <input type="checkbox" id="'.$key.'" name="'.$key.'"';
- if (getStringFromRequest($key))
+ if (getStringFromRequest($key)) {
$return .= ' checked="checked"';
+ }
$return .= ' class="childCheckBox" /><label for="'.$key.'">'.$group_subsection_names[$key].'</label><br />'."\n";
} else {
foreach($section as $underkey => $undersection) {
$return .= ' <input type="checkbox" id="'.$key.$underkey.'" name="'.$key.$underkey.'"';
- if (getStringFromRequest($key.$underkey))
+ if (getStringFromRequest($key.$underkey)) {
$return .= ' checked="checked"';
+ }
$return .= ' /><label for="'.$key.$underkey.'">'.$undersection.'</label><br />'."\n";
}
}