Added null checks and made requested changes

Author: jasonlessenich

src/main/java/net/javadiscord/javabot/systems/notification/GuildNotificationService.java

@@ -1,5 +1,6 @@
 package net.javadiscord.javabot.systems.notification;
 
+import lombok.extern.slf4j.Slf4j;
 import net.dv8tion.jda.api.entities.Guild;
 import net.dv8tion.jda.api.entities.MessageEmbed;
 import net.javadiscord.javabot.Bot;
@@ -8,6 +9,7 @@
 /**
  * Sends notifications within a single {@link Guild}.
  */
+@Slf4j
 public final class GuildNotificationService extends NotificationService {
 
 	private final Guild guild;
@@ -18,19 +20,57 @@ public GuildNotificationService(Guild guild) {
 		this.config = Bot.config.get(guild);
 	}
 
+	/**
+	 * Sends a {@link MessageEmbed} to the Guild's log channel.
+	 *
+	 * @param embed The {@link MessageEmbed} to send.
+	 */
 	public void sendLogChannelNotification(MessageEmbed embed) {
+		if (config.getModeration().getLogChannel() == null) {
+			log.warn("Could not find Log Channel for Guild {}", guild.getName());
+			return;
+		}
 		this.sendMessageChannelNotification(config.getModeration().getLogChannel(), embed);
 	}
 
+	/**
+	 * Sends a simple Message to the Guild's log channel.
+	 *
+	 * @param string The message that should be sent.
+	 * @param args Optional args for formatting.
+	 */
 	public void sendLogChannelNotification(String string, Object... args) {
+		if (config.getModeration().getLogChannel() == null) {
+			log.warn("Could not find Log Channel for Guild {}", guild.getName());
+			return;
+		}
 		this.sendMessageChannelNotification(config.getModeration().getLogChannel(), string, args);
 	}
 
+	/**
+	 * Sends a {@link MessageEmbed} to the Guild's message log channel.
+	 *
+	 * @param embed The {@link MessageEmbed} to send.
+	 */
 	public void sendMessageLogChannelNotification(MessageEmbed embed) {
+		if (config.getMessageCache().getMessageCacheLogChannel() == null) {
+			log.warn("Could not find Message Log Channel for Guild {}", guild.getName());
+			return;
+		}
 		this.sendMessageChannelNotification(config.getMessageCache().getMessageCacheLogChannel(), embed);
 	}
 
+	/**
+	 * Sends a simple Message to the Guild's message log channel.
+	 *
+	 * @param string The message that should be sent.
+	 * @param args Optional args for formatting.
+	 */
 	private void sendMessageLogChannelNotification(String string, Object... args) {
+		if (config.getMessageCache().getMessageCacheLogChannel() == null) {
+			log.warn("Could not find Message Log Channel for Guild {}", guild.getName());
+			return;
+		}
 		this.sendMessageChannelNotification(config.getMessageCache().getMessageCacheLogChannel(), string, args);
 	}
 }

src/main/java/net/javadiscord/javabot/systems/notification/QOTWNotificationService.java

@@ -46,7 +46,7 @@ public QOTWNotificationService(User user, Guild guild) {
 			QuestionPointsRepository dao = new QuestionPointsRepository(connection);
 			account = dao.getAccountByUserId(user.getIdLong());
 		} catch (SQLException e) {
-			e.printStackTrace();
+			log.error("Could not find Account with user Id: {}", user.getIdLong(), e);
 			account = null;
 		}
 		this.account = account;

src/main/java/net/javadiscord/javabot/systems/qotw/QOTWJob.java

@@ -45,7 +45,7 @@ protected void execute(JobExecutionContext context, JDA jda) throws JobExecution
 				} else {
 					QOTWQuestion question = nextQuestion.get();
 					QOTWConfig qotw = config.getQotw();
-					qotw.getSubmissionChannel().getThreadChannels().forEach(thread -> thread.getManager().setArchived(true).queue());
+					qotw.getSubmissionChannel().getThreadChannels().forEach(thread -> thread.getManager().setLocked(true).queue());
 					qotw.getSubmissionChannel().getManager()
 							.putRolePermissionOverride(guild.getIdLong(), Collections.singleton(Permission.MESSAGE_SEND_IN_THREADS), Collections.emptySet())
 							.queue();