Skip to content

Commit 3f34291

Browse files
committed
minor #131 Remove some deprecations (phansys)
This PR was merged into the master branch. Discussion ---------- |Q |A | |--- |--- | |Branch |master| |Bug fix? |no | |New feature? |no | |BC breaks? |no | |Deprecations?|no | |Tests pass? |yes | |Fixed tickets|n/a | |License |MIT | |Doc PR |n/a | Commits ------- 40545ad Fixed some deprecations
2 parents 9b80773 + 40545ad commit 3f34291

4 files changed

Lines changed: 24 additions & 4 deletions

File tree

Command/AutoClosingCommand.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,15 @@
1111

1212
class AutoClosingCommand extends ContainerAwareCommand
1313
{
14+
protected static $defaultName = 'ticket:autoclosing';
15+
1416
/**
1517
* {@inheritdoc}
1618
*/
1719
protected function configure()
1820
{
1921
$this
20-
->setName('ticket:autoclosing')
22+
->setName(static::$defaultName) // BC for symfony/console < 3.4.0
2123
->setDescription('Automatically close resolved tickets still opened')
2224
->addArgument(
2325
'username',

Command/TicketManagerCommand.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,15 @@
1111

1212
class TicketManagerCommand extends ContainerAwareCommand
1313
{
14+
protected static $defaultName = 'ticket:create';
15+
1416
/**
1517
* {@inheritdoc}
1618
*/
1719
protected function configure()
1820
{
1921
$this
20-
->setName('ticket:create')
22+
->setName(static::$defaultName) // BC for symfony/console < 3.4.0
2123
->setDescription('Create a new Ticket')
2224
->addArgument(
2325
'subject',

DependencyInjection/Configuration.php

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,15 @@ class Configuration implements ConfigurationInterface
1717
*/
1818
public function getConfigTreeBuilder()
1919
{
20-
$treeBuilder = new TreeBuilder();
20+
$treeBuilder = new TreeBuilder('hackzilla_ticket');
21+
if (method_exists($treeBuilder, 'getRootNode')) {
22+
$rootNode = $treeBuilder->getRootNode();
23+
} else {
24+
// BC layer for symfony/config 4.1 and older
25+
$rootNode = $treeBuilder->root('hackzilla_ticket');
26+
}
2127

22-
$treeBuilder->root('hackzilla_ticket')
28+
$rootNode
2329
->children()
2430
->scalarNode('user_class')->isRequired()->cannotBeEmpty()->end()
2531
->scalarNode('ticket_class')->cannotBeEmpty()->defaultValue('Hackzilla\Bundle\TicketBundle\Entity\Ticket')->end()

Resources/config/services.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,3 +70,13 @@ services:
7070
class: Hackzilla\Bundle\TicketBundle\EventListener\FileSubscriber
7171
tags:
7272
- { name: kernel.event_subscriber }
73+
74+
hackzilla_ticket.command.autoclosing:
75+
class: Hackzilla\Bundle\TicketBundle\Command\AutoClosingCommand
76+
tags:
77+
- { name: console.command, command: ticket:autoclosing }
78+
79+
hackzilla_ticket.command.create:
80+
class: Hackzilla\Bundle\TicketBundle\Command\TicketManagerCommand
81+
tags:
82+
- { name: console.command, command: ticket:create }

0 commit comments

Comments
 (0)