facebook / facebook-php-ads-sdk

src/FacebookAds/Object/AbstractArchivableCrudObject.php

Braces +4 added lines, -8 removed lines

@@ -26,8 +26,7 @@ 
 
 use FacebookAds\Http\RequestInterface;
 
-abstract class AbstractArchivableCrudObject extends AbstractCrudObject
-{
+abstract class AbstractArchivableCrudObject extends AbstractCrudObject {
     /**
      * @var string
      */
@@ -56,8 +55,7 @@ 
     /**
      * @return string
      */
-    public function getStatusParamName()
-    {
+    public function getStatusParamName() {
         return self::STATUS_PARAM_NAME;
     }
 
@@ -67,8 +65,7 @@ 
      * @param array $params
      * @return void
      */
-    public function archive(array $params = array())
-    {
+    public function archive(array $params = array()) {
         $this->getApi()->call(
             $this->getNodePath(),
             RequestInterface::METHOD_POST,
@@ -84,8 +81,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_POST,

src/FacebookAds/Object/CustomAudience.php

Braces +5 added lines, -10 removed lines

@@ -30,8 +30,7 @@ 
 use FacebookAds\Object\Fields\CustomAudienceFields;
 use FacebookAds\Object\Values\CustomAudienceTypes;
 
-class CustomAudience extends AbstractCrudObject
-{
+class CustomAudience extends AbstractCrudObject {
     /**
      * @var string
      * @deprecated use HashNormalizer::HASH_TYPE_SHA256
@@ -41,16 +40,14 @@ 
     /**
      * @return string
      */
-    protected function getEndpoint()
-    {
+    protected function getEndpoint() {
         return 'customaudiences';
     }
 
     /**
      * @return CustomAudienceFields
      */
-    public static function getFieldsEnum()
-    {
+    public static function getFieldsEnum() {
         return CustomAudienceFields::getInstance();
     }
 
@@ -182,8 +179,7 @@ 
      * @param array $act_ids Array of account IDs
      * @return boolean Returns true on success
      */
-    public function addSharedAccounts($act_ids)
-    {
+    public function addSharedAccounts($act_ids) {
         return $this->getApi()->call(
             '/'.$this->assureId().'/adaccounts',
             RequestInterface::METHOD_POST,
@@ -197,8 +193,7 @@ 
      * @param    array $act_ids Array of Account IDs to remove
      * @return boolean Returns true on success
      */
-    public function removeSharedAccounts($act_ids)
-    {
+    public function removeSharedAccounts($act_ids) {
         return $this->getApi()->call(
             '/'.$this->assureId().'/adaccounts',
             RequestInterface::METHOD_DELETE,

src/FacebookAds/Object/AdAccountGroupUser.php

Braces +11 added lines, -22 removed lines

@@ -29,8 +29,7 @@ 
 use FacebookAds\Object\Fields\AdAccountGroupUserFields;
 use FacebookAds\Object\Traits\FieldValidation;
 
-class AdAccountGroupUser extends AbstractObject
-{
+class AdAccountGroupUser extends AbstractObject {
     use FieldValidation;
 
     /**
@@ -62,8 +61,7 @@ 
      * @param string $ad_account_group_id
      * @param Api $api
      */
-    public function __construct($ad_account_group_id, Api $api = null)
-    {
+    public function __construct($ad_account_group_id, Api $api = null) {
         $this->adAccountGroupId = $ad_account_group_id;
         $this->api = $api;
     }
@@ -71,24 +69,21 @@ 
     /**
      * @return AdAccountGroupUserFields
      */
-    public static function getFieldsEnum()
-    {
+    public static function getFieldsEnum() {
         return AdAccountGroupUserFields::getInstance();
     }
 
     /**
      * @return Api
      */
-    public function getApi()
-    {
+    public function getApi() {
         return $this->api;
     }
 
     /**
      * @return string
      */
-    public function getParentId()
-    {
+    public function getParentId() {
         return $this->adAccountGroupId;
     }
 
@@ -96,8 +91,7 @@ 
      * @param array $params
      * @return $this
      */
-    public function create(array $params = array())
-    {
+    public function create(array $params = array()) {
         return $this->save($params);
     }
 
@@ -105,8 +99,7 @@ 
      * @param array $params
      * @return $this
      */
-    public function update(array $params = array())
-    {
+    public function update(array $params = array()) {
         return $this->save($params);
     }
 
@@ -114,8 +107,7 @@ 
      * @return string
      * @throws \Exception
      */
-    protected function assureId()
-    {
+    protected function assureId() {
         if (!$this->data[AdAccountGroupUserFields::UID]) {
             throw new \Exception(AdAccountGroupUserFields::UID.' field must be set');
         }
@@ -127,8 +119,7 @@ 
      * @param array $params
      * @return $this
      */
-    public function save(array $params = array())
-    {
+    public function save(array $params = array()) {
         $this->assureId();
 
         $this->getApi()->call(
@@ -143,8 +134,7 @@ 
      * @param array $params
      * @throws \Exception
      */
-    public function delete(array $params = array())
-    {
+    public function delete(array $params = array()) {
         if (!$this->data[AdAccountGroupUserFields::UID]) {
             throw new \Exception("UID field must be set");
         }
@@ -160,8 +150,7 @@ 
     /**
      * @return AdUser
      */
-    public function getAdUser()
-    {
+    public function getAdUser() {
         return new AdUser($this->{AdAccountGroupUserFields::UID});
     }
 }

src/FacebookAds/Enum/AbstractEnum.php

Braces +13 added lines, -26 removed lines

@@ -24,8 +24,7 @@ 
 
 namespace FacebookAds\Enum;
 
-abstract class AbstractEnum implements EnumInstanceInterface
-{
+abstract class AbstractEnum implements EnumInstanceInterface {
     /**
      * @var array|null
      */
@@ -54,16 +53,14 @@ 
     /**
      * @return string
      */
-    public static function className()
-    {
+    public static function className() {
         return get_called_class();
     }
 
     /**
      * @return AbstractEnum
      */
-    public static function getInstance()
-    {
+    public static function getInstance() {
         $fqn = get_called_class();
         if (!array_key_exists($fqn, static::$instances)) {
             static::$instances[$fqn] = new static();
@@ -75,8 +72,7 @@ 
     /**
      * @return array
      */
-    public function getArrayCopy()
-    {
+    public function getArrayCopy() {
         if ($this->map === null) {
             $this->map = (new \ReflectionClass(get_called_class()))
                 ->getConstants();
@@ -88,8 +84,7 @@ 
     /**
      * @return array
      */
-    public function getNames()
-    {
+    public function getNames() {
         if ($this->names === null) {
             $this->names = array_keys($this->getArrayCopy());
         }
@@ -100,8 +95,7 @@ 
     /**
      * @return array
      */
-    public function getValues()
-    {
+    public function getValues() {
         if ($this->values === null) {
             $this->values = array_values($this->getArrayCopy());
         }
@@ -112,8 +106,7 @@ 
     /**
      * @return array
      */
-    public function getValuesMap()
-    {
+    public function getValuesMap() {
         if ($this->valuesMap === null) {
             $this->valuesMap = array_fill_keys($this->getValues(), null);
         }
@@ -125,8 +118,7 @@ 
      * @param string|int|float $name
      * @return mixed
      */
-    public function getValueForName($name)
-    {
+    public function getValueForName($name) {
         return array_key_exists($name, $this->getArrayCopy())
             ? $this->getArrayCopy()[$name]
             : null;
@@ -137,8 +129,7 @@ 
      * @return mixed
      * @throws \InvalidArgumentException
      */
-    public function assureValueForName($name)
-    {
+    public function assureValueForName($name) {
         $value = $this->getValueForName($name);
         if ($value === null) {
             throw new \InvalidArgumentException('Unknown name "'.$name.'" in '.static::className());
@@ -151,8 +142,7 @@ 
      * @param string|int|float $name
      * @return bool
      */
-    public function isValid($name)
-    {
+    public function isValid($name) {
         return array_key_exists($name, $this->getArrayCopy());
     }
 
@@ -160,8 +150,7 @@ 
      * @param string|int|float $name
      * @throws \InvalidArgumentException
      */
-    public function assureIsValid($name)
-    {
+    public function assureIsValid($name) {
         if (!array_key_exists($name, $this->getArrayCopy())) {
             throw new \InvalidArgumentException('Unknown name "'.$name.'" in '.static::className());
         }
@@ -171,8 +160,7 @@ 
      * @param string|int|float $value
      * @return bool
      */
-    public function isValidValue($value)
-    {
+    public function isValidValue($value) {
         return array_key_exists($value, $this->getValuesMap());
     }
 
@@ -180,8 +168,7 @@ 
      * @param mixed $value
      * @throws \InvalidArgumentException
      */
-    public function assureIsValidValue($value)
-    {
+    public function assureIsValidValue($value) {
         if (!$this->isValidValue($value)) {
             throw new \InvalidArgumentException('"'.$value.'", not a valid value in '.static::className());
         }

src/FacebookAds/Enum/EnumInstanceInterface.php

Braces +1 added lines, -2 removed lines

@@ -24,8 +24,7 @@
 
 namespace FacebookAds\Enum;
 
-interface EnumInstanceInterface
-{
+interface EnumInstanceInterface {
     /**
      * @return EnumInstanceInterface
      */

src/FacebookAds/Enum/EmptyEnum.php

Braces +5 added lines, -10 removed lines

@@ -24,37 +24,32 @@
 
 namespace FacebookAds\Enum;
 
-class EmptyEnum extends AbstractEnum
-{
+class EmptyEnum extends AbstractEnum {
     /**
      * @return array
      */
-    public function getArrayCopy()
-    {
+    public function getArrayCopy() {
         return array();
     }
 
     /**
      * @return array
      */
-    public function getNames()
-    {
+    public function getNames() {
         return array();
     }
 
     /**
      * @return array
      */
-    public function getValues()
-    {
+    public function getValues() {
         return array();
     }
 
     /**
      * @return array
      */
-    public function getValuesMap()
-    {
+    public function getValuesMap() {
         return array();
     }
 }

src/FacebookAds/Cursor.php

Braces +37 added lines, -74 removed lines

@@ -84,8 +84,7 @@ 
      * @param array $object_data
      * @return AbstractObject
      */
-    protected function createObject(array $object_data)
-    {
+    protected function createObject(array $object_data) {
         $object = clone $this->objectPrototype;
         $object->setDataWithoutValidation($object_data);
 
@@ -97,8 +96,7 @@ 
      * @return array
      * @throws \InvalidArgumentException
      */
-    protected function assureResponseData(ResponseInterface $response)
-    {
+    protected function assureResponseData(ResponseInterface $response) {
         $content = $response->getContent();
         if (!isset($content['data']) || !is_array($content['data'])) {
             throw new \InvalidArgumentException("Malformed response data");
@@ -110,8 +108,7 @@ 
     /**
      * @param ResponseInterface $response
      */
-    protected function prependResponse(ResponseInterface $response)
-    {
+    protected function prependResponse(ResponseInterface $response) {
         $this->response = $response;
         $data = $this->assureResponseData($response);
         if (empty($data)) {
@@ -130,8 +127,7 @@ 
     /**
      * @param ResponseInterface $response
      */
-    protected function appendResponse(ResponseInterface $response)
-    {
+    protected function appendResponse(ResponseInterface $response) {
         $this->response = $response;
         $data = $this->assureResponseData($response);
         if (empty($data)) {
@@ -154,24 +150,21 @@ 
     /**
      * @return bool
      */
-    public static function getDefaultUseImplicitFetch()
-    {
+    public static function getDefaultUseImplicitFetch() {
         return static::$defaultUseImplicitFetch;
     }
 
     /**
      * @param bool $use_implicit_fectch
      */
-    public static function setDefaultUseImplicitFetch($use_implicit_fectch)
-    {
+    public static function setDefaultUseImplicitFetch($use_implicit_fectch) {
         static::$defaultUseImplicitFetch = $use_implicit_fectch;
     }
 
     /**
      * @return bool
      */
-    public function getUseImplicitFetch()
-    {
+    public function getUseImplicitFetch() {
         return $this->useImplicitFectch !== null
             ? $this->useImplicitFectch
             : static::$defaultUseImplicitFetch;
@@ -180,16 +173,14 @@ 
     /**
      * @param bool $use_implicit_fectch
      */
-    public function setUseImplicitFetch($use_implicit_fectch)
-    {
+    public function setUseImplicitFetch($use_implicit_fectch) {
         $this->useImplicitFectch = $use_implicit_fectch;
     }
 
     /**
      * @return string|null
      */
-    public function getBefore()
-    {
+    public function getBefore() {
         $content = $this->getLastResponse()->getContent();
 
         return isset($content['paging']['cursors']['before'])
@@ -200,8 +191,7 @@ 
     /**
      * @return string|null
      */
-    public function getAfter()
-    {
+    public function getAfter() {
         $content = $this->getLastResponse()->getContent();
 
         return isset($content['paging']['cursors']['after'])
@@ -212,8 +202,7 @@ 
     /**
      * @return RequestInterface
      */
-    protected function createUndirectionalizedRequest()
-    {
+    protected function createUndirectionalizedRequest() {
         $request = $this->getLastResponse()->getRequest()->createClone();
         $params = $request->getQueryParams();
         if (array_key_exists('before', $params)) {
@@ -229,8 +218,7 @@ 
     /**
      * @return string|null
      */
-    public function getPrevious()
-    {
+    public function getPrevious() {
         $content = $this->getLastResponse()->getContent();
         if (isset($content['paging']['previous'])) {
             return $content['paging']['previous'];
@@ -249,8 +237,7 @@ 
     /**
      * @return string|null
      */
-    public function getNext()
-    {
+    public function getNext() {
         $content = $this->getLastResponse()->getContent();
         if (isset($content['paging']['next'])) {
             return $content['paging']['next'];
@@ -270,8 +257,7 @@ 
      * @param string $url
      * @return RequestInterface
      */
-    protected function createRequestFromUrl($url)
-    {
+    protected function createRequestFromUrl($url) {
         $components = parse_url($url);
         $request = $this->getLastResponse()->getRequest()->createClone();
         $request->setDomain($components['host']);
@@ -286,8 +272,7 @@ 
     /**
      * @return RequestInterface|null
      */
-    public function createBeforeRequest()
-    {
+    public function createBeforeRequest() {
         $url = $this->getPrevious();
         return $url !== null ? $this->createRequestFromUrl($url) : null;
     }
@@ -295,14 +280,12 @@ 
     /**
      * @return RequestInterface|null
      */
-    public function createAfterRequest()
-    {
+    public function createAfterRequest() {
         $url = $this->getNext();
         return $url !== null ? $this->createRequestFromUrl($url) : null;
     }
 
-    public function fetchBefore()
-    {
+    public function fetchBefore() {
         $request = $this->createBeforeRequest();
         if (!$request) {
             return;
@@ -311,8 +294,7 @@ 
         $this->prependResponse($request->execute());
     }
 
-    public function fetchAfter()
-    {
+    public function fetchAfter() {
         $request = $this->createAfterRequest();
         if (!$request) {
             return;
@@ -325,8 +307,7 @@ 
      * @deprecated Use getArrayCopy()
      * @return AbstractObject[]
      */
-    public function getObjects()
-    {
+    public function getObjects() {
         return $this->objects;
     }
 
@@ -334,8 +315,7 @@ 
      * @param bool $ksort
      * @return AbstractObject[]
      */
-    public function getArrayCopy($ksort = false)
-    {
+    public function getArrayCopy($ksort = false) {
         if ($ksort) {
             // Sort the main array to improve best case performance in future
             // invocations
@@ -349,50 +329,43 @@ 
      * @deprecated Use getLastResponse()
      * @return ResponseInterface
      */
-    public function getResponse()
-    {
+    public function getResponse() {
         return $this->response;
     }
 
     /**
      * @return ResponseInterface
      */
-    public function getLastResponse()
-    {
+    public function getLastResponse() {
         return $this->response;
     }
 
     /**
      * @return int
      */
-    public function getIndexLeft()
-    {
+    public function getIndexLeft() {
         return $this->indexLeft;
     }
 
     /**
      * @return int
      */
-    public function getIndexRight()
-    {
+    public function getIndexRight() {
         return $this->indexRight;
     }
 
-    public function rewind()
-    {
+    public function rewind() {
         $this->position = $this->indexLeft;
     }
 
-    public function end()
-    {
+    public function end() {
         $this->position = $this->indexRight;
     }
 
     /**
      * @param int $position
      */
-    public function seekTo($position)
-    {
+    public function seekTo($position) {
         $position = array_key_exists($position, $this->objects) ? $position : null;
         $this->position = $position;
     }
@@ -400,8 +373,7 @@ 
     /**
      * @return AbstractObject|bool
      */
-    public function current()
-    {
+    public function current() {
         return isset($this->objects[$this->position])
             ? $this->objects[$this->position]
             : false;
@@ -410,13 +382,11 @@ 
     /**
      * @return int
      */
-    public function key()
-    {
+    public function key() {
         return $this->position;
     }
 
-    public function prev()
-    {
+    public function prev() {
         if ($this->position == $this->getIndexLeft()) {
             if ($this->getUseImplicitFetch()) {
                 $this->fetchBefore();
@@ -433,8 +403,7 @@ 
         }
     }
 
-    public function next()
-    {
+    public function next() {
         if ($this->position == $this->getIndexRight()) {
             if ($this->getUseImplicitFetch()) {
                 $this->fetchAfter();
@@ -454,16 +423,14 @@ 
     /**
      * @return bool
      */
-    public function valid()
-    {
+    public function valid() {
         return isset($this->objects[$this->position]);
     }
 
     /**
      * @return int
      */
-    public function count()
-    {
+    public function count() {
         return count($this->objects);
     }
 
@@ -471,8 +438,7 @@ 
      * @param mixed $offset
      * @param mixed $value
      */
-    public function offsetSet($offset, $value)
-    {
+    public function offsetSet($offset, $value) {
         if ($offset === null) {
             $this->objects[] = $value;
         } else {
@@ -484,16 +450,14 @@ 
      * @param mixed $offset
      * @return bool
      */
-    public function offsetExists($offset)
-    {
+    public function offsetExists($offset) {
         return isset($this->objects[$offset]);
     }
 
     /**
      * @param mixed $offset
      */
-    public function offsetUnset($offset)
-    {
+    public function offsetUnset($offset) {
         unset($this->objects[$offset]);
     }
 
@@ -501,8 +465,7 @@ 
      * @param mixed $offset
      * @return mixed
      */
-    public function offsetGet($offset)
-    {
+    public function offsetGet($offset) {
         return isset($this->objects[$offset]) ? $this->objects[$offset] : null;
     }
 }

src/FacebookAds/Session.php

Braces +6 added lines, -12 removed lines

@@ -24,8 +24,7 @@ 
 
 namespace FacebookAds;
 
-class Session
-{
+class Session {
     /**
      * @var string
      */
@@ -51,8 +50,7 @@ 
      * @param string $app_secret
      * @param string $access_token
      */
-    public function __construct($app_id, $app_secret, $access_token)
-    {
+    public function __construct($app_id, $app_secret, $access_token) {
         $this->appId = $app_id;
         $this->appSecret = $app_secret;
         $this->accessToken = $access_token;
@@ -61,32 +59,28 @@ 
     /**
      * @return string
      */
-    public function getAppId()
-    {
+    public function getAppId() {
         return $this->appId;
     }
 
     /**
      * @return string
      */
-    public function getAppSecret()
-    {
+    public function getAppSecret() {
         return $this->appSecret;
     }
 
     /**
      * @return string
      */
-    public function getAccessToken()
-    {
+    public function getAccessToken() {
         return $this->accessToken;
     }
 
     /**
      * @return string
      */
-    public function getAppSecretProof()
-    {
+    public function getAppSecretProof() {
         if ($this->appSecretProof === null) {
             $this->appSecretProof = hash_hmac('sha256', $this->getAccessToken(), $this->getAppSecret());
         }

src/FacebookAds/Http/Util.php

Braces +2 added lines, -4 removed lines

@@ -24,8 +24,7 @@ 
 
 namespace FacebookAds\Http;
 
-abstract class Util
-{
+abstract class Util {
     /**
      * Avoid parse_str() for HHVM compatibility
      * This implementation is not a complete sobstitute, but covers all the
@@ -35,8 +34,7 @@ 
      * @param $query_string
      * @return array
      */
-    public static function parseUrlQuery($query_string)
-    {
+    public static function parseUrlQuery($query_string) {
         $query = array();
         $pairs = explode('&', $query_string);
         foreach ($pairs as $pair) {