From e962fb6f47e8992bf69f011a9be1e5b9e83251fe Mon Sep 17 00:00:00 2001
From: Gustavo Martin Morcuende <gu.martinm@gmail.com>
Date: Sun, 2 Dec 2012 20:07:01 +0100
Subject: [PATCH] Google maps window: open once per click

Bonus: now you can edit directly the coordinates without opening an annoying new window every time
you click in the GPS coodinates box. After closing the goolge maps window it can be opened agais just clicking
again in the GPS coordinates box.
---
 apps/companyfront/modules/ad/templates/_form.php   | 33 +++++++++++++---------
 .../modules/admap/templates/admapSuccess.php       |  6 +++-
 .../modules/admap/templates/officemapSuccess.php   |  7 ++++-
 .../modules/office/templates/_formCustomOffice.php | 31 ++++++++++++--------
 4 files changed, 49 insertions(+), 28 deletions(-)

diff --git a/apps/companyfront/modules/ad/templates/_form.php b/apps/companyfront/modules/ad/templates/_form.php
index e8f0041..789be45 100644
--- a/apps/companyfront/modules/ad/templates/_form.php
+++ b/apps/companyfront/modules/ad/templates/_form.php
@@ -2,30 +2,35 @@
 <?php use_javascripts_for_form($form) ?>
 
 <script type="text/javascript">
+    var alreadyOpened = false;
 
     $(document).ready(function(){
-        var longitude = document.getElementById('ad_longitude');
-        var latitude = document.getElementById('ad_latitude');
-        var adMapURL = 'http://localhost/companyfront_dev.php/admap/admap?latitude='+latitude.value+'&longitude='+longitude.value;
         $('#ad_longitude').click(function() {
-        newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
-            if (window.focus) {
-                newwindow.focus()
+            if (!window.alreadyOpened) {
+                window.alreadyOpened = true;
+                var longitude = document.getElementById('ad_longitude');
+                var latitude = document.getElementById('ad_latitude');
+                var adMapURL = 'http://localhost/companyfront_dev.php/admap/admap?latitude='+latitude.value+'&longitude='+longitude.value;
+                newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
+                if (window.focus) {
+                    newwindow.focus()
+                }
             }
-            return false;
         });
     });
 
     $(document).ready(function(){
-        var longitude = document.getElementById('ad_longitude');
-        var latitude = document.getElementById('ad_latitude');
-        var adMapURL = 'http://localhost/companyfront_dev.php/admap/admap?latitude='+latitude.value+'&longitude='+longitude.value;
         $('#ad_latitude').click(function() {
-        newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
-            if (window.focus) {
-                newwindow.focus()
+            if (!window.alreadyOpened) {
+                window.alreadyOpened = true;
+                var longitude = document.getElementById('ad_longitude');
+                var latitude = document.getElementById('ad_latitude');
+                var adMapURL = 'http://localhost/companyfront_dev.php/admap/admap?latitude='+latitude.value+'&longitude='+longitude.value;
+                newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
+                if (window.focus) {
+                    newwindow.focus()
+                }
             }
-            return false;
         });
     });
 
diff --git a/apps/companyfront/modules/admap/templates/admapSuccess.php b/apps/companyfront/modules/admap/templates/admapSuccess.php
index 54be0ca..5381521 100644
--- a/apps/companyfront/modules/admap/templates/admapSuccess.php
+++ b/apps/companyfront/modules/admap/templates/admapSuccess.php
@@ -62,9 +62,13 @@
             document.body.appendChild(script);
         }
 
+        function taskOnExit() {
+            self.opener.alreadyOpened = false;
+        }
+
         window.onload = loadScript;
 
-        google.maps.event.addDomListener(window, 'load', initialize);
+        window.onbeforeunload = taskOnExit;
     </script>
   </head>
 
diff --git a/apps/companyfront/modules/admap/templates/officemapSuccess.php b/apps/companyfront/modules/admap/templates/officemapSuccess.php
index 4f23690..a8737a6 100644
--- a/apps/companyfront/modules/admap/templates/officemapSuccess.php
+++ b/apps/companyfront/modules/admap/templates/officemapSuccess.php
@@ -62,9 +62,14 @@
             document.body.appendChild(script);
         }
 
+        function taskOnExit() {
+            self.opener.alreadyOpened = false;
+        }
+
         window.onload = loadScript;
 
-        google.maps.event.addDomListener(window, 'load', initialize);
+        window.onbeforeunload = taskOnExit;
+
     </script>
   </head>
 
diff --git a/apps/companyfront/modules/office/templates/_formCustomOffice.php b/apps/companyfront/modules/office/templates/_formCustomOffice.php
index f2f5054..b5793b3 100644
--- a/apps/companyfront/modules/office/templates/_formCustomOffice.php
+++ b/apps/companyfront/modules/office/templates/_formCustomOffice.php
@@ -34,27 +34,34 @@
 </script>
 
 <script type="text/javascript">
+    var alreadyOpened = false;
 
     $(document).ready(function(){
-        var longitude = document.getElementById('office_longitude');
-        var latitude = document.getElementById('office_latitude');
-        var adMapURL = 'http://localhost/companyfront_dev.php/admap/officemap?latitude='+latitude.value+'&longitude='+longitude.value;
         $('#office_longitude').click(function() {
-            newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
-            if (window.focus) {
-                newwindow.focus()
+            if (!window.alreadyOpened) {
+                window.alreadyOpened = true;
+                var longitude = document.getElementById('office_longitude');
+                var latitude = document.getElementById('office_latitude');
+                var adMapURL = 'http://localhost/companyfront_dev.php/admap/officemap?latitude='+latitude.value+'&longitude='+longitude.value;
+                newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
+                if (window.focus) {
+                    newwindow.focus()
+                }
             }
         });
     });
 
     $(document).ready(function(){
-        var longitude = document.getElementById('office_longitude');
-        var latitude = document.getElementById('office_latitude');
-        var adMapURL = 'http://localhost/companyfront.php/admap/officemap?latitude='+latitude.value+'&longitude='+longitude.value;
         $('#office_latitude').click(function() {
-            newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
-            if (window.focus) {
-                newwindow.focus()
+            if (!window.alreadyOpened) {
+                window.alreadyOpened = true;
+                var longitude = document.getElementById('office_longitude');
+                var latitude = document.getElementById('office_latitude');
+                var adMapURL = 'http://localhost/companyfront_dev.php/admap/officemap?latitude='+latitude.value+'&longitude='+longitude.value;
+                newwindow=window.open(adMapURL, '', 'menubar=no,height=600,width=600');
+                if (window.focus) {
+                    newwindow.focus()
+                }
             }
         });
     });
-- 
2.1.4