Merge pull request #17 from DripEmail/16-make-methods-protected

Make methods protected for subclassability

Author: will-in-wi

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ## [Unreleased]
 
 * [#15](https://github.com/DripEmail/drip-php/pull/15) - Added `\Drip\Client#create_or_update_subscribers` method (@j831)
+* Make private methods protected so that people can subclass. Fixes [#16](https://github.com/DripEmail/drip-php/issues/16)
 * Your improvement here!
 
 ## 1.0.0

src/Client.php

@@ -17,18 +17,18 @@ class Client
     const VERSION = '1.0.0';
 
     /** @var string */
-    private $api_token = '';
+    protected $api_token = '';
     /** @var string */
-    private $account_id = '';
+    protected $account_id = '';
     /** @var string */
-    private $api_end_point = 'https://api.getdrip.com/v2/';
+    protected $api_end_point = 'https://api.getdrip.com/v2/';
     /** @var integer */
-    private $timeout = 30;
+    protected $timeout = 30;
     /** @var integer */
-    private $connect_timeout = 30;
+    protected $connect_timeout = 30;
 
     /** @var callable */
-    private $guzzle_stack_constructor;
+    protected $guzzle_stack_constructor;
 
     const GET    = "GET";
     const POST   = "POST";
@@ -288,7 +288,7 @@ public function record_event($params)
     /**
      * @return string
      */
-    private function user_agent()
+    protected function user_agent()
     {
         return "Drip API PHP Wrapper (getdrip.com). Version " . self::VERSION;
     }
@@ -299,7 +299,7 @@ private function user_agent()
      * @param int $code
      * @return boolean
      */
-    private function is_success_response($code)
+    protected function is_success_response($code)
     {
         return $code >= 200 && $code <= 299;
     }
@@ -312,7 +312,7 @@ private function is_success_response($code)
      * @return \Drip\ResponseInterface
      * @throws Exception
      */
-    private function make_request($url, $params = array(), $req_method = self::GET)
+    protected function make_request($url, $params = array(), $req_method = self::GET)
     {
         if ($this->guzzle_stack_constructor) {
             // This can be replaced with `($this->guzzle_stack_constructor)()` once we drop PHP5 support.

src/Error.php

@@ -8,9 +8,9 @@
 class Error
 {
     /** @var string */
-    private $code;
+    protected $code;
     /** @var string */
-    private $message;
+    protected $message;
 
     /**
      * @param string $code      Coded error reason