From d55bdab24e378821d45424bda36b0b603e9c4ae0 Mon Sep 17 00:00:00 2001
From: XwZ <lugagne.jeremy@gmail.com>
Date: Thu, 9 Jul 2015 02:06:45 +0200
Subject: [PATCH 1/2] Fix loop map issue, normalize the  in request to stay in
 the api's range

---
 staticmap.php | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/staticmap.php b/staticmap.php
index a139984..819423d 100644
--- a/staticmap.php
+++ b/staticmap.php
@@ -33,6 +33,7 @@
 
     protected $maxWidth = 1024;
     protected $maxHeight = 1024;
+    protected $maxtiles = 0;
 
     protected $tileSize = 256;
     protected $tileSrcUrl = array('mapnik' => 'http://tile.openstreetmap.org/{Z}/{X}/{Y}.png',
@@ -182,6 +183,10 @@ public function initCoords()
         $this->offsetY = floor((floor($this->centerY) - $this->centerY) * $this->tileSize);
     }
 
+    public function normalizeTileCoord($position){
+	return ($position < 0) ? $this->maxtiles + $position : $position%$this->maxtiles;
+    }
+
     public function createBaseMap()
     {
         $this->image = imagecreatetruecolor($this->width, $this->height);
@@ -195,10 +200,11 @@ public function createBaseMap()
         $this->offsetY += floor($this->height / 2);
         $this->offsetX += floor($startX - floor($this->centerX)) * $this->tileSize;
         $this->offsetY += floor($startY - floor($this->centerY)) * $this->tileSize;
-
+        $this->maxtiles = sqrt(pow(2,(2*$this->zoom)));
+	
         for ($x = $startX; $x <= $endX; $x++) {
             for ($y = $startY; $y <= $endY; $y++) {
-                $url = str_replace(array('{Z}', '{X}', '{Y}'), array($this->zoom, $x, $y), $this->tileSrcUrl[$this->maptype]);
+                $url = str_replace(array('{Z}', '{X}', '{Y}'), array($this->zoom, $this->normalizeTileCoord($x), $y), $this->tileSrcUrl[$this->maptype]);
                 $tileData = $this->fetchTile($url);
                 if ($tileData) {
                     $tileImage = imagecreatefromstring($tileData);

From 94784b38706c0fe66cd2ebf83a13f6050ac3c9ed Mon Sep 17 00:00:00 2001
From: Dennis Glittenberg <glittenberg.d@gmail.com>
Date: Sun, 22 May 2016 15:04:43 +0200
Subject: [PATCH 2/2] changed Cache behaviour

Only the map is cached not the markers.
---
 staticmap.php | 43 +++++++++++++++----------------------------
 1 file changed, 15 insertions(+), 28 deletions(-)

diff --git a/staticmap.php b/staticmap.php
index a9855e5..715a6a2 100644
--- a/staticmap.php
+++ b/staticmap.php
@@ -215,6 +215,8 @@ public function createBaseMap()
                 imagecopy($this->image, $tileImage, $destX, $destY, 0, 0, $this->tileSize, $this->tileSize);
             }
         }
+        $this->mkdir_recursive(dirname($this->mapCacheIDToFilename()), 0777);
+        imagepng($this->image, $this->mapCacheIDToFilename(), 9);
     }
 
 
@@ -307,7 +309,7 @@ public function checkMapCache()
 
     public function serializeParams()
     {
-        return join("&", array($this->zoom, $this->lat, $this->lon, $this->width, $this->height, serialize($this->markers), $this->maptype));
+        return join("&", array($this->zoom, $this->lat, $this->lon, $this->width, $this->height, $this->maptype));
     }
 
     public function mapCacheIDToFilename()
@@ -367,7 +369,15 @@ public function sendHeader()
     public function makeMap()
     {
         $this->initCoords();
-        $this->createBaseMap();
+        if ($this->useMapCache){
+            if($this->checkMapCache()){
+                $this->image = imagecreatefrompng($this->mapCacheIDToFilename());
+            } else{
+                $this->createBaseMap();
+            }
+        } else{
+            $this->createBaseMap();
+        }
         if (count($this->markers)) $this->placeMarkers();
         if ($this->osmLogo) $this->copyrightNotice();
     }
@@ -375,32 +385,9 @@ public function makeMap()
     public function showMap()
     {
         $this->parseParams();
-        if ($this->useMapCache) {
-            // use map cache, so check cache for map
-            if (!$this->checkMapCache()) {
-                // map is not in cache, needs to be build
-                $this->makeMap();
-                $this->mkdir_recursive(dirname($this->mapCacheIDToFilename()), 0777);
-                imagepng($this->image, $this->mapCacheIDToFilename(), 9);
-                $this->sendHeader();
-                if (file_exists($this->mapCacheIDToFilename())) {
-                    return file_get_contents($this->mapCacheIDToFilename());
-                } else {
-                    return imagepng($this->image);
-                }
-            } else {
-                // map is in cache
-                $this->sendHeader();
-                return file_get_contents($this->mapCacheIDToFilename());
-            }
-
-        } else {
-            // no cache, make map, send headers and deliver png
-            $this->makeMap();
-            $this->sendHeader();
-            return imagepng($this->image);
-
-        }
+        $this->makeMap();
+        $this->sendHeader();
+        return imagepng($this->image);
     }
 
 }