Permalink
Browse files

Merge pull request #138 from MioVisman/strict-null

Strict null
2 parents 00ae015 + 018aee5 commit 4f14a20720196fb6e1ea2a7464af48e18b80decc @ashcs ashcs committed on GitHub Nov 27, 2016
View
@@ -851,7 +851,7 @@
//Check up for order_by and direction values
$order_by = isset($_GET['order_by']) ? forum_trim($_GET['order_by']) : null;
$direction = isset($_GET['direction']) ? forum_trim($_GET['direction']) : null;
- if ($order_by == null || $direction == null)
+ if ($order_by === null || $direction === null)
message($lang_common['Bad request']);
if (!in_array($order_by, array('username', 'email', 'num_posts', 'num_posts', 'registered')) || !in_array($direction, array('ASC', 'DESC')))
View
@@ -65,7 +65,7 @@ function generate_config_cache()
global $forum_db;
$return = ($hook = get_hook('ch_fn_generate_config_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Get the forum config from the DB
@@ -97,7 +97,7 @@ function generate_bans_cache()
global $forum_db;
$return = ($hook = get_hook('ch_fn_generate_bans_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Get the ban list from the DB
@@ -136,7 +136,7 @@ function generate_ranks_cache()
global $forum_db;
$return = ($hook = get_hook('ch_fn_generate_ranks_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Get the rank list from the DB
@@ -171,7 +171,7 @@ function generate_stats_cache()
$stats = array();
$return = ($hook = get_hook('ch_fn_generate_stats_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Collect some statistics from the database
@@ -245,7 +245,7 @@ function generate_censors_cache()
global $forum_db;
$return = ($hook = get_hook('ch_fn_generate_censors_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Get the censor list from the DB
@@ -278,7 +278,7 @@ function generate_quickjump_cache($group_id = false)
global $forum_db, $lang_common, $forum_url, $forum_config, $forum_user, $base_url;
$return = ($hook = get_hook('ch_fn_generate_quickjump_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
$groups = array();
@@ -391,7 +391,7 @@ function clean_quickjump_cache($group_id = false)
global $forum_db;
$return = ($hook = get_hook('ch_fn_clean_quickjump_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
$groups = array();
@@ -438,7 +438,7 @@ function generate_hooks_cache()
global $forum_db, $forum_config, $base_url;
$return = ($hook = get_hook('ch_fn_generate_hooks_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Get the hooks from the DB
@@ -502,7 +502,7 @@ function generate_updates_cache()
global $forum_db, $forum_config;
$return = ($hook = get_hook('ch_fn_generate_updates_cache_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Get a list of installed hotfix extensions
@@ -20,7 +20,7 @@ function generate_admin_menu($submenu)
global $forum_config, $forum_url, $forum_user, $lang_admin_common, $db_type;
$return = ($hook = get_hook('ca_fn_generate_admin_menu_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return $return;
if ($submenu)
@@ -110,7 +110,7 @@ function prune($forum_id, $prune_sticky, $prune_date)
global $forum_db, $db_type;
$return = ($hook = get_hook('ca_fn_prune_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return;
// Fetch topics to prune
@@ -371,7 +371,7 @@ function add_field($table_name, $field_name, $field_type, $allow_null, $default_
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -385,7 +385,7 @@ function alter_field($table_name, $field_name, $field_type, $allow_null, $defaul
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -399,7 +399,7 @@ function add_field($table_name, $field_name, $field_type, $allow_null, $default_
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -413,7 +413,7 @@ function alter_field($table_name, $field_name, $field_type, $allow_null, $defaul
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -371,7 +371,7 @@ function add_field($table_name, $field_name, $field_type, $allow_null, $default_
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -385,7 +385,7 @@ function alter_field($table_name, $field_name, $field_type, $allow_null, $defaul
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -399,7 +399,7 @@ function add_field($table_name, $field_name, $field_type, $allow_null, $default_
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' ADD '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
@@ -413,7 +413,7 @@ function alter_field($table_name, $field_name, $field_type, $allow_null, $defaul
if ($default_value !== null && !is_int($default_value) && !is_float($default_value))
$default_value = '\''.$this->escape($default_value).'\'';
- $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field != null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
+ $this->query('ALTER TABLE '.($no_prefix ? '' : $this->prefix).$table_name.' MODIFY '.$field_name.' '.$field_type.($allow_null ? ' ' : ' NOT NULL').($default_value !== null ? ' DEFAULT '.$default_value : ' ').($after_field !== null ? ' AFTER '.$after_field : '')) or error(__FILE__, __LINE__);
}
View
@@ -19,7 +19,7 @@
function is_valid_email($email)
{
$return = ($hook = get_hook('em_fn_is_valid_email_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return $return;
if (strlen($email) > 80)
@@ -37,7 +37,7 @@ function is_banned_email($email)
global $forum_db, $forum_bans;
$return = ($hook = get_hook('em_fn_is_banned_email_start')) ? eval($hook) : null;
- if ($return != null)
+ if ($return !== null)
return $return;
foreach ($forum_bans as $cur_ban)
Oops, something went wrong. Retry.

0 comments on commit 4f14a20

Please sign in to comment.