[WIP] Fixed the possibility to have a smilies list customizable

Decoda/Hook/EmoticonHook.php

@@ -161,29 +161,6 @@ public function getEmoticonCollection()
 
         $this->collection = new EmoticonCollection();
 
-        // Convert a default decoda emoticons array to an EmoticonCollection
-        $collection = new EmoticonCollection();
-
-        if (!$this->getEmoticons()) {
-            if (null === $this->getParser()) {
-                $this->setParser(new Decoda());
-            }
-
-            $this->startup();
-        }
-
-        foreach ($this->getEmoticons() as $name => $smilies) {
-            $emoticon = new Emoticon();
-            foreach ($smilies as $smiley) {
-                $emoticon->setSmiley($smiley);
-            }
-            $collection->add($name, $emoticon);
-        }
-
-
-        $this->collection->addCollection($collection);
-
-
         if (null !== $this->options['resource']) {
             $subCollection = $this->loader->load($this->options['resource'], $this->options['resource_type']);
             $this->collection->addCollection($subCollection);

DependencyInjection/Configuration.php

@@ -113,7 +113,7 @@ private function addEmoticonSection(ArrayNodeDefinition $rootNode)
                     ->canBeUnset()
                     ->addDefaultsIfNotSet()
                     ->children()
-                        ->scalarNode('resource')->end()
+                        ->scalarNode('resource')->defaultValue('@FMBbcodeBundle/Resources/config/emoticons.yml')->end()
                         ->scalarNode('type')->end()
                         ->scalarNode('path')
                             ->defaultValue('/emoticons/')

README.md

@@ -220,6 +220,7 @@ fm_bbcode:
 ```yaml
 # path/to/emoticons.yml
 imports:
+  - { resource: '@FMBbcodeBundle/Resources/config/emoticons.yml' } # Add the emoticons configuration from Decoda
   - { resource: path/to/another/emoticons.yml }
 
 emoticons:

Resources/config/emoticons.yml

@@ -0,0 +1,17 @@
+emoticons:
+    angry: { smilies: ['>(', '>:(', '>[', '>:[', ':angry:'] }
+    aw: { smilies: [':aw:'] }
+    cool: { smilies: [8), '8]', ':cool:'] }
+    ecstatic: { smilies: [':D', 8D, ':ecstatic:'] }
+    furious: { smilies: ['>:D', '><', ':furious:'] }
+    gah: { smilies: ['D:', ':O', ':gah:'] }
+    happy: { smilies: [':)', ':]', ':happy:'] }
+    heart: { smilies: ['<3', ':heart:'] }
+    hm: { smilies: [':/', ':\', ':hm:'] }
+    kiss: { smilies: [':3', ':kiss:'] }
+    meh: { smilies: [':|', '-.-', '<_<', '>_>', ':meh:'] }
+    mmf: { smilies: [':x', ':X', ':mmf:'] }
+    sad: { smilies: [':(', ':[', ;(, ';[', ':''(', ':''[', ';''(', ';''[', ':sad:'] }
+    tongue: { smilies: [':P', ':p', ':tongue:'] }
+    what: { smilies: [':o', ':?', ':what:'] }
+    wink: { smilies: [;), ';]', ;D, ':wink:'] }

Tests/Templating/BbcodeExtensionTest.php

@@ -200,10 +200,10 @@ public function testEmoticonHook($value, $expected)
     public function dataEmoticonHook()
     {
         return array(
-            array(':)', '<img src="/emoticons/happy.png" alt="" />'),
+            array(':)', ':)'),
             array(':tester:', '<img src="/emoticons/tester.png" alt="" />'),
             array(':url_tester:', '<img src="/foo.gif" alt="" />'),
-            array(':\\', '<img src="/emoticons/hm.png" alt="" />'),
+            array(':\\', ':\\'),
             //array(':place_holder:', '<img src="/emoticons//foo.gif" alt="" />'), // FIXME Decoda Bug: result => <img src="/emoticons/tongue.png" alt="" />lace_holder:
             array(':_lace_holder:', '<img src="/emoticons//foo.gif" alt="" />'),
             array(':my_emoticon:', '<img src="/emoticons//my_emoticon.png" alt="" title="my_emoticon" />'),