From f595e7247b919393822537ce4ebf1ebb994b0cda Mon Sep 17 00:00:00 2001 From: Reinhard Pointner Date: Tue, 21 Feb 2017 16:43:55 +0800 Subject: [PATCH] Update feelingLucky --- source/net/filebot/ui/SupportDialog.java | 30 ++++++++++++++++++------ 1 file changed, 23 insertions(+), 7 deletions(-) diff --git a/source/net/filebot/ui/SupportDialog.java b/source/net/filebot/ui/SupportDialog.java index f0b2535d..56c2055d 100644 --- a/source/net/filebot/ui/SupportDialog.java +++ b/source/net/filebot/ui/SupportDialog.java @@ -86,18 +86,34 @@ public enum SupportDialog { }; public boolean feelingLucky(int sessionRenameCount, int totalRenameCount, int currentRevision, int lastSupportRevision, int supportRevisionCount) { - // ask for reviews only once per revision - if (currentRevision <= lastSupportRevision && isAppStore()) { - return false; + if (isAppStore()) { + // ask for reviews at most once per revision + if (currentRevision <= lastSupportRevision) { + return false; + } + + // ask for reviews only when a significant number of files have been processed + if (sessionRenameCount <= 10 || totalRenameCount <= 5000) { + return false; + } + + // ask for reviews at most every once in a while + if (Math.random() <= 0.777) { + return false; + } } - // always lucky if many files are processed in a single session - if (sessionRenameCount >= 5000 * Math.pow(2, supportRevisionCount)) { + // lucky if many files are processed in a single session + if (sessionRenameCount >= 2000 * Math.pow(2, supportRevisionCount)) { return true; } - // sometimes lucky if many files have been processed over time - return totalRenameCount >= 2000 * Math.pow(4, supportRevisionCount) && Math.random() >= 0.777; + // lucky if many many files have been processed over time + if (totalRenameCount >= 2000 * Math.pow(4, supportRevisionCount)) { + return true; + } + + return false; } public boolean show(int totalRenameCount, boolean first) {