Fixing bad model checking

Issues googleapis#120 and googleapis#115 both address the issue of invalid null checks. I added a test and caught another, so this looks good.

Author: Ian Barber

src/Google/Model.php

@@ -111,7 +111,7 @@ public function toSimpleObject()
  // Process all other data.
  foreach ($this->data as $key => $val) {
  $result = $this->getSimpleValue($val);
- if ($result != null) {
+ if ($result !== null) {
  $object->$key = $result;
  }
  }
@@ -122,7 +122,7 @@ public function toSimpleObject()
  foreach ($props as $member) {
  $name = $member->getName();
  $result = $this->getSimpleValue($this->$name);
- if ($result != null) {
+ if ($result !== null) {
  $object->$name = $result;
  }
  }
@@ -142,7 +142,7 @@ private function getSimpleValue($value)
  $return = array();
  foreach ($value as $key => $a_value) {
  $a_value = $this->getSimpleValue($a_value);
- if ($a_value != null) {
+ if ($a_value !== null) {
  $return[$key] = $a_value;
  }
  }

tests/general/ApiModelTest.php

@@ -80,7 +80,9 @@ public function testJsonStructure()
  $model3 = new Google_Model();
  $model3->publicE = 54321;
  $model3->publicF = null;
- $model->publicG = array($model3, "hello");
+ $model->publicG = array($model3, "hello", false);
+ $model->publicH = false;
+ $model->publicI = 0;
  $string = json_encode($model->toSimpleObject());
  $data = json_decode($string, true);
  $this->assertEquals(12345, $data['publicB']['publicC']);
@@ -89,7 +91,10 @@ public function testJsonStructure()
  $this->assertArrayHasKey("publicE", $data['publicG'][0]);
  $this->assertArrayNotHasKey("publicF", $data['publicG'][0]);
  $this->assertEquals("hello", $data['publicG'][1]);
+ $this->assertEquals(false, $data['publicG'][2]);
  $this->assertArrayNotHasKey("data", $data);
+ $this->assertEquals(false, $data['publicH']);
+ $this->assertEquals(0, $data['publicI']);
  }
 
  public function testIssetPropertyOnModel()