Beiträge: 1.136
Themen: 39
Registriert seit: 27.08.2014
08.04.2019, 13:13
(Dieser Beitrag wurde zuletzt bearbeitet: 08.04.2019, 14:04 von itsmeJAY.)
Ich kann natürlich das Plugin einmal komplett aktualisieren. Ich würde dann noch einige kleine "Unsauberheiten" anpassen (u.a die language Dateien für die neuen Einstellungen) und das Plugin anschließend auf MyBB.com veröffentlichen. Dann kann es auch jeder andere downloaden, installen und es funktioniert auf Anhieb. Könnte sich jemand hier um die Readme kümmern?
@Gerti: Bezüglich dem Verkettungsoperator und deinem letzten Beitrag: "kann" in dem Fall schon, würde allerdings bereits etwas in der Variable stehen würdest du den Inhalt nur hinten dran hängen und nicht ersetzen. Das kann unter Umständen im weiteren Verlauf des Skripts zu Fehlermeldungen führen. In dem Fall, so wie ich es auf den ersten Blick sehe, macht es keinen Unterschied (wobei es halt ohne . sauberer ist). Ich wollte damit im Grunde nur ausdrücken, dass ich den "." nicht in die Plugin-Version übernehme (wenn ich es dann veröffentliche). Zugleich wollte ich dir die Begründung geben. Ist absolut nicht böse gemeint. Ich stehe natürlich für Kritik und Anregungen immer offen.
Beiträge: 1.721
Themen: 177
Registriert seit: 08.02.2018
MyBB-Version: 1.8.37
(08.04.2019, 13:13)itsmeJAY schrieb: Ich kann natürlich das Plugin einmal komplett aktualisieren. Ich würde dann noch einige kleine "Unsauberheiten" anpassen (u.a die language Dateien für die neuen Einstellungen) und das Plugin anschließend auf MyBB.com veröffentlichen. Dann kann es auch jeder andere downloaden, installen und es funktioniert auf Anhieb. Könnte sich jemand hier um die Readme kümmern? Um die "read.me" kümmere ich mich, denke Schnapsnase wird mir mit seiner Erfahrung dabei sicherlich behilflich sein. Ich werde die derzeit im Plugin vorhandene "Anleitung" als Grundlage übernehmen. Nach Fertigstellung (dauert einige Tage) sende ich Dir die "read.me" via PN zu damit Du sie nach Prüfung in das ZIP-File mit einbinden und dann das Plugin hochladen kannst.
Bitte in der ./inc.plugins/usermap.php die Version 1.4. 5 eintragen. Ich werde Dir bei mir vorhandene Datei (V1.4.4) zur Verfügung stellen damit Du die dortigen Änderungen mit Deiner Datei (Ausgangversion 1.4.3) verleichen und bei Bedarf die Änderungen mit einbinden kannst.
(08.04.2019, 13:13)itsmeJAY schrieb: ...Verkettungsoperator... Danke für die Erläuterung, s versteht man wenigsten das "Warum".
Sollte ich der Ansicht sein helfen zu können biete ich Hilfe(n) an! ...ich bitte jedoch nicht darum helfen zu dürfen!
Tools ◀ [ Unixzeit ⇔ Realzeit] ♦ [ BOM-Finder] ♦ [ SQL-Prefix-Changer] ♦ [ USV-Rechner] ♦ [ PlugIns]
Beiträge: 1.136
Themen: 39
Registriert seit: 27.08.2014
09.04.2019, 09:37
(Dieser Beitrag wurde zuletzt bearbeitet: 09.04.2019, 10:37 von itsmeJAY.)
Morgen,
im Grunde brauchen wir ja 3 APIs:
JavaScript Maps,
Geocoding,
Maps Elevation
Derzeit ist Key 1 für JavaScript Maps und Elevation zuständig und Key 2 für Geocoding. Ich würde das aktualisierte Plugin dahingehend anpassen, das wir für alle 3 APIs jeweils einen Key angeben können. Somit können wir im ACP 3 Keys hinterlegen. Der Benutzer kann natürlich für alle 3 APIs den selben Key verwenden, hat aber auch die Möglichkeit für jede API einen einzelnen Key zu verwenden. Was haltet ihr davon?
Grüße
/Edit: Hab soeben den Bug bezüglich der Höhe gefixxt. Die funktioniert nun auch wieder und wird korrekt angezeigt.
Anhang:
Ich würde den Bugfix dann sobald die readme fertig ist mit in die Version packen.
Beiträge: 2.267
Themen: 33
Registriert seit: 25.05.2018
MyBB-Version: 1.8.38
(09.04.2019, 09:37)itsmeJAY schrieb: Ich würde das aktualisierte Plugin dahingehend anpassen, das wir für alle 3 APIs jeweils einen Key angeben können. Das wäre nicht unbedingt nötig. Lediglich die Geocoding-API benötigt aus bekannten Gründen ihren eigenen Key.
JavaScript und Elevation können eigentlich weiterhin einen gemeinsamen Key haben.
Wer sich bei Google die Trace-Daten anschauen möchte, bekommt sie sowieso pro API einzeln angezeigt, unabhängig vom Key.
itsmeJAY schrieb:Der Benutzer kann natürlich für alle 3 APIs den selben Key verwenden,..... Ja, dann aber nicht mit einer URL-Einschränkung!
Ansonsten alles top!! Vielen Dank!
Beiträge: 1.136
Themen: 39
Registriert seit: 27.08.2014
09.04.2019, 12:00
(Dieser Beitrag wurde zuletzt bearbeitet: 09.04.2019, 12:02 von itsmeJAY.)
(09.04.2019, 11:06)Schnapsnase schrieb: (09.04.2019, 09:37)itsmeJAY schrieb: Ich würde das aktualisierte Plugin dahingehend anpassen, das wir für alle 3 APIs jeweils einen Key angeben können. Das wäre nicht unbedingt nötig. Lediglich die Geocoding-API benötigt aus bekannten Gründen ihren eigenen Key.
JavaScript und Elevation können eigentlich weiterhin einen gemeinsamen Key haben.
Wer sich bei Google die Trace-Daten anschauen möchte, bekommt sie sowieso pro API einzeln angezeigt, unabhängig vom Key.
itsmeJAY schrieb:Der Benutzer kann natürlich für alle 3 APIs den selben Key verwenden,..... Ja, dann aber nicht mit einer URL-Einschränkung!
Ansonsten alles top!! Vielen Dank!
Ich weiß das nur die Geocoding-API einen eigenen Key braucht aufgrund der Einschränkungen. Ich finde nur persönlich, es ist sauberer (programmierseitig) den Benutzer die Möglichkeit zu geben bei 3 APIs auch 3 Keys eingeben zu lassen. Ob er dann 2 mal den gleichen nimmt für die Map und Elevation, ist ja egal. Er hat zumindest die Möglichkeit so flexibel zu sein, wie er will. Er könnte auch nur einen Key (den selben) für alle 3 Felder angeben im ACP und diesen nicht einschränken, alles wie der Benutzer will.
Beiträge: 2.267
Themen: 33
Registriert seit: 25.05.2018
MyBB-Version: 1.8.38
(09.04.2019, 12:00)itsmeJAY schrieb: Ich finde nur persönlich, es ist sauberer (programmierseitig)......
Jepp, dann mach`s doch so. Schaden tut`s ja auch nicht...
Beiträge: 1.136
Themen: 39
Registriert seit: 27.08.2014
jau, sobald die readme seitens dir und Gerti fertig ist, setze ich mich dran..
Beiträge: 1.721
Themen: 177
Registriert seit: 08.02.2018
MyBB-Version: 1.8.37
(09.04.2019, 12:36)Schnapsnase schrieb: (09.04.2019, 12:00)itsmeJAY schrieb: Ich finde nur persönlich, es ist sauberer (programmierseitig)......
Jepp, dann mach`s doch so. Schaden tut`s ja auch nicht... dann jedoch bräuchten die derzeitigen Anwender der Usermap nochmals ein angepasstes Uppdate damit die derzeitigen PINs erhalten bleiben.
Anmerkung:
Nach dem Update ist im ACP das Eingabefeld zum 2. Key nach der Breiteneinstellung der Karte und vor der Höheneinstellung der Karte. Das ist unlogisch.
Die zusätzlichen Punkte 2. und 3. Key sollen direkt nach dem ersten Eingabefeld "API-Schlüssel" definiert werden.
Sollte ich der Ansicht sein helfen zu können biete ich Hilfe(n) an! ...ich bitte jedoch nicht darum helfen zu dürfen!
Tools ◀ [ Unixzeit ⇔ Realzeit] ♦ [ BOM-Finder] ♦ [ SQL-Prefix-Changer] ♦ [ USV-Rechner] ♦ [ PlugIns]
Beiträge: 1.136
Themen: 39
Registriert seit: 27.08.2014
danke, notiert für das nächste Update.
Beiträge: 1.721
Themen: 177
Registriert seit: 08.02.2018
MyBB-Version: 1.8.37
(09.04.2019, 14:08)itsmeJAY schrieb: danke, notiert für das nächste Update. gibbet vorab eine kleine Updateanleitung sowie die erforderlichen Dateien, damit von jetzt zwei Keys auf drei Kkeys erweitert werden kann? Auch sollte der 2. Key damit gleich eien Stufe höher gesetzt werden.
Oder wie kann sonst bei der bestehender/aktiver Usermap (mit 2 Key-Definitonen im ACP) auf 3 Keys updaten?
Sollte ich der Ansicht sein helfen zu können biete ich Hilfe(n) an! ...ich bitte jedoch nicht darum helfen zu dürfen!
Tools ◀ [ Unixzeit ⇔ Realzeit] ♦ [ BOM-Finder] ♦ [ SQL-Prefix-Changer] ♦ [ USV-Rechner] ♦ [ PlugIns]
|