Make services private.

Resources/config/admin.yaml

@@ -14,9 +14,6 @@ parameters:
     darvin_menu.admin.slug_map_item_link_view_widget.class: Darvin\MenuBundle\Admin\View\SlugMapItemLinkWidget
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.admin.form_type.menu:
         class: '%darvin_menu.admin.form_type.menu.class%'
         arguments:
@@ -59,8 +56,7 @@ services:
             - { name: form.type }
 
     darvin_menu.admin.menu_item_factory:
-        class:  '%darvin_menu.admin.menu_item_factory.class%'
-        public: false
+        class: '%darvin_menu.admin.menu_item_factory.class%'
         arguments:
             - '@darvin_admin.router'
             - '@security.authorization_checker'
@@ -70,15 +66,15 @@ services:
             - { name: darvin_admin.menu_item_factory }
 
     darvin_menu.admin.menu_item_sorter:
-        class: '%darvin_menu.admin.menu_item_sorter.class%'
+        class:  '%darvin_menu.admin.menu_item_sorter.class%'
+        public: true
         arguments:
             - '@darvin_menu.slug_map.object_loader'
             - '@darvin_utils.tree.sorter'
 
     darvin_menu.admin.slug_map_item_link_view_widget:
         class:  '%darvin_menu.admin.slug_map_item_link_view_widget.class%'
         parent: darvin_admin.view.widget.abstract
-        public: false
         arguments:
             - '@darvin_admin.entity_namer'
             - '@darvin_admin.view.widget.show_link'

Resources/config/breadcrumbs.yaml

@@ -5,12 +5,8 @@ parameters:
     darvin_menu.breadcrumbs.twig_extension.default_options: { currentAsLink: false }
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.breadcrumbs.builder:
-        class:  '%darvin_menu.breadcrumbs.builder.class%'
-        public: false
+        class: '%darvin_menu.breadcrumbs.builder.class%'
         arguments:
             - '@doctrine.orm.entity_manager'
             - '@darvin_menu.item_factory.pool'
@@ -21,8 +17,7 @@ services:
             - '%darvin_menu.breadcrumbs.slug_parameter_name%'
 
     darvin_menu.breadcrumbs.twig_extension:
-        class:  '%darvin_menu.breadcrumbs.twig_extension.class%'
-        public: false
+        class: '%darvin_menu.breadcrumbs.twig_extension.class%'
         arguments:
             - '@darvin_menu.breadcrumbs.builder'
             - '@knp_menu.helper'

Resources/config/builder.yaml

@@ -2,13 +2,9 @@ parameters:
     darvin_menu.builder.abstract.class: Darvin\MenuBundle\Builder\Builder
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.builder.abstract:
         class:    '%darvin_menu.builder.abstract.class%'
         abstract: true
-        public:   false
         arguments:
             - '@doctrine.orm.entity_manager'
             - '@darvin_utils.orm.entity_resolver'

Resources/config/configuration.yaml

@@ -2,10 +2,8 @@ parameters:
     darvin_menu.configuration.menu.class: Darvin\MenuBundle\Configuration\MenuConfiguration
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.configuration.menu:
-        class: '%darvin_menu.configuration.menu.class%'
+        class:  '%darvin_menu.configuration.menu.class%'
+        public: true
         arguments:
             - '%darvin_menu.menus%'

Resources/config/dev/fixture.yaml

@@ -5,12 +5,8 @@ parameters:
     darvin_menu.fixture.menu_item.max_level: 3
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.fixture.menu_item:
-        class:  '%darvin_menu.fixture.menu_item.class%'
-        public: false
+        class: '%darvin_menu.fixture.menu_item.class%'
         arguments:
             - '%darvin_menu.fixture.menu_item.min_count%'
             - '%darvin_menu.fixture.menu_item.max_count%'

Resources/config/item_factory.yaml

@@ -12,27 +12,21 @@ parameters:
     darvin_menu.item_factory.pool.class: Darvin\MenuBundle\Item\Factory\Pool\ItemFactoryPool
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.item_factory.abstract:
         class:    '%darvin_menu.item_factory.abstract.class%'
         abstract: true
-        public:   false
         calls:
             - [ setGenericItemFactory, [ '@knp_menu.factory' ] ]
 
     darvin_menu.item_factory.string:
         class:  '%darvin_menu.item_factory.string.class%'
         parent: darvin_menu.item_factory.abstract
-        public: false
         tags:
             - { name: darvin_menu.item_factory }
 
     darvin_menu.item_factory.abstract_entity:
         class:    '%darvin_menu.item_factory.abstract_entity.class%'
         parent:   darvin_menu.item_factory.abstract
-        public:   false
         abstract: true
         calls:
             - [ setEntityManager, [ '@doctrine.orm.entity_manager' ] ]
@@ -42,7 +36,6 @@ services:
     darvin_menu.item_factory.menu_item_entity:
         class:  '%darvin_menu.item_factory.menu_item_entity.class%'
         parent: darvin_menu.item_factory.abstract_entity
-        public: false
         arguments:
             - '@request_stack'
         tags:
@@ -51,10 +44,8 @@ services:
     darvin_menu.item_factory.slug_map_item_entity:
         class:  '%darvin_menu.item_factory.slug_map_item_entity.class%'
         parent: darvin_menu.item_factory.abstract_entity
-        public: false
         tags:
             - { name: darvin_menu.item_factory }
 
     darvin_menu.item_factory.pool:
-        class:  '%darvin_menu.item_factory.pool.class%'
-        public: false
+        class: '%darvin_menu.item_factory.pool.class%'

Resources/config/matcher.yaml

@@ -2,9 +2,6 @@ parameters:
     darvin_menu.matcher.voter.uri.class: Darvin\MenuBundle\Matcher\Voter\UriVoter
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.matcher.voter.uri:
         class: '%darvin_menu.matcher.voter.uri.class%'
         arguments:

Resources/config/provider.yaml

@@ -1,6 +1,3 @@
 services:
-    _defaults:
-        public: true
-
     darvin_menu.provider:
         alias: knp_menu.menu_provider

Resources/config/slug_map.yaml

@@ -2,12 +2,8 @@ parameters:
     darvin_menu.slug_map.object_loader.class: Darvin\MenuBundle\Slug\SlugMapObjectLoader
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.slug_map.object_loader:
-        class:  '%darvin_menu.slug_map.object_loader.class%'
-        public: false
+        class: '%darvin_menu.slug_map.object_loader.class%'
         arguments:
             - '@darvin_utils.custom_object.loader'
             - '@darvin_image.orm.image_joiner'

Resources/config/switcher.yaml

@@ -6,9 +6,6 @@ parameters:
     darvin_menu.switcher.switch_event_subscriber.class: Darvin\MenuBundle\EventListener\SwitchMenuSubscriber
 
 services:
-    _defaults:
-        public: true
-
     darvin_menu.switcher.admin.form_type:
         class: '%darvin_menu.switcher.admin.form_type.class%'
         arguments:
@@ -18,16 +15,14 @@ services:
             - { name: form.type }
 
     darvin_menu.switcher:
-        class:  '%darvin_menu.switcher.class%'
-        public: false
+        class: '%darvin_menu.switcher.class%'
         arguments:
             - '@doctrine.orm.entity_manager'
             - '@darvin_utils.orm.entity_resolver'
             - '%darvin_menu.switcher.default_menus%'
 
     darvin_menu.switcher.switch_event_subscriber:
-        class:  '%darvin_menu.switcher.switch_event_subscriber.class%'
-        public: false
+        class: '%darvin_menu.switcher.switch_event_subscriber.class%'
         arguments:
             - '@darvin_utils.orm.entity_resolver'
             - '@darvin_menu.switcher'