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

Opened 9 years ago

Last modified 9 years ago

基本情報管理>配送方法設定の伝票No.URLがmaxlength="50"だと少ない

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

Description

開発コミュニティーサイトより下記報告あり。

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=9105&forum=9

【管理画面】~2.11.3 基本情報管理>配送方法設定の伝票No.URLがmaxlength="50"だと少ない
軽微ですが、題の通り。

基本情報管理>配送方法設定のフォームで伝票No.URLに maxlength="50" が付加されており、例えば佐川のURL
http://k2k.sagawa-exp.co.jp/p/sagawa/web/okurijoinput.jsp
だと入力できなくなります。

試しにmaxlengthを無効にして入力してみると文字数エラーが。


確認したら、STEXT_LEN が割り当てられているのですね。


/data/class/pages/admin/basis/LC_Page_Admin_Basis_Delivery_Input.php
        $this->objFormParam->addParam("伝票No.確認URL", "confirm_url", STEXT_LEN, "n", array("URL_CHECK", "MAX_LENGTH_CHECK"), "http://");

(コードは2.4.4から抜粋)

URLで50文字(デフォルト)は少ないと思います。
大したことじゃないのですが、コード触れない人にとっては STEXT_LEN の値を増やすしか対処法がないわけですので。


        $this->objFormParam->addParam("伝票No.確認URL", "confirm_url", URL_LEN, "n", array("URL_CHECK", "MAX_LENGTH_CHECK"), "http://");

と URL_LEN あたりを使うのがベストでしょうか。

Change History

comment:1 Changed 9 years ago by shutta

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

comment:2 Changed 9 years ago by shutta

  • 修正済み set

clicktx さんの指摘のそのままに、 r21289 にて、「伝票No.URL」項目の文字数制限をSTEXT_LENからURL_LENに変更した。

comment:3 Changed 9 years ago by h_yoshimoto

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

確認しました。closeさせて頂きます。

Note: See TracTickets for help on using tickets.