updates for the simple installation

youshido-php · May 9, 2016 · f6c5682 · f6c5682
1 parent 00f4b33
commit f6c5682
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 13 deletions.
diff --git a/Command/GenerateSchemaCommand.php b/Command/GenerateSchemaCommand.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace AppBundle\Command;
+namespace Youshido\GraphQLBundle\Command;
 
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Input\InputArgument;
@@ -14,7 +14,7 @@ class GenerateSchemaCommand extends ContainerAwareCommand
     protected function configure()
     {
         $this
-            ->setName('graphql:generate-schema')
+            ->setName('graphql:schema:generate')
             ->setDescription('Generates GraphQL Schema class')
             ->addArgument('bundle', InputArgument::REQUIRED, 'Bundle to generate class to');
     }

diff --git a/Controller/GraphQLController.php b/Controller/GraphQLController.php
@@ -41,7 +41,8 @@ public function defaultAction()
         }
 
         $processor = $this->get('youshido.graphql.processor');
-        if ($schemaClass = $this->getParameter('youshido.graphql.schema_class')) {
+        if ($this->container->hasParameter('youshido.graphql.schema_class')) {
+            $schemaClass = $this->getParameter('youshido.graphql.schema_class');
             if (!class_exists($schemaClass)) {
                 throw new ConfigurationException('Schema class ' . $schemaClass . ' does not exist');
             }
@@ -52,4 +53,4 @@ public function defaultAction()
         return new JsonResponse($processor->getResponseData(), 200, $this->getParameter('youshido.graphql.response_headers'));
     }
 
-}
+}
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -23,7 +23,7 @@ public function getConfigTreeBuilder()
 
         $rootNode
             ->children()
-                ->scalarNode('query_schema')->cannotBeEmpty()->end()
+                ->scalarNode('schema_class')->cannotBeEmpty()->end()
                 ->scalarNode('logger')->defaultValue(null)->end()
                 ->arrayNode('response_headers')
                     ->prototype('array')

diff --git a/DependencyInjection/GraphQLExtension.php b/DependencyInjection/GraphQLExtension.php
@@ -29,7 +29,7 @@ public function load(array $configs, ContainerBuilder $container)
             $responseHeaders[$responseHeader['name']] = $responseHeader['value'];
         }
 
-        $container->setParameter('youshido.graphql.project_schema', $this->getConfig('query_schema', null));
+        $container->setParameter('youshido.graphql.schema_class', $this->getConfig('schema_class', null));
         $container->setParameter('youshido.graphql.response_headers', $responseHeaders);
         $container->setParameter('youshido.graphql.logger', $this->getConfig('logger', null));
 
@@ -40,7 +40,7 @@ public function load(array $configs, ContainerBuilder $container)
     private function getDefaultHeaders()
     {
         return [
-            ['name'  => 'Access-Control-Allow-Origin', 'value' => '*'],
+            ['name' => 'Access-Control-Allow-Origin', 'value' => '*'],
         ];
     }
 

diff --git a/Resources/config/services.yml b/Resources/config/services.yml
@@ -6,10 +6,5 @@ services:
     youshido.graphql.processor:
         class: %youshido.graphql.processor.class%
         calls:
-            - [ setSchema, [@youshido.graphql.schema]]
-            - [ setContainer, [@service_container]]
+            - [ setContainer, ['@service_container']]
             - [ setLogger, [%youshido.graphql.logger%] ]
-
-
-    youshido.graphql.schema:
-        class: %youshido.graphql.project_schema%