PR

【3/27】ゲソてんプロモーション4月 #4579

wada opened this at 2026-03-24T10:45:28; closed

Description

opened by wada at 2026-03-24T10:45:28; updated 2026-03-30T10:10:09

概要

仕様書

3step
リワードガチャ
vポイント

更新内容

  • vポイントを新しく追加
  • 加算条件をborder_typeに変更
  • 3月はquest_id、4月はborder_typeを参照するように更新

quest_idとborder_type:11が被るため2回加算される

DB更新内容

【3/27】ゲソてんプロモーション4月 jolly/jrye-db#1369

テスト方法、確認した内容

各条件の加算(3月、4月)

wada add_label at 2026-03-24T13:41:53: レビュー待ち

Comment

#44911 by wada at 2026-03-24T13:42:00; updated 2026-03-24T13:42:00

@kouho
レビューお願いします。

Comment

#44924 by kouho at 2026-03-24T14:27:25; updated 2026-03-24T14:36:26

@wada
今回分の更新ではないですが、user_gesoten_questのapi_send_flgは更新されますが、log_gesoten_questのsend_apiに0が固定で入ってしまっていました。
Gesoten_campaign_repository.php 164行目
$user_quest['send_api']の部分は$user_quest['send_api_flg'] が正と思われますので確認をお願いいたします。

kouho add_label at 2026-03-24T14:27:28: コメントあり

Comment

#44940 by wada at 2026-03-25T10:33:28; updated 2026-03-25T10:33:51

@kouho
重複対応しました。

> user_gesoten_questのapi_send_flgは更新されますが、log_gesoten_questのsend_apiに0が固定で入ってしまっていました。
> CP内で条件が重複すると1クエストしか加算されないため

kouho add_label at 2026-03-25T16:26:39: 皇甫 LGTM

Comment

#44961 by kouho at 2026-03-25T16:27:08; updated 2026-03-25T16:27:08

@wada
対応ありがとうございます。確認OKです。
> 重複対応しました。

kouho delete_label at 2026-03-27T08:57:49: コメントあり
wada merge at 2026-03-30T10:10:05: 【3/27】ゲソてんプロモーション4月
wada close at 2026-03-30T10:10:05: Close
wada delete_branch at 2026-03-30T10:10:09: vpoint_gesoten_cp_2604

commit comment

#345 by kouho at 2026-03-24T15:02:16; lib/models/Gesoten_campaign.php / old:118 new:None
e9480c053120873f926af6127eea6d3f1774aea7

@wada
コード内TODOだけでなく、仕様書にもこの制約を明記しておきたいです。
CP内で条件が重複すると1クエストしか加算されないため、将来の仕様変更時にも見落とさないよう、分かる形で残しておいてほしいです。

Commit range

base10606798bc6d472c49faa7938d300ed0ef978c9f
heade9480c053120873f926af6127eea6d3f1774aea7
base branchjolly/jr-web:master
head branchjolly/jr-web:vpoint_gesoten_cp_2604

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed4 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/class/const/QuestConst.class.php
modifiedlib/models/Gesoten_campaign.php
modifiedlib/models/dao/Gesoten_campaign_repository.php
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#345 by kouho at 2026-03-24T15:02:16; lib/models/Gesoten_campaign.php / old:118 new:None
e9480c053120873f926af6127eea6d3f1774aea7

@wada
コード内TODOだけでなく、仕様書にもこの制約を明記しておきたいです。
CP内で条件が重複すると1クエストしか加算されないため、将来の仕様変更時にも見落とさないよう、分かる形で残しておいてほしいです。