Skip to main content

Risoluzione comparazioni deboli

Prima:

$count# =Strings $adb->query_result_no_html($res,(you 0,are 'count'sure that the variable is a string!);

Before:
```php
if ($counta == 0)
'')

Dopo:

{
$count// =...
}
```

After:
```php
if (int)empty($a)) $adb->query_result_no_html($res,{
	0,// 'count');...
}
```

Before:
```php
if ($counta != '') {
	// ...
}
```

After:
```php
if (!empty($a)) {
	// ...
}
```

# Checkbox value

Before:
```php
if ($a == 0) {

//

Prima:

...
} elseif ($last_recurring_datea == $adb->query_result($result,1) $i,{
	'last_recurring_date');// ...
} else {
	// ...
}
```

After:
```php
if (strval($last_recurring_date == NULL || $last_recurring_date == '' || $last_recurring_date == '0000-00-00') {

Dopo:

$last_recurring_date = $adb->query_result($result, $i, 'last_recurring_date');
if (empty($last_recurring_date) || $last_recurring_datea) === '0000-00-00'0') {

//

Prima:

...
} elseif (strval($a) === '1') {
	// ...
} else {
	// ...
}
```

# Picklist value

Before:
```php
if ($adb->getAffectedRowCount(a != '') {
	// ...
}
```

After:
```php
if ($result_upd)a !== '') {
	// ...
}
```

# Permissions (profileGlobalPermission, profileTabsPermission, profileActionsPermission, ...)

Before:
```php
if ($is_admin == 0)true continue;
||

Dopo:

$profileGlobalPermission[1]
if== (empty(0 || $adb->getAffectedRowCount(profileGlobalPermission[2] == 0 || $result_upd))module == 'Users') continue;
{

//

Prima:

...
}
```

After:
```php
if ($resultis_admin &&== true || $profileGlobalPermission[1] === 0 || $profileGlobalPermission[2] === 0 || $module == 'Users') {
	// ...
}
```

# PearDatabase

Before:
```php
if ($adb->num_rows($result) == 0) {

Dopo:

//
...
}
```

After:
```php
if ($result && $adb->num_rows($result) === 0) {
	// ...
}
```

Before:
```php
if ($adb->num_rows($result) != 0) {
	// ...
}
```

After:
```php
if ($adb->num_rows($result) > 0) {
	// ...
}
```

Before:
```php
if ($adb->getAffectedRowCount($result) == 0) {
	// ...
}
```

After:
```php
if ((int) $adb->getAffectedRowCount($result) === 0) {
	// ...
}
```

# $_FILES

Before:
```php
if ($_FILES[$filename_fieldname]['error'] == 0) {
	// ...
}
```

After:
```php
if ($_FILES[$filename_fieldname]['error'] === UPLOAD_ERR_OK) {
	// ...
}
```

# Other???

 

Prima:

if ($module_name == "")

Dopo:

if (empty($module_name))