Skip to content

Commit 6e6ad22

Browse files
committed
Simplify pugModule handle
1 parent e485a4a commit 6e6ad22

File tree

2 files changed

+4
-20
lines changed

2 files changed

+4
-20
lines changed

src/JsPhpize/JsPhpizePhugFormatter.php

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,12 @@ public function injectFormatter(Formatter $formatter)
2727
$compiler = $this->compiler;
2828
// 'dependencies_storage'
2929
$formatter->setOptionsRecursive([
30-
'dependencies_storage_getter' => function ($phpCode) {
31-
$phpCode = ltrim($phpCode);
32-
33-
return substr($phpCode, 0, 1) === '$'
34-
? substr($phpCode, 1)
35-
: $phpCode;
36-
},
37-
'patterns' => [
38-
'transform_expression' => function ($jsCode) use (&$compiler) {
30+
'patterns' => [
31+
'transform_expression' => function ($jsCode) use (&$compiler, &$formatter) {
3932
/** @var JsPhpize $jsPhpize */
4033
$jsPhpize = $compiler->getOption('jsphpize_engine');
34+
$pugModuleName = $formatter->getOption('dependencies_storage');
35+
$jsCode = str_replace('$'.$pugModuleName, $pugModuleName, $jsCode);
4136

4237
try {
4338
return rtrim(trim(preg_replace(

tests/JsPhpize/JsPhpizePhugTest.php

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -53,15 +53,4 @@ public function testPlug()
5353
$compiler->compile('a(foo?!=array("foo" => "bar")[\'foo\'])')
5454
);
5555
}
56-
57-
public function testDependencyStorageFixer()
58-
{
59-
$formatter = new Formatter([
60-
'modules' => [new JsPhpizePhugFormatter(new Compiler())],
61-
]);
62-
$getter = $formatter->getOption('dependencies_storage_getter');
63-
64-
self::assertSame('foo', $getter(' foo'));
65-
self::assertSame('foo', $getter(' $foo'));
66-
}
6756
}

0 commit comments

Comments
 (0)