Commit f76acb2d authored by webchick's avatar webchick
Browse files

#282405 by Damien Tournoud, lilou, Dave Reid: Enforce coding standard on elseif.

parent a765e882
......@@ -168,7 +168,7 @@ function cache_clear_all($cid = NULL, $table = NULL, $wildcard = FALSE) {
// This is the first request to clear the cache, start a timer.
variable_set('cache_flush', REQUEST_TIME);
}
else if (REQUEST_TIME > ($cache_flush + variable_get('cache_lifetime', 0))) {
elseif (REQUEST_TIME > ($cache_flush + variable_get('cache_lifetime', 0))) {
// Clear the cache for everyone, cache_flush_delay seconds have
// passed since the first request to clear the cache.
db_delete($table)
......
......@@ -1247,25 +1247,25 @@ function format_date($timestamp, $type = 'medium', $format = '', $timezone = NUL
if (strpos('AaDlM', $c) !== FALSE) {
$date .= t(gmdate($c, $timestamp), array(), $langcode);
}
else if ($c == 'F') {
elseif ($c == 'F') {
// Special treatment for long month names: May is both an abbreviation
// and a full month name in English, but other languages have
// different abbreviations.
$date .= trim(t('!long-month-name ' . gmdate($c, $timestamp), array('!long-month-name' => ''), $langcode));
}
else if (strpos('BdgGhHiIjLmnsStTUwWYyz', $c) !== FALSE) {
elseif (strpos('BdgGhHiIjLmnsStTUwWYyz', $c) !== FALSE) {
$date .= gmdate($c, $timestamp);
}
else if ($c == 'r') {
elseif ($c == 'r') {
$date .= format_date($timestamp - $timezone, 'custom', 'D, d M Y H:i:s O', $timezone, $langcode);
}
else if ($c == 'O') {
elseif ($c == 'O') {
$date .= sprintf('%s%02d%02d', ($timezone < 0 ? '-' : '+'), abs($timezone / 3600), abs($timezone % 3600) / 60);
}
else if ($c == 'Z') {
elseif ($c == 'Z') {
$date .= $timezone;
}
else if ($c == '\\') {
elseif ($c == '\\') {
$date .= $format[++$i];
}
else {
......@@ -1789,7 +1789,7 @@ function drupal_get_css($css = NULL) {
}
// If a CSS file is not to be preprocessed and it's a theme CSS file, it needs to *always* appear at the *bottom*,
// regardless of whether preprocessing is on or off.
else if (!$preprocess && $type == 'theme') {
elseif (!$preprocess && $type == 'theme') {
$no_theme_preprocess .= '<link type="text/css" rel="stylesheet" media="' . $media . '" href="' . base_path() . $file . $query_string . '" />' . "\n";
}
else {
......@@ -2559,7 +2559,7 @@ function page_set_cache() {
if (zlib_get_coding_type() == 'deflate') {
$cache = FALSE;
}
else if (zlib_get_coding_type() == FALSE) {
elseif (zlib_get_coding_type() == FALSE) {
$data = gzencode($data, 9, FORCE_GZIP);
}
// The remaining case is 'gzip' which means the data is
......
......@@ -128,11 +128,11 @@ function file_create_path($destination = NULL) {
}
// Check if the destination is instead inside the Drupal temporary files
// directory.
else if (file_check_location($destination, file_directory_temp())) {
elseif (file_check_location($destination, file_directory_temp())) {
return $destination;
}
// Not found, try again with prefixed directory path.
else if (file_check_location($file_path . '/' . $destination, $file_path)) {
elseif (file_check_location($file_path . '/' . $destination, $file_path)) {
return $file_path . '/' . $destination;
}
// File not found.
......
......@@ -488,7 +488,7 @@ function drupal_prepare_form($form_id, &$form, &$form_state) {
$form['form_token'] = array('#type' => 'token', '#default_value' => drupal_get_token($form['#token']));
}
}
else if (isset($user->uid) && $user->uid && !$form['#programmed']) {
elseif (isset($user->uid) && $user->uid && !$form['#programmed']) {
$form['#token'] = $form_id;
$form['form_token'] = array(
'#id' => form_clean_id('edit-' . $form_id . '-form-token'),
......@@ -1375,7 +1375,7 @@ function form_options_flatten($array, $reset = TRUE) {
if (is_object($value)) {
form_options_flatten($value->option, FALSE);
}
else if (is_array($value)) {
elseif (is_array($value)) {
form_options_flatten($value, FALSE);
}
else {
......@@ -1479,12 +1479,12 @@ function form_get_options($element, $key) {
if (is_array($choice)) {
return FALSE;
}
else if (is_object($choice)) {
elseif (is_object($choice)) {
if (isset($choice->option[$key])) {
$keys[] = $index;
}
}
else if ($index == $key) {
elseif ($index == $key) {
$keys[] = $index;
}
}
......
......@@ -1320,7 +1320,7 @@ function _locale_import_one_string_db(&$report, $langcode, $source, $translation
db_query("INSERT INTO {locales_target} (lid, language, translation, plid, plural) VALUES (%d, '%s', '%s', %d, %d)", $lid, $langcode, $translation, $plid, $plural);
$report[0]++;
}
else if ($mode == LOCALE_IMPORT_OVERWRITE) {
elseif ($mode == LOCALE_IMPORT_OVERWRITE) {
// Translation exists, only overwrite if instructed.
db_query("UPDATE {locales_target} SET translation = '%s', plid = %d, plural = %d WHERE language = '%s' AND lid = %d", $translation, $plid, $plural, $langcode, $lid);
$report[1]++;
......
......@@ -476,7 +476,7 @@ function module_invoke_all() {
if (isset($result) && is_array($result)) {
$return = array_merge_recursive($return, $result);
}
else if (isset($result)) {
elseif (isset($result)) {
$return[] = $result;
}
}
......
......@@ -401,7 +401,7 @@ function theme_pager_link($text, $page_new, $element, $parameters = array(), $at
if (isset($titles[$text])) {
$attributes['title'] = $titles[$text];
}
else if (is_numeric($text)) {
elseif (is_numeric($text)) {
$attributes['title'] = t('Go to page @number', array('@number' => $text));
}
}
......
......@@ -1128,7 +1128,7 @@ function theme_links($links, $attributes = array('class' => 'links')) {
// Pass in $link as $options, they share the same keys.
$output .= l($link['title'], $link['href'], $link);
}
else if (!empty($link['title'])) {
elseif (!empty($link['title'])) {
// Some links are actually not links, but we wrap these in <span> for adding title and class attributes
if (empty($link['html'])) {
$link['title'] = check_plain($link['title']);
......@@ -1457,7 +1457,7 @@ function theme_mark($type = MARK_NEW) {
if ($type == MARK_NEW) {
return ' <span class="marker">' . t('new') . '</span>';
}
else if ($type == MARK_UPDATED) {
elseif ($type == MARK_UPDATED) {
return ' <span class="marker">' . t('updated') . '</span>';
}
}
......@@ -1639,7 +1639,7 @@ function theme_username($object) {
$output = check_plain($name);
}
}
else if ($object->name) {
elseif ($object->name) {
// Sometimes modules display content composed by people who are
// not registered members of the site (e.g. mailing list or news
// aggregator modules). This clause enables modules to display
......
......@@ -174,10 +174,10 @@ function drupal_convert_to_utf8($data, $encoding) {
if (function_exists('iconv')) {
$out = @iconv($encoding, 'utf-8', $data);
}
else if (function_exists('mb_convert_encoding')) {
elseif (function_exists('mb_convert_encoding')) {
$out = @mb_convert_encoding($data, 'utf-8', $encoding);
}
else if (function_exists('recode_string')) {
elseif (function_exists('recode_string')) {
$out = @recode_string($encoding . '..utf-8', $data);
}
else {
......@@ -370,16 +370,16 @@ function _decode_entities($prefix, $codepoint, $original, &$table, &$exclude) {
if ($codepoint < 0x80) {
$str = chr($codepoint);
}
else if ($codepoint < 0x800) {
elseif ($codepoint < 0x800) {
$str = chr(0xC0 | ($codepoint >> 6))
. chr(0x80 | ($codepoint & 0x3F));
}
else if ($codepoint < 0x10000) {
elseif ($codepoint < 0x10000) {
$str = chr(0xE0 | ( $codepoint >> 12))
. chr(0x80 | (($codepoint >> 6) & 0x3F))
. chr(0x80 | ( $codepoint & 0x3F));
}
else if ($codepoint < 0x200000) {
elseif ($codepoint < 0x200000) {
$str = chr(0xF0 | ( $codepoint >> 18))
. chr(0x80 | (($codepoint >> 12) & 0x3F))
. chr(0x80 | (($codepoint >> 6) & 0x3F))
......@@ -488,7 +488,7 @@ function drupal_substr($text, $start, $length = NULL) {
}
}
}
else if ($start < 0) {
elseif ($start < 0) {
// Count all the continuation bytes from the end until we have found
// abs($start) characters
$start = abs($start);
......@@ -507,7 +507,7 @@ function drupal_substr($text, $start, $length = NULL) {
if ($length === NULL) {
$bytes = $strlen - 1;
}
else if ($length > 0) {
elseif ($length > 0) {
// Count all the continuation bytes from the starting index until we have
// found $length + 1 characters. Then backtrack one byte.
$bytes = $istart; $chars = 0;
......@@ -520,7 +520,7 @@ function drupal_substr($text, $start, $length = NULL) {
}
$bytes--;
}
else if ($length < 0) {
elseif ($length < 0) {
// Count all the continuation bytes from the end until we have found
// abs($length) characters
$length = abs($length);
......
......@@ -801,7 +801,7 @@ function aggregator_parse_feed(&$data, $feed) {
if (!empty($guid)) {
$entry = db_fetch_object(db_query("SELECT iid, timestamp FROM {aggregator_item} WHERE fid = %d AND guid = '%s'", $feed['fid'], $guid));
}
else if ($link && $link != $feed['link'] && $link != $feed['url']) {
elseif ($link && $link != $feed['link'] && $link != $feed['url']) {
$entry = db_fetch_object(db_query("SELECT iid, timestamp FROM {aggregator_item} WHERE fid = %d AND link = '%s'", $feed['fid'], $link));
}
else {
......
......@@ -19,7 +19,7 @@ function blog_page_user($account) {
if (($account->uid == $user->uid) && user_access('create blog content')) {
$items[] = l(t('Post new blog entry.'), "node/add/blog");
}
else if ($account->uid == $user->uid) {
elseif ($account->uid == $user->uid) {
$items[] = t('You are not allowed to post a new blog entry.');
}
......
......@@ -88,7 +88,7 @@ function comment_reply($node, $pid = NULL) {
}
}
// This is the case where the comment is in response to a node. Display the node.
else if (user_access('access content')) {
elseif (user_access('access content')) {
$output .= node_view($node);
}
......@@ -97,7 +97,7 @@ function comment_reply($node, $pid = NULL) {
drupal_set_message(t("This discussion is closed: you can't post new comments."), 'error');
drupal_goto("node/$node->nid");
}
else if (user_access('post comments')) {
elseif (user_access('post comments')) {
$output .= comment_form_box(array('pid' => $pid, 'nid' => $node->nid), t('Reply'));
}
else {
......
......@@ -160,7 +160,7 @@ function contact_user_page($account) {
if (!valid_email_address($user->mail)) {
$output = t('You need to provide a valid e-mail address to contact other users. Please update your <a href="@url">user information</a> and try again.', array('@url' => url("user/$user->uid/edit", array('query' => 'destination=' . drupal_get_destination()))));
}
else if (!flood_is_allowed('contact', variable_get('contact_hourly_threshold', 3)) && !user_access('administer site-wide contact form')) {
elseif (!flood_is_allowed('contact', variable_get('contact_hourly_threshold', 3)) && !user_access('administer site-wide contact form')) {
$output = t("You cannot send more than %number messages per hour. Please try again later.", array('%number' => variable_get('contact_hourly_threshold', 3)));
}
else {
......
......@@ -880,12 +880,12 @@ function _filter_autop($text) {
}
}
// Only allow a matching tag to close it.
else if (!$open && $ignoretag == $tag) {
elseif (!$open && $ignoretag == $tag) {
$ignore = FALSE;
$ignoretag = '';
}
}
else if (!$ignore) {
elseif (!$ignore) {
$chunk = preg_replace('|\n*$|', '', $chunk) . "\n\n"; // just to make things a little easier, pad the end
$chunk = preg_replace('|<br />\s*<br />|', "\n\n", $chunk);
$chunk = preg_replace('!(<' . $block . '[^>]*>)!', "\n$1", $chunk); // Space things out a little
......@@ -1001,7 +1001,7 @@ function _filter_xss_split($m, $store = FALSE) {
// We matched a lone ">" character
return '&gt;';
}
else if (strlen($string) == 1) {
elseif (strlen($string) == 1) {
// We matched a lone "<" character
return '&lt;';
}
......
......@@ -162,7 +162,7 @@ class FilterAdminTestCase extends DrupalWebTestCase {
if ($format->name == 'Filtered HTML') {
$filtered = $format->format;
}
else if ($format->name == 'Full HTML') {
elseif ($format->name == 'Full HTML') {
$full = $format->format;
}
}
......
......@@ -285,7 +285,7 @@ function _forum_parent_select($tid, $title, $child_type) {
if ($child_type == 'container') {
$description = t('Containers are usually placed at the top (root) level, but may also be placed inside another container or forum.');
}
else if ($child_type == 'forum') {
elseif ($child_type == 'forum') {
$description = t('Forums may be placed at the top (root) level, or inside another container or forum.');
}
......
......@@ -562,7 +562,7 @@ function locale_update_js_files() {
}
// If no refresh was attempted, but we have new source files, we need
// to store them too. This occurs if current page is in English.
else if ($new_files) {
elseif ($new_files) {
variable_set('javascript_parsed', $parsed);
}
......
......@@ -272,7 +272,7 @@ function menu_block($op = 'list', $delta = '') {
}
return $blocks;
}
else if ($op == 'view') {
elseif ($op == 'view') {
$data['subject'] = check_plain($menus[$delta]);
$data['content'] = menu_tree($delta);
return $data;
......
......@@ -224,7 +224,7 @@ function node_filter_form() {
$value = module_invoke('taxonomy', 'get_term', $value);
$value = $value->name;
}
else if ($type == 'language') {
elseif ($type == 'language') {
$value = empty($value) ? t('Language neutral') : module_invoke('locale', 'language_name', $value);
}
else {
......
......@@ -718,7 +718,7 @@ function node_invoke_nodeapi(&$node, $op, $a3 = NULL, $a4 = NULL) {
if (isset($result) && is_array($result)) {
$return = array_merge($return, $result);
}
else if (isset($result)) {
elseif (isset($result)) {
$return[] = $result;
}
}
......@@ -1716,7 +1716,7 @@ function node_block($op = 'list', $delta = '') {
$blocks['syndicate']['cache'] = BLOCK_NO_CACHE;
return $blocks;
}
else if ($op == 'view') {
elseif ($op == 'view') {
$block['subject'] = t('Syndicate');
$block['content'] = theme('feed_icon', url('rss.xml'), t('Syndicate'));
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment