fix: bluetooth still entered notificationcenter

[18202781743]

Filting bluetooth notification in DataAccessorProxy.

pms: BUG-311617

Summary by Sourcery

Improve handling of Bluetooth file transfer notifications to prevent them from appearing in the notification center

Bug Fixes:

• Fix Bluetooth file transfer notifications incorrectly entering the notification center by adding a filter mechanism

Enhancements:

• Introduce a new filterToSource method in DataAccessorProxy to detect and filter out Bluetooth file transfer notifications

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request filters Bluetooth notifications related to file transfers, preventing them from being displayed in the notification center. This is achieved by adding a filter in DataAccessorProxy and removing the redundant filter in NotificationManager.

No diagrams generated as the changes look simple and do not need a visual representation.

File-Level Changes

Change	Details	Files
Implemented a filter to prevent Bluetooth notifications related to file transfers from being displayed in the notification center.	Added a filterToSource method to check if a notification should be filtered based on its content and actions. Modified addEntity and updateEntityProcessedType to use the new filter. Removed bluetooth filter logic from NotificationManager::updateEntityProcessed.	panels/notification/common/dataaccessorproxy.cpp panels/notification/server/notificationmanager.cpp panels/notification/common/dataaccessorproxy.h

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!
• Generate a plan of action for an issue: Comment @sourcery-ai plan on
  an issue to generate a plan of action for it.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey @18202781743 - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider adding a unit test for the filterToSource method to ensure it correctly identifies Bluetooth notifications.
• It might be helpful to add a comment explaining why the bluetooth notification is being filtered.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 在DataAccessorProxy::addEntity函数中，filterToSource函数的调用应该放在m_source->addEntity(entity)之前，以确保只有在m_source有效且实体通过过滤的情况下才尝试添加实体。

2. 在DataAccessorProxy::updateEntityProcessedType函数中，filterToSource函数的调用应该放在m_source->addEntity(entity)之前，以确保只有在实体通过过滤的情况下才尝试添加实体。

3. 在DataAccessorProxy::updateEntityProcessedType函数中，如果filterToSource返回true，则应该直接返回，而不是继续执行后续代码。

4. 在DataAccessorProxy::filterToSource函数中，bluetooth变量的命名不够直观，建议使用更具描述性的名称，如isSendingBluetoothFile。

5. 在NotificationManager::updateEntityProcessed函数中，注释掉的代码应该被移除，以保持代码的整洁。

6. 在NotificationManager::updateEntityProcessed函数中，bluetooth变量的计算应该放在if (removed || !showInCenter)条件判断之前，以避免不必要的计算。

7. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (removed || !showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

8. 在NotificationManager::updateEntityProcessed函数中，showInCenter变量的计算应该放在if (entity.hints().contains("x-deepin-ShowInNotifyCenter"))条件判断之前，以避免不必要的计算。

9. 在NotificationManager::updateEntityProcessed函数中，removed变量的计算应该放在if (entity.hints().contains("x-deepin-Removed"))条件判断之前，以避免不必要的计算。

10. 在NotificationManager::updateEntityProcessed函数中，if (removed || !showInCenter || bluetooth)条件判断应该被简化为if (!showInCenter)，因为removed和bluetooth已经在之前的条件判断中处理过了。

11. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

12. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

13. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

14. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

15. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

16. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

17. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

18. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

19. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

20. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

21. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

22. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

23. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

24. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

25. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

26. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

27. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

28. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

29. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

30. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

31. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

32. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

33. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

34. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

35. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

36. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

37. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

38. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

39. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

40. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

41. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

42. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

43. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

44. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

45. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

46. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

47. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

48. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

49. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

50. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

51. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

52. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

53. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

54. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

55. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

56. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

57. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

58. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

59. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

60. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

61. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

62. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

63. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

64. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

65. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

66. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

67. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

68. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

69. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

70. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

71. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

72. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

73. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

74. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

75. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

76. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

77. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

78. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

79. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

80. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

81. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

82. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

83. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

84. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

85. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除实体时才执行删除操作。

86. 在NotificationManager::updateEntityProcessed函数中，m_persistence->removeEntity(id);应该放在if (!showInCenter)条件判断之后，以确保只有在需要删除

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, yixinshark

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[18202781743]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)