mirror of
https://github.com/openharmony/notification_distributed_notification_service.git
synced 2026-08-24 07:03:05 -04:00
a175392373
Group the boolean behavior flags previously scattered across function signatures into the... Created-by: let_it_rot Commit-by: let_it_rot Merged-by: openharmony_ci Description: ### 一、内容说明(相关的Issue) Group the boolean behavior flags previously scattered across function signatures into the ShowLimit struct, and use a new isUpdateDeliveryTime flag to control whether the notification delivery time is refreshed when showing a reminder. - Add isPlaySound/isSysTimeChanged/isCloseDefaultSound/isSlienceNotification fields to ShowLimit in reminder_data_manager.h - Add isUpdateDeliveryTime field (default true) to ShowLimit - ShowReminder: signature changes to (reminder, const ShowLimit& limits); pass limits.isUpdateDeliveryTime through to ReminderRequest::UpdateNotificationRequest - HandleImmediatelyShow: signature changes to (vector, ShowLimit&, size_t index = 0); OnShow and the recursive ffrt callback now read isSysTimeChanged/isSlienceNotification from limits - Update all call sites (OnLanguageChanged, LoadShareReminders, UpdateShareReminders, OnDataShareInsertOrDelete, InitShareReminders, boot complete, system time change, ShowActiveReminder) to populate the relevant flags on a local ShowLimit before invoking - isUpdateDeliveryTime is true only for normal timer trigger (ShowActiveReminder) and system time change trigger; false for all other cases (boot, init, data share, language change) - Update unit test and both fuzz harnesses to the new signatures  ### 二、建议测试周期和提测地址 建议测试完成时间:xxxx.xx.xx 投产上线时间:xxxx.xx.xx 提测地址:CI环境/压测环境 测试账号: ### 三、变更内容 * 3.1 关联PR列表 * 3.2 数据库和部署说明 1. 常规更新 2. 重启unicorn 3. 重启sidekiq 4. 迁移任务:是否有迁移任务,没有写 "无" 5. rake脚本:`bundle exec xxx RAILS_ENV = production`;没有写 "无" * 3.4 其他技术优化内容(做了什么,变更了什么) - 重构了 xxxx 代码 - xxxx 算法优化 * 3.5 废弃通知(什么字段、方法弃用?) * 3.6 后向不兼容变更(是否有无法向后兼容的变更?) ### 四、研发自测点(自测哪些?冒烟用例全部自测?) 自测测试结论: ### 五、测试关注点(需要提醒QA重点关注的、可能会忽略的地方) 检查点: | 需求名称 | 是否影响xx公共模块 | 是否需要xx功能 | 需求升级是否依赖其他子产品 | |------|------------|----------|---------------| | xxx | 否 | 需要 | 不需要 | | | | | | 接口测试: 性能测试: 并发测试: 其他: See merge request: openharmony/notification_distributed_notification_service!4975