ルーティングのBINDのキーを修正 #35

[sw-takaya-murata-zz]

No description provided.

[okazy]

@sw-takaya-murata
PRありがとうございます。
EC-CUBE本体のissueに関連のものが報告されておりました。
https://github.com/EC-CUBE/ec-cube/issues/3096

修正いただいた内容でissueは解決しますが、根本解決としてはEC-CUBE本体側での以下のような対応を検討していますので、こちらのPRはクローズさせていただいてよろしいでしょうか？
https://github.com/EC-CUBE/ec-cube/issues/3096#issuecomment-390135707

もしくは本体の根本対応のPRをいただけますでしょうか。

よろしくお願いいたします。

[sw-takaya-murata-zz]

@okazy
ご確認いただきありがとうございます。
今回のissueについてですが、弊社としては、マージしていただきたいと考えております。

理由としましては以下の通りです。

• 本体のリリースまでの期間が長くなる傾向があるため、バージョンアップが実行されるまで解決されない。
• 本体ver3.16以下の場合にイベントが発火しないバグが存在し続ける。
  （本プラグインが、ver3.16以下を対応バージョンとしない場合は問題ありません。）

こちらのPRのクローズかマージかのご判断につきましては、貴社にお任せいたします。
また、本体の根本対応のPRについては、検討いたします。

よろしくお願いいたします。

[okazy]

@sw-takaya-murata

ご意見ありがとうございます。
確かにそうですね、こちらのPRを取り込む方向で進めたいと思います。

取り込むに当たって、１点修正をお願いできますでしょうか。
すでに本プラグインを利用中の互換性を考慮して、既存のルーティングを残したまま、新しいルーティングを追加できればと思っています。

ServiceProvider/OrderPdfServiceProvider.php

// 帳票の作成
$admin->match('/plugin/order-pdf', '\\Plugin\\OrderPdf\\Controller\\OrderPdfController::index')
    ->bind('admin_plugin_order_pdf');
$admin->match('/plugin/order-pdf', '\\Plugin\\OrderPdf\\Controller\\OrderPdfController::index')
    ->bind('plugin_admin_order_pdf'); // deprecated
// PDFファイルダウンロード
$admin->post('/plugin/order-pdf/download', '\\Plugin\\OrderPdf\\Controller\\OrderPdfController::download')
    ->bind('admin_plugin_order_pdf_download');
$admin->post('/plugin/order-pdf/download', '\\Plugin\\OrderPdf\\Controller\\OrderPdfController::download')
    ->bind('plugin_admin_order_pdf_download'); // deprecated

※ルーティングは先に書いた方が優先されるようです。

お手数をおかけしますがよろしくお願いいたします。

[sw-takaya-murata-zz]

@okazy
ご確認いただきありがとうございます。

ご提案のありました既存のルーティングを残すことについて、
先ほど修正しましたので、ご確認ください。

よろしくお願いいたします。

5e99575

[yosshieee]

@sw-takaya-murata
travis-ciがエラーとなっております。
お手数ですが、ご確認お願いできますでしょうか？

[sw-takaya-murata-zz]

@t-nagahashi

弊社の環境におけるテストでは、エラーが出ませんでした。
またエラーの内容を確認するとPRとは関係ないところで
エラーになっていると思われます。

今回のPR以前から同様のエラーが確認できており、
解決の為のPRもクローズされており、1.0.1でマージ予定だと思われます。

お手数ですが、ご確認お願いいたします。

今回のエラーに対するPR
#30

同様のエラーが確認できるPR
https://travis-ci.org/EC-CUBE/order-pdf-plugin/builds/364033439

[ryo-endo]

ご確認ありがとうございます。

こちらでも調査させていただいたのですが、本体のmaster管理画面テンプレートが一部変更されている部分があり、[帳票出力]メニューの差し込みが上手くできなくなっているようです。(テストNGはこれが原因)
#37

テストが通るかどうか判断できないので、上記不具合をFixさせてからこちら取り込ませていただきたいと思います。