Moved lib/API to lib/Router

- renamed lib/API/API.php to lib/Router/Front.php
- updated namespaces in various file to account for namespace change

lib/Config/Initializer.php

@@ -120,8 +120,12 @@ class Initializer {
     }
 
     try {
+      // legacy router
       $this->setupRouter();
+      // new api (will replace legacy router completely)
       $this->setupAPI();
+
+      $this->setupFrontRouter();
       $this->setupPages();
     } catch(\Exception $e) {
       $this->handleFailedInitialization($e);
@@ -187,8 +191,12 @@ class Initializer {
   }
 
   function setupAPI() {
-    $API = new \MailPoet\API\API();
+
-    $API->init();
+  }
+
+  function setupFrontRouter() {
+    $router = new Router\Front();
+    $router->init();
   }
 
   function runQueueSupervisor() {

lib/Cron/CronHelper.php

@@ -1,8 +1,8 @@
 <?php
 namespace MailPoet\Cron;
 
-use MailPoet\API\API;
+use MailPoet\Router\Front as FrontRouter;
-use MailPoet\API\Endpoints\Queue as QueueAPI;
+use MailPoet\Router\Queue as QueueEndpoint;
 use MailPoet\Models\Setting;
 use MailPoet\Util\Security;
 
@@ -40,9 +40,9 @@ class CronHelper {
 
   static function accessDaemon($token, $timeout = self::DAEMON_REQUEST_TIMEOUT) {
     $data = array('token' => $token);
-    $url = API::buildRequest(
+    $url = FrontRouter::buildRequest(
-      QueueAPI::ENDPOINT,
+      QueueEndpoint::ENDPOINT,
-      QueueAPI::ACTION_RUN,
+      QueueEndpoint::ACTION_RUN,
       $data
     );
     $args = array(

lib/Newsletter/Links/Links.php

@@ -1,8 +1,8 @@
 <?php
 namespace MailPoet\Newsletter\Links;
 
-use MailPoet\API\API;
+use MailPoet\Router\Front as FrontRouter;
-use MailPoet\API\Endpoints\Track as TrackAPI;
+use MailPoet\Router\Track as TrackEndpoint;
 use MailPoet\Models\NewsletterLink;
 use MailPoet\Newsletter\Shortcodes\Shortcodes;
 use MailPoet\Util\Security;
@@ -115,12 +115,12 @@ class Links {
         'queue' => $queue_id,
         'hash' => $hash
       );
-      $API_action = ($matches[2][$index] === self::DATA_TAG_CLICK) ?
+      $router_action = ($matches[2][$index] === self::DATA_TAG_CLICK) ?
-        TrackAPI::ACTION_CLICK :
+        TrackEndpoint::ACTION_CLICK :
-        TrackAPI::ACTION_OPEN;
+        TrackEndpoint::ACTION_OPEN;
-      $link = API::buildRequest(
+      $link = FrontRouter::buildRequest(
-        TrackAPI::ENDPOINT,
+        TrackEndpoint::ENDPOINT,
-        $API_action,
+        $router_action,
         $data
       );
       $content = str_replace($match, $link, $content);

lib/Newsletter/Url.php

@@ -1,8 +1,8 @@
 <?php
 namespace MailPoet\Newsletter;
 
-use MailPoet\API\API;
+use MailPoet\Router\Front as FrontRouter;
-use MailPoet\API\Endpoints\ViewInBrowser as ViewInBrowserAPI;
+use MailPoet\Router\ViewInBrowser as ViewInBrowserEndpoint;
 use MailPoet\Models\Subscriber;
 
 class Url {
@@ -34,9 +34,9 @@ class Url {
         $queue['id'] :
         $queue
     );
-    return API::buildRequest(
+    return FrontRouter::buildRequest(
-      ViewInBrowserAPI::ENDPOINT,
+      ViewInBrowserEndpoint::ENDPOINT,
-      ViewInBrowserAPI::ACTION_VIEW,
+      ViewInBrowserEndpoint::ACTION_VIEW,
       $data
     );
   }

lib/Router/Front.php

@@ -1,17 +1,16 @@
 <?php
-namespace MailPoet\API;
+namespace MailPoet\Router;
 
 use MailPoet\Util\Helpers;
 
 if(!defined('ABSPATH')) exit;
 
-class API {
+class Front {
   public $api_request;
   public $endpoint;
   public $action;
   public $data;
   const NAME = 'mailpoet_api';
-  const ENDPOINT_NAMESPACE = '\MailPoet\API\Endpoints\\';
   const RESPONSE_ERROR = 404;
 
   function __construct($api_data = false) {
@@ -29,10 +28,10 @@ class API {
   }
 
   function init() {
-    $endpoint = self::ENDPOINT_NAMESPACE . ucfirst($this->endpoint);
+    $endpoint = ucfirst($this->endpoint);
     if(!$this->api_request) return;
     if(!$this->endpoint || !class_exists($endpoint)) {
-      self::terminateRequest(self::RESPONSE_ERROR, __('Invalid API endpoint.'));
+      self::terminateRequest(self::RESPONSE_ERROR, __('Invalid Router endpoint.'));
     }
     $this->callEndpoint(
       $endpoint,
@@ -43,7 +42,7 @@ class API {
 
   function callEndpoint($endpoint, $action, $data) {
     if(!method_exists($endpoint, $action)) {
-      self::terminateRequest(self::RESPONSE_ERROR, __('Invalid API action.'));
+      self::terminateRequest(self::RESPONSE_ERROR, __('Invalid Router action.'));
     }
     call_user_func(
       array(

lib/Router/Queue.php

@@ -1,5 +1,5 @@
 <?php
-namespace MailPoet\API\Endpoints;
+namespace MailPoet\Router;
 
 use MailPoet\Cron\Daemon;
 

lib/Router/Subscription.php

@@ -1,5 +1,5 @@
 <?php
-namespace MailPoet\API\Endpoints;
+namespace MailPoet\Router;
 
 use MailPoet\Subscription as UserSubscription;
 

lib/Router/Track.php

@@ -1,5 +1,5 @@
 <?php
-namespace MailPoet\API\Endpoints;
+namespace MailPoet\Router;
 
 use MailPoet\Statistics\Track\Clicks;
 use MailPoet\Statistics\Track\Opens;

lib/Router/ViewInBrowser.php

@@ -1,5 +1,5 @@
 <?php
-namespace MailPoet\API\Endpoints;
+namespace MailPoet\Router;
 
 use MailPoet\Newsletter\ViewInBrowser as NewsletterViewInBrowser;
 

lib/Subscription/Url.php

@@ -1,8 +1,8 @@
 <?php
 namespace MailPoet\Subscription;
 
-use MailPoet\API\API;
+use MailPoet\Router\Front as FrontRouter;
-use MailPoet\API\Endpoints\Subscription;
+use MailPoet\Router\Subscription as SubscriptionEndpoint;
 use MailPoet\Models\Subscriber;
 use MailPoet\Models\Setting;
 
@@ -45,10 +45,10 @@ class Url {
     }
 
     $params = array(
-      API::NAME,
+      FrontRouter::NAME,
-      'endpoint='.Subscription::ENDPOINT,
+      'endpoint='.SubscriptionEndpoint::ENDPOINT,
       'action='.$action,
-      'data='.API::encodeRequestData($data)
+      'data='.FrontRouter::encodeRequestData($data)
     );
 
     // add parameters