Fix documentation, grammar, and spelling in bindings W-Z

[lsiepel]

No description provided.

[ccutrer]

In WiZ, you changed the light#mode channel from a string to a number. The handler provides a StringType when sending updates to that channel, even though its value will be the string representation of a number. For commands, it treats it as a string and will look for values matching it as a number, or various string forms. Creating the channel as a Number circumvents that's extra logic. If your contention is that the channel definition already handles the mapping of string named modes to the number representation, that's probably fine, but I think that would be better as a separate PR that also removes the string matching in the handler. And updates the handler to provide a DecimalType instead of a StringType when sending updates to the channel.

[lsiepel]

In WiZ, you changed the light#mode channel from a string to a number.

That one slipped trough, Thanks for checking. I'll revert that itemtype. When i see the option list, i must say a number type would be a better fit, but that should be changed in a separate PR.

Edit: Fixed.

[ccutrer]

LGTM for WiZ

[marcelrv]

LGTM wrt windcentrale

[jlaur]

Otherwise LGTM for WeMo.

[lsiepel]

Build fails due to #19650

[hmerk]

LGTM

[Copilot]

Pull request overview

This pull request provides comprehensive documentation, grammar, and spelling improvements across openHAB bindings with names starting with letters W through Z. The changes focus on improving clarity, consistency, and correctness in user-facing text.

Key Changes

• Capitalization consistency: Standardized "Thing" capitalization throughout documentation and configuration files
• Grammar and punctuation improvements: Fixed sentence structure, added missing periods, improved comma usage, and corrected verb forms
• Spelling corrections: Fixed typos like "sendComand" → "sendCommand", brand names like "WeatherUnderground" → "Weather Underground"
• Terminology improvements: Clarified technical terms (e.g., "WiFi" → "Wi‑Fi", "WebSocket server" → "web service")
• Documentation clarity: Improved descriptions to be more precise and user-friendly across configuration parameters and channel descriptions

Reviewed changes

Copilot reviewed 67 out of 67 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
zway binding	Capitalized "Thing" consistently, improved grammar in descriptions
zwavejs binding	Fixed punctuation, improved parameter descriptions, capitalized "Command Class" and "Node", changed "required=true" to "required=false" for optional parameters
zoneminder binding	Improved channel descriptions, fixed grammar, standardized terminology
yioremote binding	Enhanced description clarity, fixed grammar
yeelight binding	Improved channel descriptions for better clarity
yamahareceiver binding	Fixed capitalization ("Receiver" → "receiver"), improved descriptions with better punctuation
yamahamusiccast binding	Enhanced clarity of MusicCast-related descriptions
xmppclient binding	Improved XMPP terminology descriptions, better formatting
xmltv binding	Capitalized "XMLTV" consistently, improved channel descriptions
x binding	Improved Twitter/X binding descriptions for clarity
wundergroundupdatereceiver	Capitalized "Thing" consistently
wolfsmartset binding	Improved table formatting and descriptions
wled binding	Enhanced channel descriptions and table formatting
wlanthermo binding	Capitalized "Thing" consistently
wiz binding	Fixed "WiFi" → "Wi‑Fi", improved descriptions
windcentrale binding	Improved refresh interval descriptions
wifiled binding	Fixed "WiFi" → "Wi‑Fi", improved program names (e.g., "Red,Green" → "Red/Green")
wemo binding	Fixed numerous typos (e.g., "DemoLoght2" → "DemoLight2"), improved channel descriptions, capitalized "WeMo" correctly
webthing binding	Improved table formatting
webexteams binding	Rebranded "WebexTeams" → "Webex Teams" consistently
weatherunderground binding	Fixed capitalization, improved descriptions and table formatting
weathercompany binding	Fixed label errors ("Day 3" → "Day 4"), improved descriptions, fixed pattern formatting
warmup binding	Fixed "WiFi" → "Wi‑Fi", improved channel descriptions

The changes are exclusively documentation improvements with no functional code modifications, making this a safe enhancement to the user experience.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jlaur]

Thanks! Just a few comments.

[jlaur]

I think it should rather be changed to required="false" here, since it has a default value:

openhab-addons/bundles/org.openhab.binding.warmup/src/main/resources/OH-INF/thing/thing-types.xml

Lines 63 to 67 in 211f0e2

	<parameter name="overrideDuration" type="integer" unit="m" required="true">
	<label>Override Duration</label>
	<description>Duration in minutes of override when target temperature is changed</description>
	<default>60</default>
	</parameter>

Suggested change

	| overrideDuration | Integer | Duration in minutes of override when target temperature is changed | true | 60 |
	| overrideDuration | Integer | Duration in minutes of override when target temperature is changed | false | 60 |

[lsiepel]

Are you sure?
Up to now, i thought that having a default or not. Required also signals the UI to have a value.
But now i wonder if that is true, How ould one unset a default value. E.g. remove it from the jsondb to stick to the default value from the binding. aargh :-/

[jlaur]

Example (defaults to required="false"):

openhab-addons/bundles/org.openhab.binding.energidataservice/src/main/resources/OH-INF/config/service.xml

Lines 17 to 25 in 211f0e2

	<parameter name="currencyCode" type="text">
	<label>Currency Code</label>
	<description>Currency code in which to obtain spot prices.</description>
	<default>DKK</default>
	<options>
	<option value="DKK">Danish Krone</option>
	<option value="EUR">Euro</option>
	</options>
	</parameter>

openhab-addons/bundles/org.openhab.binding.energidataservice/src/main/java/org/openhab/binding/energidataservice/internal/config/EnergiDataServiceConfiguration.java

Lines 34 to 37 in 211f0e2

	/**
	* Currency code for the prices.
	*/
	public String currencyCode = EnergiDataServiceBindingConstants.CURRENCY_DKK.getCurrencyCode();

Never set:

Click to upper right corner to clear:

Save - and it's back to default. Code tab:

version: 1
things:
  energidataservice:service:311fd75536:
    config:
      priceArea: DK1
      currencyCode: DKK
      hourlySpotPrices: false
      gridCompanyGLN: "5790000705184"
      energinetGLN: "5790000432752"
      reducedElectricityTax: false

Remove it:

version: 1
things:
  energidataservice:service:311fd75536:
    config:
      priceArea: DK1
      hourlySpotPrices: false
      gridCompanyGLN: "5790000705184"
      energinetGLN: "5790000432752"
      reducedElectricityTax: false

And save. It's back.

Warmup binding:

Deleting the "0", now trying to delete "6", but that instantly causes it to go back to 60:

If I delete it in the code tab, it automatically is restored. File-based configuration seems to work as well:

Thing warmup:room:test"Room" (warmup:my-warmup:test) [serialNumber="123"]

So here the only problem seems to be that you can't actually delete it in the UI.

I think it's most logical to leave parameters having a default value as optional, since the binding can then just use the default value. I think there used to be more annoying issues, but right now can't remember or reproduce.

[jlaur]

Anyway, that's how the binding is now, and you just updated the documentation to reflect that, so it's fine for this PR.

[jlaur]

LGTM