Ticket #543 (closed バグ指摘: 修正済)

Opened 12 years ago

Last modified 12 years ago

住所の文字長上限が統一されていない

Reported by: Seasoft Owned by: Yammy
Priority: Milestone: EC-CUBE2.4.3
Component: フロント Version: 2.4.1
Keywords: Cc:
修正済み:

Description

フォーラムにて medk 様の報告。  http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=4521&forum=4

  • STEXT_LEN と MTEXT_LEN の箇所がある。
  • mtb_constants_init.php を見る限り、MTEXT_LEN が正しい。
  • MTEXT_LEN では、PDF 帳票出力で問題が発生するケースがある。

想定される影響範囲

  • フロント機能 PCサイト「会員登録」
  • フロント機能 PCサイト「MYページ - 会員登録内容変更」
  • フロント機能 PCサイト「MYページ - お届け先追加・変更」
  • フロント機能 PCサイト「注文工程」
  • フロント機能 モバイルサイト (詳細未確認だが、PCサイトと同等か?)
  • 管理機能「顧客管理 - 顧客管理」
  • 管理機能「受注管理 - 受注管理」
  • 管理機能「受注管理 - 新規受注入力」
  • 管理機能 PDF 帳票出力 (納品書)

Change History

comment:1 Changed 12 years ago by Yammy

STEXT_LEN で指定されている箇所を MTEXT_LEN に指定する?!

comment:2 Changed 12 years ago by Yammy

  • Owner changed from somebody to Yammy

comment:3 Changed 12 years ago by Yammy

ざっと調べた感じ、影響範囲は以下のようです。とりあえず、STEXT_LEN の指定箇所をMTEXT_LENに変えるべき?

STEXT_LEN
./class/pages/shopping/LC_Page_Shopping.php
./class/pages/shopping/LC_Page_Shopping_Deliv.php
./class/pages/admin/basis/LC_Page_Admin_Basis.php
./class/pages/admin/basis/LC_Page_Admin_Basis_Tradelaw.php
./class/pages/admin/order/LC_Page_Admin_Order_Edit.php

MTEXT_LEN
./class/pages/campaign/LC_Page_CampaignEntry.php
./class/pages/mypage/LC_Page_Mypage_DeliveryAddr.php
./class/pages/admin/customer/LC_Page_Admin_Customer_Edit.php
./class/pages/contact/LC_Page_Contact.php
./class/pages/inquiry/LC_Page_Inquiry.php
./class/pages/entry/LC_Page_Entry.php

STEXT_LEN, MTEXT_LEN混在
./class/pages/mypage/LC_Page_Mypage_Change.php
./class/pages/shopping/LC_Page_Shopping_DelivAddr.php

comment:4 Changed 12 years ago by Yammy

  • Status changed from new to closed
  • Resolution set to 修正済

comment:5 Changed 12 years ago by Yammy

STEXT_LEN の指定になっている部分を MTEXT_LEN に変更して r18474 としてコミットしてみました。

comment:6 Changed 12 years ago by kajiwara

  • Status changed from closed to reopened
  • Resolution 修正済 deleted

version-2_4 へのマージがまだなため、一旦差し戻しします。

version-2_4へのマージが完了後、改めて、解決済にして下さいませ。

comment:7 Changed 12 years ago by nanasess

  • Status changed from reopened to closed
  • Resolution set to 修正済

スミマセン. r18523 で version-2_4 へマージ済みです.

Note: See TracTickets for help on using tickets.