fixed errors VC

pandora_console/include/javascript/pandora_visual_console.js

@@ -1181,8 +1181,8 @@ function createOrUpdateVisualConsoleItem(
       }
 
       if (item.itemProps.id) {
-        item.setMeta({ isUpdating: false });
         visualConsole.updateElement(data);
+        item.setMeta({ isUpdating: false });
       } else {
         data["receivedAt"] = new Date();
         var newItem = visualConsole.addElement(data);
@@ -1548,6 +1548,11 @@ function linkedVisualConsoleTypeChange() {
 // eslint-disable-next-line no-unused-vars
 function imageVCChange(baseUrl, vcId, only) {
   var nameImg = document.getElementById("imageSrc").value;
+  if (nameImg == 0) {
+    $("#li-image-item label").empty();
+    return;
+  }
+
   if (!only) {
     only = 0;
   }

pandora_console/include/rest-api/models/VisualConsole/Item.php

@@ -2124,9 +2124,11 @@ class Item extends CachedModel
     /**
      * List images for Vc Icons.
      *
+     * @param boolean|null $service If service item.
+     *
      * @return array
      */
-    public function getListImagesVC():array
+    public function getListImagesVC(?bool $service=false):array
     {
         global $config;
 
@@ -2167,6 +2169,10 @@ class Item extends CachedModel
             }
         }
 
+        if ($service === true) {
+            \array_unshift($result, ['name' => __('None')]);
+        }
+
         return $result;
     }
 
@@ -2382,6 +2388,8 @@ class Item extends CachedModel
         string $image,
         ?bool $only=false
     ):string {
+        $images = '';
+        if ($image !== '0') {
             $type_image = [''];
             if ($only === false) {
                 $type_image = [
@@ -2392,7 +2400,6 @@ class Item extends CachedModel
                 ];
             }
 
-        $images = '';
             foreach ($type_image as $k => $v) {
                 $type = '';
                 if ($v !== '') {
@@ -2409,6 +2416,7 @@ class Item extends CachedModel
                     ]
                 );
             }
+        }
 
         return $images;
     }

pandora_console/include/rest-api/models/VisualConsole/View.php

@@ -489,7 +489,12 @@ class View extends \HTML
             break;
 
             case SERVICE:
-                $data['imageSrc'] = \get_parameter('imageSrc');
+                $imageSrc = \get_parameter('imageSrc');
+                if ($imageSrc === '0') {
+                    $imageSrc = '';
+                }
+
+                $data['imageSrc'] = $imageSrc;
                 $data['serviceId'] = \get_parameter('serviceId');
             break;
 

visual_console_client/src/Item.ts

@@ -775,11 +775,7 @@ abstract class VisualConsoleItem<Props extends ItemProps> {
       }
     }
     // Change link.
-    if (
+    if (prevProps && prevProps.isLinkEnabled !== this.props.isLinkEnabled) {
-      prevProps &&
-      (prevProps.isLinkEnabled !== this.props.isLinkEnabled ||
-        (this.props.isLinkEnabled && prevProps.link !== this.props.link))
-    ) {
       const container = this.createContainerDomElement();
       // Add the children of the old element.
       container.innerHTML = this.elementRef.innerHTML;
@@ -787,7 +783,12 @@ abstract class VisualConsoleItem<Props extends ItemProps> {
       const attrs = this.elementRef.attributes;
       for (let i = 0; i < attrs.length; i++) {
         if (attrs[i].nodeName !== "id") {
-          container.setAttributeNode(attrs[i]);
+          let cloneIsNeeded = this.elementRef.getAttributeNode(
+            attrs[i].nodeName
+          );
+          if (cloneIsNeeded !== null) {
+            container.setAttributeNode(<any>cloneIsNeeded.cloneNode());
+          }
         }
       }
       // Replace the reference.
@@ -799,6 +800,15 @@ abstract class VisualConsoleItem<Props extends ItemProps> {
       this.elementRef = container;
     }
 
+    if (
+      prevProps &&
+      (this.props.isLinkEnabled && prevProps.link !== this.props.link)
+    ) {
+      if (this.props.link !== null) {
+        this.elementRef.setAttribute("href", this.props.link);
+      }
+    }
+
     // Change metadata related things.
     if (!prevMeta || prevMeta.editMode !== this.meta.editMode) {
       if (this.meta.editMode) {

visual_console_client/src/items/Service.ts

@@ -81,9 +81,12 @@ export default class Service extends Item<ServiceProps> {
    */
   protected updateDomElement(element: HTMLElement): void {
     if (this.props.statusImageSrc !== null) {
-      const imgSrc = this.props.statusImageSrc || this.props.imageSrc;
+      element.style.background = `url(${this.props.statusImageSrc}) no-repeat`;
-      element.style.backgroundImage = `url(${imgSrc})`;
+      element.style.backgroundSize = "contain";
+      element.style.backgroundPosition = "center";
+      element.innerHTML = "";
     } else if (this.props.encodedTitle !== null) {
+      element.innerHTML = decodeBase64(this.props.encodedTitle);
     }
   }
 }