From c5ef4d38613199d7a4fde0d776e5a33f762238bc Mon Sep 17 00:00:00 2001
From: Eugene Burmakin <iamfrey@gmail.com>
Date: Sat, 6 Dec 2025 20:42:52 +0100
Subject: [PATCH] Remove some console.log statements

---
 app/javascript/controllers/area_creation_v2_controller.js | 6 ------
 app/javascript/controllers/area_drawer_controller.js      | 6 ------
 app/javascript/controllers/public_stat_map_controller.js  | 8 --------
 app/javascript/maps_maplibre/services/api_client.js       | 4 ----
 4 files changed, 24 deletions(-)

diff --git a/app/javascript/controllers/area_creation_v2_controller.js b/app/javascript/controllers/area_creation_v2_controller.js
index 8566e8d3..fc4502bd 100644
--- a/app/javascript/controllers/area_creation_v2_controller.js
+++ b/app/javascript/controllers/area_creation_v2_controller.js
@@ -33,7 +33,6 @@ export default class extends Controller {
    */
   setupEventListeners() {
     document.addEventListener('area:drawn', (e) => {
-      console.log('[Area Creation V2] area:drawn event received:', e.detail)
       this.open(e.detail.center, e.detail.radius)
     })
   }
@@ -42,8 +41,6 @@ export default class extends Controller {
    * Open the modal with area data
    */
   open(center, radius) {
-    console.log('[Area Creation V2] open() called with center:', center, 'radius:', radius)
-
     // Store area data
     this.area = { center, radius }
 
@@ -156,9 +153,6 @@ export default class extends Controller {
    * Show success message
    */
   showSuccess(message) {
-    // You can replace this with a toast notification if available
-    console.log(message)
-
     // Try to use the Toast component if available
     if (window.Toast) {
       window.Toast.show(message, 'success')
diff --git a/app/javascript/controllers/area_drawer_controller.js b/app/javascript/controllers/area_drawer_controller.js
index d1005866..e52d60cf 100644
--- a/app/javascript/controllers/area_drawer_controller.js
+++ b/app/javascript/controllers/area_drawer_controller.js
@@ -22,13 +22,11 @@ export default class extends Controller {
    * @param {maplibregl.Map} map - The MapLibre map instance
    */
   startDrawing(map) {
-    console.log('[Area Drawer] startDrawing called with map:', map)
     if (!map) {
       console.error('[Area Drawer] Map instance not provided')
       return
     }
 
-    console.log('[Area Drawer] Starting drawing mode')
     this.isDrawing = true
     this.map = map
     map.getCanvas().style.cursor = 'crosshair'
@@ -97,13 +95,9 @@ export default class extends Controller {
 
     if (!this.center) {
       // First click - set center
-      console.log('[Area Drawer] First click - setting center:', e.lngLat)
       this.center = [e.lngLat.lng, e.lngLat.lat]
     } else {
       // Second click - finish drawing
-      console.log('[Area Drawer] Second click - finishing drawing')
-
-      console.log('[Area Drawer] Dispatching area:drawn event')
       document.dispatchEvent(new CustomEvent('area:drawn', {
         detail: {
           center: this.center,
diff --git a/app/javascript/controllers/public_stat_map_controller.js b/app/javascript/controllers/public_stat_map_controller.js
index b0d9918a..c218fd0c 100644
--- a/app/javascript/controllers/public_stat_map_controller.js
+++ b/app/javascript/controllers/public_stat_map_controller.js
@@ -72,9 +72,7 @@ export default class extends BaseController {
   }
 
   async loadHexagons() {
-    console.log('🎯 loadHexagons started - checking overlay state');
     const initialLoadingElement = document.getElementById('map-loading');
-    console.log('📊 Initial overlay display:', initialLoadingElement?.style.display || 'default');
 
     try {
       // Use server-provided data bounds
@@ -94,9 +92,6 @@ export default class extends BaseController {
           // Fallback timeout in case moveend doesn't fire
           setTimeout(resolve, 1000);
         });
-        console.log('✅ Map fitBounds complete - checking overlay state');
-        const afterFitBoundsElement = document.getElementById('map-loading');
-        console.log('📊 After fitBounds overlay display:', afterFitBoundsElement?.style.display || 'default');
       }
 
       // Load hexagons only if they are pre-calculated and data exists
@@ -138,7 +133,6 @@ export default class extends BaseController {
       loadingElement.style.display = 'flex';
       loadingElement.style.visibility = 'visible';
       loadingElement.style.zIndex = '9999';
-      console.log('👁️ Loading overlay ENSURED visible - should be visible now');
     }
 
     // Disable map interaction during loading
@@ -187,7 +181,6 @@ export default class extends BaseController {
       }
 
       const geojsonData = await response.json();
-      console.log(`✅ Loaded ${geojsonData.features?.length || 0} hexagons`);
 
       // Add hexagons directly to map as a static layer
       if (geojsonData.features && geojsonData.features.length > 0) {
@@ -210,7 +203,6 @@ export default class extends BaseController {
       const loadingElement = document.getElementById('map-loading');
       if (loadingElement) {
         loadingElement.style.display = 'none';
-        console.log('🚫 Loading overlay hidden - hexagons are fully loaded');
       }
     }
   }
diff --git a/app/javascript/maps_maplibre/services/api_client.js b/app/javascript/maps_maplibre/services/api_client.js
index 5cd756f3..661f5f0e 100644
--- a/app/javascript/maps_maplibre/services/api_client.js
+++ b/app/javascript/maps_maplibre/services/api_client.js
@@ -123,15 +123,11 @@ export class ApiClient {
     })
 
     const url = `${this.baseURL}/photos?${params}`
-    console.log('[ApiClient] Fetching photos from:', url)
-    console.log('[ApiClient] With headers:', this.getHeaders())
 
     const response = await fetch(url, {
       headers: this.getHeaders()
     })
 
-    console.log('[ApiClient] Photos response status:', response.status)
-
     if (!response.ok) {
       throw new Error(`Failed to fetch photos: ${response.statusText}`)
     }