PL-7445 Remove empty class comments

modules/core/src/com/haulmont/cuba/core/app/ClusterManager.java

@@ -228,6 +228,7 @@ public class ClusterManager implements ClusterManagerAPI, AppContext.Listener {
         return globalConfig.getWebContextName() + ".jgroups";
     }
 
+    @Override
     public int getActiveThreadsCount() {
         return executor.getActiveCount();
     }
@@ -294,6 +295,7 @@ public class ClusterManager implements ClusterManagerAPI, AppContext.Listener {
         return clusterStateStat.toString();
     }
 
+    @Override
     public String printMessagesStat() {
         StringBuilder messagesStats = new StringBuilder();
         for (Map.Entry<String, MessageStat> entry : messagesStat.entrySet()) {

modules/core/src/com/haulmont/cuba/core/app/EmailSender.java

@@ -59,6 +59,7 @@ public class EmailSender implements EmailSenderAPI {
         this.mailSender = mailSender;
     }
 
+    @Override
     public void sendEmail(SendingMessage sendingMessage) throws MessagingException {
         MimeMessage msg = createMimeMessage(sendingMessage);
 

modules/core/src/com/haulmont/cuba/core/app/dynamicattributes/DynamicAttributesManager.java

@@ -182,6 +182,7 @@ public class DynamicAttributesManager implements DynamicAttributesManagerAPI {
         return targetMetaClass;
     }
 
+    @Override
     public void storeDynamicAttributes(BaseGenericIdEntity entity) {
         if (persistence.isInTransaction()) {
             doStoreDynamicAttributes(entity);
@@ -196,6 +197,7 @@ public class DynamicAttributesManager implements DynamicAttributesManagerAPI {
         }
     }
 
+    @Override
     public <E extends BaseGenericIdEntity> void fetchDynamicAttributes(List<E> entities) {
         if (persistence.isInTransaction()) {
             doFetchDynamicAttributes(entities);

modules/core/src/com/haulmont/cuba/core/jmx/Emailer.java

@@ -94,6 +94,7 @@ public class Emailer implements EmailerMBean {
         return config.getSmtpStarttlsEnable();
     }
 
+    @Override
     public boolean getSmtpSslEnabled() {
         return config.getSmtpSslEnabled();
     }

modules/core/src/com/haulmont/cuba/core/sys/entitycache/EntityCacheRemoteConnection.java

@@ -97,17 +97,21 @@ public class EntityCacheRemoteConnection extends BroadcastRemoteConnection {
         processReceivedObject(object, "");
     }
 
+    @Override
     protected boolean areAllResourcesFreedOnClose() {
         return !isLocal();
     }
 
+    @Override
     protected void closeInternal() {
     }
 
+    @Override
     protected void createDisplayString() {
         this.displayString = Helper.getShortClassName(this) + "[" + serviceId.toString() + "]";
     }
 
+    @Override
     protected boolean shouldCheckServiceId() {
         return false;
     }
@@ -131,4 +135,4 @@ public class EntityCacheRemoteConnection extends BroadcastRemoteConnection {
                     '}';
         }
     }
-}
+}

modules/core/src/com/haulmont/cuba/security/sys/UserSessionManager.java

@@ -239,6 +239,7 @@ public class UserSessionManager implements UserSessionFinder {
      * @param sessionId session's ID
      * @return          session instance or null if not found
      */
+    @Override
     public UserSession findSession(UUID sessionId) {
         return sessions.get(sessionId, false);
     }

modules/core/test/com/haulmont/cuba/core/listener/TestListener.java

@@ -39,10 +39,12 @@ public class TestListener implements
 
     public static final List<String> events = new ArrayList<>();
 
+    @Override
     public void onAfterInsert(Server entity) {
         events.add("onAfterInsert: " + entity.getId());
     }
 
+    @Override
     public void onAfterUpdate(Server entity) {
         events.add("onAfterUpdate: " + entity.getId());
 
@@ -75,7 +77,8 @@ public class TestListener implements
         }
     }
 
+    @Override
     public void onAfterDelete(Server entity) {
         events.add("onAfterDelete: " + entity.getId());
     }
-}
+}

modules/core/test/com/haulmont/cuba/soft_delete/SoftDeleteMany2ManyTest.java

@@ -35,8 +35,6 @@ import java.util.HashSet;
 
 import static org.junit.Assert.*;
 
-/**
- */
 public class SoftDeleteMany2ManyTest {
 
     @ClassRule

modules/core/test/com/haulmont/cuba/testmodel/many2many/Many2ManyA.java

@@ -23,8 +23,6 @@ import com.haulmont.cuba.core.global.DeletePolicy;
 import javax.persistence.*;
 import java.util.Set;
 
-/**
- */
 @Entity(name = "test$Many2ManyA")
 @Table(name = "TEST_MANY2MANY_A")
 public class Many2ManyA extends StandardEntity {

modules/core/test/com/haulmont/cuba/testmodel/many2many/Many2ManyB.java

@@ -25,8 +25,6 @@ import javax.persistence.ManyToMany;
 import javax.persistence.Table;
 import java.util.Set;
 
-/**
- */
 @Entity(name = "test$Many2ManyB")
 @Table(name = "TEST_MANY2MANY_B")
 public class Many2ManyB extends StandardEntity {

modules/desktop/src/com/haulmont/cuba/desktop/gui/components/DesktopAbstractUploadComponent.java

@@ -95,10 +95,12 @@ public abstract class DesktopAbstractUploadComponent<T extends JComponent>
         return dropZone;
     }
 
+    @Override
     public String getDropZonePrompt() {
         return dropZonePrompt;
     }
 
+    @Override
     public void setDropZonePrompt(String dropZonePrompt) {
         this.dropZonePrompt = dropZonePrompt;
     }

modules/desktop/src/com/haulmont/cuba/desktop/gui/components/DesktopTokenList.java

@@ -650,11 +650,13 @@ public class DesktopTokenList extends DesktopAbstractField<JPanel> implements To
             return editable;
         }
 
+        @Override
         public void setEnabled(boolean enabled) {
             this.enabled = enabled;
             removeButton.setEnabled(enabled);
         }
 
+        @Override
         public boolean isEnabled() {
             return enabled;
         }

modules/desktop/src/com/haulmont/cuba/desktop/sys/vcl/DatePicker/DatePicker.java

@@ -52,6 +52,7 @@ public class DatePicker extends JXDatePicker {
         setUI(new CustomDatePickerUI());
     }
 
+    @Override
     public void setEditor(final JFormattedTextField editor) {
         final int ENTER_CODE = 10;
 
@@ -89,6 +90,7 @@ public class DatePicker extends JXDatePicker {
             setFormats(format);
     }
 
+    @Override
     public void setLinkDay(Date linkDay) {
         MessageFormat todayFormat = new MessageFormat(AppBeans.get(Messages.class).getMessage("com.haulmont.cuba.desktop", "DatePicker.linkFormat"));
         todayFormat.setFormat(0, new SimpleDateFormat(Datatypes.getFormatStrings(AppBeans.get(UserSessionSource.class).getLocale()).getDateFormat()));
@@ -96,6 +98,7 @@ public class DatePicker extends JXDatePicker {
         super.setLinkDay(linkDay);
     }
 
+    @Override
     public void setFormats(String... formats) {
         super.setFormats(formats);
         format = formats[0];
@@ -108,6 +111,7 @@ public class DatePicker extends JXDatePicker {
             getEditor().setValue(prevVal);
     }
 
+    @Override
     public void setFormats(DateFormat... formats) {
         if (formats != null) {
             Contract.asNotNull(formats, "the array of formats " + "must not contain null elements");
@@ -133,6 +137,7 @@ public class DatePicker extends JXDatePicker {
         return mask.toString();
     }
 
+    @Override
     public DateFormat[] getFormats() {
         if (getEditor() != null) {
             return super.getFormats();
@@ -141,6 +146,7 @@ public class DatePicker extends JXDatePicker {
     }
 
     public class CustomDatePickerFormatter extends DatePickerFormatter {
+        @Override
         public void install(final JFormattedTextField ftf) {
             try {
                 if (valueToString(ftf.getValue()) == null && format != null) {
@@ -158,6 +164,7 @@ public class DatePicker extends JXDatePicker {
             super(formats, locale);
         }
 
+        @Override
         public Object stringToValue(String text) throws ParseException {
             if (text == null || text.trim().length() == 0 || ObjectUtils.equals(getMask(format), text)) {
                 return null;

modules/desktop/src/com/haulmont/cuba/desktop/sys/vcl/DatePicker/DatePickerDocument.java

@@ -162,6 +162,7 @@ public class DatePickerDocument extends PlainDocument {
                                     result.setCharAt(i + shift + 1, '0');
                                 else result.insert(i + shift + 1, '0');
                                 SwingUtilities.invokeLater(new Runnable() {
+                                    @Override
                                     public void run() {
                                         field.setCaretPosition(field.getCaretPosition() - 1);
                                     }
@@ -204,6 +205,7 @@ public class DatePickerDocument extends PlainDocument {
                                     result.setCharAt(i + shift, text.charAt(i));
                                     result.insert(i + shift + 1, '0');
                                     SwingUtilities.invokeLater(new Runnable() {
+                                        @Override
                                         public void run() {
                                             field.setCaretPosition(field.getCaretPosition() - 1);
                                         }
@@ -245,6 +247,7 @@ public class DatePickerDocument extends PlainDocument {
                             result.insert(i + shift + 1 + 1, '0');
                             shift++;
                             SwingUtilities.invokeLater(new Runnable() {
+                                @Override
                                 public void run() {
                                     field.setCaretPosition(field.getCaretPosition() - 1);
                                 }
@@ -269,6 +272,7 @@ public class DatePickerDocument extends PlainDocument {
                             result.insert(i + shift + 2, '0');
                             shift++;
                             SwingUtilities.invokeLater(new Runnable() {
+                                @Override
                                 public void run() {
                                     field.setCaretPosition(field.getCaretPosition() - 1);
                                 }
@@ -290,4 +294,4 @@ public class DatePickerDocument extends PlainDocument {
         }
         return result.toString();
     }
-}
+}

modules/desktop/src/com/haulmont/cuba/desktop/sys/vcl/SearchAutoCompleteSupport.java

@@ -180,6 +180,7 @@ import java.util.Comparator;
  * </ul>
  *
  */
+@SuppressWarnings("override")
 public final class SearchAutoCompleteSupport<E> {
 
     private static final ParsePosition PARSE_POSITION = new ParsePosition(0);

modules/desktop/test/com/haulmont/cuba/desktop/sys/vcl/VclTestApp.java

@@ -344,6 +344,7 @@ public class VclTestApp extends JFrame {
         final Timer timer = new Timer(3000, null);
         timer.addActionListener(
                 new ActionListener() {
+                    @Override
                     public void actionPerformed(ActionEvent e) {
                         if (!MouseInfo.getPointerInfo().getLocation().equals(location)) {
                             System.out.println("Hide popup " + popup);

modules/global/src/com/haulmont/cuba/core/sys/jpql/EntityPath.java

@@ -20,11 +20,6 @@ package com.haulmont.cuba.core.sys.jpql;
 import com.haulmont.cuba.core.sys.jpql.pointer.EntityPointer;
 import com.haulmont.cuba.core.sys.jpql.pointer.Pointer;
 
-/**
- * User: Alex Chevelev
- * Date: 13.10.2010
- * Time: 23:27:26
- */
 public class EntityPath {
     public String topEntityVariableName;
     public String[] traversedFields;

modules/global/src/com/haulmont/cuba/core/sys/jpql/model/AttributeImpl.java

@@ -17,11 +17,6 @@
 
 package com.haulmont.cuba.core.sys.jpql.model;
 
-/**
- * User: Alex Chevelev
- * Date: 13.10.2010
- * Time: 23:12:41
- */
 public class AttributeImpl implements Attribute {
     private final Class simpleType;
     private final String name;
@@ -83,6 +78,7 @@ public class AttributeImpl implements Attribute {
         this.userFriendlyName = userFriendlyName;
     }
 
+    @Override
     public boolean isEmbedded() {
         return isEmbedded;
     }

modules/global/src/com/haulmont/cuba/core/sys/jpql/model/EntityBuilder.java

@@ -21,11 +21,6 @@ import com.haulmont.chile.core.model.MetaClass;
 import com.haulmont.cuba.core.global.AppBeans;
 import com.haulmont.cuba.core.global.MessageTools;
 
-/**
- * User: Alex Chevelev
- * Date: 13.10.2010
- * Time: 20:41:42
- */
 public class EntityBuilder {
     private EntityImpl result;
 

modules/global/src/com/haulmont/cuba/core/sys/jpql/transform/QueryTransformerAstBased.java

@@ -264,6 +264,7 @@ public class QueryTransformerAstBased implements QueryTransformer {
         addedParams.clear();
     }
 
+    @Override
     public void replaceInCondition(String paramName) {
         getQueryTransformer().replaceInCondition(paramName);
     }

modules/gui/src/com/haulmont/cuba/gui/components/EditorWindowDelegate.java

@@ -238,6 +238,7 @@ public class EditorWindowDelegate extends WindowDelegate {
         return ((DatasourceImplementation) ds).getParent();
     }
 
+    @Override
     public boolean isModified() {
         if (readOnly)
             return false;

modules/gui/src/com/haulmont/cuba/gui/components/TokenList.java

@@ -33,6 +33,7 @@ public interface TokenList extends Field, Component.BelongToFrame, Component.Has
     CaptionMode getCaptionMode();
     void setCaptionMode(CaptionMode captionMode);
 
+    @Override
     CollectionDatasource getDatasource();
     void setDatasource(CollectionDatasource datasource);
 

modules/gui/src/com/haulmont/cuba/gui/components/Window.java

@@ -282,6 +282,7 @@ public interface Window extends Frame, Component.HasCaption {
          * Called by the framework to validate, commit and close the screen if commit was successful.
          * <p/> Passes {@link #COMMIT_ACTION_ID} to associated {@link CloseListener}s.
          */
+        @Override
         void commitAndClose();
 
         /**

modules/gui/src/com/haulmont/cuba/gui/components/filter/addcondition/ConditionDescriptorsTreeBuilder.java

@@ -44,7 +44,6 @@ import java.util.regex.Pattern;
 /**
  * Builds a {@link com.haulmont.bali.datastruct.Tree} of {@link com.haulmont.cuba.gui.components.filter.descriptor.AbstractConditionDescriptor}.
  * These descriptors are used in a new condition dialog.
- *
  */
 @Component(ConditionDescriptorsTreeBuilderAPI.NAME)
 @Scope("prototype")
@@ -79,6 +78,7 @@ public class ConditionDescriptorsTreeBuilder implements ConditionDescriptorsTree
         excludedProperties = new ArrayList<>();
     }
 
+    @Override
     public Tree<AbstractConditionDescriptor> build() {
         Messages messages = AppBeans.get(Messages.class);
         String messagesPack = filter.getFrame().getMessagesPack();
@@ -274,4 +274,4 @@ public class ConditionDescriptorsTreeBuilder implements ConditionDescriptorsTree
             return cd1.getLocCaption().compareTo(cd2.getLocCaption());
         }
     }
-}
+}

modules/gui/src/com/haulmont/cuba/gui/data/aggregation/Aggregation.java

@@ -19,10 +19,6 @@ package com.haulmont.cuba.gui.data.aggregation;
 
 import java.util.Collection;
 
-/**
- *
- * @param <T>
- */
 public interface Aggregation<T> {
 
     T sum(Collection<T> items);

modules/gui/src/com/haulmont/cuba/gui/data/impl/RuntimePropsDatasourceImpl.java

@@ -85,6 +85,7 @@ public class RuntimePropsDatasourceImpl
         throw new UnsupportedOperationException();
     }
 
+    @Override
     public MetaClass resolveCategorizedEntityClass() {
         if (categorizedEntityClass == null) {
             return mainDs.getMetaClass();

modules/gui/test/com/haulmont/cuba/gui/autocomplete/impl/HintProviderTest.java

@@ -36,11 +36,6 @@ import java.util.Set;
 import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
 
-/**
- * User: Alex Chevelev
- * Date: 13.10.2010
- * Time: 18:01:03
- */
 public class HintProviderTest {
     @Test
     public void requestHint_entityNameHint_simple() throws RecognitionException {

modules/rest-api/src/com/haulmont/cuba/restapi/ConverterHelper.java

@@ -32,6 +32,7 @@ import java.util.*;
 
 public class ConverterHelper {
     public static final Comparator<MetaProperty> PROPERTY_COMPARATOR = new Comparator<MetaProperty>() {
+        @Override
         public int compare(MetaProperty p1, MetaProperty p2) {
             return p1.getName().compareTo(p2.getName());
         }
@@ -71,4 +72,4 @@ public class ConverterHelper {
             }
         }
     }
-}
+}

modules/rest-api/src/com/haulmont/cuba/restapi/MyJSONObject.java

@@ -50,6 +50,7 @@ public class MyJSONObject implements MyJSON {
         return asString(0).toString();
     }
 
+    @Override
     public StringBuilder asString(int indent) {
         StringBuilder buf = new StringBuilder().append(OBJECT_START);
         if (!Strings.isNullOrEmpty(_id)) {
@@ -86,7 +87,6 @@ public class MyJSONObject implements MyJSON {
         return new StringBuilder().append(tabs);
     }
 
-
     String ior() {
         return _id;
     }
@@ -122,6 +122,7 @@ public class MyJSONObject implements MyJSON {
             return asString(0).toString();
         }
 
+        @Override
         public StringBuilder asString(int indent) {
             StringBuilder buf = new StringBuilder().append(ARRAY_START);
             StringBuilder tab = MyJSONObject.newIndent(indent+1);
@@ -152,6 +153,7 @@ public class MyJSONObject implements MyJSON {
             return asString(0).toString();
         }
 
+        @Override
         public StringBuilder asString(int indent) {
             StringBuilder buf = new StringBuilder().append(ARRAY_START);
             StringBuilder tab = MyJSONObject.newIndent(indent+1);

modules/web/src/com/haulmont/cuba/web/gui/components/WebAbstractUploadComponent.java

@@ -93,10 +93,12 @@ public abstract class WebAbstractUploadComponent<T extends com.vaadin.ui.Compone
         this.dropZone = dropZone;
     }
 
+    @Override
     public String getDropZonePrompt() {
         return dropZonePrompt;
     }
 
+    @Override
     public void setDropZonePrompt(String dropZonePrompt) {
         this.dropZonePrompt = dropZonePrompt;
     }

modules/web/src/com/haulmont/cuba/web/gui/components/WebTwinColumn.java

@@ -187,6 +187,7 @@ public class WebTwinColumn extends WebAbstractOptionsField<CubaTwinColSelect> im
         this.styleProvider = styleProvider;
         if (styleProvider != null) {
             component.setStyleGenerator(new CubaTwinColSelect.OptionStyleGenerator() {
+                @Override
                 public String generateStyle(AbstractSelect source, Object itemId, boolean selected) {
                     final Entity item = optionsDatasource.getItem(itemId);
                     return styleProvider.getStyleName(item, itemId, component.isSelected(itemId));

modules/web/src/com/haulmont/cuba/web/sys/singleapp/SingleAppWebContextLoader.java

@@ -97,6 +97,7 @@ public class SingleAppWebContextLoader extends WebAppContextLoader {
         filterReg.addMappingForUrlPatterns(EnumSet.of(DispatcherType.REQUEST), false, "/dispatch/*");
     }
 
+    @Override
     protected ClassPathXmlApplicationContext createClassPathXmlApplicationContext(String[] locations) {
         return new CubaClassPathXmlApplicationContext(locations) {
             /**