Skip to content

Commit

Permalink
[FrameworkBundle] display actual target for error in AssetsInstallCom…
Browse files Browse the repository at this point in the history
…mand

When assets:install fails because the target directory does not exist, it should display the actual directory it wanted to have instead of the configuration directive. In most cases, the target directory is retrieved from the kernel config and thus differs from the argument.
  • Loading branch information
NerdyProjects authored and nicolas-grekas committed May 8, 2020
1 parent 45a69c2 commit 5e5c982
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Command/AssetsInstallCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
if (is_dir(\dirname($targetArg).'/web')) {
$targetArg = \dirname($targetArg).'/web';
} else {
throw new InvalidArgumentException(sprintf('The target directory "%s" does not exist.', $input->getArgument('target')));
throw new InvalidArgumentException(sprintf('The target directory "%s" does not exist.', $targetArg));
}
}
}
Expand Down

0 comments on commit 5e5c982

Please sign in to comment.