Deprecate TargetingSearchTypes::CITY and TargetingSearchTypes::REGION

Summary: deprecate /search?type=[adcity|adregion] in favor of /search?type=adgeolocation

Test Plan: ./vendor/bin/phpunit -vc test/ test/FacebookAdsTest/Object/TargetingSearchTest.php

Author: Luca Bruno

src/FacebookAds/Object/Search/TargetingSearchTypes.php

@@ -31,7 +31,6 @@
  */
 class TargetingSearchTypes extends AbstractEnum {
 
-  const CITY = 'adcity';
   const COUNTRY = 'adcountry';
   const EDUCATION = 'adeducationschool';
   const EMPLOYER = 'adworkemployer';
@@ -45,7 +44,6 @@ class TargetingSearchTypes extends AbstractEnum {
   const MAJOR = 'adeducationmajor';
   const POSITION = 'adworkposition';
   const RADIUS_SUGGESTION = 'adradiussuggestion';
-  const REGION = 'adregion';
   const TARGETING_CATEGORY = 'adTargetingCategory';
   const ZIPCODE = 'adzipcode';
 }

test/FacebookAdsTest/Object/TargetingSearchTest.php

@@ -31,7 +31,8 @@
 class TargetingSearchTest extends AbstractCrudObjectTestCase {
 
   public function testCrudAccess() {
-    $cursor = TargetingSearch::search(TargetingSearchTypes::CITY, null, 'Lon');
+    $cursor = TargetingSearch::search(
+      TargetingSearchTypes::GEOLOCATION, null, 'Lon');
 
     /* @var $category AbstractCrudObject */
     $result = $cursor->current();