Project

General

Profile

Bug(バグ) #1309

Unused template files are existing in mobile_mail_frontend (使われていない不要なメールテンプレートが存在している)

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

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-07-08
Due date:
% Done:

100%

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

Description

Overview

The mail template files that are called by "opMailSend::sendTemplateMail()" for mobile are "templates/mail/mobile/*".
However, "_requestRegister.php" is existing below:

メール送信時のテンプレートは opMailSend::sendTemplateMail() によって呼ばれ、モバイル版に関してはそれらは templates/mail/mobile/ 以下のファイルに限られます。
しかしながら、メールテンプレートが以下の場所にも存在しています。

  • apps/pc_backend/templates/mail/_requestRegisterURL.php
  • apps/mobile_mail_frontend/templates/mail/_requestRegisterURL.php

refs:

$ ack -g requestRegisterURL.php
apps/pc_backend/templates/mail/_requestRegisterURL.php
apps/mobile_mail_frontend/templates/mail/_requestRegisterURL.php
templates/mail/mobile/_requestRegisterURL.php
templates/mail/pc/_requestRegisterURL.php

Note

This design of mail is different between lte 3.4.x and gte 3.5.x (opAuthMailAddressPlugin)

メールテンプレートなど、メール周りの設計は 3.4.x 以前と 3.5.x 以降で異なっています。
これは新規登録時の opAuthMailAddressPlugin の依存性などの改善によるものと考えられます。

Causes

Defective design about mail.

メールに関する設計不備です。

Way to fix

Delete unused files.

不要なファイルを削除してください。


Related issues

Related to OpenPNE 3 - Bug(バグ) #1304: Mail templates for mobile are defective (モバイル版のメールテンプレートに不備がある) Fixed(完了) 2010-07-07
Related to OpenPNE 3 - Enhancement(機能追加・改善) #1329: Change opMailSend::sendTemplateMail() to not use super global template Fixed(完了) 2010-07-13

Associated revisions

Revision 11444bde (diff)
Added by Shogo Kawahara over 10 years ago

removed some unnecessarily mail templates (fixes #1309)

History

#1 Updated by Shogo Kawahara over 10 years ago

  • Assignee set to Shogo Kawahara

#2 Updated by Shogo Kawahara over 10 years ago

  • Tracker changed from Enhancement(機能追加・改善) to Bug(バグ)

#3 Updated by Shogo Kawahara over 10 years ago

This issue happen from 3.2.x.

#4 Updated by Shogo Kawahara over 10 years ago

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

更新履歴 11444bde55eb42a25edf76bba5797e2686b41cce で適用されました。

#5 Updated by Rimpei Ogawa over 10 years ago

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

#6 Updated by Kousuke Ebihara over 10 years ago

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

Also available in: Atom PDF