Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

On Sonoff B1 with Version 5.11.1g SetOption20 i got {"Command":"Unknown"} #1741

Closed
store2704 opened this issue Jan 29, 2018 · 2 comments
Closed

On Sonoff B1 with Version 5.11.1g SetOption20 i got {"Command":"Unknown"} #1741

store2704 opened this issue Jan 29, 2018 · 2 comments
Labels
bug Type - Confirmated Bug fixed Result - The work on the issue has ended

Comments

@store2704
Copy link

Hi,

i upraded to version 5.11.1g to verify the adjust Dimmer without turning on device funktion. In the release notes i read that option 20 is used. But when i type at the console the SetOption20 command i got the following error message? Is there something missing to activate Option 20?
08:57:58 CMD: SetOption20
08:57:58 MQT: /Sonoff/Schlafen/Sz.LampeSchrank/stat/RESULT = {"Command":"Unknown"}

best regadrs

Stefan
@arendst arendst added the bug Type - Confirmated Bug label Jan 29, 2018
@arendst
Copy link
Owner

arendst commented Jan 29, 2018

Bug. Quick fix is change line 1002 in sonoff.ino from:

    else if ((CMND_SETOPTION == command_code) && ((index >= 0) && (index <= 19)) || ((index > 31) && (index <= P_MAX_PARAM8 +31))) {

in to

    else if ((CMND_SETOPTION == command_code) && ((index >= 0) && (index <= 20)) || ((index > 31) && (index <= P_MAX_PARAM8 +31))) {

@store2704
Copy link
Author

Change the line as suggested and it works. The test if the bulb stays off i will test later at home. Thanks for the quick support

arendst added a commit that referenced this issue Jan 30, 2018
@arendst
v5.11.1h - Weblog redesign saving RAM
b8f323a 
5.11.1h
 * Rewrite webserver argument processing gaining 5k code space
(#1705)
 * Redesign weblog storage (#1730)
 * Fix command SetOption20
(#1741)
@arendst arendst added the fixed Result - The work on the issue has ended label Jan 30, 2018
@arendst arendst closed this as completed Jan 30, 2018
arendst added a commit that referenced this issue Feb 9, 2018
@arendst
v5.12.0
600c4b1 
5.12.0 20180209
* Change library PubSubClient.h define MQTT_MAX_PACKET_SIZE from 512 to
1000 for Home Assistant  support
* Change relation of define MESSZ being dependent on PubSubClient.h
define MQTT_MAX_PACKET_SIZE
* Change command color parameter input checks to less strict for Home
Assistant support
* Change command Ina219Mode into command Sensor13
* Change commands HlwPCal, HlwUCal and HlwICal to PowerCal, VoltageCal
and CurrentCal to be used for both Pow and S31 calibration
* Change commands HlwPSet, HlwUSet and HlwISet to PowerSet, VoltageSet
and CurrentSet to be used for both Pow and S31 calibration
* Change uptime from hour to second resulting in a display of
123T13:45:21 where 123 is days
* Change module name Wemos D1 mini into Generic (#1220)
* Change HTML from width=100% to style=width:100% supporting HTML5
(#1358)
* Change OSWATCH_RESET_TIME (Blocked loop) from 30 to 120 seconds to
allow slow networks (#1556)
* Change WIFI_MANAGER_SEC into WIFI_CONFIG_SEC (#1616)
* Change function pointers code to save code space and memory (#1683)
* Change webserver argument processing gaining 5k code space (#1705)
* Change weblog memory usage (#1730, #1793, #1819)
* Update TasmotaSerial library to 1.1.0
* Update language files Italian (#1594), Dutch (#1723) and Spanish
(#1722)
* Fix Non-English JSON temperature unit attachement
* Fix Arilux RF induced exception by moving interrupt handler to iram on
non ESP8266/Arduino lib v2.3.0
* Fix truncated command names and wrong response for DomoticzSwitchIdx
(#1571)
* Fix %-sign issue as printf escape character in Humidity and Sonoff SC
(#1579)
* Fix DS18B20 temperature JSON decimal dot (#1561)
* Fix Energy JSON message (#1621)
* Fix IRSend parameter translation (#1636)
* Fix TSL2561 device detection (#1644, #1825)
* Fix BME680 teleperiod resistance measuring (#1647)
* Fix Energy Monitoring Energy Today and Energy Total reading after
restart (#1648)
* Fix IRReceive Data value (#1663)
* Fix Energy Monitoring Energy Period roll-over (#1688)
* Fix compiler warnings (#1774)
* Fix command PWM response if no PWM channel is configured (#1783)
* Add locale Decimal Separator to Web sensor page
* Add ColorTemperature to light status message
* Add command PowerOnState option 5 which inverts PulseTime and allows
for delayed always on after power on
* Add OtaMagic two step Web server OTA upgrade using filename-minimal
image if OTA free space is too small
* Add support for PMS5003 and PMS7003 particle concentration sensor
* Add command SetOption21 1 to allow Energy Monitoring when power is off
on Sonoff Pow and Sonoff S31 (#1420)
* Add Chinese language file (#1551)
* Add French language file (#1561)
* Add Spanish language file (#1589)
* Add HTTP Allow Cross Origin removed from ESP8266/Arduino lib v2.4.0
(#1572)
* Add Home Assistant MQTT Discovery for switch and light to be enabled
by command SetOption19 1 (#1534) or define
HOME_ASSISTANT_DISCOVERY_ENABLE in user_config.h (#1685)
* Add command State to retrieve device state information (same data as
teleperiod state and status 11 in slightly different JSON format)
* Add optional login to Webserver AP mode (#1587, #1635)
* Add command Sensor15 2 to start MHZ19(B) Zero Point Calibration
(#1643)
* Add support for Sonoff S31 Smart Socket with Power Consumption
Detection (#1626)
* Add command SetOption20 to allow update of Dimmer/Color/Ct without
turning power on (#1719, #1741)
* Add NTP sync time slot based on chip id (#1773)
* Add cursor pointer to web button (#1836)
curzon01 pushed a commit to curzon01/Tasmota that referenced this issue Sep 6, 2018
@arendst
v5.11.1h - Weblog redesign saving RAM
0d0c490 
5.11.1h
 * Rewrite webserver argument processing gaining 5k code space
(arendst#1705)
 * Redesign weblog storage (arendst#1730)
 * Fix command SetOption20
(arendst#1741)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Type - Confirmated Bug fixed Result - The work on the issue has ended
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arendst @store2704