You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openmeetings.apache.org by so...@apache.org on 2024/03/01 09:04:19 UTC
(openmeetings) branch master updated: [OPENMEETINGS-2783] some issues are addressed
This is an automated email from the ASF dual-hosted git repository.
solomax pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/openmeetings.git
The following commit(s) were added to refs/heads/master by this push:
new 5a81c2964 [OPENMEETINGS-2783] some issues are addressed
5a81c2964 is described below
commit 5a81c2964f3bfaa64593a202f68c62dcb579a247
Author: Maxim Solodovnik <so...@gmail.com>
AuthorDate: Fri Mar 1 16:04:09 2024 +0700
[OPENMEETINGS-2783] some issues are addressed
---
.../apache/openmeetings/core/converter/ImageConverter.java | 10 +++++++---
.../org/apache/openmeetings/db/dao/label/LabelDao.java | 5 +++--
.../java/org/apache/openmeetings/db/dao/user/UserDao.java | 14 +++++++-------
.../apache/openmeetings/mediaserver/KurentoHandler.java | 3 +--
.../service/mail/template/InvitationTemplate.html | 6 +-----
.../service/mail/template/RegisterUserTemplate.html | 4 +---
.../service/mail/template/ResetPasswordTemplate.html | 6 +-----
.../mail/template/subject/InvitedAppointmentTemplate.java | 2 +-
.../apache/openmeetings/web/pages/auth/SignInDialog.java | 4 +++-
.../openmeetings/web/pages/install/InstallWizard.java | 2 +-
.../org/apache/openmeetings/web/db/dao/TestMessageDao.java | 2 +-
11 files changed, 27 insertions(+), 31 deletions(-)
diff --git a/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/ImageConverter.java b/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/ImageConverter.java
index ce92aad1d..02f374d7e 100644
--- a/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/ImageConverter.java
+++ b/openmeetings-core/src/main/java/org/apache/openmeetings/core/converter/ImageConverter.java
@@ -33,7 +33,9 @@ import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStream;
+import java.nio.file.DirectoryStream;
import java.nio.file.Files;
+import java.nio.file.Path;
import java.util.List;
import java.util.Optional;
import java.util.function.DoubleConsumer;
@@ -93,10 +95,12 @@ public class ImageConverter extends BaseConverter {
ProcessResultList returnMap = new ProcessResultList();
// User Profile Update
- Files.newDirectoryStream(
+ try (DirectoryStream<Path> dirStream = Files.newDirectoryStream(
getUploadProfilesUserDir(userId).toPath()
- , fi -> fi.toString().endsWith(EXTENSION_PNG))
- .forEach(path -> FileUtils.deleteQuietly(path.toFile()));
+ , fi -> fi.toString().endsWith(EXTENSION_PNG)))
+ {
+ dirStream.forEach(path -> FileUtils.deleteQuietly(path.toFile()));
+ }
File destinationFile = OmFileHelper.getNewFile(getUploadProfilesUserDir(userId), PROFILE_FILE_NAME, EXTENSION_PNG);
returnMap.add(resize(file, destinationFile, 250, 250, true));
diff --git a/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/label/LabelDao.java b/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/label/LabelDao.java
index aabe63062..3b211292f 100644
--- a/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/label/LabelDao.java
+++ b/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/label/LabelDao.java
@@ -193,8 +193,9 @@ public class LabelDao implements IDataProviderDao<StringLabel>{
}
List<StringLabel> result = new ArrayList<>(labelCache.containsKey(l) ? labelCache.get(l) : new ArrayList<StringLabel>());
if (!Strings.isEmpty(search)) {
- result = result.stream().filter(o -> o != null && (o.getKey().contains(search) || o.getValue().contains(search)))
- .collect(Collectors.toList());
+ result = result.stream()
+ .filter(o -> o != null && (o.getKey().contains(search) || o.getValue().contains(search)))
+ .toList();
}
return result;
}
diff --git a/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java b/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
index 64212363e..8341562a4 100644
--- a/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
+++ b/openmeetings-db/src/main/java/org/apache/openmeetings/db/dao/user/UserDao.java
@@ -434,8 +434,8 @@ public class UserDao implements IGroupAdminDataProviderDao<User> {
return !Strings.isEmpty(login) && login.length() >= getMinLoginLength();
}
- public User getByLogin(String _login, Type type, Long domainId) {
- String login = _login == null ? null : _login.trim().toLowerCase(Locale.ROOT);
+ public User getByLogin(String inLogin, Type type, Long domainId) {
+ String login = inLogin == null ? null : inLogin.trim().toLowerCase(Locale.ROOT);
return single(fillLazy(em
, oem -> oem.createNamedQuery("getUserByLogin", User.class)
.setParameter("login", login)
@@ -448,8 +448,8 @@ public class UserDao implements IGroupAdminDataProviderDao<User> {
return getByEmail(email, User.Type.USER, null);
}
- public User getByEmail(String _email, User.Type type, Long domainId) {
- String email = _email == null ? null : _email.trim().toLowerCase(Locale.ROOT);
+ public User getByEmail(String inEmail, User.Type type, Long domainId) {
+ String email = inEmail == null ? null : inEmail.trim().toLowerCase(Locale.ROOT);
return single(fillLazy(em
, oem -> oem.createNamedQuery("getUserByEmail", User.class)
.setParameter(PARAM_EMAIL, email)
@@ -598,13 +598,13 @@ public class UserDao implements IGroupAdminDataProviderDao<User> {
/**
* login logic
*
- * @param _userOrEmail - login or email of the user being tested
+ * @param inUserOrEmail - login or email of the user being tested
* @param userpass - password of the user being tested
* @return User object in case of successful login
* @throws OmException in case of any issue
*/
- public User login(String _userOrEmail, String userpass) throws OmException {
- String userOrEmail = _userOrEmail == null ? null : _userOrEmail.trim().toLowerCase(Locale.ROOT);
+ public User login(String inUserOrEmail, String userpass) throws OmException {
+ String userOrEmail = inUserOrEmail == null ? null : inUserOrEmail.trim().toLowerCase(Locale.ROOT);
List<User> users = em.createNamedQuery("getUserByLoginOrEmail", User.class)
.setParameter("userOrEmail", userOrEmail)
.setParameter("type", Type.USER)
diff --git a/openmeetings-mediaserver/src/main/java/org/apache/openmeetings/mediaserver/KurentoHandler.java b/openmeetings-mediaserver/src/main/java/org/apache/openmeetings/mediaserver/KurentoHandler.java
index d3989f7fe..6cbc29a75 100644
--- a/openmeetings-mediaserver/src/main/java/org/apache/openmeetings/mediaserver/KurentoHandler.java
+++ b/openmeetings-mediaserver/src/main/java/org/apache/openmeetings/mediaserver/KurentoHandler.java
@@ -317,8 +317,7 @@ public class KurentoHandler {
return rooms.computeIfAbsent(roomId, k -> {
log.debug("Room {} does not exist. Will create now!", roomId);
Room r = roomDao.get(roomId);
- KRoom kRoom = new KRoom(r);
- return kRoom;
+ return new KRoom(r);
});
}
diff --git a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/InvitationTemplate.html b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/InvitationTemplate.html
index 6f1f93ea2..c5d69d489 100644
--- a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/InvitationTemplate.html
+++ b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/InvitationTemplate.html
@@ -21,11 +21,7 @@
<html xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-9.xsd">
<wicket:extend>
- <h2>
- <strong>
- <span wicket:id="titleLbl"></span>
- </strong>
- </h2>
+ <h2 wicket:id="titleLbl"></h2>
<span wicket:id="userLbl"></span> <span wicket:id="user"></span><br />
<span wicket:id="messageLbl"></span> <span wicket:id="message"></span>
<br /><br />
diff --git a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/RegisterUserTemplate.html b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/RegisterUserTemplate.html
index 8707592a8..7fca3edb8 100644
--- a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/RegisterUserTemplate.html
+++ b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/RegisterUserTemplate.html
@@ -21,9 +21,7 @@
<html xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-9.xsd">
<wicket:extend>
- <h2>
- <span wicket:id="registrationLbl"></span>
- </h2>
+ <h2 wicket:id="registrationLbl"></h2>
<strong>
<wicket:message key="507" />
</strong>
diff --git a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/ResetPasswordTemplate.html b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/ResetPasswordTemplate.html
index 92baebd97..8685e55d9 100644
--- a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/ResetPasswordTemplate.html
+++ b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/ResetPasswordTemplate.html
@@ -21,11 +21,7 @@
<html xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-9.xsd">
<wicket:extend>
- <h2>
- <strong>
- <span wicket:id="reset-header"></span>
- </strong>
- </h2>
+ <h2 wicket:id="reset-header"></h2>
<p>
<wicket:message key="514" />
</p>
diff --git a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/subject/InvitedAppointmentTemplate.java b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/subject/InvitedAppointmentTemplate.java
index ac4823e81..358af85fe 100644
--- a/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/subject/InvitedAppointmentTemplate.java
+++ b/openmeetings-service/src/main/java/org/apache/openmeetings/service/mail/template/subject/InvitedAppointmentTemplate.java
@@ -28,7 +28,7 @@ public abstract class InvitedAppointmentTemplate extends AppointmentTemplate {
private static final long serialVersionUID = 1L;
protected final String invitorName;
- public InvitedAppointmentTemplate(Locale locale, Appointment a, TimeZone tz, String invitorName) {
+ protected InvitedAppointmentTemplate(Locale locale, Appointment a, TimeZone tz, String invitorName) {
super(locale, a, tz);
this.invitorName = invitorName;
}
diff --git a/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java b/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
index 962bd8d2d..51b94b59e 100644
--- a/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
+++ b/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
@@ -26,6 +26,7 @@ import static org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DEFAULT_
import static org.apache.openmeetings.util.OpenmeetingsVariables.isOtpEnabled;
import java.util.List;
+import java.util.Random;
import org.apache.openmeetings.db.dao.basic.ConfigurationDao;
import org.apache.openmeetings.db.dao.server.LdapConfigDao;
@@ -83,6 +84,7 @@ import jakarta.inject.Inject;
public class SignInDialog extends Modal<String> {
private static final long serialVersionUID = 1L;
private static final Logger log = LoggerFactory.getLogger(SignInDialog.class);
+ private static final Random rnd = new Random();
private final PasswordTextField passField = new PasswordTextField("pass", Model.of(""));
private final RequiredTextField<String> loginField = new RequiredTextField<>("login", Model.of(""));
private boolean rememberMe = false;
@@ -319,7 +321,7 @@ public class SignInDialog extends Modal<String> {
public static void penalty() {
// add random timeout
try {
- Thread.sleep(6 + (long)(10 * Math.random() * 1000));
+ Thread.sleep(6 + rnd.nextLong(10 * 1000));
} catch (InterruptedException e) {
log.error("Unexpected exception while sleeping", e);
Thread.currentThread().interrupt();
diff --git a/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java b/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
index c544135d6..1a931e1c7 100644
--- a/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
+++ b/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/install/InstallWizard.java
@@ -278,7 +278,7 @@ public class InstallWizard extends BootstrapWizard {
private final Form<ConnectionProperties> form = new Form<>("form", new CompoundPropertyModel<>(getProps(null))) {
private static final long serialVersionUID = 1L;
private final DropDownChoice<DbType> db = new DropDownChoice<>("dbType", List.of(DbType.values())
- , new LambdaChoiceRenderer<>(db -> getString("install.wizard.db.step." + db.dbName() + ".name"), DbType::name));
+ , new LambdaChoiceRenderer<>(dtb -> getString("install.wizard.db.step." + dtb.dbName() + ".name"), DbType::name));
@Override
protected void onInitialize() {
diff --git a/openmeetings-web/src/test/java/org/apache/openmeetings/web/db/dao/TestMessageDao.java b/openmeetings-web/src/test/java/org/apache/openmeetings/web/db/dao/TestMessageDao.java
index 35625acd0..73f908bbc 100644
--- a/openmeetings-web/src/test/java/org/apache/openmeetings/web/db/dao/TestMessageDao.java
+++ b/openmeetings-web/src/test/java/org/apache/openmeetings/web/db/dao/TestMessageDao.java
@@ -29,7 +29,7 @@ import org.apache.openmeetings.db.entity.user.User;
import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.annotation.Autowired;
-public class TestMessageDao extends AbstractOmServerTest {
+class TestMessageDao extends AbstractOmServerTest {
@Autowired
private PrivateMessageFolderDao msgFolderDao;