Merge branch 'current' into next

plugins/configuration.rb

@@ -7,6 +7,11 @@ module Jekyll
       'icon'         => '/docs/configuration/customizing-devices/#icon',
     }
 
+    TYPES = [
+      'action', 'boolean', 'string', 'integer', 'float', 'time', 'template',
+      'device_class', 'icon', 'map', 'list', 'date', 'datetime'
+    ]
+
     def initialize(tag_name, text, tokens)
       super
       @component, @platform = text.strip.split('.', 2)
@@ -48,7 +53,7 @@ module Jekyll
       end
     end
 
-    def render_config_vars(vars:, component:, platform:, classes: nil)
+    def render_config_vars(vars:, component:, platform:, converter:, classes: nil, parent_type: nil)
       result = Array.new
       result << "<dl class='#{classes}'>"
 
@@ -57,20 +62,56 @@ module Jekyll
         markup << "<dt><a class='title-link' name='#{slug(key)}' href='\##{slug(key)}'></a> #{key}</dt>"
         markup << "<dd>"
         markup << "<p class='desc'>"
+
         if attr.key? 'type'
+
+          # Check if the type (or list of types) are valid
+          if attr['type'].kind_of? Array
+            attr['type'].each do |type|
+              raise ArgumentError, "Configuration type '#{type}' for key '#{key}' is not a valid type in the documentation."\
+              " See: https://developers.home-assistant.io/docs/en/documentation_create_page.html#configuration" unless \
+                TYPES.include? type
+            end
+          else          
+            raise ArgumentError, "Configuration type '#{attr['type']}' for key '#{key}' is not a valid type in the documentation."\
+            " See: https://developers.home-assistant.io/docs/en/documentation_create_page.html#configuration" unless \
+              TYPES.include? attr['type']
+          end
+
           markup << "<span class='type'>(<span class='#{type_class(attr['type'])}'>"
           markup << "#{type_link(attr['type'], component: component)}</span>)</span>"
+        else
+          # Type is missing, which is required (unless we are in a list or template)
+          raise ArgumentError, "Configuration key '#{key}' is missing a type definition" \
+            unless ['list', 'template'].include? parent_type
         end
+
+        
         if attr.key? 'required'
+          # Check if required is a valid value
+          raise ArgumentError, "Configuration key '#{key}' required field must be specified as: "\
+            "true, false, inclusive or exclusive."\
+            unless [true, false, 'inclusive', 'exclusive'].include? attr['required']
+
           markup << "<span class='required'>(#{required_value(attr['required'])})</span>"
         end
+
         if attr.key? 'description'
-          markup << "<span class='description'>#{attr['description']}</span>"
+          markup << "<span class='description'>#{converter.convert(attr['description'].to_s)}</span>"
+        else
+          # Description is missing
+          raise ArgumentError, "Configuration key '#{key}' is missing a description."
         end
         markup << "</p>"
-        if attr.key? 'default'
+
-          markup << "<p class='default'>Default value: #{attr['default']}</p>"
+        if attr.key? 'default' and not attr['default'].to_s.empty?
+          markup << "<p class='default'>\nDefault value: #{converter.convert(attr['default'].to_s)}</p>"
+        elsif attr['type'].to_s.include? 'boolean'
+          # Is the type is a boolean, a default key is required
+          raise ArgumentError, "Configuration key '#{key}' is a boolean type and"\
+            " therefore, requires a default."
         end
+
         markup << "</dd>"
 
         # Check for nested configuration variables
@@ -78,7 +119,8 @@ module Jekyll
           markup << "<dd>"
           markup << render_config_vars(
             vars: attr['keys'], component: component,
-            platform: platform, classes: 'nested')
+            platform: platform, converter: converter,
+            classes: 'nested', parent_type: attr['type'])
           markup << "</dd>"
         end
 
@@ -100,12 +142,20 @@ module Jekyll
       component = Liquid::Template.parse(@component).render context
       platform  = Liquid::Template.parse(@platform).render context
 
+      site = context.registers[:site]
+      converter = site.find_converter_instance(::Jekyll::Converters::Markdown)
+
       vars = SafeYAML.load(contents)
 
       <<~MARKUP
         <div class="config-vars">
           <h3><a class="title-link" name="configuration-variables" href="#configuration-variables"></a> Configuration Variables</h3>
-          #{render_config_vars(vars: vars, component: component, platform: platform)}
+          #{render_config_vars(
+            vars: vars,
+            component: component,
+            platform: platform,
+            converter: converter
+          )}
         </div>
       MARKUP
     end

source/_components/climate.mqtt.markdown

@@ -107,7 +107,7 @@ mode_state_template:
   required: false
   type: template
 modes:
-  description: A list of supported modes.
+  description: A list of supported modes. Needs to be a subset of the default values.
   required: false
   default: ['auto', 'off', 'cool', 'heat', 'dry', 'fan_only']
   type: list

source/_components/sensibo.markdown

@@ -59,16 +59,16 @@ switch:
     switches:
       ac:
         friendly_name: "AC"
-        value_template: "{{ is_state('climate.ac', 'cool') or is_state('climate.ac', 'heat') or is_state('climate.ac', 'dry') or is_state('climate.ac', 'heat')}}"
+        value_template: "{{ is_state('climate.ac', 'cool') or is_state('climate.ac', 'heat') or is_state('climate.ac', 'dry') or is_state('climate.ac', 'fan_only') }}"
         turn_on:
           service: climate.set_havc_mode
           data:
             entity_id: climate.ac
-            hvac_mode: off
+            hvac_mode: cool
         turn_off:
           service: climate.set_havc_mode
           data:
             entity_id: climate.ac
-            hvac_mode: 
+            hvac_mode: off
 ```
 {% endraw %}