diff --git a/adapter/ohos/capability/udmf/udmf_impl.cpp b/adapter/ohos/capability/udmf/udmf_impl.cpp index 4ef1eab6f94c81cc0cc066499662c5bbc3b5cbe6..e826cdb4941527275c26c0ba98a643854aeb36cc 100644 --- a/adapter/ohos/capability/udmf/udmf_impl.cpp +++ b/adapter/ohos/capability/udmf/udmf_impl.cpp @@ -258,12 +258,6 @@ void UnifiedDataImpl::SetUnifiedData(std::shared_ptr unifiedD unifiedData_ = unifiedData; } -uint32_t DataLoadParamsImpl::GetRecordCount() -{ - CHECK_NULL_RETURN(dataLoadParams_, 0); - return dataLoadParams_->dataLoadInfo.recordCount; -} - std::shared_ptr DataLoadParamsImpl::GetDataLoadParams() const { return dataLoadParams_; diff --git a/adapter/ohos/capability/udmf/udmf_impl.h b/adapter/ohos/capability/udmf/udmf_impl.h index 9b9f9375c09822ed264ec306ea9ba882f5b2d012..935eb884df555654a1486a04244136e62440afdd 100644 --- a/adapter/ohos/capability/udmf/udmf_impl.h +++ b/adapter/ohos/capability/udmf/udmf_impl.h @@ -105,7 +105,6 @@ public: DataLoadParamsImpl() = default; ~DataLoadParamsImpl() = default; - uint32_t GetRecordCount() override; std::shared_ptr GetDataLoadParams() const; void SetDataLoadParams(const std::shared_ptr& dataLoadParams); diff --git a/frameworks/core/common/udmf/data_load_params.h b/frameworks/core/common/udmf/data_load_params.h index 2f9704a5a2131a109121051bc36632f0dda310f5..3c02bd9c0882ee9c8d920ca1c2ce91544604c3b2 100644 --- a/frameworks/core/common/udmf/data_load_params.h +++ b/frameworks/core/common/udmf/data_load_params.h @@ -25,7 +25,6 @@ class DataLoadParams : public AceType { public: DataLoadParams() = default; ~DataLoadParams() = default; - virtual uint32_t GetRecordCount() = 0; }; } // namespace OHOS::Ace #endif diff --git a/frameworks/core/components_ng/event/gesture_event_hub_drag.cpp b/frameworks/core/components_ng/event/gesture_event_hub_drag.cpp index 7f59ad63d168f43f8b7ec5dd195d98783468aff0..f7c712ee00f1ff466299caa39868477e5e9617d4 100644 --- a/frameworks/core/components_ng/event/gesture_event_hub_drag.cpp +++ b/frameworks/core/components_ng/event/gesture_event_hub_drag.cpp @@ -1689,9 +1689,6 @@ int32_t GestureEventHub::GetBadgeNumber(const RefPtr& drag } else if (unifiedData && !isUseDataLoadParams) { auto recordSize = unifiedData->GetSize(); badgeNumber = recordSize > 1 ? recordSize : 1; - } else if (dataLoadParams && isUseDataLoadParams) { - auto recodeCount = dataLoadParams->GetRecordCount(); - badgeNumber = (recodeCount == 0 || recodeCount > INT32_MAX) ? 1 : static_cast(recodeCount); } auto dragPreviewOptions = frameNode->GetDragPreviewOption(); diff --git a/frameworks/core/components_ng/manager/drag_drop/drag_drop_func_wrapper.cpp b/frameworks/core/components_ng/manager/drag_drop/drag_drop_func_wrapper.cpp index 9ee41bf0b40964e07c21d38855a6bb03746d7bcc..951b55a74d4547c6bdb9b814aa95be06c710befc 100644 --- a/frameworks/core/components_ng/manager/drag_drop/drag_drop_func_wrapper.cpp +++ b/frameworks/core/components_ng/manager/drag_drop/drag_drop_func_wrapper.cpp @@ -196,8 +196,6 @@ void DragDropFuncWrapper::EnvelopedData(std::shared_ptrdataLoadParams->GetRecordCount(); - dataSize = (recodeCount == 0 || recodeCount > INT32_MAX) ? 1 : static_cast(recodeCount); } if (ret == 0) { ret = UdmfClient::GetInstance()->GetSummary(udKey, summary, detailedSummary); diff --git a/interfaces/napi/kits/drag_controller/js_drag_controller.cpp b/interfaces/napi/kits/drag_controller/js_drag_controller.cpp index 096ccb6f65e6c3e55bd6ddffb7ecb380d1aff5de..eadf13b2fce8acb0452233fabaf71e365a5e37f8 100644 --- a/interfaces/napi/kits/drag_controller/js_drag_controller.cpp +++ b/interfaces/napi/kits/drag_controller/js_drag_controller.cpp @@ -799,8 +799,6 @@ int32_t SetUnifiedData(std::shared_ptr asyncCtx, std::st if (ret != 0) { TAG_LOGI(AceLogTag::ACE_DRAG, "udmf set delayInfo failed, return value is %{public}d", ret); } - auto recodeCount = asyncCtx->dataLoadParams->GetRecordCount(); - dataSize = (recodeCount == 0 || recodeCount > INT32_MAX) ? 1 : static_cast(recodeCount); } if (asyncCtx->unifiedData) { ret = UdmfClient::GetInstance()->SetData(asyncCtx->unifiedData, udKey); diff --git a/test/mock/core/common/mock_udmf.h b/test/mock/core/common/mock_udmf.h index 4d12c122441ad57cb6b76ba71694e690d64067a3..cc70a74397ffbfc9331c4f214ec3688bcb3da6d1 100644 --- a/test/mock/core/common/mock_udmf.h +++ b/test/mock/core/common/mock_udmf.h @@ -102,7 +102,6 @@ class MockDataLoadParams : public DataLoadParams { public: MockDataLoadParams() {}; ~MockDataLoadParams() = default; - MOCK_METHOD(uint32_t, GetRecordCount, (), (override)); }; } // namespace OHOS::Ace #endif // FOUNDATION_ACE_TEST_MOCK_CORE_COMMON_MOCK_UDMF_H diff --git a/test/unittest/core/event/gesture_event_hub_test_three_ng.cpp b/test/unittest/core/event/gesture_event_hub_test_three_ng.cpp index 696d16aff29ab7a963bf2c7d265ce1fc82e12521..cdd3a2e8749326c86defbe32d1dc68bd6616bf66 100644 --- a/test/unittest/core/event/gesture_event_hub_test_three_ng.cpp +++ b/test/unittest/core/event/gesture_event_hub_test_three_ng.cpp @@ -883,26 +883,6 @@ HWTEST_F(GestureEventHubTestNg, GetBadgeNumber_001, TestSize.Level1) dragEvent->SetUseDataLoadParams(false); auto ret1 = guestureEventHub->GetBadgeNumber(dragEvent); EXPECT_EQ(ret1, 5); - - dragEvent->SetUseDataLoadParams(true); - RefPtr mockDataLoadParams = AceType::MakeRefPtr(); - ASSERT_NE(mockDataLoadParams, nullptr); - dragEvent->SetDataLoadParams(mockDataLoadParams); - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(-1)); - auto ret2 = guestureEventHub->GetBadgeNumber(dragEvent); - EXPECT_EQ(ret2, 1); - - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(0)); - auto ret3 = guestureEventHub->GetBadgeNumber(dragEvent); - EXPECT_EQ(ret3, 1); - - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(INT32_MAX + 1)); - auto ret4 = guestureEventHub->GetBadgeNumber(dragEvent); - EXPECT_EQ(ret4, 1); - - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(10)); - auto ret5 = guestureEventHub->GetBadgeNumber(dragEvent); - EXPECT_EQ(ret5, 10); } /** diff --git a/test/unittest/core/manager/drag_drop/drag_drop_func_wrapper_test_ng_coverage.cpp b/test/unittest/core/manager/drag_drop/drag_drop_func_wrapper_test_ng_coverage.cpp index dc1f656b4358d3bba6a0834cbbb534d858027a6f..50ce97aec37ef0c72450b3b91ceb3e4a86de431e 100644 --- a/test/unittest/core/manager/drag_drop/drag_drop_func_wrapper_test_ng_coverage.cpp +++ b/test/unittest/core/manager/drag_drop/drag_drop_func_wrapper_test_ng_coverage.cpp @@ -1507,23 +1507,5 @@ HWTEST_F(DragDropFuncWrapperTestNgCoverage, DragDropFuncWrapperTestNgCoverage042 EXPECT_CALL(*mockUdmfClient, SetDelayInfo(_, _)).WillRepeatedly(testing::Return(1)); DragDropFuncWrapper::EnvelopedData(dragAction, udKey, summary, detailedSummary, dataSize); EXPECT_EQ(dataSize, 1); - - EXPECT_CALL(*mockUdmfClient, SetDelayInfo(_, _)).WillRepeatedly(testing::Return(0)); - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(10)); - EXPECT_CALL(*mockUdmfClient, GetSummary(_, _, _)).WillRepeatedly(testing::Return(1)); - DragDropFuncWrapper::EnvelopedData(dragAction, udKey, summary, detailedSummary, dataSize); - EXPECT_EQ(dataSize, 10); - - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(-1)); - DragDropFuncWrapper::EnvelopedData(dragAction, udKey, summary, detailedSummary, dataSize); - EXPECT_EQ(dataSize, 1); - - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(0)); - DragDropFuncWrapper::EnvelopedData(dragAction, udKey, summary, detailedSummary, dataSize); - EXPECT_EQ(dataSize, 1); - - EXPECT_CALL(*mockDataLoadParams, GetRecordCount()).WillRepeatedly(testing::Return(INT32_MAX + 1)); - DragDropFuncWrapper::EnvelopedData(dragAction, udKey, summary, detailedSummary, dataSize); - EXPECT_EQ(dataSize, 1); } } // namespace OHOS::Ace::NG