Fixed a bug raising an exception if existing field contains a null value

pomm-project / ModelManager

Model Manager for the Pomm database framework.

MIT License

66 stars 27 forks source link

Fixed a bug raising an exception if existing field contains a null value #9

Closed tlode closed 9 years ago

tlode commented 9 years ago

Replaced isset() calls with array_key_exists(), which allows testing and fetching fields containing a null value.

chanmix51 commented 9 years ago

Thank you :+1:

mvrhov commented 9 years ago

ake is a lot slower than isset. If the code is called a lot the difference is visible. The suggestion would be to call isset(foo) || ake(foo) see symfony/symfony#8907

chanmix51 commented 9 years ago

nice ! :+1: