Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JobテーブルがTraitでカラム追加できない #6273

Open
beststore-tokyo opened this issue Aug 29, 2024 · 9 comments
Open

JobテーブルがTraitでカラム追加できない #6273

beststore-tokyo opened this issue Aug 29, 2024 · 9 comments
Labels

Comments

@beststore-tokyo
Copy link

概要(Overview)

プラグインで mtb_job テーブルに Traitでカラムを追加し、プラグインのインストールを実行したところ、カラムが追加できません。

再現手順(Procedure)

①プラグインでTraitを作成する
②プラグインをインストールする

Eccube\Doctrine\ORM\Mapping\Driver\ReloadSafeAnnotationDriver.phpファイルの103行の問題と思います。
コードを下記のように変更すると解決できると思います。

//$proxyFile = str_replace($projectDir, $this->trait_proxies_directory, $path).'/'.basename($sourceFile);
$proxyFile = str_replace($projectDir, $this->trait_proxies_directory, $path).(basename(dirname($sourceFile))=="Master" ? "/Master" : '').'/'.basename($sourceFile);

環境(Environment)

EC-CUBE: 4.2.3

@nanasess
Copy link
Contributor

現在 mtb テーブルを拡張すると、以下のように自動で schema update が実行されない問題があります。こちらが解決すれば、以下の PR を取り込みできると思います。
#5401 (comment)

@beststore-tokyo beststore-tokyo mentioned this issue Aug 29, 2024
15 tasks
@beststore-tokyo
Copy link
Author

@nanasess

こちらの件のPRも作りました、プラグインの開発&公開などに影響範囲が大きいので、早急にマージしていただきたいと思います。

@nanasess
Copy link
Contributor

@beststore-tokyo ありがとうございます。この修正で自動で schema update が実行されない問題も解消されますでしょうか?

@beststore-tokyo
Copy link
Author

@nanasess
解消されると思います

@nanasess
Copy link
Contributor

nanasess commented Sep 2, 2024

@beststore-tokyo #6275 の問題の影響で、E2Eテストが停止しております。解消までもうしばらくお待ちください

@nanasess
Copy link
Contributor

@beststore-tokyo
E2Eテストが復旧できましたので、再現テストをしてみました。
自動で schema update が実行されない問題は解消できていないようです。ご確認お願いできますでしょうか🙇‍♂️
https://github.com/nanasess/ec-cube/actions/runs/10803395290/job/29972441998

@beststore-tokyo
Copy link
Author

@nanasess
ありがとうございます。
確認致します

@beststore-tokyo
Copy link
Author

@nanasess
nanasess@4f7b522#r146601757
E2Eテストがよくわからないのですが、テストケースには問題なかったのでしょうか?

@nanasess
Copy link
Contributor

@beststore-tokyo
テストケースのほうも指摘いただいたので確認中です🙇‍♂️

もう1点、本修正を適用すると、以下のテストもエラーになってしまうようですが、ご確認いただけますでしょうか?
https://github.com/nanasess/ec-cube/actions/runs/10826327207/job/30037051615#step:15:102
#4638

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants