Project

General

Profile

Bug（バグ） #1304

Mail templates for mobile are defective (モバイル版のメールテンプレートに不備がある)

Added by Minoru Takai about 10 years ago. Updated about 10 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Shogo Kawahara
Target version:
OpenPNE 3.6beta1
Start date:
2010-07-07
Due date:
% Done:

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

Overview

Contents of Templates-PHP files are different from YAML file.

  • Templates PHP: templates/mail/mobile/*
  • YAML: lib/config/config/mail_template.yml

PHPのデフォルトテンプレートに不足（不備）があるため、
管理画面からメール文面を保存しない限り適切なメール本文が送信されません。
署名もその一つです。

また、管理画面上ではYAML側の文面が表示されるため、
メール文面がきちんと設定されているように見えてしまいますが、
実際は送信されないので実装を知らないSNS管理者が混乱してしまいます。

Causes

Defective Implementation.

Way to fix

Prepare proper default (php) mail templates.

Related issues

Related to OpenPNE 3 - Bug（バグ） #1038: 携帯版の退会処理で500エラー Invalid（無効） 2010-05-10
Related to OpenPNE 3 - Bug（バグ） #1309: Unused template files are existing in mobile_mail_frontend (使われていない不要なメールテンプレートが存在している) Fixed（完了） 2010-07-08
Related to OpenPNE 3 - Enhancement（機能追加・改善） #1329: Change opMailSend::sendTemplateMail() to not use super global template Fixed（完了） 2010-07-13

Associated revisions

Revision 61da4bbb (diff)
Added by Shogo Kawahara about 10 years ago

added an template for mobile mail signature (fixes #1304)

History

#1 Updated by Minoru Takai about 10 years ago

このチケットで示した問題に直接は関係していませんが、

http://github.com/openpne/OpenPNE3/blob/OpenPNE-3.4.6/lib/config/config/mail_template.yml#LC276

        {% app_url_for('pc_frontend', 'opAuthMailAddress/register?token='~token~'&authMode='~authMode, true) %}

OpenPNE 3.4.6 時点ではymlファイルの中で、モバイル版の requestRegisterURL テンプレートについて、
'pc_frontend' というアプリケーション名を用いているのが不自然に見えるのですが、これは問題ないものでしょうか。

#2 Updated by Shogo Kawahara about 10 years ago

  • Assignee set to Shogo Kawahara

#3 Updated by Shogo Kawahara about 10 years ago

  • Status changed from New（新規） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

更新履歴 61da4bbb6a57886d01d03e3f7df43ae25dd916fe で適用されました。

#4 Updated by Rimpei Ogawa about 10 years ago

  • Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70

#5 Updated by Kousuke Ebihara about 10 years ago

  • Status changed from Pending Testing（テスト待ち） to Fixed（完了）
  • % Done changed from 70 to 100

Also available in: Atom PDF