PluginService で config.yml をチェックするコードが残っている

[nanasess]

概要(Overview)

おそらく3系の名残り。4系に config.yml は存在しない

ec-cube/src/Eccube/Service/PluginService.php

Lines 418 to 444 in e9edc3b

	public function checkPluginArchiveContent($dir, array $config_cache = [])
	{
	try {
	if (!empty($config_cache)) {
	$meta = $config_cache;
	} else {
	$meta = $this->readConfig($dir);
	}
	} catch (\Symfony\Component\Yaml\Exception\ParseException $e) {
	throw new PluginException($e->getMessage(), $e->getCode(), $e);
	}
	if (!is_array($meta)) {
	throw new PluginException('config.yml not found or syntax error');
	}
	if (!isset($meta['code']) || !$this->checkSymbolName($meta['code'])) {
	throw new PluginException('config.yml code empty or invalid_character(\W)');
	}
	if (!isset($meta['name'])) {
	// nameは直接クラス名やPATHに使われるわけではないため文字のチェックはなしし
	throw new PluginException('config.yml name empty');
	}
	if (!isset($meta['version'])) {
	// versionは直接クラス名やPATHに使われるわけではないため文字のチェックはなしし
	throw new PluginException('config.yml version invalid_character(\W) ');
	}
	}

再現手順(Procedure)

コードレビューで発見

環境(Environment)

EC-CUBE 4.2

[dotani1111]

@nanasess
Issueありがとうございます。
こちら調査を進めたいと思います。

[dotani1111]

@nanasess
readConfigの関数を見ると、内部的にはcomposer.jsonを見ているようでした。
エラーメッセージのみ修正必要になります。
またいくつかのコメントやエラーメッセージでconfig.ymlの記載があるので、合わせて修正します。

ec-cube/src/Eccube/Service/PluginService.php

Lines 453 to 479 in e9edc3b

	public function readConfig($pluginDir)
	{
	$composerJsonPath = $pluginDir.DIRECTORY_SEPARATOR.'composer.json';
	if (file_exists($composerJsonPath) === false) {
	throw new PluginException("${composerJsonPath} not found.");
	}
	$json = json_decode(file_get_contents($composerJsonPath), true);
	if ($json === null) {
	throw new PluginException("Invalid json format. [${composerJsonPath}]");
	}
	if (!isset($json['version'])) {
	throw new PluginException("`version` is not defined in ${composerJsonPath}");
	}
	if (!isset($json['extra']['code'])) {
	throw new PluginException("`extra.code` is not defined in ${composerJsonPath}");
	}
	return [
	'code' => $json['extra']['code'],
	'name' => isset($json['description']) ? $json['description'] : $json['extra']['code'],
	'version' => $json['version'],
	'source' => isset($json['extra']['id']) ? $json['extra']['id'] : 0,
	];
	}