ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 11 years ago

Closed 11 years ago

#4049 closed task (fixed)

upgrade scriptにREADMEを追加する

Reported by: kiwa Owned by: kiwa
Priority: minor Milestone: OpenPNE2.14RC1
Component: 指定しない Version:
Keywords: Cc:

Change History (15)

comment:1 Changed 11 years ago by kiwa

Type: enhancementtask

comment:2 Changed 11 years ago by kiwa

Owner: changed from nobody to kiwa
Status: newassigned

やります。

comment:3 Changed 11 years ago by kiwa

Description: modified (diff)
Keywords: 確認中 added

以下のリビジョンで対応しました。ご確認下さい。

comment:4 Changed 11 years ago by shingo

  • 以下の箇所がおかしい(今回追記されたわけではありませんが)
    setup/script/README.txt
    3	・用途
    4	により分類されて別ディレクトリに格納されているので、適切なファイルを選択してご使用ください。
    
  • 同一ディレクトリで実行する必要がある胸を銘記する必要があるのではないでしょうか?
     php update01-for2.13.8-insert_image_size.php
    
  • どういう機能なのか・update(upgrade)する必要があるのかどうかが不鮮明な気がします
    setup/script/update/README.txt
    既存のアップロード画像容量を画像アップロード容量制限に反映させるスクリプトです
    

comment:5 Changed 11 years ago by shingo

Keywords: 差し戻し added; 確認中 removed

comment:6 in reply to:  4 Changed 11 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

Replying to shingo:

以下のリビジョンで内容を少し詳しくしました。

どういう機能なのか・update(upgrade)する必要があるのかどうかが不鮮明な気がします

updateする必要があるかどうかについて、確かにupdateしなくても動作に支障はありませんが、update scriptはアップデート時に必ず実行するものと考えます。アップグレード時については説明が無かったのでドキュメントの方に追加してもらう形にしました。

comment:ticket:4048:7 > 今回はアップグレードスクリプトがあるので、それについての説明を追加していただきたいです。

comment:7 Changed 11 years ago by shingo

  • 文章がわかりづらく感じた為、参考としての例を考えてみました(元の文章のほうが良いと感じるなら修正の必要はありません)
     	6	  各メンバーがアップロードできる画像の容量を管理者側で制限するアップロード制限機能にて、 
     	7	  既にアップロードされている画像の容量を画像アップロード容量制限に反映させるスクリプトです。
    
    【例】
    2.13.8で追加された「画像アップロードサイズ制限機能」へ対応するために
    アップロード済み画像のサイズデータを取得するスクリプトです
    

comment:8 Changed 11 years ago by shingo

Keywords: 差し戻し added; 確認中 removed

comment:9 in reply to:  7 Changed 11 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

Replying to shingo: 以下のリビジョンでアップグレードスクリプトの説明を変更しました。

comment:10 Changed 11 years ago by shingo

細かいんですが「upgrade」での説明でも「2.13.8で追加された」との記述でもいいのでしょうか?(念のための確認です)

comment:11 Changed 11 years ago by shingo

Keywords: 差し戻し added; 確認中 removed

comment:12 in reply to:  10 Changed 11 years ago by kiwa

Replying to shingo:

細かいんですが「upgrade」での説明でも「2.13.8で追加された」との記述でもいいのでしょうか?(念のための確認です)

upgrade では不要ですね。とりけします。

comment:13 Changed 11 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンでアップグレードスクリプトのREADMEに入っていた「2.13.8」を「2.14」に変更しました。

comment:14 Changed 11 years ago by shingo

Keywords: テスト待ち added; 確認中 removed

確認しました

comment:15 Changed 11 years ago by imamura623

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.