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

大量データによりCSVのダウンロードメモリオーバー #4435

Closed
oywc410 opened this issue Jan 14, 2020 · 1 comment
Closed
Labels
Milestone

Comments

@oywc410
Copy link
Contributor

oywc410 commented Jan 14, 2020

概要(Overview)

CSVダウンロードときにメモリ回収されない。
大量データダウンロードするときにメモリオーバーエラーになりやすい。

会員CSVダウンロードを調べたところ下記の修正でオーバーを解消できます

oywc410@9052e3c

問題個所として

src/Eccube/Service/CsvExportService.php::isNotEmptyとisEmpty内は

@trigger_error('The '.__METHOD__.' method is deprecated.', E_USER_DEPRECATED);

が利用されている

もう一つはCSVダウンロードときにイベント処理です。

$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_CUSTOMER_CSV_EXPORT, $event);

期待する内容(Expect) or 要望 (Requirement)

CSVダウンロード時イベント制御と@trigger_error使いを見直したい

再現手順(Procedure)

管理画面画面から大量データをダウンロードするときにメモリ回収せずにオーバーされる

環境 (environment)

  • EC-CUBE: 4.0.x

関連情報 (Ref)

#4336

@chihiro-adachi
Copy link
Contributor

#4815 で対応

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

4 participants