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

Opened 12 years ago

Last modified 10 years ago

#1483 new enhancement

BIZ:スケジュール登録の「繰り返し」の基準が一定でない

Reported by: kiwa Owned by: nobody
Priority: minor Milestone:
Component: pne-biz Version: 2.12.x & 2.14.x
Keywords: Cc:

Description (last modified by kiwa)

■現象

スケジュール登録・編集時の繰り返しの基準が状況によって変化してしまう。(詳細は関連情報を参照)

■原因

■修正内容

■関連情報

さて本題ですが、BIZ以下の条件でスケジュールで条件と色々変えて登録してみました。

(1)
繰り返し設定:繰り返す(1週間)
開始日:本日(11月2日)
繰り返し曜日:日~土すべて
結果->登録完了画面で違うスケジュールが表示される。カレンダーには表示されない。初めての登録の場合はメンテナンス中画面になる。

(2)
繰り返し設定:繰り返す(1週間)
開始日:本日(11月2日)
繰り返し曜日:登録した金曜以外すべて
結果->11月3日(土)から11月8日(木)まで登録される。

(3)
繰り返し設定:繰り返す(2週間)
開始日:本日(11月2日)
繰り返し曜日:日~土すべて
結果->11月2日(金)から11月9日(金)まで登録される。

(4)
繰り返し設定:繰り返す(2週間)
開始日:本日(11月2日)
繰り返し曜日:登録した金曜以外すべて
結果->11月3日(土)から11月15日(木)まで登録される。(11月9日は除く)

(5)
繰り返し設定:繰り返す(2週間)
開始日:11月3日
繰り返し曜日:日~土すべて
結果->11月3日(土)から11月10日(土)まで登録される。

(6)
繰り返し設定:繰り返す(2週間)
開始日:11月3日
繰り返し曜日:登録した金曜以外すべて
結果->11月3日(土)から11月10日(土)まで登録される。(11月9日は除く)

週の解釈を (a)7日間 と考えるか、(b)月曜日から始まる週 と考えるかで変わるでしょう。
(a)と考えると(1)は11月8日まで、(3)は11月15日まで、(5)は11月16日まで、(6)は11月15日までになるように思います。
(b)と考えると(1)、(2)は11月3日まで、(3)・(4)は11月10日までになるように思います。
ほかにも週の解釈があるかもしれませんが、(a)が自然かなと思い、以下のように変更して期待通りの結果になりました。

[変更]
webapp_biz\modules\biz\do\fhg_biz_schedule_add.php 150行目あたり
webapp_biz\modules\biz\do\h_biz_schedule_edit.php 103行目あたり

(前)$finish_date = date("Y-m-d", strtotime($begin_date . ' + ' . ($requests['sc_rcount'] - 1) * 7 . 'days'));
(後)$finish_date = date("Y-m-d", strtotime($begin_date . ' + ' . (($requests['sc_rcount']) * 7-1) . 'days'));

[結果]
(1)11/2-11/8
(2)11/3-11/8
(3)11/2-11/15
(4)11/3-11/15(11/9を除く)
(5)11/3-11/16
(6)11/3-11/15(11/9を除く)

終了日を選択できる方が、ややこしくないのかなあ。

Change History (25)

comment:1 Changed 12 years ago by kiwa

Priority: minormajor

comment:2 Changed 12 years ago by kiwa

コメントに追加情報をいただきました

すみません、報告漏れです。

上の変更だけでは#713が復活してしまいます。
また、携帯版BIZでは「繰り返し予定」が1週間マイナス表示されます。

以下の箇所を変更しました。

(1)webapp_biz\modules\biz\page\fh_biz_schedule_edit.php 195行目
(2)webapp_biz\modules\biz\page\fh_biz_schedule_view.php 96行目

(前)$daycount = $repeat_term / (24 * 60 * 60) / 6;
(後)$daycount = ceil((($repeat_term / (24 * 60 * 60))+1) / 7);

また、その数行後
(前)$this->set('repeat_term', ceil($daycount));
(後)$this->set('repeat_term', intval($daycount));

(3)webapp_biz\modules\ktai_biz\page\fh_biz_schedule_view.php  99行目
(前)$daycount = $repeat_term / (24 * 60 * 60) / 7;
(後)$daycount = ceil((($repeat_term / (24 * 60 * 60))+1) / 7);

comment:3 Changed 12 years ago by imoto

Keywords: 再現済み added; 再現待ち removed

再現済み

■再現手順[[BR]]
下記の項目についてスケジュールを入力[[BR]]

(1)
繰り返し設定:繰り返す(1週間)
開始日:本日(11月7日)
繰り返し曜日:日~土すべて
結果->カレンダーには表示されない

(2)
繰り返し設定:繰り返す(1週間)
開始日:本日(11月7日)
繰り返し曜日:登録した水曜以外すべて
結果->11月8日(木)から11月13日(火)まで登録される。

(3)
繰り返し設定:繰り返す(2週間)
開始日:本日(11月7日)
繰り返し曜日:日~土すべて
結果->11月7日(水)から11月15日(水)まで登録される。

(4)
繰り返し設定:繰り返す(2週間)
開始日:本日(11月7日)
繰り返し曜日:登録した水曜以外すべて
結果->11月8日(木)から11月20日(火)まで登録される。(11月14日は除く)

(5)
繰り返し設定:繰り返す(2週間)
開始日:11月8日
繰り返し曜日:日~土すべて
結果->11月8日(木)から11月15日(木)まで登録される。

(6)
繰り返し設定:繰り返す(2週間)
開始日:11月8日
繰り返し曜日:登録した金曜以外すべて
結果->11月8日(木)から11月15日(木)まで登録される。(11月14日は除く)

comment:4 Changed 12 years ago by kiwa

Description: modified (diff)
Milestone: OpenPNE2.10.2
Summary: BIZ:スケジュール登録の「繰り返し」についてBIZ:スケジュール登録の「繰り返し」の基準が一定でない

comment:5 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.7 added

comment:6 Changed 12 years ago by ogawa

Keywords: OpenPNE2.8.7 removed
Milestone: OpenPNE2.10.2
Priority: majorminor

comment:7 Changed 12 years ago by ebihara

Milestone: OpenPNE2.11.3

開発版の修正項目にします

comment:8 Changed 12 years ago by ogawa

Milestone: OpenPNE2.11.3

comment:9 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 added; 再現済み removed
Milestone: OpenPNE2.10.5

comment:10 Changed 12 years ago by kiwa

Priority: minormajor

comment:11 Changed 12 years ago by ebihara

Priority: majorcritical

修正内容を明確にしていただいているので、是非対応したいところです。

プライオリティを上げます。

comment:12 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

対応します。

comment:13 Changed 11 years ago by ebihara

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.5OpenPNE2.11.5

正常動作するかどうかの検証が困難なので、安定版では取り込まずに、 comment:7 のとおり、開発版の対応項目とします。

comment:14 Changed 11 years ago by ebihara

Owner: changed from ebihara to nobody
Status: assignednew

comment:15 Changed 11 years ago by ebihara

Milestone: OpenPNE2.11.5OpenPNE2.11.6

comment:16 Changed 11 years ago by ogawa

Priority: criticalmajor

comment:17 Changed 11 years ago by imamura623

Milestone: OpenPNE2.11.6OpenPNE2.11.7

2.11.6では見送ります。

comment:18 Changed 11 years ago by kiwa

Milestone: OpenPNE2.11.7

2.11.7では見送ります。

comment:19 Changed 11 years ago by ogawa

Version: 2.8.x & 2.10.x & 2.11.x2.10.x & 2.12.x & 2.13.x

comment:20 Changed 11 years ago by ebihara

Version: 2.10.x & 2.12.x & 2.13.x2.13.x

comment:21 Changed 11 years ago by ebihara

Priority: majorminor

comment:22 Changed 11 years ago by ebihara

Type: defectenhancement
Version: 2.13.x2.10.x & 2.12.x & 2.13.x

安定版でも発生するが開発版でのみ対応する項目ということで、 version と type を変更します。

comment:23 Changed 11 years ago by ebihara

Version: 2.10.x & 2.12.x & 2.13.x2.10.x & 2.12.x

comment:24 Changed 10 years ago by kiwa

Version: 2.10.x & 2.12.x2.10.x & 2.12.x & 2.14.x

comment:25 Changed 10 years ago by kiwa

Version: 2.10.x & 2.12.x & 2.14.x2.12.x & 2.14.x
Note: See TracTickets for help on using tickets.