Project

General

Profile

Actions

Enhancement(機能追加・改善) #1620

open

opMailSend::sendTemplateMailで、$params['subject']の扱いが適切でない

Added by Hidenori Goto about 14 years ago. Updated almost 5 years ago.

Status:
New(新規)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2010-09-27
Due date:
% Done:

0%

Estimated time:

Description

現象

1

lib/util/opMailSend.class.phpで、yamlテンプレートを使ってメールを送信する場合、yamlテンプレートにメールの件名も記述しているため、呼び出し側ではパラメーターを省略したいが、パラメーターを省略すると警告(Notice)が発生する。

Notice: Undefined index: subject in ***\lib\util\opMailSend.class.php on line ---

2

呼び出し側で$params['subject']を指定しても、メールの件名を変更できない。

修正案

  • $params['subject']が設定されていない場合を考慮する。
  • $params['subject']が設定されている場合はテンプレートよりも$params['subject']を優先する?(コアのポリシーによる)
  • 上で$params['subject']を優先するとした場合、$params['subject']が設定されていればテンプレートからの読み込みは行わない

パッチ

添付参照
($params['subject']が設定されている場合はこれを優先するという処理にしてあります)

また、このパッチでは最終的に$subjectが空だった場合もエラーにする(falseを返す)という処理も追加していますが、これは別チケットとして扱った方がよいのかもしれません。

関連

http://redmine.openpne.jp/issues/471


Files

Actions #1

Updated by kaoru n almost 5 years ago

  • Target version set to OpenPNE 3.10.x
Actions

Also available in: Atom PDF