[sitemap] Remove snedFrequency parameter for Slider/Colorpicker widge…

…ts (openhab#4347) Related to openhab#4338 Signed-off-by: Laurent Garnier <[email protected]>
holgerfriedrich · Sep 7, 2024 · edf5b62 · edf5b62
1 parent e3d7885
commit edf5b62
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 10 deletions.
diff --git a/...rest.sitemap/src/main/java/org/openhab/core/io/rest/sitemap/internal/SitemapResource.java b/...rest.sitemap/src/main/java/org/openhab/core/io/rest/sitemap/internal/SitemapResource.java
@@ -656,7 +656,6 @@ private PageDTO createPageBean(String sitemapName, @Nullable String title, @Null
             bean.inputHint = inputWidget.getInputHint();
         }
         if (widget instanceof Slider sliderWidget) {
-            bean.sendFrequency = sliderWidget.getFrequency();
             bean.switchSupport = sliderWidget.isSwitchEnabled();
             bean.releaseOnly = sliderWidget.isReleaseOnly();
             bean.minValue = sliderWidget.getMinValue();

diff --git a/...re.io.rest.sitemap/src/main/java/org/openhab/core/io/rest/sitemap/internal/WidgetDTO.java b/...re.io.rest.sitemap/src/main/java/org/openhab/core/io/rest/sitemap/internal/WidgetDTO.java
@@ -57,7 +57,6 @@ public class WidgetDTO {
     public final List<MappingDTO> mappings = new ArrayList<>();
     public Boolean switchSupport;
     public Boolean releaseOnly;
-    public Integer sendFrequency;
     public Integer refresh;
     public Integer height;
     public BigDecimal minValue;

diff --git a/bundles/org.openhab.core.model.sitemap/src/org/openhab/core/model/sitemap/Sitemap.xtext b/bundles/org.openhab.core.model.sitemap/src/org/openhab/core/model/sitemap/Sitemap.xtext
@@ -126,9 +126,8 @@ Slider:
     (('icon=' icon=Icon) |
         ('icon=[' (IconRules+=IconRule (',' IconRules+=IconRule)*) ']') |
         ('staticIcon=' staticIcon=Icon))? &
-    ('sendFrequency=' frequency=INT)? & (switchEnabled?='switchSupport')? &
-    (releaseOnly?='releaseOnly')? & ('minValue=' minValue=Number)? &
-    ('maxValue=' maxValue=Number)? & ('step=' step=Number)? &
+    (switchEnabled?='switchSupport')? & (releaseOnly?='releaseOnly')? &
+    ('minValue=' minValue=Number)? & ('maxValue=' maxValue=Number)? & ('step=' step=Number)? &
     ('labelcolor=[' (LabelColor+=ColorArray (',' LabelColor+=ColorArray)*) ']')? &
     ('valuecolor=[' (ValueColor+=ColorArray (',' ValueColor+=ColorArray)*) ']')? &
     ('iconcolor=[' (IconColor+=ColorArray (',' IconColor+=ColorArray)*) ']')? &
@@ -161,7 +160,6 @@ Colorpicker:
     (('icon=' icon=Icon) |
         ('icon=[' (IconRules+=IconRule (',' IconRules+=IconRule)*) ']') |
         ('staticIcon=' staticIcon=Icon))? &
-    ('sendFrequency=' frequency=INT)? &
     ('labelcolor=[' (LabelColor+=ColorArray (',' LabelColor+=ColorArray)*) ']')? &
     ('valuecolor=[' (ValueColor+=ColorArray (',' ValueColor+=ColorArray)*) ']')? &
     ('iconcolor=[' (IconColor+=ColorArray (',' IconColor+=ColorArray)*) ']')? &

diff --git a/....ui/src/main/java/org/openhab/core/ui/internal/components/UIComponentSitemapProvider.java b/....ui/src/main/java/org/openhab/core/ui/internal/components/UIComponentSitemapProvider.java
@@ -243,8 +243,6 @@ protected Sitemap buildSitemap(RootUIComponent rootComponent) {
                         SitemapPackage.SLIDER__SWITCH_ENABLED);
                 setWidgetPropertyFromComponentConfig(widget, component, "releaseOnly",
                         SitemapPackage.SLIDER__RELEASE_ONLY);
-                setWidgetPropertyFromComponentConfig(widget, component, "sendFrequency",
-                        SitemapPackage.SLIDER__FREQUENCY);
                 break;
             case "Selection":
                 SelectionImpl selectionWidget = (SelectionImpl) SitemapFactory.eINSTANCE.createSelection();
@@ -266,8 +264,6 @@ protected Sitemap buildSitemap(RootUIComponent rootComponent) {
             case "Colorpicker":
                 ColorpickerImpl colorpickerWidget = (ColorpickerImpl) SitemapFactory.eINSTANCE.createColorpicker();
                 widget = colorpickerWidget;
-                setWidgetPropertyFromComponentConfig(widget, component, "frequency",
-                        SitemapPackage.COLORPICKER__FREQUENCY);
                 break;
             case "Buttongrid":
                 ButtongridImpl buttongridWidget = (ButtongridImpl) SitemapFactory.eINSTANCE.createButtongrid();