cakephp/cakephp — pull request #357 by fitorec (2011-12-05 19:07:23+00:00)
Title: Solving the small defect in the array of behaviors(actsAs)

Reviewing the code I found a comparison that is always true:

```php
actsAs !== null || $this->actsAs !== false)
```

I [commented](https://github.com/cakephp/cakephp/commit/15753ab6416bdc679aa48079cfd640819d61e668#commitcomment-732091) a few days , I thought some solutions pusibles:

The first is to change the logical comparator.

```php
actsAs !== null && $this->actsAs !== false)
```

When thinking why does not generate any errors when testing, check the `Object::_mergeVars`, Another solution that comes to mind is to check that the element is not empty:

```php
actsAs))
```
Finally, if the condition is always true, we can avoid the `if` starting the array of `merge`:

```php
actsAs !== null || $this->actsAs !== false) {
$merge[] = 'actsAs';
}*/
```

I honestly do not know which is the ideal solution.

>p.s. That makes a request while where I'm wrong, sorry.