diff --git a/src/main/java/io/kamax/mxisd/config/threepid/ThreePidConfig.java b/src/main/java/io/kamax/mxisd/config/threepid/ThreePidConfig.java index 33e6ef4..6012c61 100644 --- a/src/main/java/io/kamax/mxisd/config/threepid/ThreePidConfig.java +++ b/src/main/java/io/kamax/mxisd/config/threepid/ThreePidConfig.java @@ -20,7 +20,6 @@ package io.kamax.mxisd.config.threepid; -import com.google.gson.JsonObject; import io.kamax.matrix.ThreePidMedium; import io.kamax.matrix.json.GsonUtil; import io.kamax.mxisd.config.threepid.medium.EmailConfig; @@ -31,18 +30,18 @@ import java.util.Map; public class ThreePidConfig { - private Map medium = new HashMap<>(); + private Map medium = new HashMap<>(); - public ThreePidConfig() { + public ThreePidConfig() { // TODO Check if this is still needed medium.put(ThreePidMedium.Email.getId(), GsonUtil.makeObj(new EmailConfig())); medium.put(ThreePidMedium.PhoneNumber.getId(), GsonUtil.makeObj(new PhoneConfig())); } - public Map getMedium() { + public Map getMedium() { return medium; } - public void setMedium(Map medium) { + public void setMedium(Map medium) { this.medium = medium; } diff --git a/src/main/java/io/kamax/mxisd/threepid/notification/BuiltInNotificationHandlerSupplier.java b/src/main/java/io/kamax/mxisd/threepid/notification/BuiltInNotificationHandlerSupplier.java index 92b4de6..7c137bc 100644 --- a/src/main/java/io/kamax/mxisd/threepid/notification/BuiltInNotificationHandlerSupplier.java +++ b/src/main/java/io/kamax/mxisd/threepid/notification/BuiltInNotificationHandlerSupplier.java @@ -63,9 +63,9 @@ public class BuiltInNotificationHandlerSupplier implements NotificationHandlerSu private void acceptEmail(String handler, Mxisd mxisd) { if (StringUtils.equals(EmailRawNotificationHandler.ID, handler)) { - JsonObject emailCfgJson = mxisd.getConfig().getThreepid().getMedium().get(ThreePidMedium.Email.getId()); - if (Objects.nonNull(emailCfgJson)) { - EmailConfig emailCfg = GsonUtil.get().fromJson(emailCfgJson, EmailConfig.class); + Object o = mxisd.getConfig().getThreepid().getMedium().get(ThreePidMedium.Email.getId()); + if (Objects.nonNull(o)) { + EmailConfig emailCfg = GsonUtil.get().fromJson(GsonUtil.makeObj(o), EmailConfig.class); if (org.apache.commons.lang.StringUtils.isBlank(emailCfg.getGenerator())) { throw new ConfigurationException("notification.email.generator"); @@ -105,9 +105,9 @@ public class BuiltInNotificationHandlerSupplier implements NotificationHandlerSu private void acceptPhone(String handler, Mxisd mxisd) { if (StringUtils.equals(PhoneNotificationHandler.ID, handler)) { - JsonObject cfgJson = mxisd.getConfig().getThreepid().getMedium().get(ThreePidMedium.PhoneNumber.getId()); - if (Objects.nonNull(cfgJson)) { - PhoneConfig cfg = GsonUtil.get().fromJson(cfgJson, PhoneConfig.class); + Object o = mxisd.getConfig().getThreepid().getMedium().get(ThreePidMedium.PhoneNumber.getId()); + if (Objects.nonNull(o)) { + PhoneConfig cfg = GsonUtil.get().fromJson(GsonUtil.makeObj(o), PhoneConfig.class); List generators = StreamSupport .stream(ServiceLoader.load(PhoneGeneratorSupplier.class).spliterator(), false)