]> git.mxchange.org Git - friendica-addons.git/blobdiff - openstreetmap/openstreetmap.php
[various] Set config loaded from default values to SOURCE_STATIC
[friendica-addons.git] / openstreetmap / openstreetmap.php
index 26d51e1a5ebbae398bf1b858ba55df2b6a0d97e2..e83e02c27f5c315d3c36f2f9b4b4931194464b1b 100644 (file)
@@ -8,12 +8,15 @@
  * Author: Klaus Weidenbach
  *
  */
-use Friendica\Core\Addon;
-use Friendica\Core\Cache;
-use Friendica\Core\Config;
-use Friendica\Core\L10n;
-use Friendica\Core\System;
-use Friendica\Util\Network;
+
+use Friendica\App;
+use Friendica\Core\Cache\Enum\Duration;
+use Friendica\Core\Hook;
+use Friendica\Core\Logger;
+use Friendica\Core\Renderer;
+use Friendica\DI;
+use Friendica\Core\Config\Util\ConfigFileLoader;
+use Friendica\Util\Strings;
 
 const OSM_TMS = 'https://www.openstreetmap.org';
 const OSM_NOM = 'https://nominatim.openstreetmap.org/search.php';
@@ -22,37 +25,25 @@ const OSM_MARKER = 0;
 
 function openstreetmap_install()
 {
-       Addon::registerHook('load_config',     'addon/openstreetmap/openstreetmap.php', 'openstreetmap_load_config');
-       Addon::registerHook('render_location', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_location');
-       Addon::registerHook('generate_map', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_generate_map');
-       Addon::registerHook('generate_named_map', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_generate_named_map');
-       Addon::registerHook('Map::getCoordinates', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_get_coordinates');
-       Addon::registerHook('page_header', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_alterheader');
-
-       logger("installed openstreetmap");
-}
-
-function openstreetmap_uninstall()
-{
-       Addon::unregisterHook('load_config',     'addon/openstreetmap/openstreetmap.php', 'openstreetmap_load_config');
-       Addon::unregisterHook('render_location', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_location');
-       Addon::unregisterHook('generate_map', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_generate_map');
-       Addon::unregisterHook('generate_named_map', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_generate_named_map');
-       Addon::unregisterHook('Map::getCoordinates', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_get_coordinates');
-       Addon::unregisterHook('page_header', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_alterheader');
-
-       logger("removed openstreetmap");
+       Hook::register('load_config',     'addon/openstreetmap/openstreetmap.php', 'openstreetmap_load_config');
+       Hook::register('render_location', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_location');
+       Hook::register('generate_map', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_generate_map');
+       Hook::register('generate_named_map', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_generate_named_map');
+       Hook::register('Map::getCoordinates', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_get_coordinates');
+       Hook::register('page_header', 'addon/openstreetmap/openstreetmap.php', 'openstreetmap_alterheader');
+
+       Logger::notice("installed openstreetmap");
 }
 
-function openstreetmap_load_config(\Friendica\App $a)
+function openstreetmap_load_config(App $a, ConfigFileLoader $loader)
 {
-       $a->loadConfigFile(__DIR__. '/config/openstreetmap.ini.php');
+       $a->getConfigCache()->load($loader->loadAddonConfig('openstreetmap'), \Friendica\Core\Config\ValueObject\Cache::SOURCE_STATIC);
 }
 
-function openstreetmap_alterheader($a, &$navHtml)
+function openstreetmap_alterheader(App $a, &$navHtml)
 {
-       $addScriptTag = '<script type="text/javascript" src="' . $a->get_baseurl() . '/addon/openstreetmap/openstreetmap.js"></script>' . "\r\n";
-       $a->page['htmlhead'] .= $addScriptTag;
+       $addScriptTag = '<script type="text/javascript" src="' . DI::baseUrl()->get() . '/addon/openstreetmap/openstreetmap.js"></script>' . "\r\n";
+       DI::page()['htmlhead'] .= $addScriptTag;
 }
 
 /**
@@ -64,7 +55,7 @@ function openstreetmap_alterheader($a, &$navHtml)
  * @param mixed $a
  * @param array& $item
  */
-function openstreetmap_location($a, &$item)
+function openstreetmap_location(App $a, &$item)
 {
        if (!(strlen($item['location']) || strlen($item['coord']))) {
                return;
@@ -78,19 +69,19 @@ function openstreetmap_location($a, &$item)
         * ?mlat=lat&mlon=lon for markers.
         */
 
-       $tmsserver = Config::get('openstreetmap', 'tmsserver', OSM_TMS);
-       $nomserver = Config::get('openstreetmap', 'nomserver', OSM_NOM);
-       $zoom = Config::get('openstreetmap', 'zoom', OSM_ZOOM);
-       $marker = Config::get('openstreetmap', 'marker', OSM_MARKER);
+       $tmsserver = DI::config()->get('openstreetmap', 'tmsserver', OSM_TMS);
+       $nomserver = DI::config()->get('openstreetmap', 'nomserver', OSM_NOM);
+       $zoom = DI::config()->get('openstreetmap', 'zoom', OSM_ZOOM);
+       $marker = DI::config()->get('openstreetmap', 'marker', OSM_MARKER);
 
        // This is needed since we stored an empty string in the config in previous versions
        if (empty($nomserver)) {
                $nomserver = OSM_NOM;
        }
 
-       if ($item['coord'] != "") {
+       if ($item['coord'] != '') {
                $coords = explode(' ', $item['coord']);
-               if (count($coords) > 1) {
+               if ((count($coords) > 1) && is_numeric($coords[0]) && is_numeric($coords[1])) {
                        $lat = urlencode(round($coords[0], 5));
                        $lon = urlencode(round($coords[1], 5));
                        $target = $tmsserver;
@@ -105,18 +96,18 @@ function openstreetmap_location($a, &$item)
                $target = $nomserver.'?q='.urlencode($item['location']);
        }
 
-       if ($item['location'] != "") {
+       if ($item['location'] != '') {
                $title = $item['location'];
        } else {
                $title = $item['coord'];
        }
 
-       $item['html'] = '<a target="map" title="'.$title.'" href= "'.$target.'">'.$title.'</a>';
+       $item['html'] = '<a target="map" title="' . $title . '" href= "' . $target . '">' . $title . '</a>';
 }
 
-function openstreetmap_get_coordinates($a, &$b)
+function openstreetmap_get_coordinates(App $a, array &$b)
 {
-       $nomserver = Config::get('openstreetmap', 'nomserver', OSM_NOM);
+       $nomserver = DI::config()->get('openstreetmap', 'nomserver', OSM_NOM);
 
        // This is needed since we stored an empty string in the config in previous versions
        if (empty($nomserver)) {
@@ -125,14 +116,14 @@ function openstreetmap_get_coordinates($a, &$b)
 
        $args = '?q=' . urlencode($b['location']) . '&format=json';
 
-       $cachekey = "openstreetmap:" . $b['location'];
-       $j = Cache::get($cachekey);
+       $cachekey = 'openstreetmap:' . $b['location'];
+       $j = DI::cache()->get($cachekey);
 
        if (is_null($j)) {
-               $x = Network::curl($nomserver . $args);
-               if ($x['success']) {
-                       $j = json_decode($x['body'], true);
-                       Cache::set($cachekey, $j, CACHE_MONTH);
+               $curlResult = DI::httpClient()->get($nomserver . $args);
+               if ($curlResult->isSuccess()) {
+                       $j = json_decode($curlResult->getBody(), true);
+                       DI::cache()->set($cachekey, $j, Duration::MONTH);
                }
        }
 
@@ -142,7 +133,7 @@ function openstreetmap_get_coordinates($a, &$b)
        }
 }
 
-function openstreetmap_generate_named_map(&$a, &$b)
+function openstreetmap_generate_named_map(App $a, array &$b)
 {
        openstreetmap_get_coordinates($a, $b);
 
@@ -151,22 +142,22 @@ function openstreetmap_generate_named_map(&$a, &$b)
        }
 }
 
-function openstreetmap_generate_map(&$a, &$b)
+function openstreetmap_generate_map(App $a, array &$b)
 {
-       $tmsserver = Config::get('openstreetmap', 'tmsserver', OSM_TMS);
+       $tmsserver = DI::config()->get('openstreetmap', 'tmsserver', OSM_TMS);
 
-       if (strpos(z_root(), 'https:') !== false) {
+       if (strpos(DI::baseUrl()->get(true), 'https:') !== false) {
                $tmsserver = str_replace('http:','https:',$tmsserver);
        }
 
-       $zoom = Config::get('openstreetmap', 'zoom', OSM_ZOOM);
-       $marker = Config::get('openstreetmap', 'marker', OSM_MARKER);
+       $zoom = DI::config()->get('openstreetmap', 'zoom', OSM_ZOOM);
+       $marker = DI::config()->get('openstreetmap', 'marker', OSM_MARKER);
 
        $lat = $b['lat']; // round($b['lat'], 5);
        $lon = $b['lon']; // round($b['lon'], 5);
 
-       logger('lat: ' . $lat, LOGGER_DATA);
-       logger('lon: ' . $lon, LOGGER_DATA);
+       Logger::debug('lat: ' . $lat);
+       Logger::debug('lon: ' . $lon);
 
        $cardlink = '<a href="' . $tmsserver;
 
@@ -174,7 +165,7 @@ function openstreetmap_generate_map(&$a, &$b)
                $cardlink .= '?mlat=' . $lat . '&mlon=' . $lon;
        }
 
-       $cardlink .= '#map=' . $zoom . '/' . $lat . '/' . $lon . '">' . ($b['location'] ? escape_tags($b['location']) : L10n::t('View Larger')) . '</a>';
+       $cardlink .= '#map=' . $zoom . '/' . $lat . '/' . $lon . '">' . ($b['location'] ??0? Strings::escapeHtml($b['location']) : DI::l10n()->t('View Larger')) . '</a>';
        if (empty($b['mode'])) {
                $b['html'] = '<iframe style="width:100%; height:300px; border:1px solid #ccc" src="' . $tmsserver .
                                '/export/embed.html?bbox=' . ($lon - 0.01) . '%2C' . ($lat - 0.01) . '%2C' . ($lon + 0.01) . '%2C' . ($lat + 0.01) .
@@ -184,42 +175,35 @@ function openstreetmap_generate_map(&$a, &$b)
                $b['html'] .= '<br/>' . $cardlink;
        }
 
-       logger('generate_map: ' . $b['html'], LOGGER_DATA);
+       Logger::debug('generate_map: ' . $b['html']);
 }
 
-function openstreetmap_addon_admin(&$a, &$o)
+function openstreetmap_addon_admin(App $a, string &$o)
 {
-       $t = get_markup_template("admin.tpl", "addon/openstreetmap/");
-       $tmsserver = Config::get('openstreetmap', 'tmsserver', OSM_TMS);
-       $nomserver = Config::get('openstreetmap', 'nomserver', OSM_NOM);
-       $zoom = Config::get('openstreetmap', 'zoom', OSM_ZOOM);
-       $marker = Config::get('openstreetmap', 'marker', OSM_MARKER);
+       $t = Renderer::getMarkupTemplate('admin.tpl', 'addon/openstreetmap/');
+       $tmsserver = DI::config()->get('openstreetmap', 'tmsserver', OSM_TMS);
+       $nomserver = DI::config()->get('openstreetmap', 'nomserver', OSM_NOM);
+       $zoom = DI::config()->get('openstreetmap', 'zoom', OSM_ZOOM);
+       $marker = DI::config()->get('openstreetmap', 'marker', OSM_MARKER);
 
        // This is needed since we stored an empty string in the config in previous versions
        if (empty($nomserver)) {
                $nomserver = OSM_NOM;
        }
 
-       $o = replace_macros($t, [
-                       '$submit' => L10n::t('Submit'),
-                       '$tmsserver' => ['tmsserver', L10n::t('Tile Server URL'), $tmsserver, L10n::t('A list of <a href="http://wiki.openstreetmap.org/wiki/TMS" target="_blank">public tile servers</a>')],
-                       '$nomserver' => ['nomserver', L10n::t('Nominatim (reverse geocoding) Server URL'), $nomserver, L10n::t('A list of <a href="http://wiki.openstreetmap.org/wiki/Nominatim" target="_blank">Nominatim servers</a>')],
-                       '$zoom' => ['zoom', L10n::t('Default zoom'), $zoom, L10n::t('The default zoom level. (1:world, 18:highest, also depends on tile server)')],
-                       '$marker' => ['marker', L10n::t('Include marker on map'), $marker, L10n::t('Include a marker on the map.')],
+       $o = Renderer::replaceMacros($t, [
+                       '$submit' => DI::l10n()->t('Submit'),
+                       '$tmsserver' => ['tmsserver', DI::l10n()->t('Tile Server URL'), $tmsserver, DI::l10n()->t('A list of <a href="http://wiki.openstreetmap.org/wiki/TMS" target="_blank" rel="noopener noreferrer">public tile servers</a>')],
+                       '$nomserver' => ['nomserver', DI::l10n()->t('Nominatim (reverse geocoding) Server URL'), $nomserver, DI::l10n()->t('A list of <a href="http://wiki.openstreetmap.org/wiki/Nominatim" target="_blank" rel="noopener noreferrer">Nominatim servers</a>')],
+                       '$zoom' => ['zoom', DI::l10n()->t('Default zoom'), $zoom, DI::l10n()->t('The default zoom level. (1:world, 18:highest, also depends on tile server)')],
+                       '$marker' => ['marker', DI::l10n()->t('Include marker on map'), $marker, DI::l10n()->t('Include a marker on the map.')],
        ]);
 }
 
-function openstreetmap_addon_admin_post(&$a)
+function openstreetmap_addon_admin_post(App $a)
 {
-       $urltms = defaults($_POST, 'tmsserver', OSM_TMS);
-       $urlnom = defaults($_POST, 'nomserver', OSM_NOM);
-       $zoom = defaults($_POST, 'zoom', OSM_ZOOM);
-       $marker = defaults($_POST, 'marker', OSM_MARKER);
-
-       Config::set('openstreetmap', 'tmsserver', $urltms);
-       Config::set('openstreetmap', 'nomserver', $urlnom);
-       Config::set('openstreetmap', 'zoom', $zoom);
-       Config::set('openstreetmap', 'marker', $marker);
-
-       info(L10n::t('Settings updated.') . EOL);
+       DI::config()->set('openstreetmap', 'tmsserver', ($_POST['tmsserver'] ?? '') ?: OSM_TMS);
+       DI::config()->set('openstreetmap', 'nomserver', ($_POST['nomserver'] ?? '') ?: OSM_NOM);
+       DI::config()->set('openstreetmap', 'zoom', ($_POST['zoom'] ?? '') ?: OSM_ZOOM);
+       DI::config()->set('openstreetmap', 'marker', ($_POST['marker'] ?? '') ?: OSM_MARKER);
 }