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

Opened 7 years ago

Last modified 6 years ago

「システム設定>パラメーター設定」画面にて、rankが同じデータがあると意図していない設定が保存される場合がある

Reported by: habu Owned by: somebody
Priority: Milestone: EC-CUBE2.13.2
Component: 管理画面 Version: 2.13.0
Keywords: Cc:
修正済み: yes

Description (last modified by habu) (diff)

2.13.0にて FORM_COUNTRY_ENABLE と USE_NORMALIZE_HOSTNAME のrankがどちらも1414なので、不具合が顕在化した。

以下のキャプチャの様に、FORM_COUNTRY_ENABLE と USE_NORMALIZE_HOSTNAME の説明欄が逆になっている。

また、この状態でデータを更新すると、IDとは違うパラメータとして値が保存されるので、パラメーター名で検索して値を更新すると、思わぬ設定間違いが発生する事になる。

パラメータ設定画面と、pgAdminⅢの画面キャプチャ

まずは rank が重複しないようにSQLを変更するとして、最終的には、rankが重複していても意図していない設定が保存される事がなくなるようにすべき。

また、インストール時のSQLのrank値の設定について、現在は1刻みになっていますが、機能ごとにグルーピングして、〇〇機能は1000番台、△△機能は2000番台というように定めて、それぞれの項目は10番刻みくらいで設定しておくようにすると、パラメーターを新たに追加する際に、途中の場所に入れこみやすくなるのではないかとも考えています。(これは、他の方にも要確認ですね)

例:

1010 SAMPLE_ADDRESS1
1020 SAMPLE_ADDRESS2
2010 LARGE_IMAGE_WIDTH
2020 LARGE_IMAGE_HEIGHT

Attachments

2013-10-03_181247.png Download (34.6 KB) - added by habu 7 years ago.
パラメータ設定画面と、pgAdminⅢの画面キャプチャ

Change History

Changed 7 years ago by habu

パラメータ設定画面と、pgAdminⅢの画面キャプチャ

comment:1 Changed 7 years ago by habu

  • Owner changed from somebody to habu
  • Status changed from new to assigned
  • Description modified (diff)

comment:2 Changed 7 years ago by habu

  • Description modified (diff)

comment:3 Changed 7 years ago by habu

  • Owner habu deleted
  • Status changed from assigned to new
  • Description modified (diff)

まずは r23239 にて、rankが重複しない様にSQLを修正しました。

最終的には、rankが重複していても意図していない設定が保存される事がなくなるようにパラメーター設定画面の挙動を修正すべきだと思います。

ただ、時間が無いため、一旦手放します。

comment:4 Changed 7 years ago by m_uehara

  • Owner set to somebody
  • Milestone changed from EC-CUBE 2.13.1 to EC-CUBE 2.13.2

comment:5 Changed 6 years ago by h_yoshimoto

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

不具合自体は修正されているのでクローズさせていただきます。

ありがとうございました。

Note: See TracTickets for help on using tickets.