Skip to content

Commit

Permalink
IBX-1431: Fixed incorrect merge of ezsystems/ezplatform-design-engine…
Browse files Browse the repository at this point in the history
… changes

The support for phpstorm semantic configuration for Twig auto-completion
helper has been dropped via
ezsystems/ezplatform-design-engine#40
however due to rebranding changes was not merged properly into the upstream

In v4.0 that deprecated configuration is dropped along with its service
container parameters
  • Loading branch information
alongosz committed Jan 27, 2022
1 parent f2e7164 commit b376b7f
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 80 deletions.
63 changes: 0 additions & 63 deletions src/bundle/DependencyInjection/Compiler/PHPStormPass.php

This file was deleted.

10 changes: 0 additions & 10 deletions src/bundle/DependencyInjection/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -43,16 +43,6 @@ public function getConfigTreeBuilder()
->info('If set to true, assets path won\'t be pre-resolved at compile time.')
->defaultValue('%kernel.debug%')
->end()
->arrayNode('phpstorm')
->addDefaultsIfNotSet()
->children()
->booleanNode('enabled')->defaultValue('%kernel.debug%')->info('Activates PHPStorm support')->end()
->scalarNode('twig_config_path')
->info('Path where to store PHPStorm configuration file for additional Twig namespaces (ide-twig.json).')
->defaultValue('%kernel.project_dir%')
->end()
->end()
->end()
->end();

return $treeBuilder;
Expand Down
4 changes: 0 additions & 4 deletions src/bundle/DependencyInjection/IbexaDesignEngineExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -47,10 +47,6 @@ private function configureDesigns(array $config, ConfigurationProcessor $process
$container->setParameter('ezdesign.templates_override_paths', $config['templates_override_paths']);
$container->setParameter('ezdesign.templates_path_map', $config['templates_theme_paths']);
$container->setParameter('ezdesign.asset_resolution.disabled', $config['disable_assets_pre_resolution']);

// PHPStorm settings
$container->setParameter('ezdesign.phpstorm.enabled', $config['phpstorm']['enabled']);
$container->setParameter('ezdesign.phpstorm.twig_config_path', $config['phpstorm']['twig_config_path']);
}
}

Expand Down
2 changes: 0 additions & 2 deletions src/bundle/IbexaDesignEngineBundle.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@

use Ibexa\Bundle\DesignEngine\DependencyInjection\Compiler\AssetPathResolutionPass;
use Ibexa\Bundle\DesignEngine\DependencyInjection\Compiler\AssetThemePass;
use Ibexa\Bundle\DesignEngine\DependencyInjection\Compiler\PHPStormPass;
use Ibexa\Bundle\DesignEngine\DependencyInjection\Compiler\TwigThemePass;
use Ibexa\Bundle\DesignEngine\DependencyInjection\DesignConfigParser;
use Ibexa\Bundle\DesignEngine\DependencyInjection\IbexaDesignEngineExtension;
Expand All @@ -30,7 +29,6 @@ public function build(ContainerBuilder $container)
$container->addCompilerPass(new TwigThemePass());
$container->addCompilerPass(new AssetThemePass(), PassConfig::TYPE_OPTIMIZE);
$container->addCompilerPass(new AssetPathResolutionPass(), PassConfig::TYPE_OPTIMIZE);
$container->addCompilerPass(new PHPStormPass(), PassConfig::TYPE_OPTIMIZE);
}

public function getContainerExtension()
Expand Down
1 change: 0 additions & 1 deletion src/bundle/Resources/config/services.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ parameters:
ezdesign.assets_override_paths: []
ezdesign.asset_resolved_paths: {}
ezdesign.asset_resolution.disabled: '%kernel.debug%'
ezdesign.phpstorm.enabled: true

services:
Ibexa\DesignEngine\Templating\ThemeTemplateNameResolver:
Expand Down

0 comments on commit b376b7f

Please sign in to comment.