Skip to content

Commit

Permalink
removed unnecessary semicolons (except after enum constants)
Browse files Browse the repository at this point in the history
  • Loading branch information
virgo47 committed Mar 8, 2020
1 parent bc00e55 commit dbc04d9
Show file tree
Hide file tree
Showing 30 changed files with 31 additions and 33 deletions.
Expand Up @@ -62,7 +62,7 @@ protected IColumn<SelectableBean<O>, String> createCheckboxColumn() {
protected void onUpdateRow(AjaxRequestTarget target, DataTable table, IModel<SelectableBean<O>> rowModel, IModel<Boolean> selected) {
super.onUpdateRow(target, table, rowModel, selected);
onUpdateCheckbox(target, rowModel);
};
}

@Override
protected void onUpdateHeader(AjaxRequestTarget target, boolean selected, DataTable table) {
Expand Down
Expand Up @@ -84,7 +84,7 @@ private ItemStatus getStatus(I childItem) {
public IW createWrapper(Item childItem, ItemStatus status, WrapperContext context) throws SchemaException {
return createWrapper(null, (I) childItem, status, context);

};
}

private IW createWrapper(PrismContainerValueWrapper<?> parent, I childItem, ItemStatus status, WrapperContext context) throws SchemaException {
IW itemWrapper = createWrapper(parent, childItem, status);
Expand Down
Expand Up @@ -76,7 +76,7 @@ public boolean isCreateIfEmpty() {

public void setCreateIfEmpty(boolean createIfEmpty) {
this.createIfEmpty = createIfEmpty;
};
}

public Boolean getReadOnly() {
return readOnly;
Expand Down
Expand Up @@ -98,7 +98,7 @@ protected void populateItem(ListItem<ContainersPopupDto> item) {

protected boolean getLabelVisible() {
return false;
};
}

};
checkFormGroup.getCheck().add(new EmptyOnChangeAjaxFormUpdatingBehavior());
Expand Down
Expand Up @@ -490,7 +490,7 @@ public boolean isEnabled(){
body.add(allowTransitive);

addAjaxOnUpdateBehavior(body);
};
}

private void addPrivilegesPanel(WebMarkupContainer body){
privilegesNames = getPrivilegesNamesList();
Expand Down
Expand Up @@ -141,7 +141,7 @@ private int computeRefreshInterval() {

public int getAutoRefreshInterval() {
return 0;
};
}

public boolean isAutoRefreshEnabled() {
return false;
Expand Down
Expand Up @@ -116,7 +116,7 @@ public boolean isSecondEnabled(IModel<T> model){
}

public void firstClicked(AjaxRequestTarget target, IModel<T> model){}
public void secondClicked(AjaxRequestTarget target, IModel<T> model){};
public void secondClicked(AjaxRequestTarget target, IModel<T> model){}

public String getFirstSizeCssClass(){
return ButtonSizeClass.SMALL.toString();
Expand Down
Expand Up @@ -53,15 +53,15 @@ public void close(AjaxRequestTarget target) {
protected void onAfterRender() {
opResult.setAlreadyShown(true);
super.onAfterRender();
};
}
};
panel.add(new VisibleEnableBehaviour() {

private static final long serialVersionUID = 1L;

public boolean isVisible() {
return !opResult.isAlreadyShown();
};
}
});

panel.setOutputMarkupId(true);
Expand Down
Expand Up @@ -92,7 +92,7 @@ public String getExcludeOid() {

public IModel<String> getLabel() {
return createStringResource("FocusType.delegations");
};
}

@Override
protected List<InlineMenuItem> createAssignmentMenu() {
Expand Down
Expand Up @@ -80,7 +80,7 @@ public void onClick(AjaxRequestTarget target) {
public void yesPerformed(AjaxRequestTarget target) {
getPageBase().getCounterManager().removeCounter(counter);
target.add(InternalsCountersPanel.this);
};
}
};
getPageBase().showMainPopup(confirmPanel, target);
target.add(InternalsCountersPanel.this);
Expand Down
Expand Up @@ -63,7 +63,7 @@ protected void onError(AjaxRequestTarget target) {
}
};
form.add(update);
};
}

private void updateDebugPerformed(AjaxRequestTarget target){
getModelObject().saveDebugUtil();
Expand Down
Expand Up @@ -83,7 +83,7 @@ protected void onError(AjaxRequestTarget target) {
}
};
form.add(update);
};
}

private void updateTraces(AjaxRequestTarget target){
for (Entry<String, Boolean> entry: tracesMap.entrySet()) {
Expand Down
Expand Up @@ -29,7 +29,7 @@ public class PasswordQuestionsDto implements Serializable{
}*/
public PasswordQuestionsDto(){

};
}

public PasswordQuestionsDto(String passwQuestion, String passAnswer){
this.passwQuestion = passwQuestion;
Expand Down
Expand Up @@ -227,7 +227,7 @@ protected void runConfirmPerformed(AjaxRequestTarget target, ReportType reportTy
PageReports.this.runConfirmPerformed(target, reportType, reportParam);
hideMainPopup(target);

};
}
};
showMainPopup(runReportPopupPanel, target);

Expand Down
Expand Up @@ -525,7 +525,7 @@ private List<TaskType> getTasksForKind(List<PrismObject<TaskType>> tasks) {

protected void initCustomLayout() {
// Nothing to do, for subclass extension
};
}

protected ObjectQuery createQuery() {
ObjectQuery baseQuery = null;
Expand Down
Expand Up @@ -855,7 +855,7 @@ protected String getNullValidDisplayValue(){

protected void onUpdate(AjaxRequestTarget target) {
refreshAll(target);
};
}

});

Expand Down
Expand Up @@ -203,7 +203,7 @@ private void initLayout(){

public boolean isVisible() {
return (level != null && level.getFormRef() != null && level.getFormRef().getOid() != null);
};
}
});

if (level != null && level.getFormRef() != null && level.getFormRef().getOid() != null) {
Expand Down
Expand Up @@ -144,7 +144,7 @@ public boolean isVisible() {

public boolean isVisible() {
return getResetPasswordPolicy().getResetMethod() == ResetMethod.SECURITY_QUESTIONS;
};
}
});

WebMarkupContainer emailContainer = new WebMarkupContainer(ID_EMAIL_CONTAINER);
Expand All @@ -161,7 +161,7 @@ public boolean isVisible() {
public boolean isVisible() {
ResetMethod resetMethod = getResetPasswordPolicy().getResetMethod();
return resetMethod == ResetMethod.SECURITY_QUESTIONS || resetMethod == ResetMethod.MAIL;
};
}
});

}
Expand Down
Expand Up @@ -112,7 +112,7 @@ public boolean isVisible() {

public boolean isVisible() {
return oldPasswordVisible;
};
}
});

PasswordPanel passwordPanel = new PasswordPanel(ID_PASSWORD_PANEL, new PropertyModel<>(model, MyPasswordsDto.F_PASSWORD));
Expand Down
Expand Up @@ -99,7 +99,7 @@ protected void initLayout() {

public IModel<?> getBody() {
return searchTypes.get(selectedSearchType);
};
}

@Override
protected void onSubmit(AjaxRequestTarget target) {
Expand Down
Expand Up @@ -102,7 +102,7 @@ protected AuthenticationProvider getProvider(AbstractCredentialAuthenticationMod
}

return getObjectObjectPostProcessor().postProcess(createProvider(usedPolicy));
};
}

protected abstract ModuleAuthentication createEmptyModuleAuthentication(AbstractAuthenticationModuleType moduleType, C configuration);

Expand Down
Expand Up @@ -123,7 +123,7 @@ protected AuthenticationProvider getProvider(AuthenticationModuleLdapType module
getObjectObjectPostProcessor().postProcess(provider);

return provider;
};
}

protected ModuleWebSecurityConfig createModule(ModuleWebSecurityConfiguration configuration) {
return getObjectObjectPostProcessor().postProcess(new LdapWebSecurityConfig((LdapModuleWebSecurityConfiguration) configuration));
Expand Down
Expand Up @@ -17,7 +17,7 @@ public class LegacyValidationMessage {

public enum Type {
WARNING, ERROR
};
}

public Type type;
public String oid;
Expand Down
Expand Up @@ -36,7 +36,7 @@ public interface LocalDefinitionStore {
@SuppressWarnings("unchecked")
default <ID extends ItemDefinition> ID findLocalItemDefinition(@NotNull QName name) {
return (ID) findLocalItemDefinition(name, ItemDefinition.class, false);
};
}

@SuppressWarnings("unchecked")
default <ID extends ItemDefinition> ID findItemDefinition(@NotNull ItemPath path) {
Expand Down
Expand Up @@ -138,6 +138,6 @@ protected Trace getLogger() {

protected String getContentType() {
return "text/html";
};
}

}
Expand Up @@ -264,7 +264,7 @@ private Object getVariableValue(String name, Mode mode) {
}
}

enum Mode { DEFAULT, OLD, ESTIMATED };
enum Mode { DEFAULT, OLD, ESTIMATED }

// NOT USED

Expand Down
Expand Up @@ -360,7 +360,7 @@ void assertsAfterImmediateExecutionFinished(CaseType task, OperationResult resul
void assertsRootCaseFinishes(CaseType aCase, List<CaseType> subcases, Task opTask,
OperationResult result) throws Exception { }
boolean decideOnApproval(CaseType subcase, ApprovalContextType wfContext) throws Exception { return true; }
String getObjectOid(CaseType task, OperationResult result) throws SchemaException { return null; };
String getObjectOid(CaseType task, OperationResult result) throws SchemaException { return null; }
boolean removeAssignmentsBeforeTest() { return true; }
}

Expand Down
Expand Up @@ -77,7 +77,7 @@ public class ProvisioningUtil {
private static final QName FAKE_SCRIPT_ARGUMENT_NAME = new QName(SchemaConstants.NS_C, "arg");
public static final Duration DEFAULT_OPERATION_RETRY_PERIOD_DURATION = XmlTypeConverter.createDuration("PT30M");
public static final int DEFAULT_OPERATION_RETRY_MAX_ATTEMPTS = 3;
private static final Duration DEFAULT_PENDING_OPERATION_RETENTION_PERIOD_DURATION = XmlTypeConverter.createDuration("P1D");;
private static final Duration DEFAULT_PENDING_OPERATION_RETENTION_PERIOD_DURATION = XmlTypeConverter.createDuration("P1D");
public static final Duration DEFAULT_DEAD_SHADOW_RETENTION_PERIOD_DURATION = XmlTypeConverter.createDuration("P7D");

private static final Trace LOGGER = TraceManager.getTrace(ProvisioningUtil.class);
Expand Down
Expand Up @@ -450,7 +450,7 @@ private void assertLdapObject(PrismObject<OrgType> org, ShadowKindType kind, Str
if (LDAP_OU_VIP_INTENT.equals(intent)) search = "ou="+orgName+"-vip";
}
Entry objEntry = openDJController.searchSingle(search);
assertNotNull("No LDAP entry for "+orgName, objEntry);;
assertNotNull("No LDAP entry for "+orgName, objEntry);
display("LDAP entry kind " +kind + " inten " + intent +" ldapObj", objEntry);

if (kind.equals(ShadowKindType.ENTITLEMENT)){
Expand Down
Expand Up @@ -147,8 +147,6 @@ public boolean isTestFailure() {
return isFailure;
}

;

public T clickOk() {

$(Schrodinger.byDataId("ok"))
Expand Down

0 comments on commit dbc04d9

Please sign in to comment.