| @@ -69,6 +69,9 @@ | ||
| 69 | 69 | $this->deletedObject = true; | 
| 70 | 70 | } | 
| 71 | 71 | |
| 72 | + /** | |
| 73 | + * @param string $field | |
| 74 | + */ | |
| 72 | 75 | private function editField ($field, $value) | 
| 73 | 76 |      { | 
| 74 | 77 | $this->checkInvalid(); | 
| @@ -233,7 +233,7 @@ | ||
| 233 | 233 | * | 
| 234 | 234 | * @throws \allejo\DaPulse\Exceptions\CurlException If cURL is misconfigured or encounters an error | 
| 235 | 235 | * | 
| 236 | - * @return mixed | |
| 236 | + * @return string | |
| 237 | 237 | */ | 
| 238 | 238 | private function executeCurl () | 
| 239 | 239 |      { | 
| @@ -18,7 +18,7 @@ discard block | ||
| 18 | 18 | protected $pulse_id; | 
| 19 | 19 | protected $column_value; | 
| 20 | 20 | |
| 21 | - public function __construct($array) | |
| 21 | + public function __construct ($array) | |
| 22 | 22 |      { | 
| 23 | 23 | $this->arrayConstructionOnly = true; | 
| 24 | 24 | |
| @@ -45,6 +45,6 @@ discard block | ||
| 45 | 45 |          throw new InvalidObjectException("'$type' is an unsupported column type to modify."); | 
| 46 | 46 | } | 
| 47 | 47 | |
| 48 | - abstract public function getValue(); | |
| 49 | - abstract public function updateValue($updateValue); | |
| 48 | + abstract public function getValue (); | |
| 49 | + abstract public function updateValue ($updateValue); | |
| 50 | 50 | } | 
| 51 | 51 | \ No newline at end of file | 
| @@ -51,7 +51,7 @@ | ||
| 51 | 51 | * | 
| 52 | 52 | * @return int|mixed|string | 
| 53 | 53 | */ | 
| 54 | - public static function array_search_column($array, $column, $search) | |
| 54 | + public static function array_search_column ($array, $column, $search) | |
| 55 | 55 |      { | 
| 56 | 56 |          if (function_exists('array_column')) | 
| 57 | 57 |          { | 
| @@ -25,7 +25,7 @@ | ||
| 25 | 25 | return (isset($this->apiToken)); | 
| 26 | 26 | } | 
| 27 | 27 | |
| 28 | - public function getApiToken() | |
| 28 | + public function getApiToken () | |
| 29 | 29 |      { | 
| 30 | 30 | return $this->apiToken; | 
| 31 | 31 | } | 
| @@ -136,7 +136,7 @@ discard block | ||
| 136 | 136 | * | 
| 137 | 137 | * @return string | 
| 138 | 138 | */ | 
| 139 | - public function getUrl() | |
| 139 | + public function getUrl () | |
| 140 | 140 |      { | 
| 141 | 141 | return $this->url; | 
| 142 | 142 | } | 
| @@ -146,7 +146,7 @@ discard block | ||
| 146 | 146 | * | 
| 147 | 147 | * @return int | 
| 148 | 148 | */ | 
| 149 | - public function getId() | |
| 149 | + public function getId () | |
| 150 | 150 |      { | 
| 151 | 151 | return $this->id; | 
| 152 | 152 | } | 
| @@ -156,7 +156,7 @@ discard block | ||
| 156 | 156 | * | 
| 157 | 157 | * @return string | 
| 158 | 158 | */ | 
| 159 | - public function getName() | |
| 159 | + public function getName () | |
| 160 | 160 |      { | 
| 161 | 161 | return $this->name; | 
| 162 | 162 | } | 
| @@ -166,7 +166,7 @@ discard block | ||
| 166 | 166 | * | 
| 167 | 167 | * @return int | 
| 168 | 168 | */ | 
| 169 | - public function getUpdatesCount() | |
| 169 | + public function getUpdatesCount () | |
| 170 | 170 |      { | 
| 171 | 171 | return $this->updates_count; | 
| 172 | 172 | } | 
| @@ -176,7 +176,7 @@ discard block | ||
| 176 | 176 | * | 
| 177 | 177 | * @return int | 
| 178 | 178 | */ | 
| 179 | - public function getBoardId() | |
| 179 | + public function getBoardId () | |
| 180 | 180 |      { | 
| 181 | 181 | return $this->board_id; | 
| 182 | 182 | } | 
| @@ -186,7 +186,7 @@ discard block | ||
| 186 | 186 | * | 
| 187 | 187 | * @return \DateTime | 
| 188 | 188 | */ | 
| 189 | - public function getCreatedAt() | |
| 189 | + public function getCreatedAt () | |
| 190 | 190 |      { | 
| 191 | 191 | return $this->created_at; | 
| 192 | 192 | } | 
| @@ -196,7 +196,7 @@ discard block | ||
| 196 | 196 | * | 
| 197 | 197 | * @return \DateTime | 
| 198 | 198 | */ | 
| 199 | - public function getUpdatedAt() | |
| 199 | + public function getUpdatedAt () | |
| 200 | 200 |      { | 
| 201 | 201 | return $this->updated_at; | 
| 202 | 202 | } | 
| @@ -214,7 +214,7 @@ discard block | ||
| 214 | 214 | * @since 0.1.0 | 
| 215 | 215 | * @return string | 
| 216 | 216 | */ | 
| 217 | - public function getGroupId($forceFetch = false) | |
| 217 | + public function getGroupId ($forceFetch = false) | |
| 218 | 218 |      { | 
| 219 | 219 | if (empty($this->group_id) || $forceFetch) | 
| 220 | 220 |          { | 
| @@ -582,7 +582,7 @@ discard block | ||
| 582 | 582 | * @since 0.1.0 | 
| 583 | 583 | * @return PulseNote | 
| 584 | 584 | */ | 
| 585 | - public function addNote ($title, $content, $owners_only = false, $user_id = NULL, $create_update = false) | |
| 585 | + public function addNote ($title, $content, $owners_only = false, $user_id = null, $create_update = false) | |
| 586 | 586 |      { | 
| 587 | 587 | $url = sprintf($this->urlSyntax, parent::apiEndpoint(), $this->id, "notes"); | 
| 588 | 588 | $postParams = array( | 
| @@ -648,7 +648,7 @@ discard block | ||
| 648 | 648 | * | 
| 649 | 649 | * @since 0.1.0 | 
| 650 | 650 | */ | 
| 651 | - public function createUpdate ($user, $text, $announceToAll = NULL) | |
| 651 | + public function createUpdate ($user, $text, $announceToAll = null) | |
| 652 | 652 |      { | 
| 653 | 653 | PulseUpdate::createUpdate($user, $this->getId(), $text, $announceToAll); | 
| 654 | 654 | } | 
| @@ -48,14 +48,14 @@ discard block | ||
| 48 | 48 | * | 
| 49 | 49 | * @internal | 
| 50 | 50 | */ | 
| 51 | - const API_VERSION = "v1"; | |
| 51 | + const API_VERSION = "v1"; | |
| 52 | 52 | |
| 53 | 53 | /** | 
| 54 | 54 | * The suffix that is appended to the URL to access functionality for certain objects | 
| 55 | 55 | * | 
| 56 | 56 | * @internal | 
| 57 | 57 | */ | 
| 58 | - const API_PREFIX = ""; | |
| 58 | + const API_PREFIX = ""; | |
| 59 | 59 | |
| 60 | 60 | /** | 
| 61 | 61 | * The API key used to make the URL calls | 
| @@ -137,7 +137,7 @@ discard block | ||
| 137 | 137 | */ | 
| 138 | 138 | protected function assignResults () | 
| 139 | 139 |      { | 
| 140 | - foreach($this->jsonResponse as $key => $val) | |
| 140 | + foreach ($this->jsonResponse as $key => $val) | |
| 141 | 141 |          { | 
| 142 | 142 | if (property_exists(get_called_class(), $key)) | 
| 143 | 143 |              { | 
| @@ -162,7 +162,7 @@ discard block | ||
| 162 | 162 | /** | 
| 163 | 163 | * Overload this function if any class variables need to be initialized to a default value | 
| 164 | 164 | */ | 
| 165 | -    protected function initializeValues() {} | |
| 165 | +    protected function initializeValues () {} | |
| 166 | 166 | |
| 167 | 167 | /** | 
| 168 | 168 | * Inject data into the array that will be mapped into individual instance variables. This function must be called | 
| @@ -162,7 +162,9 @@ | ||
| 162 | 162 | /** | 
| 163 | 163 | * Overload this function if any class variables need to be initialized to a default value | 
| 164 | 164 | */ | 
| 165 | -    protected function initializeValues() {} | |
| 165 | + protected function initializeValues() | |
| 166 | +    { | |
| 167 | +} | |
| 166 | 168 | |
| 167 | 169 | /** | 
| 168 | 170 | * Inject data into the array that will be mapped into individual instance variables. This function must be called | 
| @@ -454,7 +454,7 @@ | ||
| 454 | 454 | * | 
| 455 | 455 | * @return string The base URL to call | 
| 456 | 456 | */ | 
| 457 | - final protected static function apiEndpoint ($apiPrefix = NULL) | |
| 457 | + final protected static function apiEndpoint ($apiPrefix = null) | |
| 458 | 458 |      { | 
| 459 | 459 | $apiSection = isset($apiPrefix) ? $apiPrefix : static::API_PREFIX; | 
| 460 | 460 | |
| @@ -112,7 +112,7 @@ discard block | ||
| 112 | 112 | * | 
| 113 | 113 | * @return string | 
| 114 | 114 | */ | 
| 115 | - public function getUrl() | |
| 115 | + public function getUrl () | |
| 116 | 116 |      { | 
| 117 | 117 | return $this->url; | 
| 118 | 118 | } | 
| @@ -122,7 +122,7 @@ discard block | ||
| 122 | 122 | * | 
| 123 | 123 | * @return int | 
| 124 | 124 | */ | 
| 125 | - public function getId() | |
| 125 | + public function getId () | |
| 126 | 126 |      { | 
| 127 | 127 | return $this->id; | 
| 128 | 128 | } | 
| @@ -132,7 +132,7 @@ discard block | ||
| 132 | 132 | * | 
| 133 | 133 | * @return string | 
| 134 | 134 | */ | 
| 135 | - public function getName() | |
| 135 | + public function getName () | |
| 136 | 136 |      { | 
| 137 | 137 | return $this->name; | 
| 138 | 138 | } | 
| @@ -142,7 +142,7 @@ discard block | ||
| 142 | 142 | * | 
| 143 | 143 | * @return string | 
| 144 | 144 | */ | 
| 145 | - public function getEmail() | |
| 145 | + public function getEmail () | |
| 146 | 146 |      { | 
| 147 | 147 | return $this->email; | 
| 148 | 148 | } | 
| @@ -152,7 +152,7 @@ discard block | ||
| 152 | 152 | * | 
| 153 | 153 | * @return string | 
| 154 | 154 | */ | 
| 155 | - public function getPhotoUrl() | |
| 155 | + public function getPhotoUrl () | |
| 156 | 156 |      { | 
| 157 | 157 | return $this->photo_url; | 
| 158 | 158 | } | 
| @@ -162,7 +162,7 @@ discard block | ||
| 162 | 162 | * | 
| 163 | 163 | * @return string | 
| 164 | 164 | */ | 
| 165 | - public function getTitle() | |
| 165 | + public function getTitle () | |
| 166 | 166 |      { | 
| 167 | 167 | return $this->title; | 
| 168 | 168 | } | 
| @@ -172,7 +172,7 @@ discard block | ||
| 172 | 172 | * | 
| 173 | 173 | * @return string | 
| 174 | 174 | */ | 
| 175 | - public function getPosition() | |
| 175 | + public function getPosition () | |
| 176 | 176 |      { | 
| 177 | 177 | return $this->position; | 
| 178 | 178 | } | 
| @@ -182,7 +182,7 @@ discard block | ||
| 182 | 182 | * | 
| 183 | 183 | * @return string | 
| 184 | 184 | */ | 
| 185 | - public function getPhone() | |
| 185 | + public function getPhone () | |
| 186 | 186 |      { | 
| 187 | 187 | return $this->phone; | 
| 188 | 188 | } | 
| @@ -192,7 +192,7 @@ discard block | ||
| 192 | 192 | * | 
| 193 | 193 | * @return string | 
| 194 | 194 | */ | 
| 195 | - public function getLocation() | |
| 195 | + public function getLocation () | |
| 196 | 196 |      { | 
| 197 | 197 | return $this->location; | 
| 198 | 198 | } | 
| @@ -202,7 +202,7 @@ discard block | ||
| 202 | 202 | * | 
| 203 | 203 | * @return string | 
| 204 | 204 | */ | 
| 205 | - public function getStatus() | |
| 205 | + public function getStatus () | |
| 206 | 206 |      { | 
| 207 | 207 | return $this->status; | 
| 208 | 208 | } | 
| @@ -212,7 +212,7 @@ discard block | ||
| 212 | 212 | * | 
| 213 | 213 | * @return string | 
| 214 | 214 | */ | 
| 215 | - public function getBirthday() | |
| 215 | + public function getBirthday () | |
| 216 | 216 |      { | 
| 217 | 217 | return $this->birthday; | 
| 218 | 218 | } | 
| @@ -222,7 +222,7 @@ discard block | ||
| 222 | 222 | * | 
| 223 | 223 | * @return bool | 
| 224 | 224 | */ | 
| 225 | - public function getIsGuest() | |
| 225 | + public function getIsGuest () | |
| 226 | 226 |      { | 
| 227 | 227 | return $this->is_guest; | 
| 228 | 228 | } | 
| @@ -232,7 +232,7 @@ discard block | ||
| 232 | 232 | * | 
| 233 | 233 | * @return string[] | 
| 234 | 234 | */ | 
| 235 | - public function getSkills() | |
| 235 | + public function getSkills () | |
| 236 | 236 |      { | 
| 237 | 237 | return $this->skills; | 
| 238 | 238 | } | 
| @@ -242,7 +242,7 @@ discard block | ||
| 242 | 242 | * | 
| 243 | 243 | * @return \DateTime | 
| 244 | 244 | */ | 
| 245 | - public function getCreatedAt() | |
| 245 | + public function getCreatedAt () | |
| 246 | 246 |      { | 
| 247 | 247 | return $this->created_at; | 
| 248 | 248 | } | 
| @@ -252,7 +252,7 @@ discard block | ||
| 252 | 252 | * | 
| 253 | 253 | * @return \DateTime | 
| 254 | 254 | */ | 
| 255 | - public function getUpdatedAt() | |
| 255 | + public function getUpdatedAt () | |
| 256 | 256 |      { | 
| 257 | 257 | return $this->updated_at; | 
| 258 | 258 | } | 
| @@ -225,7 +225,7 @@ discard block | ||
| 225 | 225 | * | 
| 226 | 226 | * @since 0.1.0 | 
| 227 | 227 | */ | 
| 228 | - public function addSubscriber ($user_id, $as_admin = NULL) | |
| 228 | + public function addSubscriber ($user_id, $as_admin = null) | |
| 229 | 229 |      { | 
| 230 | 230 | if ($user_id instanceof PulseUser) | 
| 231 | 231 |          { | 
| @@ -359,7 +359,7 @@ discard block | ||
| 359 | 359 | * | 
| 360 | 360 | * @return PulseGroup[] | 
| 361 | 361 | */ | 
| 362 | - public function getGroups ($show_archived = NULL) | |
| 362 | + public function getGroups ($show_archived = null) | |
| 363 | 363 |      { | 
| 364 | 364 |          $url = sprintf("%s/%d/groups.json", self::apiEndpoint(), $this->getId()); | 
| 365 | 365 | $params = array(); | 
| @@ -471,7 +471,7 @@ discard block | ||
| 471 | 471 | $this->deletedObject = true; | 
| 472 | 472 | } | 
| 473 | 473 | |
| 474 | - public static function createBoard ($name, $user_id, $description = NULL) | |
| 474 | + public static function createBoard ($name, $user_id, $description = null) | |
| 475 | 475 |      { | 
| 476 | 476 |          $url        = sprintf("%s.json", self::apiEndpoint()); | 
| 477 | 477 | $postParams = array( | 
| @@ -95,7 +95,7 @@ discard block | ||
| 95 | 95 | * | 
| 96 | 96 | * @return string | 
| 97 | 97 | */ | 
| 98 | - public function getUrl() | |
| 98 | + public function getUrl () | |
| 99 | 99 |      { | 
| 100 | 100 | return $this->url; | 
| 101 | 101 | } | 
| @@ -109,7 +109,7 @@ discard block | ||
| 109 | 109 | * | 
| 110 | 110 | * @return int | 
| 111 | 111 | */ | 
| 112 | - public function getId() | |
| 112 | + public function getId () | |
| 113 | 113 |      { | 
| 114 | 114 | return $this->id; | 
| 115 | 115 | } | 
| @@ -123,7 +123,7 @@ discard block | ||
| 123 | 123 | * | 
| 124 | 124 | * @return string | 
| 125 | 125 | */ | 
| 126 | - public function getName() | |
| 126 | + public function getName () | |
| 127 | 127 |      { | 
| 128 | 128 | return $this->name; | 
| 129 | 129 | } | 
| @@ -137,7 +137,7 @@ discard block | ||
| 137 | 137 | * | 
| 138 | 138 | * @return string | 
| 139 | 139 | */ | 
| 140 | - public function getDescription() | |
| 140 | + public function getDescription () | |
| 141 | 141 |      { | 
| 142 | 142 | return $this->description; | 
| 143 | 143 | } | 
| @@ -151,7 +151,7 @@ discard block | ||
| 151 | 151 | * | 
| 152 | 152 | * @return \DateTime | 
| 153 | 153 | */ | 
| 154 | - public function getCreatedAt() | |
| 154 | + public function getCreatedAt () | |
| 155 | 155 |      { | 
| 156 | 156 | self::lazyLoad($this->created_at, "DateTime"); | 
| 157 | 157 | |
| @@ -167,7 +167,7 @@ discard block | ||
| 167 | 167 | * | 
| 168 | 168 | * @return \DateTime | 
| 169 | 169 | */ | 
| 170 | - public function getUpdatedAt() | |
| 170 | + public function getUpdatedAt () | |
| 171 | 171 |      { | 
| 172 | 172 | self::lazyLoad($this->updated_at, "DateTime"); | 
| 173 | 173 | |
| @@ -256,7 +256,7 @@ discard block | ||
| 256 | 256 | * | 
| 257 | 257 | * @return PulseColumn[] | 
| 258 | 258 | */ | 
| 259 | - public function getColumns() | |
| 259 | + public function getColumns () | |
| 260 | 260 |      { | 
| 261 | 261 | self::lazyInject($this->columns, array( | 
| 262 | 262 | "board_id" => $this->getId() |