correct definition of member function getSogoUrl()

[guggstduhier]

Hallo Herr Bieling, eine Kleinigkeit, die mir gerade auffiel.... Da ich noch nicht sehr sicher mit Git und Hub bin, bitte ich Fehler beim Pullrequest zu entschuldigen. Es wird besser werden... :-) Viele Grüße, Andreas Amann

[jobisoft]

Alles richtig gemacht. Ich könnte jetzt noch sagen "aber ich mag keine Leerzeichen vor und nach dem Gleichheitszeichen" und dann könntest du das bei dir korrigieren und commiten und diese Änderung ist dann automatisch Teil des PullRequest. Die geänderte Version akzeptiere ich dann, vielleicht :-)

[guggstduhier]

Ok. Leerzeichen vor dem '=' konsequent entfernt. :-)

[guggstduhier]

Mit Leerzeichen sieht es besser aus.... :-) Ich schließe den Pullrequest jetzt. Vielen Dank für die kleine Übungsstunde...

[jobisoft]

Jetzt waren sie zu schnell - ich wollte natürlich jetzt antworten "dass das ohne Leerzeichen doch nicht so schick aussieht" und sie dann um einen revert ihres letzten commits bitten (und erklären wie das geht). Haben Sie "von Hand" reverted oder die richtige revert Funktion von github desktop benutzt?

Ich mach den PullRequest mal wieder auf, weil es ja tatsächlich ein Fehler ist, den Sie da gefunden haben.

[jobisoft]

Oh, und ich sehe gerade, dass sie weitere Änderungen gemacht haben, die aber nicht zu dem PullRequest gehören. Um dass zu umgehen, kann man mit mehreren Branches arbeiten.

[guggstduhier]

Naja. Ich wollte Ihren Issuetracker nicht unnötig "vollspammen" und Ihre Zeit übermäßig in Anspruch nehmen. Ist ja schon sehr nett, dass Sie mir altem Mann da ein wenig unter die Arme greifen!! Ja, der Workflow ist bei mir noch nicht wirklich verinnerlicht.... :-( Die Sache mit den Branches habe ich mir gestern schon mal angesehen und auch, was zu tun ist, um mit dem Upstream (Ihr Repo) nicht allzu weit auseinander zu laufen. So langsam verschwinden ein paar Fragezeichen, aber neue kommen hinzu... Ich übe noch. Das wichtigste werde ich wohl hinbekommen, auch wenn es holpert und manchmal "zu Fuß" geht. Apropos: "Du" wäre von meiner Seite aus absolut ok...

[guggstduhier]

Mir sind in der Console noch drei JS Strict Warnings aufgefallen, die ich zusätzlich beseitigt habe. Ich hoffe, 'let' ist an den Stellen jeweils richtig, zumindest habe ich keine Verwendung der Variablen ausserhalb der umschließenden Blöcke gesehen...

[jobisoft]

Uih, dann brauche ich Nachhilfe! Wie hast du es geschafft, auf der Console Javascript-Fehler zu sehen? Ich sehe nur meine dumps!

Wenn ich Console meine, dann das "schwarze Fenster", dass durch Aufrufen von "thunderbird.exe -console" angezeigt wird. Gibts noch was anderes?

[guggstduhier]

Ah, das beruhigt ein bischen, dass Du auch nicht alles weißt und ich auch mal aushelfen kann. Ich kämpfe mich immer noch in Deinen Code ein und bin nach wie vor der Meinung, dass ich das so noch lange nicht hinbekommen hätte... ;-)

javascript.options.strict = true (siehe: https://developer.mozilla.org/de/docs/Einrichten_einer_Entwicklungsumgebung_f%C3%BCr_Erweiterungen#Entwicklungseinstellungen)

(OT: Habe mal ein bischen mit dem vcf-export gespielt... Habe Dir gerade eine Mail dazu geschrieben...)

[jobisoft]

bei mir war nicht mal "javascript.options.showInConsole" an - das macht das Finden von javascript Fehlern SO viel einfacher. Danke!

Achja, es ist and er zeit den pull request zu akzeptieren!