Rename enable0 -> checkForUpdates

This commit is contained in:
Risto Lahtela 2020-05-29 20:49:26 +03:00
parent 75847e2a0c
commit 00d48a7d20

View File

@ -23,6 +23,8 @@ import com.djrapitops.plan.settings.config.paths.PluginSettings;
import com.djrapitops.plan.settings.locale.Locale; import com.djrapitops.plan.settings.locale.Locale;
import com.djrapitops.plan.settings.locale.lang.PluginLang; import com.djrapitops.plan.settings.locale.lang.PluginLang;
import com.djrapitops.plan.utilities.java.Lists; import com.djrapitops.plan.utilities.java.Lists;
import com.djrapitops.plan.utilities.logging.ErrorContext;
import com.djrapitops.plan.utilities.logging.ErrorLogger;
import com.djrapitops.plugin.api.utility.Version; import com.djrapitops.plugin.api.utility.Version;
import com.djrapitops.plugin.logging.L; import com.djrapitops.plugin.logging.L;
import com.djrapitops.plugin.logging.console.PluginLogger; import com.djrapitops.plugin.logging.console.PluginLogger;
@ -47,6 +49,7 @@ public class VersionChecker implements SubSystem {
private final Locale locale; private final Locale locale;
private final PlanConfig config; private final PlanConfig config;
private final PluginLogger logger; private final PluginLogger logger;
private final ErrorLogger errorLogger;
private final PlanPlugin plugin; private final PlanPlugin plugin;
private VersionInfo newVersionAvailable; private VersionInfo newVersionAvailable;
@ -57,12 +60,14 @@ public class VersionChecker implements SubSystem {
Locale locale, Locale locale,
PlanConfig config, PlanConfig config,
PluginLogger logger, PluginLogger logger,
ErrorLogger errorLogger,
PlanPlugin plugin PlanPlugin plugin
) { ) {
this.currentVersion = currentVersion; this.currentVersion = currentVersion;
this.locale = locale; this.locale = locale;
this.config = config; this.config = config;
this.logger = logger; this.logger = logger;
this.errorLogger = errorLogger;
this.plugin = plugin; this.plugin = plugin;
} }
@ -70,7 +75,7 @@ public class VersionChecker implements SubSystem {
return newVersionAvailable != null; return newVersionAvailable != null;
} }
private void enable0() { private void checkForUpdates() {
try { try {
List<VersionInfo> versions = VersionInfoLoader.load(); List<VersionInfo> versions = VersionInfoLoader.load();
if (config.isFalse(PluginSettings.NOTIFY_ABOUT_DEV_RELEASES)) { if (config.isFalse(PluginSettings.NOTIFY_ABOUT_DEV_RELEASES)) {
@ -91,7 +96,10 @@ public class VersionChecker implements SubSystem {
logger.info(locale.getString(PluginLang.VERSION_NEWEST)); logger.info(locale.getString(PluginLang.VERSION_NEWEST));
} }
} catch (IOException e) { } catch (IOException e) {
logger.error(locale.getString(PluginLang.VERSION_FAIL_READ_VERSIONS)); errorLogger.log(L.WARN, e, ErrorContext.builder()
.related(locale.getString(PluginLang.VERSION_FAIL_READ_VERSIONS))
.whatToDo("Allow Plan to check for updates from Github/versions.txt or disable update check.")
.build());
} }
} }
@ -103,7 +111,7 @@ public class VersionChecker implements SubSystem {
plugin.getRunnableFactory().create("VersionChecker", new AbsRunnable() { plugin.getRunnableFactory().create("VersionChecker", new AbsRunnable() {
@Override @Override
public void run() { public void run() {
enable0(); checkForUpdates();
} }
}).runTaskAsynchronously(); }).runTaskAsynchronously();
} }