search

eckertj / MVG-Abfahrtsmonitor

MVG-Abfahrtsmonitor Widget (Scriptable)
Apache License 2.0
12 stars 3 forks source link

Code issue #1

Closed TYL87 closed 3 years ago

TYL87 commented 3 years ago

Hi when I copy the code I get an error „ Error on line 44:27: TypeError: null is not an object (evaluating 'station.replace')“

If I tip in the Name like Marienplatz it comes to an other error „ E rror on line 203:47: TypeError: null is not an object (evaluating 'station.toString')“ do not know what I should replace. Any ideas? Thanks

eckertj commented 3 years ago

Hi!

I am very sorry, that I just saw your issue. Normally this station should be passed over the Widget Parameter. That means the Script does normally not work directly from the Scriptable App but only when you configure it as a Scriptable Widget like this:

drawing

Can you please validate, if the Widget was correctly configured?

TYL87 commented 3 years ago

Hi leider funktioniert es nicht. Bekomme ein Fehler im Line 75 aber kein Plan :D

eckertj commented 3 years ago

Welche Fehlermeldung bekommst du denn da?

TYL87 commented 3 years ago

Also wenn ich den Code genauso kopiere und in Zeile 48 Marineplatz durch Johann-Clanze-Strasse ersetzte und dann auf starten klicke, dann kommt die Fehlermeldung:2021-05-14 11:12:53: Error on line 44:27: TypeError: null is not an object (evaluating 'station.replace')

TYL87 commented 3 years ago

Und wenn ich diesen Fehler ersetzte zum Beispiel durch Partnachplatz dann kommt die Fehlermeldung: Uhrzeit...Error on line 44:33: ReferenceError: Can´t find variable: Partnachplatz

eckertj commented 3 years ago

Auf welchem Gerät benutzt du Scriptable denn? Normalerweise musst du die Haltestelle gar nicht direkt im Code setzen, sondern einfach als Parameter setzen wenn du das Widget hinzufügst (Im README steht eine Anleitung).

Wenn du es dennoch im Code setzen willst, sollte da klappen wenn du die Zeilen 48 und 49 folgendermaßen änderst:

//const station = "Marienplatz"
const station = args.widgetParameter

zu

const station = "Johann-Clanze-Straße"
//const station = args.widgetParameter
TYL87 commented 3 years ago

Ich benutze es auf dem iPhone 11 Pro Version 14.6 (Beta) ging aber schon nicht als ich 14.5 drauf hatte.

Hab Zeile 48 und 49 auch schon geändert gehabt bevor ich den Post eröffnet habe, allerdings wusste ich auch nicht mehr weiter. Hab in Zeile 52 noch die Straße hinzugefügt, aber auch da kommt der selbe Fehler: Can´t find variable: Johann-Clanze-Strasse und mit Partnachplatz hab ich das auch das auch durchgespielt und Harras

TYL87 commented 3 years ago

So sieht der Code bei mir aus. Vielleicht findest du ja den Fehler:

// Variables used by Scriptable. // These must be at the very top of the file. Do not edit. // icon-color: blue; icon-glyph: subway; /**** MVG Abfahrtsmonitor Widget von Jacob Eckert 16.02.2021 GitHub für Updates: https://github.com/eckertj/MVG-Abfahrtsmonitor Für dieses Widget gilt die Apache 2.0 Lizenz https://github.com/eckertj/MVG-Abfahrtsmonitor/blob/master/LICENSE Dieses Widget zeigt aktuelle Abfahrten von Bus, Bahn und/oder Tram des Münchner Verkehrs- und Tarifverbunds für eine definierte Haltestelle an. Die Daten stammen von mvg.de. Unterstützt iOS 14 Widget in klein, mittel und groß. Im Widget-Parameter kann die Haltestelle per Namen definiert werden. Genaue Bezeichnungen der Haltestellen können hier eingesehen werden: https://www.mvg.de/dienste/abfahrtszeiten.html Installation des Widgets:

  1. Lange auf eine App auf dem Home-Screen drücken.
  2. "Home-Bildschirm bearbeiten" auswählen.
  3. Oben auf das "+" drücken
  4. Runterscrollen bis "Scriptable" in der Liste erscheint.
  5. Gewünschte Widgetgröße auswählen.
  6. Auf das neue Widget drücken.
  7. Bei Script "MVG-Abfahrtsmonitor" auswählen.
  8. Bei When Interacting 'Run Script' auswählen.
  9. Bei Parameter gewünsche Haltestelle eingeben.
  10. Fertig. Später kann die Haltestelle angepasst werden, wenn lange auf das Widget gedrückt wird und "Scriptable/Widget bearbeiten" ausgewählt wird.

Hinweis: Es können mehrere Widgets auf den Home Screen gelegt und individuell konfiguriert werden! ****/

//Set for Debug in App //const station = "Partnachplatz" const station = args.widgetParameter

//Adds "&" to combined station and replace umlauts let clearstation = Partnachplatz(" ","&").replace("ß","ss").replace("ü","ue").replace("ä","ae").replace("ö","oe")

//Get Station ID const mvgstatID = "https://www.mvg.de/api/fahrinfo/location/queryWeb?q=" + clearstation let responseID = await new Request(mvgstatID).loadJSON()

// Store the MVG ID const mvgID = responseID.locations[0].id.toString()

//Set your preferred MVG products const footway = false const bus = true const ubahn = true const sbahn = true const tram = true const zug = false

//Get departures const mvgReq = "https://www.mvg.de/api/fahrinfo/departure/" + mvgID + "?sbahn=" + sbahn + "&ubahn=" + ubahn + "&bus=" + bus + "&tram=" + tram + "&footway" + footway + "&zug=" + zug let response = await new Request(mvgReq).loadJSON()

//Calculates Departure time function calculateTimeOffset(times) { return Math.ceil((times - Date.now()) / 60000) }

//Calculates expected departure incl. delay function calculateDeparture (delay, time) { if (delay == undefined) { return time } else { return delay+time } }

//Shorten text, if length exceeds space function truncate(text, n = 22) { return (text.length > n) ? text.substr(0, n-1) + '...' : text }

function createDateString() { const now = new Date(Date.now())

let day = (now.getDate().toString().length > 1) ? now.getDate().toString() : "0" + now.getDate().toString() let monthNumb = now.getMonth() + 1 let month = (monthNumb.toString().length > 1) ? monthNumb.toString() : "0" + monthNumb.toString() let year = now.getFullYear().toString() let hours = (now.getHours().toString().length > 1) ? now.getHours().toString() : "0" + now.getHours().toString() let minutes = (now.getMinutes().toString().length > 1) ? now.getMinutes().toString() : "0" + now.getMinutes().toString()

return "Updated: " + day + "." + month + "." + year + " - " + hours + ":" + minutes }

const widgetSize = (config.widgetFamily ? config.widgetFamily : 'large'); const widget = await createWidget()

if (!config.runInWidget) { switch(widgetSize) { case 'small': await widget.presentSmall(); break;

case 'large':
await widget.presentLarge();
break;

default:
await widget.presentMedium();

} }

Script.setWidget(widget)

function createWidget() {

let ITEMS_COUNT let HEADER_SIZE let COLUMN_HEIGHT let SPACING let PADDING let LOGO_SIZE let STATION_SIZE let DEPART_SIZE let LOGO_FONT_SIZE let STATION_FONT_SIZE let DEPART_FONT_SIZE let HEADLINE_FONT_SIZE let FOOTER_HEIGHT let FOOTER_FONT_SIZE

if (widgetSize == 'small') { ITEMS_COUNT = 4 HEADER_SIZE = 20 COLUMN_HEIGHT = 15 SPACING = 3 PADDING = SPACING LOGO_SIZE = new Size(20, COLUMN_HEIGHT) STATION_SIZE = new Size(80, COLUMN_HEIGHT) DEPART_SIZE = new Size(20, COLUMN_HEIGHT) LOGO_FONT_SIZE = 12 STATION_FONT_SIZE = 14 DEPART_FONT_SIZE = 12 HEADLINE_FONT_SIZE = 16 FOOTER_HEIGHT = 20 FOOTER_FONT_SIZE = 6 } else if (widgetSize == 'medium') { ITEMS_COUNT = 3 HEADER_SIZE = 25 COLUMN_HEIGHT = 20 SPACING = 5 PADDING = SPACING LOGO_SIZE = new Size(35, COLUMN_HEIGHT) STATION_SIZE = new Size(185, COLUMN_HEIGHT) DEPART_SIZE = new Size(60, COLUMN_HEIGHT) LOGO_FONT_SIZE = 14 STATION_FONT_SIZE = 20 DEPART_FONT_SIZE = 16 HEADLINE_FONT_SIZE = 24 FOOTER_HEIGHT = 10 FOOTER_FONT_SIZE = 8 } else { ITEMS_COUNT = 8 HEADER_SIZE = 30 COLUMN_HEIGHT = 20 SPACING = 5 PADDING = SPACING LOGO_SIZE = new Size(35, COLUMN_HEIGHT) STATION_SIZE = new Size(185, COLUMN_HEIGHT) DEPART_SIZE = new Size(60, COLUMN_HEIGHT) LOGO_FONT_SIZE = 14 STATION_FONT_SIZE = 20 DEPART_FONT_SIZE = 16 HEADLINE_FONT_SIZE = 24 FOOTER_HEIGHT = 25 FOOTER_FONT_SIZE = 8 }

// Widget const widget = new ListWidget(); widget.backgroundColor = new Color("#4562a2") widget.setPadding(PADDING, PADDING, PADDING, PADDING);

// Main stack const stack = widget.addStack(); stack.layoutVertically(); stack.centerAlignContent()

// Top stack for station headline const topStack = stack.addStack(); topStack.layoutVertically(); //topStack.centerAlignContent() topStack.size = new Size(LOGO_SIZE.width + STATION_SIZE.width + DEPART_SIZE.width + 2*SPACING, HEADER_SIZE);

const stationName = topStack.addText(station.toString()); stationName.textColor = Color.white(); stationName.leftAlignText() stationName.font = Font.boldSystemFont(HEADLINE_FONT_SIZE)

// Horizontal spacer under headline (station) string stack.addSpacer(8);

for (let i = 0; i < ITEMS_COUNT; i++) { // Will be set up with 3 columns to show line, destination and departure time const bottomStack = stack.addStack(); bottomStack.spacing = SPACING bottomStack.size = new Size(LOGO_SIZE.width + STATION_SIZE.width + DEPART_SIZE.width + 2SPACING, COLUMN_HEIGHT + 2SPACING) bottomStack.layoutHorizontally(); bottomStack.centerAlignContent()

const linestack = bottomStack.addStack();
linestack.size = LOGO_SIZE
linestack.centerAlignContent()
linestack.backgroundColor = new Color(response.departures[i].lineBackgroundColor.toString())
let lineName = linestack.addText(response.departures[i].label.toString())
lineName.font = Font.boldSystemFont(LOGO_FONT_SIZE)
lineName.textColor = Color.white()
lineName.centerAlignText()
lineName.minimumScaleFactor = 0.4

const destinationStack = bottomStack.addStack();
destinationStack.size = STATION_SIZE
destinationStack.layoutVertically()
destinationStack.bottomAlignContent()
let destinationName = destinationStack.addText(truncate(response.departures[i].destination.toString()))
destinationName.font = Font.lightSystemFont(STATION_FONT_SIZE)
destinationName.textColor = Color.white()
destinationName.leftAlignText()
destinationName.minimumScaleFactor = 0.95

const departureStack = bottomStack.addStack();
departureStack.size = DEPART_SIZE
departureStack.bottomAlignContent()

// Add ' Min' extension if we have space for that
let extension = ""
if (widgetSize == 'medium' ||  widgetSize == 'large') {
  extension = " Min"
}
let departureTime = departureStack.addText(calculateDeparture(response.departures[i].delay,calculateTimeOffset(response.departures[i].departureTime)) + extension)
departureTime.font = Font.boldSystemFont(DEPART_FONT_SIZE)
departureTime.textColor = Color.white()
departureTime.rightAlignText()
departureTime.minimumScaleFactor = 0.95

}

const updatedstack = stack.addStack(); updatedstack.bottomAlignContent() updatedstack.size = new Size(LOGO_SIZE.width + STATION_SIZE.width + DEPART_SIZE.width + 2*SPACING, FOOTER_HEIGHT) let lastUpdateTime = updatedstack.addText(createDateString()) lastUpdateTime.font = Font.lightSystemFont(8) lastUpdateTime.textColor = Color.white() lastUpdateTime.rightAlignText() lastUpdateTime.textOpacity = 0.6 lastUpdateTime.minimumScaleFactor = 0.95

return widget; }

Script.complete()

eckertj commented 3 years ago

Ja in deinem File wurden leider Änderungen gemacht, die das Script kaputt machen. Wie gesagt, für die Nutzung muss das MVG-Abfahrtsmonitor.js file nicht geändert werden! Es lässt sich alles über den Widget Parameter einstellen, da man sonst ja verschiedene Scripte für verschiedene Haltestellen benötigt. Anleitung steht sowohl im Code als auch im README.

Ich habe trotzdem mal ein hardgecodetes file für den Partnachplatz angehängt. Ich hoffe damit klappt das jetzt: TYL87_Partnachplatz.txt