Ticket #1516 (closed 改善提案: 修正済)

Opened 9 years ago

Last modified 9 years ago

複数受注でまとめてメール送信

Reported by: nanasess Owned by: h_yoshimoto
Priority: Milestone: EC-CUBE2.12.0
Component: 管理画面 Version: 2.11.3
Keywords: Cc:
修正済み: yes

Description

現在は1件ずつメール送信するしかないが, 一覧にチェックを入れて, 発送完了メールなどを, まとめてメール送信したい

Change History

comment:1 Changed 9 years ago by eoogle

  • Owner changed from somebody to eoogle
  • Status changed from new to assigned

comment:2 follow-up: ↓ 3 Changed 9 years ago by eoogle

  • 修正済み set

r21687 にて対応しました。

受注検索結果(1ページ内)の受注をチェックで選択して、 送信できます。

多数の受注によるタイムタウトの制御はしていません。

comment:3 in reply to: ↑ 2 Changed 9 years ago by h_yoshimoto

  • 修正済み unset

eoogleさん への返信

r21687 にて対応しました。

受注検索結果(1ページ内)の受注をチェックで選択して、 送信できます。

多数の受注によるタイムタウトの制御はしていません。

ご対応ありますがとうございます!

実装箇所で少し修正して頂きたい箇所がありますので、一旦修正フラグを外させて頂きます。
branches/version-2_12-dev/data/Smarty/templates/admin/order/mail_confirm.tpl
【45行目付近】
<!--{if $order_id_count != 1}-->ですがこの場合、0でも通ってしまいますので"> 1"とした方が良いかと思います。

branches/version-2_12-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Mail.php
【78行目付近】
スーパーグローバル変数に関しましては直接操作しない事になっております。こちら修正お願いい致します。

【95行目付近】
if($this->order_id_array)ですが、変数に入る値が予見出来ないので演算子による判定 or 関数による判定に変更した方が良いかと思います。

コード規約はこちらになりますのでお時間ある時にご確認下さいませ
http://svn.ec-cube.net/open_trac/wiki/EC-CUBE%E6%A8%99%E6%BA%96%E8%A6%8F%E7%B4%84/%E3%83%AA%E3%83%95%E3%82%A1%E3%82%AF%E3%82%BF%E3%83%AA%E3%83%B3%E3%82%B0%E3%82%AC%E3%82%A4%E3%83%89%E3%83%A9%E3%82%A4%E3%83%B3

http://svn.ec-cube.net/open_trac/wiki/EC-CUBE%E6%A8%99%E6%BA%96%E8%A6%8F%E7%B4%84

以上、お忙しい中ご迷惑おかけしますが
ご対応よろしくお願いいたします。

comment:4 Changed 9 years ago by h_yoshimoto

  • Owner changed from eoogle to h_yoshimoto
  • 修正済み set
  • Status changed from assigned to new

r21852 にて修正しました。

comment:5 Changed 9 years ago by h_yoshimoto

  • Status changed from new to assigned

comment:6 Changed 9 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.