auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code><START> public Rectangle getHeaderBounds() { <END> return syncExec(new Result<Rectangle>() { public Rectangle run() { Point location = parent.getParent().toDisplay(parent.getLocation()); Rectangle bounds = new Rectangle(location.x, location.y, widget.getWidth(), parent.getHeaderHeight()); for (int i : parent.getColumnOrder()) { TreeColumn column = parent.getColumns()[i]; if (column.equals(widget)) { break; } else { bounds.x += column.getWidth(); } } return bounds; } }); }</code><technical_language>this need public</technical_language>

Target:
private Rectangle getHeaderBounds() { return syncExec(new Result<Rectangle>() { public Rectangle run() { Point location = parent.getParent().toDisplay(parent.getLocation()); Rectangle bounds = new Rectangle(location.x, location.y, widget.getWidth(), parent.getHeaderHeight()); for (int i : parent.getColumnOrder()) { TreeColumn column = parent.getColumn(i); if (column.equals(widget)) { break; } else { bounds.x += column.getWidth(); } } return bounds; } }); }
Source:
<code>public HostAddedToFacility(Host host, Facility facility) { this.facility = facility; this.host = host; <START> this.message = formatMessage("%s added to facility %s", host, facility); <END> }</code><technical_language>This message is one: getPerunBl().getAuditer().log(sess, "{} added {}.", host, facility);</technical_language>

Target:
public HostAddedToFacility(Host host, Facility facility) { this.facility = facility; this.host = host; this.message = formatMessage("%s added to %s.", host, facility); }
Source:
<code>public void write(JsonGenerator jg, NuxeoPrincipal nuxeoPrincipal) throws IOException { UserProfileService ups = Framework.getService(UserProfileService.class); try (RenderingContext.SessionWrapper sw = ctx.getSession(null)) { DocumentModel up = ups.getUserProfileDocument(nuxeoPrincipal.getName(), sw.getSession()); jg.writeFieldName(NAME); if (up == null) { jg.writeNull(); return; } jg.writeStartObject(); <START> ConfigurationService configurationService = Framework.getService(ConfigurationService.class); <END> if (configurationService.isBooleanPropertyTrue(COMPATIBILITY_CONFIGURATION_PARAM)) { writeCompatibilityUserProfile(jg, up); } else { writeUserProfile(jg, up); } jg.writeEndObject(); } }</code><technical_language>sure, injected instead of looked up? (Same for other services in this class.)</technical_language>

Target:
public void write(JsonGenerator jg, NuxeoPrincipal nuxeoPrincipal) throws IOException { try (RenderingContext.SessionWrapper sw = ctx.getSession(null)) { DocumentModel up = userProfileService.getUserProfileDocument(nuxeoPrincipal.getName(), sw.getSession()); jg.writeFieldName(NAME); if (up == null) { jg.writeNull(); return; } jg.writeStartObject(); if (configurationService.isBooleanPropertyTrue(COMPATIBILITY_CONFIGURATION_PARAM)) { writeCompatibilityUserProfile(jg, up); } else { writeUserProfile(jg, up); } jg.writeEndObject(); } }
Source:
<code>public List<URL> findAllRepositoriesInStream(String streamName) throws AphroditeException { if (streamName == null) { throw new AphroditeException("stream name is null in find respository stream"); } else { for (StreamService ss : streams) { <START> return ss.findAllRepositories(); <END> } return null; } }</code><technical_language>I this ss.findAllRepositories(streamName)</technical_language>

Target:
public List<URL> findAllRepositoriesInStream(String streamName) throws AphroditeException { checkStreamServiceExists(); Objects.requireNonNull(streamName,"stream name is null in find respository stream"); List<URL> urls=new ArrayList<>(); for (StreamService ss : streams) { urls.addAll(ss.findAllRepositoriesInStream(streamName)); } return urls; }
Source:
<code>private long testWriteToPipe(final Consumer<String> codeThatWritesToAFilename) throws IOException, InterruptedException, ExecutionException { final File fifo = File.createTempFile("fifo.for.testWriteToPipe", ""); Assert.assertTrue(fifo.delete()); fifo.deleteOnExit(); final Process exec = new ProcessBuilder("mkfifo", fifo.getAbsolutePath()).start(); exec.waitFor(1, TimeUnit.MINUTES); Assert.assertEquals(exec.exitValue(), 0, "mkfifo failed with exit code " + 0); ExecutorService executor = null; try { executor = Executors.newSingleThreadExecutor(); <START> Future result = executor.submit(() -> { <END> try { codeThatWritesToAFilename.accept(fifo.getAbsolutePath()); } catch (Exception x) { x.printStackTrace(); throw(x); } }); InputStream inputStream = Files.newInputStream(fifo.toPath(), StandardOpenOption.READ); int count = 0; while (inputStream.read() >= 0) { count++; } result.get(); return count; } finally { if (executor != null) executor.shutdownNow(); } }</code><technical_language>final</technical_language>

Target:
private static long testWriteToPipe(final Consumer<String> codeThatWritesToAFilename) throws IOException, InterruptedException, ExecutionException { final File fifo = makeFifo(); ExecutorService executor = Executors.newFixedThreadPool(2); try { final Future<?> writeResult = executor.submit(() -> { try { codeThatWritesToAFilename.accept(fifo.getAbsolutePath()); } catch (Exception x) { x.printStackTrace(); throw(x); } }); final Future<Integer> readResult = executor.submit(() -> { try(final InputStream inputStream = Files.newInputStream(fifo.toPath(), StandardOpenOption.READ)) { int count = 0; while (inputStream.read() >= 0) { count++; } return count; } }); writeResult.get(); return readResult.get(); } finally { executor.shutdownNow(); } }
Source:
<code>public boolean resolveConflictsOnMerge() { return resolveConflictsOnMerge && <START> clusterTopologyManager.getManagerStatus().isRunning() && <END> clusterTopologyManager.isRebalancingEnabled() && rebalancingEnabled; }</code><technical_language>CTMI status changes CTMI is stopping, caches stopped this point. IIRC conflict resolution requires cache running coordinator, need check for explicitly</technical_language>

Target:
public boolean resolveConflictsOnMerge() { return resolveConflictsOnMerge && cacheManager.getStatus().allowInvocations() && clusterTopologyManager.isRebalancingEnabled() && rebalancingEnabled; }
Source:
<code>protected Constructor<A> getEntityConstructor() { final Constructor<A> result = <START> Aggregates.getAggregatePartConstructor(getEntityClass(), rootClass); <END> return result; }</code><technical_language>get root class a generic parameter of aggregate class or, more straightforward, repository class? If can, need accept a constructor parameter</technical_language>

Target:
protected Constructor<A> getEntityConstructor() { final Constructor<A> result = Aggregates.getAggregatePartConstructor(getEntityClass()); return result; }
Source:
<code>public ClientResponse apply(ClientRequest jerseyRequest) { try { final HttpUriRequest apacheRequest = buildApacheRequest(jerseyRequest); final CloseableHttpResponse apacheResponse = client.execute(apacheRequest); final StatusLine statusLine = apacheResponse.getStatusLine(); final Response.StatusType status = Statuses.from(statusLine.getStatusCode(), firstNonNull(statusLine.getReasonPhrase(), "")); final ClientResponse jerseyResponse = new ClientResponse(status, jerseyRequest); for (Header header : apacheResponse.getAllHeaders()) { final List<String> headerValues = jerseyResponse.getHeaders().get(header.getName()); if (headerValues == null) { <START> jerseyResponse.getHeaders().put(header.getName(), new ArrayList<>(Collections.singletonList(header.getValue()))); <END> } else { headerValues.add(header.getValue()); } } final HttpEntity httpEntity = apacheResponse.getEntity(); jerseyResponse.setEntityStream(httpEntity != null ? httpEntity.getContent() : new ByteArrayInputStream(new byte[0])); return jerseyResponse; } catch (Exception e) { throw new ProcessingException(e); } }</code><technical_language>about: Lists.newArrayList(header.getValue())</technical_language>

Target:
public ClientResponse apply(ClientRequest jerseyRequest) { try { final HttpUriRequest apacheRequest = buildApacheRequest(jerseyRequest); final CloseableHttpResponse apacheResponse = client.execute(apacheRequest); final StatusLine statusLine = apacheResponse.getStatusLine(); final Response.StatusType status = Statuses.from(statusLine.getStatusCode(), firstNonNull(statusLine.getReasonPhrase(), "")); final ClientResponse jerseyResponse = new ClientResponse(status, jerseyRequest); for (Header header : apacheResponse.getAllHeaders()) { final List<String> headerValues = jerseyResponse.getHeaders().get(header.getName()); if (headerValues == null) { jerseyResponse.getHeaders().put(header.getName(), Lists.newArrayList(header.getValue())); } else { headerValues.add(header.getValue()); } } final HttpEntity httpEntity = apacheResponse.getEntity(); jerseyResponse.setEntityStream(httpEntity != null ? httpEntity.getContent() : new ByteArrayInputStream(new byte[0])); return jerseyResponse; } catch (Exception e) { throw new ProcessingException(e); } }
Source:
<code>public void deployUndeployProjectToServer(String projectToDeploy, String serverName) { new ConsoleView().clearConsole(); try { deployProject(projectToDeploy, serverName); checkDeployedProject(projectToDeploy, serverName); unDeployModule(projectToDeploy, serverName); } catch (CoreLayerException ex) { new DefaultShell("Server Error"); <START> new OkButton().click(); <END> } }</code><technical_language>please explain this? OK server error</technical_language>

Target:
public void deployUndeployProjectToServer(String projectToDeploy, String serverName) { new ConsoleView().clearConsole(); deployProject(projectToDeploy, serverName); checkDeployedProject(projectToDeploy, serverName); unDeployModule(projectToDeploy, serverName); }
Source:
<code>private boolean isGrantedForGroup(List<GroupAce> controlEntries, Authentication authentication, Permission permission, boolean isCheckAllowedGrant) { Object auth = authentication.getPrincipal(); <START> if (auth instanceof JCUser) { <END> List<Long> groupsIDs = ((JCUser) auth).getGroupsIDs(); for (GroupAce ace : controlEntries) { if (groupsIDs.contains(ace.getGroupId())){ if (isGrantedForGroup(ace, permission, isCheckAllowedGrant)) { return true; } } } } return false; }</code><technical_language>nested.. simplified: java Object auth = authentication.getPrincipal(); if (!(auth instanceof JCUser)) return false; List<Long> groupsIDs = ((JCUser) auth).getGroupsIDs(); for (GroupAce ace : controlEntries) { if (groupsIDs.contains(ace.getGroupId())) { if (isGrantedForGroup(ace, permission, isCheckAllowedGrant)) return true; } } return false;</technical_language>

Target:
private boolean isGrantedForGroup(List<GroupAce> controlEntries, Authentication authentication, Permission permission, boolean isCheckAllowedGrant) { Object auth = authentication.getPrincipal(); if (!(auth instanceof JCUser)) return false; List<Long> groupsIDs = ((JCUser) auth).getGroupsIDs(); for (GroupAce ace : controlEntries) { if (groupsIDs.contains(ace.getGroupId())) { if (isGrantedForGroup(ace, permission, isCheckAllowedGrant)) return true; } } return false; }
Source:
<code>private static void setWorkspaceNameDefault() { IPath workspaceDir = Platform.getLocation(); if (workspaceDir == null) return; String workspaceName = workspaceDir.lastSegment(); if (workspaceName == null) <START> return; <END> IPreferenceStore preferences = IDEWorkbenchPlugin.getDefault().getPreferenceStore(); preferences.setDefault(IDEInternalPreferences.WORKSPACE_NAME, workspaceName); }</code><technical_language>Oomph workspaces segment of "ws" a descriptive second-last segment. Consider a more nuanced heuristic handles this case, too. option: second-last segment if workspace name is string literal "ws". Another option: segments in circumstances. Another option: segments if segment is characters less. Another option: for a folder structure resembles oomph installation second-last segment if so. IMO, a check for string literal "ws" sufficient a pretty strong indicator workspace is part of oomph installation</technical_language>

Target:
private static void setWorkspaceNameDefault() { IPreferenceStore preferences = IDEWorkbenchPlugin.getDefault().getPreferenceStore(); String workspaceNameDefault = preferences.getDefaultString(IDEInternalPreferences.WORKSPACE_NAME); if (workspaceNameDefault != null) return; IPath workspaceDir = Platform.getLocation(); if (workspaceDir == null) return; String workspaceName = workspaceDir.lastSegment(); if (workspaceName == null) return; preferences.setDefault(IDEInternalPreferences.WORKSPACE_NAME, workspaceName); }
Source:
<code><START> public void in_stringNull() { <END> populateNoPrimaryKeyNullTypesRows(); try { realm.where(NoPrimaryKeyNullTypes.class).in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, (String[]) null).findAll(); fail(); } catch (IllegalArgumentException ignored) { } try { realm.where(NoPrimaryKeyNullTypes.class).in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{}).findAll(); fail(); } catch (IllegalArgumentException ignored) { } RealmResults<NoPrimaryKeyNullTypes> resultList = realm.where(NoPrimaryKeyNullTypes.class).in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"test data 14"}).findAll(); assertEquals(1, resultList.size()); resultList = realm.where(NoPrimaryKeyNullTypes.class).in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"test data 14", "test data 118", "test data 31", "test data 199"}).findAll(); assertEquals(4, resultList.size()); resultList = realm.where(NoPrimaryKeyNullTypes.class).in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"TEST data 14", "test data 118", "test data 31", "test DATA 199"}, Case.INSENSITIVE).findAll(); assertEquals(4, resultList.size()); resultList = realm.where(NoPrimaryKeyNullTypes.class).not().in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"TEST data 14", "test data 118", "test data 31", "test DATA 199"}, Case.INSENSITIVE).findAll(); assertEquals(196, resultList.size()); resultList = realm.where(NoPrimaryKeyNullTypes.class).not().in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"TEST data 14", "test data 118", "test data 31", "test DATA 199"}, Case.INSENSITIVE).findAll(); assertEquals(196, resultList.size()); resultList = realm.where(NoPrimaryKeyNullTypes.class).not().in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"TEST data 14", "test data 118", null, "test DATA 199"}, Case.INSENSITIVE).findAll(); assertEquals(130, resultList.size()); }</code><technical_language>in_stringNotNull() in_stringNull() identical, common method in_stringNotNull() in_stringNull() call field name parameter</technical_language>

Target:
public void in_stringNull() { doTestForInString(NoPrimaryKeyNullTypes.FIELD_STRING_NULL); RealmResults<NoPrimaryKeyNullTypes> resultList = realm.where(NoPrimaryKeyNullTypes.class).not().in(NoPrimaryKeyNullTypes.FIELD_STRING_NULL, new String[]{"TEST data 14", "test data 118", null, "test DATA 199"}, Case.INSENSITIVE).findAll(); assertEquals(130, resultList.size()); }
Source:
<code>public List<PatchLineComment> byChange(ReviewDb db, ChangeNotes notes) throws OrmException { if (!migration.readComments()) { return db.patchComments().byChange(notes.getChangeId()).toList(); } notes.load(); List<PatchLineComment> comments = Lists.newArrayList(); comments.addAll(notes.getBaseComments().values()); comments.addAll(notes.getPatchSetComments().values()); Set<String> refNames = getRefNamesAllUsers(RefNames.REFS_USER); for (String refName : refNames) { String suffix = Integer.toString(notes.getChangeId().get()); if (!refName.endsWith(suffix)) { continue; } <START> String[] parts = refName.split(RefNames.DRAFTS); <END> if (parts.length != 2) { continue; } Account.Id account = Account.Id.fromRefPart(parts[0]); comments.addAll(draftByChangeAuthor(db, notes, account)); } return comments; }</code><technical_language>need a regex here, indexOf. If want more confident input this point, change previous check to: suffix = RefNames.DRAFTS + Integer.toString(...)</technical_language>

Target:
public List<PatchLineComment> byChange(ReviewDb db, ChangeNotes notes) throws OrmException { if (!migration.readComments()) { return db.patchComments().byChange(notes.getChangeId()).toList(); } notes.load(); List<PatchLineComment> comments = Lists.newArrayList(); comments.addAll(notes.getBaseComments().values()); comments.addAll(notes.getPatchSetComments().values()); Set<String> refNames = getRefNamesAllUsers(RefNames.REFS_USER); Iterable<String> filtered = Iterables.filter(refNames, new IsDraftRefForChange(notes.getChangeId())); for (String refName : filtered) { Account.Id account = Account.Id.fromRefPart(refName); comments.addAll(draftByChangeAuthor(db, notes, account)); } return comments; }
Source:
<code>public void visit(NodeTraversal t, Node n, Node parent) { if (n.isName()) { switch (parent.getType()) { case Token.VAR: case Token.LET: case Token.CONST: case Token.FUNCTION: case Token.CLASS: case Token.PARAM_LIST: break; case Token.GETPROP: if (n == parent.getFirstChild()) { Scope scope = t.getScope(); Var var = scope.getVar(n.getString()); if (var == null) { t.report(n, UNDEFINED_EXTERN_VAR_ERROR, n.getString()); varsToDeclareInExterns.add(n.getString()); } } break; case Token.ASSIGN: if (n == parent.getLastChild() && parent.getFirstChild().isGetProp() && parent.getFirstChild().getLastChild().getString().equals(n.getString())) { break; } default: if (parent.getType() != Token.NAME || parent.getParent() == null || <START> parent.getParent().getType() != Token.VAR) { <END> t.report(n, NAME_REFERENCE_IN_EXTERNS_ERROR, n.getString()); } Scope scope = t.getScope(); Var var = scope.getVar(n.getString()); if (var == null) { varsToDeclareInExterns.add(n.getString()); } break; } } }</code><technical_language>allow CONST want allow ES6 in externs some point (though yet)</technical_language>

Target:
public void visit(NodeTraversal t, Node n, Node parent) { if (n.isName()) { switch (parent.getType()) { case Token.VAR: case Token.LET: case Token.CONST: case Token.FUNCTION: case Token.CLASS: case Token.PARAM_LIST: break; case Token.GETPROP: if (n == parent.getFirstChild()) { Scope scope = t.getScope(); Var var = scope.getVar(n.getString()); if (var == null) { t.report(n, UNDEFINED_EXTERN_VAR_ERROR, n.getString()); varsToDeclareInExterns.add(n.getString()); } } break; case Token.ASSIGN: if (n == parent.getLastChild() && parent.getFirstChild().isGetProp() && parent.getFirstChild().getLastChild().getString().equals(n.getString())) { break; } default: if (parent.getType() != Token.NAME || parent.getParent() == null || !NodeUtil.isNameDeclaration(parent.getParent())) { t.report(n, NAME_REFERENCE_IN_EXTERNS_ERROR, n.getString()); } Scope scope = t.getScope(); Var var = scope.getVar(n.getString()); if (var == null) { varsToDeclareInExterns.add(n.getString()); } break; } } }
Source:
<code>public void setLabels() { turnLabel.setText(playerList.get(turnOrder.get(currentPlayer)).getName() + "'s Turn " + timeLeft); resourceLabel.setText(playerList.get(turnOrder.get(currentPlayer)).getName() + "'s Money: " + playerList.get(turnOrder.get(currentPlayer)).getMoney() + " F: " + playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.FOOD) + " E: " + playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.ENERGY) + " S: " + playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.SMITHORE) + " C: " <START> + playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.CRYSTITE)); <END> }</code><technical_language>Consider string formatting string concatenation, refactoring this include some temporary variables, nicer</technical_language>

Target:
public void setLabels() { turnLabel.setText(playerList.get(turnOrder.get(currentPlayer)).getName() + "'s Turn " + timeLeft); String s = String.format("%1$s's Money: %2$s F: %3$s E: %4$s S: %5$s C: %6$s", playerList.get(turnOrder.get(currentPlayer)).getName(), playerList.get(turnOrder.get(currentPlayer)).getMoney(), playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.FOOD), playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.ENERGY), playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.SMITHORE), playerList.get(turnOrder.get(currentPlayer)).stockOf(ResourceType.CRYSTITE)); resourceLabel.setText(s); }
Source:
<code>public boolean Validate() { boolean isNew = getModel().getIsNew(); int maxAllowedVms = getMaxVmsInPool(); int assignedVms = getModel().getAssignedVms().AsConvertible().Integer(); getModel().getNumOfDesktops().ValidateEntity( union(new IValidation[] { new NotEmptyValidation(), new LengthValidation(4), new IntegerValidation(isNew ? 1 : 0, isNew ? maxAllowedVms : maxAllowedVms - assignedVms) }, <START> getNumOfDesktopsSpecificValidations() <END> ) ); getModel().getPrestartedVms().ValidateEntity( new IValidation[] { new NotEmptyValidation(), new IntegerValidation(0, assignedVms) }); if (getModel().getName().getIsValid()) { getModel().getName().ValidateEntity(getPoolNameSpecificValidations()); } getModel().setIsGeneralTabValid(getModel().getIsGeneralTabValid() && getModel().getName().getIsValid() && getModel().getNumOfDesktops().getIsValid() && getModel().getPrestartedVms().getIsValid()); getModel().setIsPoolTabValid(true); return super.Validate() && getModel().getName().getIsValid() && getModel().getNumOfDesktops().getIsValid() && getModel().getPrestartedVms().getIsValid(); }</code><technical_language>validation, add this line: if(getModel().getNumOfDesktops().getIsValid()){ getModel().getNumOfDesktops().ValidateEntity( } remove union (like in name validation)</technical_language>

Target:
public boolean Validate() { boolean isNew = getModel().getIsNew(); int maxAllowedVms = getMaxVmsInPool(); int assignedVms = getModel().getAssignedVms().AsConvertible().Integer(); getModel().getNumOfDesktops().ValidateEntity( new IValidation[] { new NotEmptyValidation(), new LengthValidation(4), new IntegerValidation(isNew ? 1 : 0, isNew ? maxAllowedVms : maxAllowedVms - assignedVms) }); getModel().getPrestartedVms().ValidateEntity( new IValidation[] { new NotEmptyValidation(), new IntegerValidation(0, assignedVms) }); getModel().setIsGeneralTabValid(getModel().getIsGeneralTabValid() && getModel().getName().getIsValid() && getModel().getNumOfDesktops().getIsValid() && getModel().getPrestartedVms().getIsValid()); getModel().setIsPoolTabValid(true); return super.Validate() && getModel().getName().getIsValid() && getModel().getNumOfDesktops().getIsValid() && getModel().getPrestartedVms().getIsValid(); }
Source:
<code>protected Scope decorateScope(@Nullable TraceContext currentSpan, Scope scope) { int length = scopeDecorators.size(); <START> if (length == 0) return scope; <END> for (int i = 0; i < length; i++) { scope = scopeDecorators.get(i).decorateScope(currentSpan, scope); } return scope; }</code><technical_language>I for loop duplicates this check speed 0 case</technical_language>

Target:
protected Scope decorateScope(@Nullable TraceContext currentSpan, Scope scope) { int length = scopeDecorators.size(); for (int i = 0; i < length; i++) { scope = scopeDecorators.get(i).decorateScope(currentSpan, scope); } return scope; }
Source:
<code>private String[] getHeaders() { String[] toReturn = { "Test Case", "Dry Run Results:Test Points", "Dry Run Results:Total Fails", "DemoResults:Total Test Points", "Demo: Code Problems", "Script Problems", "Requirements", "Under Investigation", "Other", "Total Fails", "Verification of RPCR", <START> "Commnets"}; <END> return toReturn; }</code><technical_language>this a constants static final array changes. need re-creating array time</technical_language>

Target:
private static String[] getHeaders() { String[] toReturn = { "Test Case", "Dry Run Results:Test Points", "Dry Run Results:Total Fails", "DemoResults:Total Test Points", "Demo: Code Problems", "Script Problems", "Requirements", "Under Investigation", "Other", "Total Fails", "Verification of RPCR", "Commnets"}; return toReturn; }
Source:
<code>public T next() { if (closed) { return null; } T res = buffer.next(); if (res != null) { return res; } else if (jobTerminated) { return null; } while (true) { if (isJobTerminated()) { jobTerminated = true; Tuple2<Long, CollectCoordinationResponse> accResults = getAccumulatorResults(); if (accResults != null) { buffer.dealWithResponse(accResults.f1, accResults.f0); } buffer.complete(); } else { <START> CollectCoordinationResponse<T> response; try { response = sendRequest(buffer.version, buffer.offset); } catch (Exception e) { LOG.warn("An exception occurs when fetching query results", e); sleepBeforeRetry(); continue; } buffer.dealWithResponse(response); <END> } res = buffer.next(); if (res != null) { return res; } else if (jobTerminated) { return null; } else { sleepBeforeRetry(); } } }</code><technical_language>suggestion long requestOffset = buffer.offset; CollectCoordinationResponse<T> response; try { response = sendRequest(buffer.version, requestOffset); } catch (Exception e) { LOG.warn("An exception occurs fetching query results", e); sleepBeforeRetry(); continue; } // response data (if any) starting requested offset buffer.dealWithResponse(response, requestOffset);</technical_language>

Target:
public T next() throws IOException { if (closed) { return null; } boolean beforeFirstTry = true; do { T res = buffer.next(); if (res != null) { return res; } else if (jobTerminated) { return null; } else if (!beforeFirstTry) { sleepBeforeRetry(); } beforeFirstTry = false; if (isJobTerminated()) { jobTerminated = true; Tuple2<Long, CollectCoordinationResponse<T>> accResults = getAccumulatorResults(); buffer.dealWithResponse(accResults.f1, accResults.f0); buffer.complete(); } else { long requestOffset = buffer.offset; CollectCoordinationResponse<T> response; try { response = sendRequest(buffer.version, requestOffset); } catch (Exception e) { LOG.warn("An exception occurs when fetching query results", e); continue; } buffer.dealWithResponse(response, requestOffset); } } while (true); }
Source:
<code>public boolean equals(Object o) { if (this == o) return true; if (!(o instanceof DiskUsage)) return false; DiskUsage diskUsage = (DiskUsage) o; if (!Objects.equals(tables, diskUsage.tables)) return false; <START> return Objects.equals(usage, diskUsage.usage); <END> }</code><technical_language>This reduced AND, like: return Objects.equals(tables, diskUsage.tables) && Objects.equals(usage, diskUsage.usage); Also, thought replacing hashCode time Objects.hash(tables, usage); - constraint equals hashCode consistent</technical_language>

Target:
public boolean equals(Object o) { if (this == o) return true; if (!(o instanceof DiskUsage)) return false; DiskUsage diskUsage = (DiskUsage) o; return Objects.equals(tables, diskUsage.tables) && Objects.equals(usage, diskUsage.usage); }
Source:
<code>public double distance(OGCGeometry another) { <START> if (another.geometryType() == OGCConcreteGeometryCollection.TYPE) { <END> return another.distance(this); } com.esri.core.geometry.Geometry geom1 = getEsriGeometry(); com.esri.core.geometry.Geometry geom2 = another.getEsriGeometry(); return com.esri.core.geometry.GeometryEngine.distance(geom1, geom2, getEsriSpatialReference()); }</code><technical_language>logic apply other methods, e.g. contains, overlaps, etc.</technical_language>

Target:
public double distance(OGCGeometry another) { if (this == another) { return isEmpty() ? Double.NaN : 0; } if (another.geometryType() == OGCConcreteGeometryCollection.TYPE) { return another.distance(this); } com.esri.core.geometry.Geometry geom1 = getEsriGeometry(); com.esri.core.geometry.Geometry geom2 = another.getEsriGeometry(); return com.esri.core.geometry.GeometryEngine.distance(geom1, geom2, getEsriSpatialReference()); }
Source:
<code>public Response updateItem(@Context HttpServletRequest request, @PathParam("id") String id, String content) { Response response = null; String mergedId = null; try (ItemManager itemManager = new ItemManager()) { Item dbItem = itemManager.load(id); Item updatedItem = Item.fromJSON(content); String trackId = null; if(dbItem != null && !updatedItem.isActiveStorm() && dbItem.isActiveStorm()) { Item trackItem = StormUtil.findTrackChildItem(updatedItem); if(trackItem != null) { trackId = trackItem.getId(); updatedItem.getChildren().remove( updatedItem.getChildren().indexOf(trackItem) ); } } Item mergedItem = Item.copyValues(updatedItem, dbItem); if (dbItem == null) { mergedId = itemManager.persist(mergedItem); } else { mergedId = itemManager.merge(mergedItem); } if (trackId != null && mergedId != null) { itemManager.delete(trackId, true); } } <START> if (null != mergedId) { <END> response = Response.ok().build(); } else { throw new BadRequestException(); } try (StatusManager statusMan = new StatusManager(); ThumbnailManager thumbMan = new ThumbnailManager()) { Status status = new Status(); status.setStatusName(Status.StatusName.ITEM_UPDATE); statusMan.save(status); thumbMan.updateDirtyBits(id); } return response; }</code><technical_language>sense move this line 213? all, if block run if mergedId is null throw early out of larger try block</technical_language>

Target:
public Response updateItem(@Context HttpServletRequest request, @PathParam("id") String id, String content) { Response response = null; String mergedId = null; try (ItemManager itemManager = new ItemManager()) { Item dbItem = itemManager.load(id); Item updatedItem = Item.fromJSON(content); String trackId = null; if(dbItem != null && !updatedItem.isActiveStorm() && dbItem.isActiveStorm()) { Item trackItem = StormUtil.findTrackChildItem(updatedItem); if(trackItem != null) { trackId = trackItem.getId(); updatedItem.getChildren().remove( updatedItem.getChildren().indexOf(trackItem) ); } } Item mergedItem = Item.copyValues(updatedItem, dbItem); if (dbItem == null) { mergedId = itemManager.persist(mergedItem); } else { mergedId = itemManager.merge(mergedItem); } if (null != mergedId) { response = Response.ok().build(); } else { throw new BadRequestException(); } if (trackId != null) { itemManager.delete(trackId, true); } } try (StatusManager statusMan = new StatusManager(); ThumbnailManager thumbMan = new ThumbnailManager()) { Status status = new Status(); status.setStatusName(Status.StatusName.ITEM_UPDATE); statusMan.save(status); thumbMan.updateDirtyBits(id); } return response; }
Source:
<code>private Class<?> getFirstInterfaceOrDefined(Object bean, Class defInterface) { Class<?>[] interfaces = bean.getClass().getInterfaces(); for (Class<?> clazz : interfaces) { if (defInterface == null) { return clazz; } else { if (clazz.equals(defInterface)) { return clazz; } } } <START> throw new IllegalStateException(bean.getClass().getCanonicalName() + " not implement interface " + defInterface <END> .getCanonicalName()); }</code><technical_language>1. "not implement" -> "does implement" 2. extend exception message "Illegal of @Publish interface" more specific descrption, stacktrace defines problem is</technical_language>

Target:
private Class<?> getFirstInterfaceOrDefined(Object bean, Class defInterface) { Class<?>[] interfaces = bean.getClass().getInterfaces(); for (Class<?> clazz : interfaces) { if (defInterface == null) { return clazz; } else { if (clazz.equals(defInterface)) { return clazz; } } } throw new PublishException( bean.getClass().getCanonicalName() + " does not implement interface " + defInterface.getCanonicalName()); }
Source:
<code>protected ActionResult execute() throws EntityDoesNotExistException { courseId = getRequestParamValue(Const.ParamsNames.COURSE_ID); feedbackSessionName = getRequestParamValue(Const.ParamsNames.FEEDBACK_SESSION_NAME); Assumption.assertPostParamNotNull(Const.ParamsNames.COURSE_ID, courseId); Assumption.assertPostParamNotNull(Const.ParamsNames.FEEDBACK_SESSION_NAME, feedbackSessionName); String regKey = getRequestParamValue(Const.ParamsNames.REGKEY); String email = getRequestParamValue(Const.ParamsNames.STUDENT_EMAIL); if (!isSpecificUserJoinedCourse()) { return createPleaseJoinCourseResponse(courseId); } FeedbackSessionAttributes feedbackSession = logic.getFeedbackSession(feedbackSessionName, courseId); if (feedbackSession == null) { statusToUser.add(new StatusMessage("The feedback session has been deleted and is no longer accessible.", StatusMessageColor.WARNING)); return createSpecificRedirectResult(); } verifyAccesibleForSpecificUser(feedbackSession); String userEmailForCourse = getUserEmailForCourse(); data = new FeedbackSubmissionEditPageData(account, student); <START> setDataBundle(userEmailForCourse); <END> data.setSessionOpenForSubmission(isSessionOpenForSpecificUser(data.bundle.feedbackSession)); setStatusToAdmin(); if (!data.isSessionOpenForSubmission()) { statusToUser.add(new StatusMessage(Const.StatusMessages.FEEDBACK_SUBMISSIONS_NOT_OPEN, StatusMessageColor.WARNING)); } data.init(regKey, email, courseId); return createSpecificShowPageResult(); }</code><technical_language>@unyoungwax, clear right this needs changed. explain</technical_language>

Target:
protected ActionResult execute() throws EntityDoesNotExistException { courseId = getRequestParamValue(Const.ParamsNames.COURSE_ID); feedbackSessionName = getRequestParamValue(Const.ParamsNames.FEEDBACK_SESSION_NAME); Assumption.assertPostParamNotNull(Const.ParamsNames.COURSE_ID, courseId); Assumption.assertPostParamNotNull(Const.ParamsNames.FEEDBACK_SESSION_NAME, feedbackSessionName); String regKey = getRequestParamValue(Const.ParamsNames.REGKEY); String email = getRequestParamValue(Const.ParamsNames.STUDENT_EMAIL); if (!isSpecificUserJoinedCourse()) { return createPleaseJoinCourseResponse(courseId); } FeedbackSessionAttributes feedbackSession = logic.getFeedbackSession(feedbackSessionName, courseId); if (feedbackSession == null) { statusToUser.add(new StatusMessage("The feedback session has been deleted and is no longer accessible.", StatusMessageColor.WARNING)); return createSpecificRedirectResult(); } verifyAccesibleForSpecificUser(feedbackSession); String userEmailForCourse = getUserEmailForCourse(); data = new FeedbackSubmissionEditPageData(account, student); data.bundle = getDataBundle(userEmailForCourse); data.setSessionOpenForSubmission(isSessionOpenForSpecificUser(data.bundle.feedbackSession)); setStatusToAdmin(); if (!data.isSessionOpenForSubmission()) { statusToUser.add(new StatusMessage(Const.StatusMessages.FEEDBACK_SUBMISSIONS_NOT_OPEN, StatusMessageColor.WARNING)); } data.init(regKey, email, courseId); return createSpecificShowPageResult(); }
Source:
<code>private static int findNextSibling(DetailAST ast, int tokenType) { <START> int token = -1; <END> DetailAST node = ast.getNextSibling(); while (node != null) { if (node.getType() == tokenType) { token = node.getType(); break; } node = node.getNextSibling(); } return token; }</code><technical_language>Return method Details object , in place check for null</technical_language>

Target:
private static DetailAST findNextSibling(DetailAST ast, int tokenType) { DetailAST token = null; DetailAST node = ast.getNextSibling(); while (node != null) { if (node.getType() == tokenType) { token = node; break; } node = node.getNextSibling(); } return token; }
Source:
<code>public abstract ExecutionResult execute(int selectedConfigurationId, @NotNull String cloudProjectId, @NotNull AndroidRunningState runningState, @NotNull Executor executor) throws ExecutionException; public <START> boolean isCloudTestingOptionEnabled(Project <END> project) { GoogleCloudTestingConfigurable.GoogleCloudTestingState googleCloudTestingState = GoogleCloudTestingSettings.getInstance(project).getState(); return googleCloudTestingState != null && googleCloudTestingState.enableCloudTesting; }</code><technical_language>static</technical_language>

Target:
public abstract ExecutionResult execute(int selectedConfigurationId, @NotNull String cloudProjectId, @NotNull AndroidRunningState runningState, @NotNull Executor executor) throws ExecutionException; public static boolean isEnabled() { CloudTestingConfigurable.cloudTestingState cloudTestingState = CloudTestingSettings.getInstance().getState(); return cloudTestingState != null && cloudTestingState.enableCloudTesting; }
Source:
<code>protected static void init(PerunSession sess) throws InternalErrorException { log.trace("Initializing AuthzResolver for [{}]", sess.getPerunPrincipal()); <START> refreshAuthz(sess); <END> log.debug("AuthzResolver: Complete PerunPrincipal: {}", sess.getPerunPrincipal()); }</code><technical_language>Please fix refreshAuthz(sess) implementation, 1:1 replacement of removed code right now</technical_language>

Target:
protected static void init(PerunSession sess) throws InternalErrorException { log.trace("Initializing AuthzResolver for [{}]", sess.getPerunPrincipal()); refreshAuthz(sess); log.trace("AuthzResolver: Complete PerunPrincipal: {}", sess.getPerunPrincipal()); }
Source:
<code>public void record(String hostname, int statusCode) { <START> hostMetrics.getUnchecked(hostname).record(statusCode); <END> }</code><technical_language>prefer fail a request record metrics. catch log</technical_language>

Target:
public void record(String hostname, int statusCode) { try { hostMetrics.getUnchecked(hostname).record(statusCode); } catch (Exception e) { log.warn("Unable to record metrics for host", UnsafeArg.of("hostname", hostname)); } }
Source:
<code>public static void withinTransaction(EntityManager em, Runnable codeblock) { EntityTransaction transaction = em.getTransaction(); transaction.begin(); try { codeblock.run(); transaction.commit(); } catch (Throwable t) { if ( transaction.isActive() ) { <START> transaction.rollback(); <END> } throw t; } }</code><technical_language>above, I this that: catch (Throwable t1) { try { if ( transaction.isActive() ) { transaction.rollback(); } } catch(Throwable t2) { someLogger.warn("Error while automatically rolling a transaction", e); // remove t1.addSuppressed(t2); } throw t1; }</technical_language>

Target:
public static void withinTransaction(EntityManager em, Runnable codeblock) { EntityTransaction transaction = em.getTransaction(); transaction.begin(); try { codeblock.run(); transaction.commit(); } catch (Throwable mainError) { try { if ( transaction.isActive() ) { transaction.rollback(); } } catch (Throwable errorOnRollBack) { mainError.addSuppressed( errorOnRollBack ); } throw mainError; } }
Source:
<code>public long transferTo(long position, long count, WritableByteChannel target) throws IOException { long bytesRead = 0; try { ensureProperFile(false, position); bytesRead = sageFileChannel.transferTo(position - fileOffsets[currFileIndex], count, target); <START> if (bytesRead < count) <END> { ensureProperFile(false, position); bytesRead += sageFileChannel.transferTo(position + bytesRead - fileOffsets[currFileIndex], count - bytesRead, target); } } finally { ensureProperFile(false); } return bytesRead; }</code><technical_language>this a 'while' instead of 'if' remove 2 lines of code...then cover spanning N number of files 2 (although, I more 2 in reality)</technical_language>

Target:
public long transferTo(long position, long count, WritableByteChannel target) throws IOException { long remaining = size() - position; if (remaining < count) count = remaining; long bytesRead = 0; try { while (bytesRead < count) { ensureProperFile(false, position); bytesRead += sageFileChannel.transferTo(position + bytesRead - fileOffsets[currFileIndex], count - bytesRead, target); } } finally { ensureProperFile(false); } return bytesRead; }
Source:
<code>public void init(Properties props, String topicName) { super.init(props, topicName); try { SchemaRegistry<Schema> registry = (SchemaRegistry<Schema>) Class.forName( props.getProperty(KafkaAvroMessageEncoder.KAFKA_MESSAGE_CODER_SCHEMA_REGISTRY_CLASS)).newInstance(); registry.init(props); this.registry = new CachedSchemaRegistry<Schema>(registry); this.latestSchema = registry.getLatestSchemaByTopic(topicName).getSchema(); timestampField = props.getProperty(JsonStringMessageDecoder.CAMUS_MESSAGE_TIMESTAMP_FIELD); <START> timestampFormat = props.getProperty(JsonStringMessageDecoder.CAMUS_MESSAGE_TIMESTAMP_FORMAT, <END> JsonStringMessageDecoder.CAMUS_MESSAGE_TIMESTAMP_FORMAT); } catch (Exception e) { throw new MessageDecoderException(e); } decoderFactory = DecoderFactory.get(); }</code><technical_language>validation timestampFormat is of format</technical_language>

Target:
public void init(Properties props, String topicName) { super.init(props, topicName); try { SchemaRegistry<Schema> registry = (SchemaRegistry<Schema>) Class.forName( props.getProperty(KafkaAvroMessageEncoder.KAFKA_MESSAGE_CODER_SCHEMA_REGISTRY_CLASS)).newInstance(); registry.init(props); this.registry = new CachedSchemaRegistry<Schema>(registry); this.latestSchema = registry.getLatestSchemaByTopic(topicName).getSchema(); timestampField = props.getProperty(CAMUS_MESSAGE_TIMESTAMP_FIELD, DEFAULT_TIMESTAMP_FIELD); timestampFormat = props.getProperty(CAMUS_MESSAGE_TIMESTAMP_FORMAT, UNIX_MILLISECONDS); } catch (Exception e) { throw new MessageDecoderException(e); } decoderFactory = DecoderFactory.get(); }
Source:
<code>public void askAKafkaConsumerWithoutServerUpTest(){ <START> SnapshotOnDemandUtils snapshotOnDemandUtils = new SnapshotOnDemandUtilsImpl(); <END> EnvConfig config = EnvConfig.getDefaultEnvConfig(); config.local(false); config.underTest(false); KafkaConsumer consumer = snapshotOnDemandUtils.getConfiguredSnapshotConsumer(config); assertNull(consumer); }</code><technical_language>create a field for this variable (and instantiate a @Before method)</technical_language>

Target:
public void askAKafkaConsumerWithoutServerUpTest(){ KafkaConsumer consumer = snapshotOnDemandUtils.getConfiguredSnapshotConsumer(config); assertNull(consumer); }
Source:
<code>public void searchRoles(@NonNull final JSONObject filters, final KuzzleOptions options, @NonNull final KuzzleResponseListener<KuzzleSecurityDocumentList> listener) throws JSONException { if (filters == null) { throw new IllegalArgumentException("KuzzleSecurity.searchRoles: filters cannot be null"); } if (listener == null) { throw new IllegalArgumentException("KuzzleSecurity.searchRoles: a callback listener is required"); } JSONObject data = new JSONObject().put("body", filters); this.kuzzle.query(buildQueryArgs("searchRoles"), data, options, new OnQueryDoneListener() { @Override public void onSuccess(JSONObject response) { try { JSONObject result = response.getJSONObject("result"); JSONArray documents = result.getJSONArray("hits"); int documentsLength = documents.length(); <START> LinkedList<AbstractKuzzleSecurityDocument> roles = new LinkedList<>(); <END> for (int i = 0; i < documentsLength; i++) { JSONObject document = documents.getJSONObject(i); roles.add(new KuzzleRole(KuzzleSecurity.this.kuzzle, document.getString("_id"), document.getJSONObject("_source"))); } listener.onSuccess(new KuzzleSecurityDocumentList(roles, result.getLong("total"))); } catch (JSONException e) { throw new RuntimeException(e); } } @Override public void onError(JSONObject error) { listener.onError(error); } }); }</code><technical_language>nitpicking: a LinkedList ArrayList? I understand, get null element in list</technical_language>

Target:
public void searchRoles(@NonNull final JSONObject filters, final KuzzleOptions options, @NonNull final KuzzleResponseListener<KuzzleSecurityDocumentList> listener) throws JSONException { if (filters == null) { throw new IllegalArgumentException("KuzzleSecurity.searchRoles: filters cannot be null"); } if (listener == null) { throw new IllegalArgumentException("KuzzleSecurity.searchRoles: a callback listener is required"); } JSONObject data = new JSONObject().put("body", filters); this.kuzzle.query(buildQueryArgs("searchRoles"), data, options, new OnQueryDoneListener() { @Override public void onSuccess(JSONObject response) { try { JSONObject result = response.getJSONObject("result"); JSONArray documents = result.getJSONArray("hits"); int documentsLength = documents.length(); ArrayList<AbstractKuzzleSecurityDocument> roles = new ArrayList<>(); for (int i = 0; i < documentsLength; i++) { JSONObject document = documents.getJSONObject(i); roles.add(new KuzzleRole(KuzzleSecurity.this.kuzzle, document.getString("_id"), document.getJSONObject("_source"))); } listener.onSuccess(new KuzzleSecurityDocumentList(roles, result.getLong("total"))); } catch (JSONException e) { throw new RuntimeException(e); } } @Override public void onError(JSONObject error) { listener.onError(error); } }); }
Source:
<code>private void bindParameterValues( long lo, long msgLimit, short parameterValueCount, @Transient ObjList<BindVariableSetter> bindVariableSetters ) throws BadProtocolException, SqlException { boolean inferTypes = parameterValueCount != bindVariableService.getIndexedVariableCount(); boolean allTextFormat = parameterFormats.size() == 0 || (parameterFormats.size() == 1 && parameterFormats.get(0) == 0); boolean allBinaryFormat = parameterFormats.size() == 1 && parameterFormats.get(0) == 1; for (int j = 0; j < parameterValueCount; j++) { if (lo + Integer.BYTES > msgLimit) { LOG.error().$("could not read parameter value length [index=").$(j).$(']').$(); throw BadProtocolException.INSTANCE; } int valueLen = getInt(lo); lo += Integer.BYTES; if (valueLen == -1) { continue; } if (lo + valueLen > msgLimit) { LOG.error() .$("value length is outside of buffer [parameterIndex=").$(j) .$(", valueLen=").$(valueLen) .$(", messageRemaining=").$(msgLimit - lo) .$(']').$(); throw BadProtocolException.INSTANCE; } ensureData(lo, valueLen, msgLimit, j); if (inferTypes) { <START> int pgType = inferParameterType(lo, valueLen); <END> setupBindVariable(bindVariableSetters, j, pgType); } if (allTextFormat || (!allBinaryFormat) && parameterFormats.get(j) == 0) { bindVariableSetters.setQuick(j * 2, bindVariableSetters.getQuick(j * 2 + 1)); } bindVariableSetters.getQuick(j * 2).set(j, lo, valueLen); lo += valueLen; } }</code><technical_language>inferParameterType() is liable return -1, is handled tested either</technical_language>

Target:
private void bindParameterValues( long lo, long msgLimit, short parameterValueCount, @Transient ObjList<BindVariableSetter> bindVariableSetters ) throws BadProtocolException, SqlException { boolean inferTypes = parameterValueCount != bindVariableService.getIndexedVariableCount(); boolean allTextFormat = parameterFormats.size() == 0 || (parameterFormats.size() == 1 && parameterFormats.get(0) == 0); boolean allBinaryFormat = parameterFormats.size() == 1 && parameterFormats.get(0) == 1; for (int j = 0; j < parameterValueCount; j++) { if (lo + Integer.BYTES > msgLimit) { LOG.error().$("could not read parameter value length [index=").$(j).$(']').$(); throw BadProtocolException.INSTANCE; } int valueLen = getInt(lo); lo += Integer.BYTES; if (valueLen == -1) { continue; } if (lo + valueLen > msgLimit) { LOG.error() .$("value length is outside of buffer [parameterIndex=").$(j) .$(", valueLen=").$(valueLen) .$(", messageRemaining=").$(msgLimit - lo) .$(']').$(); throw BadProtocolException.INSTANCE; } ensureData(lo, valueLen, msgLimit, j); if (inferTypes) { int pgType = inferParameterType(lo, valueLen); if (pgType == -1) { LOG.error().$("invalid parameter type for parameter #[").$(j).$(']').$(); throw BadProtocolException.INSTANCE; } setupBindVariable(bindVariableSetters, j, pgType); } if (allTextFormat || (!allBinaryFormat) && parameterFormats.get(j) == 0) { bindVariableSetters.setQuick(j * 2, bindVariableSetters.getQuick(j * 2 + 1)); } bindVariableSetters.getQuick(j * 2).set(j, lo, valueLen); lo += valueLen; } }
Source:
<code>public List<ServiceTaskSummary> getServiceTasks() { return repoService.getServices().stream() .map(repoData -> new ServiceTaskSummary(repoData.getId(), "fa fa-cogs", repoData.getName(), repoData.getDescription(), repoData.getActiontitle(), repoData.isEnabled(), repoData.getInstalledOn())) .collect(Collectors.toList()); <START> <END> }</code><technical_language>@mswiderski apply some default sort list of services</technical_language>

Target:
public List<ServiceTaskSummary> getServiceTasks() { return repoService.getServices().stream() .map(repoData -> new ServiceTaskSummary(repoData.getId(), "fa fa-cogs", repoData.getName(), repoData.getDescription(), repoData.getActiontitle(), repoData.isEnabled(), repoData.getInstalledOn())) .sorted((service, service2) -> service.getName().compareTo(service2.getName())) .collect(Collectors.toList()); }