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 2018/07/26 10:28:20 UTC
[openmeetings] branch 4.0.x updated: [OPENMEETINGS-1898] code
clean-up
This is an automated email from the ASF dual-hosted git repository.
solomax pushed a commit to branch 4.0.x
in repository https://gitbox.apache.org/repos/asf/openmeetings.git
The following commit(s) were added to refs/heads/4.0.x by this push:
new 65abcac [OPENMEETINGS-1898] code clean-up
new 54abcac Merge branch '4.0.x' of github.com:apache/openmeetings into 4.0.x
65abcac is described below
commit 65abcaca71ed45c6ce884dd79210b4b96bfebf32
Author: Maxim Solodovnik <so...@gmail.com>
AuthorDate: Thu Jul 26 17:27:55 2018 +0700
[OPENMEETINGS-1898] code clean-up
---
.../org/apache/openmeetings/db/entity/user/User.java | 16 ----------------
openmeetings-web/pom.xml | 1 -
.../apache/openmeetings/web/common/GeneralUserForm.html | 3 ---
.../apache/openmeetings/web/common/GeneralUserForm.java | 2 --
.../apache/openmeetings/web/pages/auth/SignInPage.java | 2 +-
5 files changed, 1 insertion(+), 23 deletions(-)
diff --git a/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/user/User.java b/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/user/User.java
index 66b1839..910cee3 100644
--- a/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/user/User.java
+++ b/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/user/User.java
@@ -273,10 +273,6 @@ public class User extends HistoricalEntity {
@Element(data = true, required = false)
private boolean forceTimeZoneCheck;
- @Column(name = "sendSMS", nullable = false)
- @Element(data = false, required = false)
- private boolean sendSMS;
-
@Column(name = "user_offers")
@Element(data = true, required = false)
private String userOffers;
@@ -518,14 +514,6 @@ public class User extends HistoricalEntity {
this.forceTimeZoneCheck = forceTimeZoneCheck;
}
- public boolean getSendSMS() {
- return sendSMS;
- }
-
- public void setSendSMS(boolean sendSMS) {
- this.sendSMS = sendSMS;
- }
-
public String getUserOffers() {
return userOffers;
}
@@ -558,10 +546,6 @@ public class User extends HistoricalEntity {
this.showContactDataToContacts = showContactDataToContacts;
}
- public String getPhoneForSMS() {
- return getSendSMS() ? getAddress().getPhone() : "";
- }
-
public Type getType() {
return type;
}
diff --git a/openmeetings-web/pom.xml b/openmeetings-web/pom.xml
index 6f8ee45..1d2331c 100644
--- a/openmeetings-web/pom.xml
+++ b/openmeetings-web/pom.xml
@@ -663,7 +663,6 @@
<dependency>
<groupId>org.apache.cxf</groupId>
<artifactId>cxf-rt-rs-client</artifactId>
- <version>${cxf.version}</version>
<scope>test</scope>
</dependency>
<!--dependency> commented for Red5 lib substitution
diff --git a/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.html b/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.html
index 100a63e..e01e6a3 100644
--- a/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.html
+++ b/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.html
@@ -41,9 +41,6 @@
<label wicket:for="address.phone"><wicket:message key="607" /></label><input type="text" wicket:id="address.phone" />
</div>
<div class="formelement">
- <label wicket:for="sendSMS"><wicket:message key="1503" /></label><input class="formcheckbox" type="checkbox" wicket:id="sendSMS" />
- </div>
- <div class="formelement">
<label wicket:for="age"><wicket:message key="138" /></label><input type="text" class="date time picker" wicket:id="age" style="width: 258px;" />
</div>
<div class="formelement">
diff --git a/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.java b/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.java
index 3209c13..f402f53 100644
--- a/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.java
+++ b/openmeetings-web/src/main/java/org/apache/openmeetings/web/common/GeneralUserForm.java
@@ -42,7 +42,6 @@ import org.apache.openmeetings.web.util.RestrictiveChoiceProvider;
import org.apache.wicket.core.request.handler.IPartialPageRequestHandler;
import org.apache.wicket.extensions.validation.validator.RfcCompliantEmailAddressValidator;
import org.apache.wicket.markup.head.IHeaderResponse;
-import org.apache.wicket.markup.html.form.CheckBox;
import org.apache.wicket.markup.html.form.ChoiceRenderer;
import org.apache.wicket.markup.html.form.DropDownChoice;
import org.apache.wicket.markup.html.form.Form;
@@ -100,7 +99,6 @@ public class GeneralUserForm extends Form<User> {
add(new DropDownChoice<>("timeZoneId", AVAILABLE_TIMEZONES));
add(new LanguageDropDown("languageId"));
add(new TextField<String>("address.phone"));
- add(new CheckBox("sendSMS"));
add(new AjaxDatePicker("age", new PropertyModel<LocalDate>(this, "age"), WebSession.get().getLocale()) {
private static final long serialVersionUID = 1L;
diff --git a/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java b/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
index 920f3e9..3e42312 100644
--- a/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
+++ b/openmeetings-web/src/main/java/org/apache/openmeetings/web/pages/auth/SignInPage.java
@@ -84,7 +84,7 @@ public class SignInPage extends BaseInitedPage {
try {
long serverId = oauthid.toLong(-1);
OAuthServer server = getBean(OAuth2Dao.class).get(serverId);
- log.debug("OAuthServer=" + server);
+ log.debug("OAuthServer={}", server);
if (server == null) {
log.warn("OAuth server id={} not found", serverId);
return;