Fix/watermarks overflow

src/Image/Composite/HelioviewerCompositeImage.php

@@ -1374,32 +1374,33 @@ private function _addScaleBar($imagickImage) {
      * @return void
      */
     private function _addWatermark($imagickImage) {
+
+        // paths of the different logos to choose from
+        $hv_logo = sprintf('%s/resources/images/watermark_circle_small_black_border.png', HV_ROOT_DIR);
+        $hv_with_url_logo = sprintf("%s/resources/images/watermark_small_black_border.png", HV_ROOT_DIR);
+
         if ( $this->width < 200 || $this->height < 200 ) {
             return;
         }
 
-        $watermark = new IMagick(  HV_ROOT_DIR
-                                 . '/resources/images/'
-                                 . 'watermark_small_black_border.png');
-
-        // If the image is too small, use only the circle, not the url, and
-        // scale it so it fits the image.
-        if ( $this->width / 300 < 2 ) {
-            $watermark->readImage(  HV_ROOT_DIR
-                                  . '/resources/images/'
-                                  . 'watermark_circle_small_black_border.png');
-            $scale = ($this->width / 2) / 300;
-            $width = $watermark->getImageWidth();
-            $watermark->scaleImage(intval($width * $scale), intval($width * $scale));
+        // Default text + URL
+        $watermark = new IMagick($hv_with_url_logo);
+
+        // If the image is too small, use only the circle, not the url, and scale it so it fits the image.
+        if ( $this->width < 600 ) {
+            $watermark->readImage($hv_logo);
+            $scale = $this->width / 600;
+            $scaled_watermark_width = intval($watermark->getImageWidth() * $scale);
+            $watermark->scaleImage($scaled_watermark_width, $scaled_watermark_width);
         }
 
         // For whatever reason, compositeImage() doesn't carry over gravity
         // settings so the offsets must be relative to the top left corner of
         // the image rather than the desired gravity.
         $x = $this->width  - $watermark->getImageWidth()  - 10;
         $y = $this->height - $watermark->getImageHeight() - 10;
-        $imagickImage->compositeImage(
-            $watermark, IMagick::COMPOSITE_DISSOLVE, intval($x), intval($y) );
+
+        $imagickImage->compositeImage($watermark, IMagick::COMPOSITE_DISSOLVE, intval($x), intval($y) );
 
         // If the image is too small, text won't fit. Don't put a date string
         // on it.
@@ -1452,19 +1453,39 @@ private function _addTimestampWatermark($imagickImage) {
         $underText->setStrokeColor($black);
         $underText->setStrokeAntialias(true);
         $underText->setStrokeWidth(2);
-        $imagickImage->annotateImage($underText,     $leftPad, $lowerPad, 0, $nameCmd);
-        $imagickImage->annotateImage($underText, 120+$leftPad, $lowerPad, 0, $timeCmd);
+        $imagickImage->annotateImage($underText, $leftPad, $lowerPad, 0, $nameCmd);
+
+        // Query font metrics to find out area width of the satellite image sources text (ex: SDO AIA 404)
+        // Then we use this width to calculate where to put timestamps outline in image
+        // last parameter true means multiline text
+        $underTextQueryMetrics = $imagickImage->queryFontMetrics($underText, $nameCmd, true);
 
-        // Write words in white over outline
+        // 10 is the default padding between source outline and timestamp outline
+        $underTextRightPad = $underTextQueryMetrics['textWidth'] ? ($underTextQueryMetrics['textWidth'] + 10) : 120; 
+
+        // Place timestamp outline 
+        $imagickImage->annotateImage($underText, $underTextRightPad + $leftPad, $lowerPad, 0, $timeCmd);
+
+        // Write text in white over the outline
         $text = new IMagickDraw();
         $text->setTextEncoding('utf-8');
         $text->setFont(HV_ROOT_DIR.'/../resources/fonts/DejaVuSans.ttf');
         $text->setFontSize(12);
         $text->setFillColor($white);
         $text->setTextAntialias(true);
         $text->setStrokeWidth(0);
-        $imagickImage->annotateImage($text,     $leftPad, $lowerPad, 0, $nameCmd);
-        $imagickImage->annotateImage($text, 120+$leftPad, $lowerPad, 0, $timeCmd);
+        $imagickImage->annotateImage($text, $leftPad, $lowerPad, 0, $nameCmd);
+
+        // Query font metrics to find out area width of the satellite image sources text (ex: SDO AIA 404)
+        // Then we use this width to calculate where to put timestamps texts in image.
+        // last parameter true means multiline text
+        $textQueryMetrics = $imagickImage->queryFontMetrics($text , $nameCmd, true);
+
+        // 10 is the default padding between source text and timestamp text
+        $textRightPad = $textQueryMetrics['textWidth'] ? ($textQueryMetrics['textWidth'] + 10) : 120; 
+
+        // Place timestamp text 
+        $imagickImage->annotateImage($text, $textRightPad + $leftPad, $lowerPad, 0, $timeCmd);
 
         // Cleanup
         $black->destroy();

src/Image/HelioviewerImage.php

@@ -96,7 +96,7 @@ private function _imageNotVisible($pixelRoi) {
      * @return string watermark name
      */
     public function getWaterMarkName() {
-        return $this->getWaterMarkName();
+        return trim(join(' ', array_column($this->uiLabels, 'name'))) . "\n";
     }
 
     /**

src/Image/ImageType/AIAImage.php

@@ -51,18 +51,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $name = $this->uiLabels[2]['name'];
-		if($this->uiLabels[2]['name'] == 'AIA'){
-			$name = $this->uiLabels[3]['name'];
-		}
-        return 'AIA '.$name."\n";
-    }
 }
 ?>

src/Image/ImageType/CORImage.php

@@ -52,16 +52,6 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
 
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string Watermark name
-     */
-    public function getWaterMarkName() {
-        $which = substr($this->uiLabels[0]['name'], -1);
-        return $this->uiLabels[2]['name'].'-'.$which."\n";
-    }
-
     /**
      * Generates a portion of an ImageMagick convert command to apply
      * an alpha mask

src/Image/ImageType/COSMOImage.php

@@ -41,15 +41,6 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
 
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string Watermark name
-     */
-    public function getWaterMarkName() {
-        return 'COSMO_K-Coronagraph '.$this->uiLabels[2]['name']."\n";
-    }
-
     /**
      * Generates a portion of an ImageMagick convert command to apply an alpha mask
      *
@@ -166,4 +157,4 @@ protected function setAlphaChannel(&$imagickImage) {
         $mask->destroy();
     }
 }
-?>
+?>

src/Image/ImageType/EITImage.php

@@ -46,16 +46,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-	    $labelName = isset($this->uiLabels[3]) ? $this->uiLabels[3]['name'] : $this->uiLabels[2]['name'];
-        return 'EIT '.$labelName."\n";
-    }
-
 }
-?>
+?>

src/Image/ImageType/EUIImage.php

@@ -44,17 +44,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $type = $this->uiLabels[2]['name'];
-        $measurement = $this->uiLabels[3]['name'];
-        $watermark = 'EUI '.$type.' '.$measurement." Å\n";
-        return $watermark;
-    }
 }
 ?>

src/Image/ImageType/EUVIImage.php

@@ -38,15 +38,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $which = substr($this->uiLabels[0]['name'], -1);
-        return 'EUVI-'.$which.' '.$this->uiLabels[3]['name']."\n";
-    }
 }
-?>
+?>

src/Image/ImageType/GONGImage.php

@@ -44,15 +44,5 @@ protected function setColorPalette(&$input) {
         return false;
     }
 
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-	    $labelName = $this->uiLabels[2]['name'];
-        return 'GONG '.$labelName."\n";
-    }
-
 }
-?>
+?>

src/Image/ImageType/HMIImage.php

@@ -47,16 +47,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
     protected function setColorPalette(&$input) {
         return false;
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-	    $labelName = isset($this->uiLabels[3]) ? $this->uiLabels[3]['name'] : $this->uiLabels[2]['name'];
-        return 'HMI '.$labelName."\n";
-    }
-
 }
-?>
+?>

src/Image/ImageType/LASCOImage.php

@@ -48,15 +48,6 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
 
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string Watermark name
-     */
-    public function getWaterMarkName() {
-        return 'LASCO '.$this->uiLabels[2]['name']."\n";
-    }
-
     /**
      * Generates a portion of an ImageMagick convert command to apply an alpha mask
      *
@@ -175,4 +166,4 @@ protected function setAlphaChannel(&$imagickImage) {
         $mask->destroy();
     }
 }
-?>
+?>

src/Image/ImageType/MDIImage.php

@@ -46,16 +46,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
     protected function setColorPalette(&$input) {
         return false;
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-	    $labelName = isset($this->uiLabels[3]) ? $this->uiLabels[3]['name'] : $this->uiLabels[2]['name'];
-        return 'MDI '.$labelName."\n";
-    }
-
 }
-?>
+?>

src/Image/ImageType/RHESSIImage.php

@@ -31,17 +31,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
         $this->setColorTable($color_table);
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $observatory = $this->uiLabels[0]['name'];
-        $energy = $this->uiLabels[1]['name'];
-        $reconstruction = $this->uiLabels[2]['name'];
-        return $observatory . " " . $energy . " " . $reconstruction;
-    }
 }
-?>
+?>

src/Image/ImageType/SJIImage.php

@@ -34,16 +34,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $measurement = $this->uiLabels[2]['name'];
-        $watermark = 'IRIS SJI '.$measurement." Å\n";
-        return $watermark;
-    }
 }
 ?>

src/Image/ImageType/SUVIImage.php

@@ -16,6 +16,7 @@
 require_once HV_ROOT_DIR.'/../src/Image/HelioviewerImage.php';
 
 class Image_ImageType_SUVIImage extends Image_HelioviewerImage {
+
     /**
      * Creates a new SUVIImage
      *
@@ -44,17 +45,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $type = $this->uiLabels[2]['name'];
-        $measurement = $this->uiLabels[3]['name'];
-        $watermark = 'SUVI '.$type.' '.$measurement."\n";
-        return $watermark;
-    }
 }
 ?>

src/Image/ImageType/SWAPImage.php

@@ -63,15 +63,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        return 'SWAP '.(isset($this->uiLabels[3]) ? $this->uiLabels[3]['name'] : $this->uiLabels[2]['name'])."\n";
-    }
-
 }
-?>
+?>

src/Image/ImageType/SXTImage.php

@@ -46,16 +46,5 @@ public function __construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY
 
         parent::__construct($jp2, $filepath, $roi, $uiLabels, $offsetX, $offsetY, $options);
     }
-
-    /**
-     * Gets a string that will be displayed in the image's watermark
-     *
-     * @return string watermark name
-     */
-    public function getWaterMarkName() {
-        $labelName = isset($this->uiLabels[3]) ? $this->uiLabels[3]['name'] : $this->uiLabels[2]['name'];
-        return 'SXT '.$labelName."\n";
-    }
-
 }
 ?>