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

【要望】PHPExcelを標準に含めて欲しい #2019

Closed
SumikoTohyama opened this issue Jan 9, 2017 · 4 comments
Closed

【要望】PHPExcelを標準に含めて欲しい #2019

SumikoTohyama opened this issue Jan 9, 2017 · 4 comments
Milestone

Comments

@SumikoTohyama
Copy link

前提

プラグインの開発で、Excelで出力する帳票を作成する。

問題・課題

①Excel出力するには、PHPExcelを使用する。
②プラグインで出力できるようにするには、PHPExcelをプラグイン内に含めないといけない。
③コンポーネントのサイズが大きいため、PHPのアップロードサイズの上限を変更する必要がある。
④他のプラグインでも使用する場合、重複する。

対応方法

PHPExcelコンポーネントをEC-Cube3の標準に含める。

@pineray
Copy link
Contributor

pineray commented Jan 9, 2017

PHPExcelを標準に含めるよりは、プラグインの機構に依存情報を含めるほうが良いと思います。
PHPExcelのみのユーティリティプラグインを別途作成して、それがインストールされている場合にのみ動作する、というようにすれば他にも応用できます。

アップロードサイズの上限については、WordPressのような管理画面からの直接ダウンロードを実現していただけるのではないかな、と株式会社ロックオンさんには期待しています。

@Yangsin Yangsin added the enhancement 機能追加 label Jan 10, 2017
@Yangsin Yangsin added this to the 3.x milestone Jan 10, 2017
@Yangsin
Copy link

Yangsin commented Jan 10, 2017

前提として本体で利用していないコンポーネントを標準に含めるのは避けたいと考えています。

その上で、ファイルサイズの問題は何かしら解決はできると思いますがPHPExcelにかぎらず、プラグインで外部のコンポーネントをどう利用するかという作法がないので、そこらへんがうまく必要になってくるなと。

こちら3.xレベルにはなってしまうかと思いますが、どのような形がのぞましいか、皆様のお知恵をお借りできればと思います。
既存のIssueやPullRequestも関連しそうですね; #1648 #1543

@nanasess
Copy link
Contributor

使ったことないですが、 Composer に関していえば、こういうの使えませんかね?
https://github.com/wikimedia/composer-merge-plugin

コマンドラインで叩けないような環境では難しいかもしれません。。。

@okazy
Copy link
Contributor

okazy commented Mar 7, 2018

#2034 でプラグインでもcomposerが使えるようになりますので、こちらのissueはクローズします。

@okazy okazy closed this as completed Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants