facebook / facebook-php-ads-sdk

src/FacebookAds/Object/BusinessAdAccount.php

Braces +1 added lines, -2 removed lines

@@ -24,6 +24,5 @@
 
 namespace FacebookAds\Object;
 
-class BusinessAdAccount extends AbstractObject
-{
+class BusinessAdAccount extends AbstractObject {
 }

src/FacebookAds/Object/AbstractCrudObject.php

Braces +25 added lines, -50 removed lines

@@ -29,8 +29,7 @@ 
 use FacebookAds\Http\RequestInterface;
 use FacebookAds\Http\ResponseInterface;
 
-abstract class AbstractCrudObject extends AbstractObject
-{
+abstract class AbstractCrudObject extends AbstractObject {
     /**
      * @var string
      */
@@ -61,8 +60,7 @@ 
      * @param string $parent_id Optional, needed for creating new objects.
      * @param Api $api The Api instance this object should use to make calls
      */
-    public function __construct($id = null, $parent_id = null, Api $api = null)
-    {
+    public function __construct($id = null, $parent_id = null, Api $api = null) {
         parent::__construct();
         $this->data[static::FIELD_ID] = $id;
         $this->parentId = $parent_id;
@@ -72,24 +70,21 @@ 
     /**
      * @param string $id
      */
-    public function setId($id)
-    {
+    public function setId($id) {
         $this->data[static::FIELD_ID] = $id;
     }
 
     /**
      * @param string $parent_id
      */
-    public function setParentId($parent_id)
-    {
+    public function setParentId($parent_id) {
         $this->parentId = $parent_id;
     }
 
     /**
      * @param Api $api The Api instance this object should use to make calls
      */
-    public function setApi(Api $api)
-    {
+    public function setApi(Api $api) {
         $this->api = static::assureApi($api);
     }
 
@@ -103,8 +98,7 @@ 
      * @return Api
      * @throws \InvalidArgumentException
      */
-    protected static function assureApi(Api $instance = null)
-    {
+    protected static function assureApi(Api $instance = null) {
         $instance = $instance ?: Api::instance();
         if (!$instance) {
             throw new \InvalidArgumentException(
@@ -118,8 +112,7 @@ 
     /**
      * @return string|null
      */
-    public function getParentId()
-    {
+    public function getParentId() {
         return $this->parentId;
     }
 
@@ -127,8 +120,7 @@ 
      * @return string
      * @throws \Exception
      */
-    protected function assureParentId()
-    {
+    protected function assureParentId() {
         if (!$this->parentId) {
             throw new \Exception("A parent ID is required.");
         }
@@ -140,8 +132,7 @@ 
      * @return string
      * @throws \Exception
      */
-    protected function assureId()
-    {
+    protected function assureId() {
         if (!$this->data[static::FIELD_ID]) {
             throw new \Exception("field '".static::FIELD_ID."' is required.");
         }
@@ -152,8 +143,7 @@ 
     /**
      * @return Api
      */
-    public function getApi()
-    {
+    public function getApi() {
         return $this->api;
     }
 
@@ -162,8 +152,7 @@ 
      *
      * @return array Key value pairs of changed variables
      */
-    public function getChangedValues()
-    {
+    public function getChangedValues() {
         return $this->changedFields;
     }
 
@@ -172,16 +161,14 @@ 
      *
      * @return array Array of changed field names
      */
-    public function getChangedFields()
-    {
+    public function getChangedFields() {
         return array_keys($this->changedFields);
     }
 
     /**
      * Get the values which have changed, converting them to scalars
      */
-    public function exportData()
-    {
+    public function exportData() {
         $data = array();
         foreach ($this->changedFields as $key => $val) {
             $data[$key] = $val instanceof AbstractObject ? $val->exportData() : $val;
@@ -193,8 +180,7 @@ 
     /**
      * @return void
      */
-    protected function clearHistory()
-    {
+    protected function clearHistory() {
         $this->changedFields = array();
     }
 
@@ -202,8 +188,7 @@ 
      * @param string $name
      * @param mixed $value
      */
-    public function __set($name, $value)
-    {
+    public function __set($name, $value) {
         if (!array_key_exists($name, $this->data)
             || $this->data[$name] !== $value) {
             $this->changedFields[$name] = $value;
@@ -214,24 +199,21 @@ 
     /**
      * @param string[] $fields
      */
-    public static function setDefaultReadFields(array $fields = array())
-    {
+    public static function setDefaultReadFields(array $fields = array()) {
         static::$defaultReadFields = $fields;
     }
 
     /**
      * @return string[]
      */
-    public static function getDefaultReadFields()
-    {
+    public static function getDefaultReadFields() {
         return static::$defaultReadFields;
     }
 
     /**
      * @return string
      */
-    protected function getNodePath()
-    {
+    protected function getNodePath() {
         return '/'.$this->assureId();
     }
 
@@ -242,8 +224,7 @@ 
      * @return $this
      * @throws \Exception
      */
-    public function create(array $params = array())
-    {
+    public function create(array $params = array()) {
         if ($this->data[static::FIELD_ID]) {
             throw new \Exception("Object has already an ID");
         }
@@ -283,8 +264,7 @@ 
      * @param array $params Additional request parameters
      * @return $this
      */
-    public function read(array $fields = array(), array $params = array())
-    {
+    public function read(array $fields = array(), array $params = array()) {
         $fields = implode(',', $fields ?: static::getDefaultReadFields());
         if ($fields) {
             $params['fields'] = $fields;
@@ -308,8 +288,7 @@ 
      * @param array $params Update parameters in assoc
      * @return $this
      */
-    public function update(array $params = array())
-    {
+    public function update(array $params = array()) {
         $this->getApi()->call(
             $this->getNodePath(),
             RequestInterface::METHOD_POST,
@@ -327,8 +306,7 @@ 
      * @param array $params
      * @return void
      */
-    public function delete(array $params = array())
-    {
+    public function delete(array $params = array()) {
         $this->getApi()->call(
             $this->getNodePath(),
             RequestInterface::METHOD_DELETE,
@@ -345,8 +323,7 @@ 
      * @param array $params
      * @return $this
      */
-    public function save(array $params = array())
-    {
+    public function save(array $params = array()) {
         if ($this->data[static::FIELD_ID]) {
             return $this->update($params);
         } else {
@@ -360,8 +337,7 @@ 
      * @return string
      * @throws \InvalidArgumentException
      */
-    protected function assureEndpoint($prototype_class, $endpoint)
-    {
+    protected function assureEndpoint($prototype_class, $endpoint) {
         if (!$endpoint) {
             $prototype = new $prototype_class(null, null, $this->getApi());
             if (!$prototype instanceof AbstractCrudObject) {
@@ -486,8 +462,7 @@ 
      * @param Api $api Api Object to use
      * @return bool Returns true on success
      */
-    public static function deleteIds(array $ids, Api $api = null)
-    {
+    public static function deleteIds(array $ids, Api $api = null) {
         $batch = array();
         foreach ($ids as $id) {
             $request = array(

src/FacebookAds/Object/AbstractObject.php

Braces +13 added lines, -26 removed lines

@@ -26,15 +26,13 @@ 
 
 use FacebookAds\Enum\EmptyEnum;
 
-abstract class AbstractObject
-{
+abstract class AbstractObject {
     /**
      * @var mixed[] set of key value pairs representing data
      */
     protected $data = array();
 
-    public function __construct()
-    {
+    public function __construct() {
         $this->data = static::getFieldsEnum()->getValuesMap();
     }
 
@@ -43,8 +41,7 @@ 
      * @return mixed
      * @throws \InvalidArgumentException
      */
-    public function __get($name)
-    {
+    public function __get($name) {
         if (array_key_exists($name, $this->data)) {
             return $this->data[$name];
         } else {
@@ -56,8 +53,7 @@ 
      * @param string $name
      * @param mixed $value
      */
-    public function __set($name, $value)
-    {
+    public function __set($name, $value) {
         $this->data[$name] = $value;
     }
 
@@ -65,8 +61,7 @@ 
      * @param string $name
      * @return boolean
      */
-    public function __isset($name)
-    {
+    public function __isset($name) {
         return array_key_exists($name, $this->data);
     }
 
@@ -74,8 +69,7 @@ 
      * @param array
      * @return $this
      */
-    public function setData(array $data)
-    {
+    public function setData(array $data) {
         foreach ($data as $key => $value) {
             $this->{$key} = $value;
         }
@@ -89,8 +83,7 @@ 
      * @param array
      * @return $this
      */
-    public function setDataWithoutValidation(array $data)
-    {
+    public function setDataWithoutValidation(array $data) {
         foreach ($data as $key => $value) {
             $this->data[$key] = $value;
         }
@@ -101,8 +94,7 @@ 
     /**
      * @return array
      */
-    public function getData()
-    {
+    public function getData() {
         return $this->data;
     }
 
@@ -110,8 +102,7 @@ 
      * @param mixed $value
      * @return mixed
      */
-    protected function exportValue($value)
-    {
+    protected function exportValue($value) {
         switch (true) {
             case $value === null:
                 break;
@@ -134,32 +125,28 @@ 
     /**
      * @return array
      */
-    public function exportData()
-    {
+    public function exportData() {
         return $this->exportValue($this->data);
     }
 
     /**
      * @return EmptyEnum
      */
-    public static function getFieldsEnum()
-    {
+    public static function getFieldsEnum() {
         return EmptyEnum::getInstance();
     }
 
     /**
      * @return array
      */
-    public static function getFields()
-    {
+    public static function getFields() {
         return static::getFieldsEnum()->getValues();
     }
 
     /**
      * @return string
      */
-    public static function className()
-    {
+    public static function className() {
         return get_called_class();
     }
 }

src/FacebookAds/Object/Search/DemographicSearchClasses.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
 /**
  * @method static DemographicSearchClasses getInstance()
  */
-class DemographicSearchClasses extends AbstractEnum
-{
+class DemographicSearchClasses extends AbstractEnum {
     const LIFE_EVENTS = 'life_events';
     const POLITICS = 'politics';
     const MARKETS = 'markets';

src/FacebookAds/Object/Search/TargetingSearchTypes.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
 /**
  * @method static TargetingSearchTypes getInstance()
  */
-class TargetingSearchTypes extends AbstractEnum
-{
+class TargetingSearchTypes extends AbstractEnum {
     const COUNTRY = 'adcountry';
     const EDUCATION = 'adeducationschool';
     const EMPLOYER = 'adworkemployer';

src/FacebookAds/Object/CanRedownloadInterface.php

Braces +1 added lines, -2 removed lines

@@ -24,8 +24,7 @@
 
 namespace FacebookAds\Object;
 
-interface CanRedownloadInterface
-{
+interface CanRedownloadInterface {
     /**
      * @string
      */

src/FacebookAds/Object/Campaign.php

Braces +7 added lines, -14 removed lines

@@ -30,8 +30,7 @@ 
 use FacebookAds\Object\Traits\FieldValidation;
 use FacebookAds\Object\Traits\ObjectValidation;
 
-class Campaign extends AbstractArchivableCrudObject
-{
+class Campaign extends AbstractArchivableCrudObject {
     use FieldValidation;
     use ObjectValidation;
     use AdLabelAwareCrudObjectTrait;
@@ -39,16 +38,14 @@ 
     /**
      * @return string
      */
-    protected function getEndpoint()
-    {
+    protected function getEndpoint() {
         return 'campaigns';
     }
 
     /**
      * @return CampaignFields
      */
-    public static function getFieldsEnum()
-    {
+    public static function getFieldsEnum() {
         return CampaignFields::getInstance();
     }
 
@@ -57,8 +54,7 @@ 
      * @param array $params
      * @return Cursor
      */
-    public function getAdSets(array $fields = array(), array $params = array())
-    {
+    public function getAdSets(array $fields = array(), array $params = array()) {
         return $this->getManyByConnection(AdSet::className(), $fields, $params);
     }
 
@@ -67,8 +63,7 @@ 
      * @param array $params
      * @return Cursor
      */
-    public function getAds(array $fields = array(), array $params = array())
-    {
+    public function getAds(array $fields = array(), array $params = array()) {
         return $this->getManyByConnection(Ad::className(), $fields, $params);
     }
 
@@ -78,8 +73,7 @@ 
      * @return Cursor
      */
     public function getInsights(
-        array $fields = array(), array $params = array())
-    {
+        array $fields = array(), array $params = array()) {
         return $this->getManyByConnection(
             Insights::classname(), $fields, $params, 'insights');
     }
@@ -90,8 +84,7 @@ 
      * @return AsyncJobInsights
      */
     public function getInsightsAsync(
-        array $fields = array(), array $params = array())
-    {
+        array $fields = array(), array $params = array()) {
         return $this->createAsyncJob(
             AsyncJobInsights::className(), $fields, $params);
     }

src/FacebookAds/Object/UserPermission.php

Braces +1 added lines, -2 removed lines

@@ -24,6 +24,5 @@
 
 namespace FacebookAds\Object;
 
-class UserPermission extends AbstractObject
-{
+class UserPermission extends AbstractObject {
 }

src/FacebookAds/Object/ProductSet.php

Braces +5 added lines, -10 removed lines

@@ -28,23 +28,20 @@ 
 use FacebookAds\Object\Fields\ProductSetFields;
 use FacebookAds\Object\Traits\FieldValidation;
 
-class ProductSet extends AbstractCrudObject
-{
+class ProductSet extends AbstractCrudObject {
     use FieldValidation;
 
     /**
      * @return string
      */
-    protected function getEndpoint()
-    {
+    protected function getEndpoint() {
         return 'product_sets';
     }
 
     /**
      * @return ProductSetFields
      */
-    public static function getFieldsEnum()
-    {
+    public static function getFieldsEnum() {
         return ProductSetFields::getInstance();
     }
 
@@ -54,8 +51,7 @@ 
      * @return Cursor
      */
     public function getProducts(
-        array $fields = array(), array $params = array())
-    {
+        array $fields = array(), array $params = array()) {
         return $this->getManyByConnection(
             Product::className(), $fields, $params);
     }
@@ -66,8 +62,7 @@ 
      * @return Cursor
      */
     public function getProductGroups(
-        array $fields = array(), array $params = array())
-    {
+        array $fields = array(), array $params = array()) {
         return $this->getManyByConnection(
             ProductGroup::className(), $fields, $params, 'product_groups');
     }