Added required configuration for facebook service

nucleos · Feb 25, 2017 · 3ec0d82 · 3ec0d82
1 parent 965a852
commit 3ec0d82
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 1 deletion.
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -38,6 +38,16 @@ public function getConfigTreeBuilder()
                     ->end()
                 ->end()
             ->end()
+            ->arrayNode('services')
+                ->children()
+                    ->arrayNode('facebook')
+                    ->children()
+                        ->scalarNode('app_id')->end()
+                        ->scalarNode('secret')->end()
+                    ->end()
+                ->end()
+            ->end()
+        ->end()
         ;
 
         return $treeBuilder;

diff --git a/DependencyInjection/Core23ShariffExtension.php b/DependencyInjection/Core23ShariffExtension.php
@@ -28,6 +28,14 @@ public function load(array $configs, ContainerBuilder $container)
         $loader->load('block.xml');
         $loader->load('services.xml');
 
-        $container->setParameter('core23_shariff.options', $config['options']);
+        $options = $config['options'];
+
+        if (empty($config['services']['facebook']['app_id']) || empty($config['services']['facebook']['secret'])) {
+            $options['services'] = array_diff($options['services'], array('Facebook'));
+        } else {
+            $options['Facebook'] = $config['services']['facebook'];
+        }
+
+        $container->setParameter('core23_shariff.options', $options);
     }
 }
diff --git a/README.md b/README.md
@@ -71,6 +71,11 @@ core23_shariff:
     options:
         domains: [ ] # Allow specific domains for shariff
         services: [ 'GooglePlus', 'Facebook', 'LinkedIn', 'Reddit', 'StumbleUpon', 'Flattr', 'Pinterest', 'Xing', 'AddThis' ]
+    services:
+        # Optional configuration when using facebook service
+        facebook:
+            app_id: 1234567890
+            secret: GENERATEDSECRET
 ```
 
 See [shariff-php] for a list of all available services.