Google Maps für Einsatzorte #114
Conversation
Development
In den Einstellungen kann dies (de)aktiviert werden. Zusätzliches Feld mit Koordinate, welche via Button bestimmt wird. Im Admin kann der Marker verschoben werden.
|
Super, das muss ich mir die Tage nochmal genauer anschauen, bevor ich es merge. Aber schon mal vielen Dank für die Arbeit :) |
…em PHP Source entfernt und in js/einsatzverwaltung-gmaps.js ausgelagert. [einsatzliste] um Parameter googlemaps=ja/nein erweitert, um am Seitenanfang eine Karte mit allen Einsätzen anzuzeigen.
|
Würde mich freuen, wenn dieses Feature gemerged wird. Meine Wehr liebäugelt auch mit diesem Plugin und da die bisherigen Einsatzverwaltung auch eine Google Maps-Integration hat, möchten sie jetzt darauf nicht mehr verzichten. |
Merge original to fork
|
Moin, |
|
Danke, dass du an dem Thema dranbleibst. Ich habe gerade mal über den Code gesehen, und mir sind noch ein paar Sachen aufgefallen. Da würde ich dann entsprechend noch Inline-Kommentare setzen, damit du weiß was ich meine. Da komme ich heute Abend aber nicht mehr dazu. Durch #130 entstand leider ein Merge Conflict, aber das sollte kein großes Problem sein. |
src/Frontend/ReportList.php
Outdated
| @@ -194,6 +204,21 @@ private function constructList($reports, $args) | |||
| $this->beginTable(false); | |||
| $this->insertTableHeader(); | |||
| } | |||
|
|
|||
| if( $this->options->isGMapActivate() != "" && $this->showMap) { | |||
There was a problem hiding this comment.
Eine Methode (isGMapActivate()), die dank toBoolean() immer einen Boolean zurückliefert, auf einen leeren String prüfen?
src/Model/IncidentReport.php
Outdated
| @@ -75,6 +75,9 @@ public static function getMetaFields() | |||
| 'einsatz_einsatzort' => array( | |||
| 'label' => 'Einsatzort' | |||
| ), | |||
| 'einsatz_location' => array( | |||
| 'label' => 'Goolemaps Position' | |||
There was a problem hiding this comment.
Google legt wert auf eine korrekte Schreibweise der Produkte: Google Maps anstatt Googlemaps, sowieso ist da ein Typo drin (das g fehlt bei Google).
src/einsatzverwaltung-admin.php
Outdated
| */ | ||
| private function echoGMap($location) | ||
| { | ||
| if($location != "") |
There was a problem hiding this comment.
Wäre empty() an dieser Stelle nicht besser geeignet?
Sollte ich jetzt mit dem Einbau in die aktuelle Version warten, oder läuft es ohne größere Probleme? |
# Conflicts: # src/einsatzverwaltung-core.php # src/einsatzverwaltung-settings.php
…on sebastianroming implementiert
|
Habe den es jetzt in Version 1.3.2 eingebaut und die Anregungen von @sebastianroming übernommen, Danke dafür ;) |
|
Wow, vielen Dank für die Anpassung! :) |
|
Huch, |
…into develop # Conflicts: # src/Frontend.php # src/Frontend/ReportList.php # src/Options.php # src/einsatzverwaltung-admin.php # src/einsatzverwaltung-core.php # src/einsatzverwaltung-data.php # src/einsatzverwaltung-settings.php # src/einsatzverwaltung-shortcodes.php # tests/Admin/ReportEditTest.php
Moin,
ich habe mir mal die Mühe gemacht und die Google Maps JavaScript API implementiert.
Diese kann in den Einsatzeinstellungen (de)aktiviert werden und ein API-Key sowie die Standardposition der Karte festgelegt werden.
Beim erstellen sowie editieren der Position wird diese via Knopfdruck anhand des Einsatzortes ermittelt und der Marker kann noch per Hand verschoben werden.
Die Markerposition ist extra gespeichert und wirkt sich nicht auf den Einsatzort aus.
Das ganze kann in Kürze auf https://www.feuerwehr-helmste.de begutachtet werden, muss aber noch mein testsystem in live mergen.
Gruß
Jan