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

Opened 12 years ago

Closed 12 years ago

#707 closed defect (fixed)

BIZ:スケジュール登録をグループ単位から人単位に直す

Reported by: kunitada Owned by: ebihara
Priority: critical Milestone: OpenPNE2.8beta7
Component: core【OpenPNEコア部分】 Version:
Keywords: Cc:

Description

bizスケジュール登録をグループ単位から人単位に直す。
グループ→人のコンバータも作る。

■コミュニティで議論した場所は下記。
http://openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=1810

Attachments (1)

2.6→2.8BIZコンバータ仕様.xls (16.0 KB) - added by ogawa 12 years ago.
コンバータの仕様作りました。

Download all attachments as: .zip

Change History (20)

comment:1 Changed 12 years ago by ogawa

Milestone: OpenPNE2.8beta7
Owner: changed from ogawa to nobody
Priority: blockercritical

comment:2 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:3 Changed 12 years ago by ebihara

以下について調査します。

  1. 2.4→2.6で、参加者がグループ単位での指定になった以外に、スケジュール関連でなにか大きな変更等はなかったか
    • 大きな変更がなければ、だいぶ戻すのが楽になるので……
  2. グループ->人コンバータを作成するだけで本当に対応ができるのかどうか
    • すでに2.6でコンバータを実行した予定、2.6から登録された予定、2.4から変更されていない予定……など、様々なパターンが考えられる
    • ソースコード面での対応も必要かどうか
    • 考えられる予定のパターン表を作成
  3. 2.4にスケジュールの方式を戻すことで、2.6以降に実装された新機能への対応が別途必要であるかどうか
    • 1.と同じ理由ですね

また、以上で誤りや不足等あればご指摘ください。

comment:4 Changed 12 years ago by ebihara

國忠さんから以下のようなアドバイスをいただきました。この方向で進めていきます。

  • 基本的にはこのコードの修正を取り込む http://trac.openpne.jp/log/OpenPNE/branches/kunitada/prj_2_6_X_biza
  • コンバータの対象は、2.4→2.6でコンバータをかけた予定、もしくは2.6から新規登録された予定のみ
    • 2.4でコンバータをかけなかった予定など、2.6での予定の仕様にそぐわないものは対象としない

comment:5 Changed 12 years ago by ebihara

國忠さんから教えていただいたコードの修正をそのまま取り込むと、

  • スケジュールの公開範囲設定など、スケジュール参加者単位仕様変更前後で実装された2.6の新機能の一部がデグレードしてしまう
  • 2.6~2.8でフィックスされたバグが再発する危険がある

という懸念点が生じます。

あくまでも今回は「bizスケジュール登録をグループ単位から人単位に直す」ですので、それ以外のデグレードや仕様変更は避けたいと思うのですが……。

とりあえず取り込みます。まずは中間報告ということで。

comment:6 Changed 12 years ago by ebihara

r2816で、 http://trac.openpne.jp/log/OpenPNE/branches/kunitada/prj_2_6_X_biza この修正を取り込みました。

懸念していたデグレードについてですが、発生したのはスケジュールの公開範囲設定のみのようです。テンプレート側でコメントアウトされていますので、コメントを外せば使えるようになるとは思いますが、今回の参加者単位の変更により、仕様を練り直す必要があるかもしれません。

スケジュールの公開範囲設定の件と、動作確認等については明日夕方頃に継続して行います。

Changed 12 years ago by ogawa

コンバータの仕様作りました。

comment:7 Changed 12 years ago by ebihara

スケジュールの公開範囲設定は以下のようにします。

  • 公開範囲を「全体まで公開」「参加者のみに公開」のふたつにする
    • 「グループまで公開」は削除
    • 既存の「グループのみ公開」予定については、コンバータで対処(グループメンバーをそのままスケジュールメンバーにし、「参加者のみに公開」に。詳細はコンバータの仕様を参照)
    • 「公開しない」→「参加者のみに公開」文言修正
  • グループ選択用プルダウンボックスは削除
  • 公開範囲設定は参加者の下に
  • 現状hiddenで仕込んでいる公開範囲設定は削除

また、以上の作業が終わり次第、

  1. 公開範囲設定が適切に動作しているかどうかのチェック
  2. 個人ブランチにコミット
  3. リリースブランチにコミット

の手順を踏みます。

comment:8 Changed 12 years ago by ebihara

PC版については、r2827で個人ブランチにコミットしました。携帯版についてはこれから着手します。

comment:9 Changed 12 years ago by ogawa

コンバータ作成は別チケットで扱い、RCへ回します。

  • #714 BIZスケジュールの2.6→2.8コンバータ作成

comment:10 Changed 12 years ago by ebihara

携帯版についての対応を、r2832で個人用ブランチにコミットしました。

これからマージし、問題がなければリリースブランチにコミットします。

comment:11 Changed 12 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

r2835でリリースブランチにコミットしました。ご確認ください。

comment:12 Changed 12 years ago by ogawa

Owner: changed from ogawa to ebihara

comment:13 Changed 12 years ago by ogawa

Resolution: fixed
Status: newclosed

テスト待ちで close します。問題あったら reopen してください。

comment:14 Changed 12 years ago by kiwa

Resolution: fixed
Status: closedreopened

http://openpne-28x.sosk.pne.jp/ で確認したところ、正しく動作しませんでした

  1. 予定登録(fh_biz_schedule_add)画面を表示→参加者が表示されていない
  2. 参加者が指定できなかったのでそのまま普通に登録
  3. 登録後に表示される予定詳細画面(fh_biz_schedule_view)が文字化けしている

comment:15 Changed 12 years ago by ogawa

r2835 でUTF-8ではない文字列が入った状態でコミットされているように見えます。確認してください。

comment:16 Changed 12 years ago by ogawa

Summary: bizスケジュール登録をグループ単位から人単位に直すBIZ:スケジュール登録をグループ単位から人単位に直す

comment:17 Changed 12 years ago by ebihara

申し訳ありません。r2835でマージミスしていました。r2880で修正してコミットしましたのでご確認ください(文字化けの件も解消しました)。

comment:18 Changed 12 years ago by ebihara

r2882でつぶしてなかった文字化けの修正をしました。

comment:19 Changed 12 years ago by ogawa

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.