Review

app/DoctrineMigrations/Version20170606155640.php

@@ -8,7 +8,8 @@ use Symfony\Component\DependencyInjection\ContainerAwareInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**
- * Add api_user_registration in craue_config_setting.
+ * Remove wallabag_url from craue_config_setting.
+ * It has been moved into the parameters.yml
  */
 class Version20170606155640 extends AbstractMigration implements ContainerAwareInterface
 {

app/config/parameters.yml.dist

@@ -23,7 +23,7 @@ parameters:
     # with PostgreSQL and SQLite, you must set "utf8"
     database_charset: utf8mb4
 
-    domain_name: wallabag.me
+    domain_name: http://your-wallabag-url-instance.com
 
     mailer_transport: smtp
     mailer_host: 127.0.0.1

app/config/parameters_test.yml

@@ -8,4 +8,3 @@ parameters:
     test_database_path: "%env(TEST_DATABASE_PATH)%"
     env(TEST_DATABASE_PATH): "%kernel.root_dir%/../data/db/wallabag_test.sqlite"
     test_database_charset: utf8
-    domain_name: 'http://v2.wallabag.org'

app/config/wallabag.yml

@@ -26,7 +26,6 @@ wallabag_core:
     fetching_error_message: |
         wallabag can't retrieve contents for this article. Please <a href="http://doc.wallabag.org/en/user/errors_during_fetching.html#how-can-i-help-to-fix-that">troubleshoot this issue</a>.
     api_limit_mass_actions: 10
-    domain_name: '%domain_name%'
     default_internal_settings:
         -
             name: share_public

src/Wallabag/CoreBundle/DependencyInjection/Configuration.php

@@ -63,8 +63,6 @@ class Configuration implements ConfigurationInterface
                         ->end()
                     ->end()
                 ->end()
-                ->scalarNode('domain_name')
-                ->end()
             ->end()
         ;
 

src/Wallabag/CoreBundle/DependencyInjection/WallabagCoreExtension.php

@@ -29,7 +29,6 @@ class WallabagCoreExtension extends Extension
         $container->setParameter('wallabag_core.fetching_error_message_title', $config['fetching_error_message_title']);
         $container->setParameter('wallabag_core.api_limit_mass_actions', $config['api_limit_mass_actions']);
         $container->setParameter('wallabag_core.default_internal_settings', $config['default_internal_settings']);
-        $container->setParameter('wallabag_core.domain_name', $config['domain_name']);
 
         $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');

src/Wallabag/CoreBundle/Resources/config/services.yml

@@ -195,7 +195,7 @@ services:
         arguments:
             - "@wallabag_core.entry.download_images.client"
             - "%kernel.root_dir%/../web/assets/images"
-            - '%wallabag_core.domain_name%'
+            - '%domain_name%'
             - "@logger"
 
     wallabag_core.entry.download_images.client:

tests/Wallabag/CoreBundle/Controller/EntryControllerTest.php

@@ -1000,7 +1000,7 @@ class EntryControllerTest extends WallabagCoreTestCase
         $this->assertEquals($url, $entry->getUrl());
         $this->assertContains('Perpignan', $entry->getTitle());
         // instead of checking for the filename (which might change) check that the image is now local
-        $this->assertContains('http://v2.wallabag.org/assets/images/', $entry->getContent());
+        $this->assertContains('http://your-wallabag-url-instance.com/assets/images/', $entry->getContent());
 
         $client->getContainer()->get('craue_config')->set('download_images_enabled', 0);
     }