add more comments about old event fields

src/Database/ImgIndex.php

@@ -49,6 +49,9 @@ public function insertScreenshot($date, $imageScale, $roi, $watermark, $layers,
 
         $this->_dbConnect();
 
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
         // old implementation removed for events strings
         // used to be $this->events->serialize();
         $old_events_layer_string = ""; 

src/Database/MovieDatabase.php

@@ -63,7 +63,9 @@ public function insertMovie($startTime, $endTime, $reqObservationDate, $imageSca
             $reqObservationDate   = "NULL";
         }
 
-        // !Attention we are keeping those fields for current rows in movies and screenshots tables
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
         // old implementation removed for events strings
         // used to be $this->events->serialize();
         $old_events_layer_string = ""; 

src/Module/Movies.php

@@ -1,5 +1,4 @@
 <?php
-/* vim: set expandtab tabstop=4 shiftwidth=4 softtabstop=4: */
 /**
  * Helioviewer Movies Module class definition
  * Movie generation and display.
@@ -308,8 +307,11 @@ public function queueMovie() {
             $event_labels = (bool)$this->_params['eventLabels'];
         }
 
-		// Events manager built from old logic
-		$events_manager = EventsStateManager::buildFromLegacyEventStrings($events_legacy_string, $event_labels);
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
+        // Events manager built from old logic
+        $events_manager = EventsStateManager::buildFromLegacyEventStrings($events_legacy_string, $event_labels);
 
         // TODO 2012/04/11
         // Discard any layers which do not share an overlap with the roi to

src/Module/WebClient.php

@@ -592,6 +592,10 @@ public function takeScreenshot() {
             $event_labels = (bool)$this->_params['eventLabels'];
         }
 
+
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
         // Events manager built from old logic
         $events_manager = EventsStateManager::buildFromLegacyEventStrings($events_legacy_string, $event_labels);
 
@@ -693,6 +697,9 @@ public function reTakeScreenshot($screenshotId) {
         $events_state_from_metadata = json_decode($metaData['eventsState'], true);
         $events_manager;
 
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
         if(!empty($events_state_from_metadata)) {
             $events_manager = EventsStateManager::buildFromEventsState($events_state_from_metadata);
         } else {
@@ -788,7 +795,6 @@ public function saveWebClientState() {
         $client_state = new ClientState();
 
         try {
-
             $state_key = $client_state->upsert($this->_params['json']);
 
             return $this->_sendResponse(200, 'OK', $state_key);
@@ -1358,6 +1364,9 @@ public function getEclipseImage() {
         $range = 6000;
         $roi = new Helper_RegionOfInterest(-$range, -$range, $range, $range, 15);
 
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
         // Create empty events object required for screenshots.
         $events_manager = EventStateManager::buildFromLegacyEventStrings('', false);
 

src/Movie/HelioviewerMovie.php

@@ -156,6 +156,9 @@ public function __construct($publicId, $format='mp4') {
         // Data Layers
         $this->_layers = new Helper_HelioviewerLayers($info['dataSourceString']);
 
+        // ATTENTION! These two fields eventsLabels and eventSourceString needs to be kept in DB schema
+        // We are keeping them to support old takeScreenshot , queueMovie requests
+
         // Events Manager
         $events_state_from_info = json_decode($info['eventsState'], true);