Jon Pugh · Jon Pugh · Jon Pugh · Jon Pugh · Jon Pugh · Jon Pugh
--- a/src/Application.php
+++ b/src/Application.php
@@ -124,7 +124,7 @@ class Application extends BaseApplication
        $commands[] = new ListCommand();
        $commands[] = new SaveCommand();
        $commands[] = new ServicesCommand();
-        $commands[] = new ShellCommand();
+//        $commands[] = new ShellCommand();
        $commands[] = new StatusCommand();
        $commands[] = new VerifyCommand();


--- a/src/Command/ServicesCommand.php
+++ b/src/Command/ServicesCommand.php
@@ -8,6 +8,7 @@ use Aegir\Provision\Context\PlatformContext;
 use Aegir\Provision\Context\ServerContext;
 use Aegir\Provision\Context\SiteContext;
 use Consolidation\AnnotatedCommand\CommandFileDiscovery;
+use Symfony\Component\Console\Exception\LogicException;
 use Symfony\Component\Console\Input\ArrayInput;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputDefinition;
@@ -123,6 +124,10 @@ class ServicesCommand extends Command

        // If server, ask which service type.
        if ($this->context->type == 'server') {
+            if (empty($this->context->getServiceTypeOptions($service))) {
+                throw new \Exception("There was no class found for service $service. Create one named \\Aegir\\Provision\\Service\\{$service}Service");
+            }
+            
            $service_type = $this->io->choice('Which service type?', $this->context->getServiceTypeOptions($service));

            // Then ask for all options.
@@ -137,6 +142,10 @@ class ServicesCommand extends Command
        }
        // All other context types are associating with servers that provide the service.
        else {
+            if (empty($this->getApplication()->getServerOptions($service))) {
+                throw new \Exception("No servers providing $service service were found. Create one with `provision save` or use `provision services` to add to an existing server.");
+            }
+            
            $server = $this->io->choice('Which server?', $this->getApplication()->getServerOptions($service));

            // Then ask for all options.

--- a/src/Context.php
+++ b/src/Context.php
@@ -136,10 +136,7 @@ class Context
        elseif (isset($this->config['service_subscriptions'])) {
            foreach ($this->config['service_subscriptions'] as $service_name => $service) {
                $this->servers[$service_name] = $server = $this->application->getContext($service['server']);
-                $service_type = ucfirst($server->services[$service_name]->type);
-                $service_name = ucfirst($service_name);
-                $service_class = "\\Aegir\\Provision\\Service\\{$service_name}\\{$service_name}{$service_type}Service";
-                $this->services[strtolower($service_name)] = new $service_class($service, $this);
+                $this->services[$service_name] = new ServiceSubscription($this, $server, $service_name);
            }
        }
        else {
@@ -160,7 +157,10 @@ class Context
        $discovery->setSearchPattern('*Service.php');
        $servicesFiles = $discovery->discover(__DIR__ .'/Service', '\Aegir\Provision\Service');
        foreach ($servicesFiles as $serviceClass) {
-            $classes[$serviceClass::SERVICE] = $serviceClass;
+            // If this is a server, show all services. If it is not, but service allows this type of context, load it.
+            if ($this->type == 'server' || in_array($this->type, $serviceClass::allowedContexts())) {
+                $classes[$serviceClass::SERVICE] = $serviceClass;
+            }
        }

        if ($service && isset($classes[$service])) {
@@ -343,9 +343,24 @@ class Context
     */
    public function showServices(DrupalStyle $io) {
        if (!empty($this->getServices())) {
+            $is_server = $this->type == 'server';
            $rows = [];
+            
+            $headers = $is_server?
+                ['Services']:
+                ['Service', 'Server', 'Type'];
+            
            foreach ($this->getServices() as $name => $service) {
-                $rows[] = [$name, $service->type];
+                if ($is_server) {
+                    $rows[] = [$name, $service->type];
+                }
+                else {
+                    $rows[] = [
+                        $name,
+                        $service->server->name,
+                        $service->server->getService($name)->type
+                    ];
+                }

                // Show all properties.
                if (!empty($service->properties )) {
@@ -354,7 +369,7 @@ class Context
                    }
                }
            }
-            $io->table(['Services'], $rows);
+            $io->table($headers, $rows);
        }
    }


--- a/src/Context/SiteContext.php
+++ b/src/Context/SiteContext.php
@@ -33,15 +33,18 @@ class SiteContext extends Context implements ConfigurationInterface
        parent::__construct($name, $console_config, $application, $options);

        // Load "db_server" context.
-        if (isset($this->config['db_server'])) {
-            $this->db_server = $application->getContext($this->config['service_subscriptions']['db']['server']);
-            $this->db_server->logger = $application->logger;
+//        if (isset($this->config['service_subscriptions']['db'])) {
+//            $this->db_server = $application->getContext($this->config['service_subscriptions']['db']['server']);
+//        }

-            $this->platform = $application->getContext($this->config['platform']);
-        }
-        else {
-            throw new \Exception('No db_server found.');
+        $this->logger = $application->logger;
+        
+        if (isset($this->config['platform'])) {
+            $this->platform = $application->getContext(
+                $this->config['platform']
+            );
        }
+    
    }

    static function option_documentation()

--- a/src/Service.php
+++ b/src/Service.php
@@ -27,6 +27,14 @@ class Service {
  function verify() {
    $this->writeConfigurations();
  }
+    
+    /**
+     * List context types that are allowed to subscribe to this service.
+     * @return array
+     */
+    static function allowedContexts() {
+        return [];
+    }
  
  /**
   * Write this service's configurations.

--- a/src/Service/DbService.php
+++ b/src/Service/DbService.php
@@ -36,7 +36,17 @@ class DbService extends Service
            'db_grant_all_hosts' => 'Grant access to site database users from any web host. If set to TRUE, any host will be allowed to connect to MySQL site databases on this server using the generated username and password. If set to FALSE, web hosts will be granted access by their detected IP address.',
        ];
    }
-
+    
+    /**
+     * List context types that are allowed to subscribe to this service.
+     * @return array
+     */
+    static function allowedContexts() {
+        return [
+            'site'
+        ];
+    }
+    
    /**
     * Implements Service::server_options()
     *

--- a/src/Service/Http/Apache/Configuration/SiteConfiguration.php
+++ b/src/Service/Http/Apache/Configuration/SiteConfiguration.php
@@ -55,5 +55,16 @@ class SiteConfiguration extends Configuration {
      $this->template = $this->disabled_template;
    }

+    $app_dir = $this->context->console_config['config_path'] . '/' . $this->service->getType();
+
+//    $this->data['http_port'] = $this->service->properties['http_port'];
+//    $this->data['include_statement'] = '# INCLUDE STATEMENT';
+//    $this->data['http_pred_path'] = "{$app_dir}/pre.d";
+//    $this->data['http_postd_path'] = "{$app_dir}/post.d";
+//    $this->data['http_platformd_path'] = "{$app_dir}/platform.d";
+//    $this->data['extra_config'] = "";
+
+    $this->data['http_vhostd_path'] = "{$app_dir}/vhost.d";
+
  }
 }
\ No newline at end of file
--- a/src/Service/HttpService.php
+++ b/src/Service/HttpService.php
@@ -40,6 +40,16 @@ class HttpService extends Service {
            'restart_command' => 'The command to reload the web server configuration;'
        ];
    }
+    
+    /**
+     * List context types that are allowed to subscribe to this service.
+     * @return array
+     */
+    static function allowedContexts() {
+        return [
+            'platform'
+        ];
+    }

    /**
   * Support the ability to cloak the database credentials using environment variables.

--- a/src/ServiceSubscription.php
+++ b/src/ServiceSubscription.php
+<?php
+/**
+ * @file
+ * A context's subscription to a service. Handles properties specific to a
+ * context for each service.
+ *
+ * @see Provision_Service
+ */
+
+namespace Aegir\Provision;
+
+class ServiceSubscription {
+  
+   public $context;
+   public $service;
+   public $server;
+   public $type;
+  
+  function __construct($context, $server, $service_name) {
+      $this->context = $context;
+      $this->server = $server;
+      $this->service = $server->getService($service_name);
+      $this->type = $server->getService($service_name)->type;
+  }
+}