Merge pull request #2514 from jupadin/log-consistency

Log consistency

CHANGELOG.md

@@ -19,6 +19,9 @@ _This release is scheduled to be released on 2021-04-01._
 
 ### Fixed
 
+- Fix calendar start function logging inconsistency.
+- Fix updatenotification start function logging inconsistency.
+
 ## [2.15.0] - 2021-04-01
 
 Special thanks to the following contributors: @EdgardosReis, @MystaraTheGreat, @TheDuffman85, @ashishtank, @buxxi, @codac, @fewieden, @khassel, @klaernie, @qu1que, @rejas, @sdetweil & @thomasrockhu.
@@ -82,7 +85,6 @@ Special thanks to the following contributors: @EdgardosReis, @MystaraTheGreat, @
 - Fix calendar time/date adjustment when time with GMT offset is different day (#2488).
 - Fix calendar daylight savings offset calculation if recurring FULL DAY start date before 2007 (#2483).
 - Fix newsreaders template, for wrong test for nowrap in 2 places (should be if not).
-- Fix calendar start function logging inconsistency
 
 ## [2.14.0] - 2021-01-01
 

modules/default/updatenotification/updatenotification.js

@@ -5,6 +5,7 @@
  * MIT Licensed.
  */
 Module.register("updatenotification", {
+	// Define module defaults
 	defaults: {
 		updateInterval: 10 * 60 * 1000, // every 10 minutes
 		refreshInterval: 24 * 60 * 60 * 1000, // one day
@@ -15,9 +16,10 @@ Module.register("updatenotification", {
 	suspended: false,
 	moduleList: {},
 
+	// Override start method.
 	start: function () {
 		var self = this;
-		Log.log("Start updatenotification");
+		Log.info("Starting module: " + this.name);
 		setInterval(() => {
 			self.moduleList = {};
 			self.updateDom(2);
@@ -32,6 +34,7 @@ Module.register("updatenotification", {
 		}
 	},
 
+	// Override socket notification handler.
 	socketNotificationReceived: function (notification, payload) {
 		if (notification === "STATUS") {
 			this.updateUI(payload);