diff options
author | Nasser Grainawi <nasser.grainawi@linaro.org> | 2024-04-10 10:39:50 -0700 |
---|---|---|
committer | Nasser Grainawi <nasser.grainawi@linaro.org> | 2024-04-10 10:39:50 -0700 |
commit | 2e41adb7b1a4367cf4dd163d1dbfc463a1460779 (patch) | |
tree | 3aea9f1d94ed72ceb6271fa13f2131c7d315b923 | |
parent | 6e1db3f9d25cd68a54727bd73f5e6e0c808b5e60 (diff) | |
parent | 26d58776626d3fc5b818f656ee4f05b2444fe86b (diff) |
Merge branch 'stable-3.8' into stable-3.9
* stable-3.8:
Fix large event_details in dashboard displayed
Avoid work in Optional.orElse() calls
Fix a bug with paper-tooltip.
Fix a bug with paper-tooltip.
Release-Notes: skip
Change-Id: I7c435a872f744314b37f3b90c7c28c98dae57467
14 files changed, 24 insertions, 22 deletions
diff --git a/java/com/google/gerrit/acceptance/GerritServer.java b/java/com/google/gerrit/acceptance/GerritServer.java index 73631e91e8..53c952e76a 100644 --- a/java/com/google/gerrit/acceptance/GerritServer.java +++ b/java/com/google/gerrit/acceptance/GerritServer.java @@ -334,7 +334,7 @@ public class GerritServer implements AutoCloseable { String configuredIndexBackend = cfg.getString("index", null, "type"); if (configuredIndexBackend == null) { // Propagate index type to pgms that run off of the gerrit.config file on local disk. - IndexType indexType = IndexType.fromEnvironment().orElse(new IndexType("fake")); + IndexType indexType = IndexType.fromEnvironment().orElseGet(() -> new IndexType("fake")); gerritConfig.setString("index", null, "type", indexType.isLucene() ? "lucene" : "fake"); } gerritConfig.save(); @@ -513,7 +513,7 @@ public class GerritServer implements AutoCloseable { IndexType indexType = (configuredIndexBackend != null) ? new IndexType(configuredIndexBackend) - : IndexType.fromEnvironment().orElse(new IndexType("fake")); + : IndexType.fromEnvironment().orElseGet(() -> new IndexType("fake")); daemon.setIndexModule(createIndexModule(indexType, baseConfig, testIndexModule)); daemon.setEnableHttpd(desc.httpd()); diff --git a/java/com/google/gerrit/acceptance/testsuite/group/GroupOperationsImpl.java b/java/com/google/gerrit/acceptance/testsuite/group/GroupOperationsImpl.java index 0a22688467..5f8f3e7ecf 100644 --- a/java/com/google/gerrit/acceptance/testsuite/group/GroupOperationsImpl.java +++ b/java/com/google/gerrit/acceptance/testsuite/group/GroupOperationsImpl.java @@ -78,7 +78,7 @@ public class GroupOperationsImpl implements GroupOperations { private InternalGroupCreation toInternalGroupCreation(TestGroupCreation groupCreation) { AccountGroup.Id groupId = AccountGroup.id(seq.nextGroupId()); - String groupName = groupCreation.name().orElse("group-with-id-" + groupId.get()); + String groupName = groupCreation.name().orElseGet(() -> "group-with-id-" + groupId.get()); AccountGroup.UUID groupUuid = GroupUuid.make(groupName, serverIdent); AccountGroup.NameKey nameKey = AccountGroup.nameKey(groupName); return InternalGroupCreation.builder() diff --git a/java/com/google/gerrit/acceptance/testsuite/project/ProjectOperationsImpl.java b/java/com/google/gerrit/acceptance/testsuite/project/ProjectOperationsImpl.java index bd3d65645f..a3112f8f94 100644 --- a/java/com/google/gerrit/acceptance/testsuite/project/ProjectOperationsImpl.java +++ b/java/com/google/gerrit/acceptance/testsuite/project/ProjectOperationsImpl.java @@ -88,7 +88,7 @@ public class ProjectOperationsImpl implements ProjectOperations { } private Project.NameKey createNewProject(TestProjectCreation projectCreation) throws Exception { - String name = projectCreation.name().orElse(RandomStringUtils.randomAlphabetic(8)); + String name = projectCreation.name().orElseGet(() -> RandomStringUtils.randomAlphabetic(8)); CreateProjectArgs args = new CreateProjectArgs(); args.setProjectName(name); diff --git a/java/com/google/gerrit/httpd/raw/StaticModule.java b/java/com/google/gerrit/httpd/raw/StaticModule.java index 587d82ae16..d7909b211f 100644 --- a/java/com/google/gerrit/httpd/raw/StaticModule.java +++ b/java/com/google/gerrit/httpd/raw/StaticModule.java @@ -226,7 +226,7 @@ public class StaticModule extends ServletModule { @GerritServerConfig Config cfg, GerritApi gerritApi, ExperimentFeatures experimentFeatures) { - String cdnPath = options.devCdn().orElse(cfg.getString("gerrit", null, "cdnPath")); + String cdnPath = options.devCdn().orElseGet(() -> cfg.getString("gerrit", null, "cdnPath")); String faviconPath = cfg.getString("gerrit", null, "faviconPath"); return new IndexServlet(canonicalUrl, cdnPath, faviconPath, gerritApi, experimentFeatures); } diff --git a/java/com/google/gerrit/server/DeadlineChecker.java b/java/com/google/gerrit/server/DeadlineChecker.java index f41b1e3c3d..9b7ffe6c3f 100644 --- a/java/com/google/gerrit/server/DeadlineChecker.java +++ b/java/com/google/gerrit/server/DeadlineChecker.java @@ -180,12 +180,14 @@ public class DeadlineChecker implements RequestStateProvider { this.timeoutName = clientedProvidedTimeout .map(clientTimeout -> "client.timeout") - .orElse( - serverSideDeadline - .map(serverDeadline -> serverDeadline.id() + ".timeout") - .orElse("timeout")); + .orElseGet( + () -> + serverSideDeadline + .map(serverDeadline -> serverDeadline.id() + ".timeout") + .orElse("timeout")); this.timeout = - clientedProvidedTimeout.orElse(serverSideDeadline.map(ServerDeadline::timeout).orElse(0L)); + clientedProvidedTimeout.orElseGet( + () -> serverSideDeadline.map(ServerDeadline::timeout).orElse(0L)); this.deadline = timeout > 0 ? Optional.of(start + timeout) : Optional.empty(); } diff --git a/java/com/google/gerrit/server/account/AccountCacheImpl.java b/java/com/google/gerrit/server/account/AccountCacheImpl.java index 65a80a21dc..d7d4938c97 100644 --- a/java/com/google/gerrit/server/account/AccountCacheImpl.java +++ b/java/com/google/gerrit/server/account/AccountCacheImpl.java @@ -105,7 +105,7 @@ public class AccountCacheImpl implements AccountCache { @Override public AccountState getEvenIfMissing(Account.Id accountId) { - return get(accountId).orElse(missing(accountId)); + return get(accountId).orElseGet(() -> missing(accountId)); } @Override diff --git a/java/com/google/gerrit/server/account/AccountManager.java b/java/com/google/gerrit/server/account/AccountManager.java index e32a0eb06f..18260a41f0 100644 --- a/java/com/google/gerrit/server/account/AccountManager.java +++ b/java/com/google/gerrit/server/account/AccountManager.java @@ -235,7 +235,7 @@ public class AccountManager { "Unable to deactivate account %s", authRequest .getUserName() - .orElse(" for external ID key " + authRequest.getExternalIdKey().get())); + .orElseGet(() -> " for external ID key " + authRequest.getExternalIdKey().get())); } } diff --git a/java/com/google/gerrit/server/change/DeleteReviewerOp.java b/java/com/google/gerrit/server/change/DeleteReviewerOp.java index ce5256927e..90cb9a9523 100644 --- a/java/com/google/gerrit/server/change/DeleteReviewerOp.java +++ b/java/com/google/gerrit/server/change/DeleteReviewerOp.java @@ -216,7 +216,7 @@ public class DeleteReviewerOp extends ReviewerOp { reviewerDeleted.fire( ctx.getChangeData(currChange), patchSet, - accountCache.get(reviewer.id()).orElse(AccountState.forAccount(reviewer)), + accountCache.get(reviewer.id()).orElseGet(() -> AccountState.forAccount(reviewer)), ctx.getAccount(), mailMessage, newApprovals, diff --git a/java/com/google/gerrit/server/git/DefaultChangeReportFormatter.java b/java/com/google/gerrit/server/git/DefaultChangeReportFormatter.java index 4f6094e685..58c3eb10b9 100644 --- a/java/com/google/gerrit/server/git/DefaultChangeReportFormatter.java +++ b/java/com/google/gerrit/server/git/DefaultChangeReportFormatter.java @@ -54,7 +54,7 @@ public class DefaultChangeReportFormatter implements ChangeReportFormatter { urlFormatter .get() .getChangeViewUrl(c.getProject(), c.getId()) - .orElse(c.getId().toString())); + .orElseGet(() -> c.getId().toString())); } protected String cropSubject(String subject) { diff --git a/java/com/google/gerrit/server/git/receive/ReceiveCommits.java b/java/com/google/gerrit/server/git/receive/ReceiveCommits.java index d27b13ad34..f60a0c7033 100644 --- a/java/com/google/gerrit/server/git/receive/ReceiveCommits.java +++ b/java/com/google/gerrit/server/git/receive/ReceiveCommits.java @@ -2031,10 +2031,10 @@ class ReceiveCommits { magicBranch.dest = BranchNameKey.create(project.getNameKey(), ref); magicBranch.perm = permissions.ref(ref); - Optional<AuthException> err = - checkRefPermission(magicBranch.perm, RefPermission.READ) - .map(Optional::of) - .orElse(checkRefPermission(magicBranch.perm, RefPermission.CREATE_CHANGE)); + Optional<AuthException> err = checkRefPermission(magicBranch.perm, RefPermission.READ); + if (err.isEmpty()) { + err = checkRefPermission(magicBranch.perm, RefPermission.CREATE_CHANGE); + } if (err.isPresent()) { rejectProhibited(cmd, err.get()); return; diff --git a/java/com/google/gerrit/server/group/db/AuditLogFormatter.java b/java/com/google/gerrit/server/group/db/AuditLogFormatter.java index 235ca4f613..3ba087e315 100644 --- a/java/com/google/gerrit/server/group/db/AuditLogFormatter.java +++ b/java/com/google/gerrit/server/group/db/AuditLogFormatter.java @@ -167,7 +167,7 @@ public class AuditLogFormatter { .map(Account::getName) // Historically, the database did not enforce relational integrity, so it is // possible for groups to have non-existing members. - .orElse("No Account for Id #" + accountId); + .orElseGet(() -> "No Account for Id #" + accountId); } private PersonIdent getParsableAuthorIdent( diff --git a/java/com/google/gerrit/server/patch/PatchFile.java b/java/com/google/gerrit/server/patch/PatchFile.java index 7a8180bd13..c3a6807fac 100644 --- a/java/com/google/gerrit/server/patch/PatchFile.java +++ b/java/com/google/gerrit/server/patch/PatchFile.java @@ -61,7 +61,7 @@ public class PatchFile { .filter(f -> f.getKey().equals(fileName)) .map(Map.Entry::getValue) .findFirst() - .orElse(FileDiffOutput.empty(fileName, ObjectId.zeroId(), ObjectId.zeroId())); + .orElseGet(() -> FileDiffOutput.empty(fileName, ObjectId.zeroId(), ObjectId.zeroId())); if (Patch.PATCHSET_LEVEL.equals(fileName)) { aTree = null; diff --git a/java/com/google/gerrit/server/project/Reachable.java b/java/com/google/gerrit/server/project/Reachable.java index 342c2bcc06..c935adf079 100644 --- a/java/com/google/gerrit/server/project/Reachable.java +++ b/java/com/google/gerrit/server/project/Reachable.java @@ -74,7 +74,7 @@ public class Reachable { Collection<Ref> filtered = optionalUserProvider .map(permissionBackend::user) - .orElse(permissionBackend.currentUser()) + .orElseGet(() -> permissionBackend.currentUser()) .project(project) .filter(refs, repo, RefFilterOptions.defaults()); Collection<RevCommit> visible = new ArrayList<>(); diff --git a/javatests/com/google/gerrit/server/group/db/AbstractGroupTest.java b/javatests/com/google/gerrit/server/group/db/AbstractGroupTest.java index 65eb5b0afb..e0f4b63150 100644 --- a/javatests/com/google/gerrit/server/group/db/AbstractGroupTest.java +++ b/javatests/com/google/gerrit/server/group/db/AbstractGroupTest.java @@ -142,7 +142,7 @@ public class AbstractGroupTest { return GroupConfig.loadForGroup(allUsersName, allUsersRepo, uuid) .getLoadedGroup() .map(InternalGroup::getName) - .orElse("Group " + uuid); + .orElseGet(() -> "Group " + uuid); } catch (IOException | ConfigInvalidException e) { return "Group " + uuid; } |