More lazy sunday cleanups of missing semicolons, unused vars and other small thins

This commit is contained in:
Veeck 2017-06-11 11:53:55 +02:00
parent a2464dce73
commit 96c338859b
5 changed files with 10 additions and 15 deletions

View File

@ -30,7 +30,7 @@ function createWindow() {
zoomFactor: config.zoom zoomFactor: config.zoom
}, },
backgroundColor: "#000000" backgroundColor: "#000000"
} };
// DEPRECATED: "kioskmode" backwards compatibility, to be removed // DEPRECATED: "kioskmode" backwards compatibility, to be removed
// settings these options directly instead provides cleaner interface // settings these options directly instead provides cleaner interface

View File

@ -165,8 +165,6 @@ var MM = (function() {
if( headerWrapper.length > 0 && newHeader) { if( headerWrapper.length > 0 && newHeader) {
headerWrapper[0].innerHTML = newHeader; headerWrapper[0].innerHTML = newHeader;
} }
}; };
/* hideModule(module, speed, callback) /* hideModule(module, speed, callback)
@ -219,7 +217,7 @@ var MM = (function() {
// remove lockString if set in options. // remove lockString if set in options.
if (options.lockString) { if (options.lockString) {
var index = module.lockStrings.indexOf(options.lockString) var index = module.lockStrings.indexOf(options.lockString);
if ( index !== -1) { if ( index !== -1) {
module.lockStrings.splice(index, 1); module.lockStrings.splice(index, 1);
} }

View File

@ -43,7 +43,7 @@ var Server = function(config, callback) {
app.use("/js", express.static(__dirname)); app.use("/js", express.static(__dirname));
var directories = ["/config", "/css", "/fonts", "/modules", "/vendor", "/translations", "/tests/configs"]; var directories = ["/config", "/css", "/fonts", "/modules", "/vendor", "/translations", "/tests/configs"];
var directory; var directory;
for (i in directories) { for (var i in directories) {
directory = directories[i]; directory = directories[i];
app.use(directory, express.static(path.resolve(global.root_path + directory))); app.use(directory, express.static(path.resolve(global.root_path + directory)));
} }

View File

@ -174,7 +174,6 @@ Module.register("calendar", {
var titleWrapper = document.createElement("td"), var titleWrapper = document.createElement("td"),
repeatingCountTitle = ""; repeatingCountTitle = "";
if (this.config.displayRepeatingCountTitle) { if (this.config.displayRepeatingCountTitle) {
repeatingCountTitle = this.countTitleForUrl(event.url); repeatingCountTitle = this.countTitleForUrl(event.url);
@ -496,9 +495,9 @@ Module.register("calendar", {
*/ */
broadcastEvents: function () { broadcastEvents: function () {
var eventList = []; var eventList = [];
for (url in this.calendarData) { for (var url in this.calendarData) {
var calendar = this.calendarData[url]; var calendar = this.calendarData[url];
for (e in calendar) { for (var e in calendar) {
var event = cloneObject(calendar[e]); var event = cloneObject(calendar[e]);
delete event.url; delete event.url;
eventList.push(event); eventList.push(event);

View File

@ -63,7 +63,7 @@ Module.register("weatherforecast",{
firstEvent: false, firstEvent: false,
// create a variable to hold the location name based on the API result. // create a variable to hold the location name based on the API result.
fetchedLocatioName: "", fetchedLocationName: "",
// Define required scripts. // Define required scripts.
getScripts: function() { getScripts: function() {
@ -175,7 +175,6 @@ Module.register("weatherforecast",{
row.style.opacity = 1 - (1 / steps * currentStep); row.style.opacity = 1 - (1 / steps * currentStep);
} }
} }
} }
return table; return table;
@ -184,7 +183,7 @@ Module.register("weatherforecast",{
// Override getHeader method. // Override getHeader method.
getHeader: function() { getHeader: function() {
if (this.config.appendLocationNameToHeader) { if (this.config.appendLocationNameToHeader) {
return this.data.header + " " + this.fetchedLocatioName; return this.data.header + " " + this.fetchedLocationName;
} }
return this.data.header; return this.data.header;
@ -200,10 +199,9 @@ Module.register("weatherforecast",{
if (notification === "CALENDAR_EVENTS") { if (notification === "CALENDAR_EVENTS") {
var senderClasses = sender.data.classes.toLowerCase().split(" "); var senderClasses = sender.data.classes.toLowerCase().split(" ");
if (senderClasses.indexOf(this.config.calendarClass.toLowerCase()) !== -1) { if (senderClasses.indexOf(this.config.calendarClass.toLowerCase()) !== -1) {
var lastEvent = this.firstEvent;
this.firstEvent = false; this.firstEvent = false;
for (e in payload) { for (var e in payload) {
var event = payload[e]; var event = payload[e];
if (event.location || event.geo) { if (event.location || event.geo) {
this.firstEvent = event; this.firstEvent = event;
@ -291,7 +289,7 @@ Module.register("weatherforecast",{
* argument data object - Weather information received form openweather.org. * argument data object - Weather information received form openweather.org.
*/ */
processWeather: function(data) { processWeather: function(data) {
this.fetchedLocatioName = data.city.name + ", " + data.city.country; this.fetchedLocationName = data.city.name + ", " + data.city.country;
this.forecast = []; this.forecast = [];
for (var i = 0, count = data.list.length; i < count; i++) { for (var i = 0, count = data.list.length; i < count; i++) {