vaadin/framework

Deprecate MultiSelectionEvent.getNewSelection

開放

#11,756 建立於 2019年10月23日

 (1 則留言) (1 個反應) (0 位負責人)Java (715 個分叉)auto 404
Help wantedStale

倉庫指標

星標
 (1,806 顆星)
PR 合併指標
 (PR 指標待抓取)

描述

Description copied from https://github.com/vaadin/flow/issues/6756

The class has three methods for the same thing:

Set<T> getValue()
Set<T> getAllSelectedItems()
Set<T> getNewSelection()

The first two are inherited, but getNewSelection exists only in MultiSelectionEvent.

Another reason to deprecate it is that the naming can cause confusion. We got feedback that a user expected the method to return the newly added items (compared to the old selection), i.e. what getAddedSelection returns.

JavaDocs should suggest to use getAllSelectedItems or getValue instead.

貢獻者指南