[xwiki/xwiki-platform] 75fab2: XWIKI-24864: Active Installs searchInstalls() retu...
Branch: refs/heads/XWIKI-24864 Home: https://github.com/xwiki/xwiki-platform Commit: 75fab2d9677dcd000874f09ba65876f8cc0d4df3 https://github.com/xwiki/xwiki-platform/commit/75fab2d9677dcd000874f09ba6587... Author: Vincent Massol <[email protected]> Date: 2026-09-11 (Fri, 11 Sep 2026) Changed paths: M xwiki-platform-core/xwiki-platform-activeinstalls2/xwiki-platform-activeinstalls2-api/src/main/java/org/xwiki/activeinstalls2/DataManager.java M xwiki-platform-core/xwiki-platform-activeinstalls2/xwiki-platform-activeinstalls2-api/src/main/java/org/xwiki/activeinstalls2/internal/DefaultDataManager.java M xwiki-platform-core/xwiki-platform-activeinstalls2/xwiki-platform-activeinstalls2-api/src/main/java/org/xwiki/activeinstalls2/script/ActiveInstallsScriptService.java Log Message: ----------- XWIKI-24864: Active Installs searchInstalls() returns at most 10 pings while its javadoc promises all of them * Document the bound instead of promising every matching ping: the javadoc of DataManager, DefaultDataManager and ActiveInstallsScriptService now states that only some of the matching pings come back, that which ones is undefined since they are not sorted, and why the bound exists * Point callers asking "how many" at countInstalls/countDistinctInstalls, which aggregate server-side rather than retrieving pings * Move the "empty json matches all the data" sentence from @return to @param, which is what it describes Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications
participants (1)
-
XWiki Notifications