Skip to content
  • MySensors
  • OpenHardware.io
  • Categories
  • Recent
  • Tags
  • Popular
Skins
  • Light
  • Brite
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Brand Logo
  1. Home
  2. Bug Reports
  3. Empty Payload not set in MySensor::request and MySensor::requestTime

Empty Payload not set in MySensor::request and MySensor::requestTime

Scheduled Pinned Locked Moved Bug Reports
2 Posts 2 Posters 1.5k Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • T Offline
    T Offline
    tekka
    Admin
    wrote on last edited by
    #1

    Payload not set in MySensors.cpp ==> When gateway receives message w/o payload (e.g. request or requestTime), payload from previous message is reported.

    Example:

    RECV: 4;4;1;0;1;44.9
    RECV: 4;5;1;0;16;0
    RECV: 4;6;1;0;16;557 <== set payload = 557
    RECV: 1;255;3;0;1;557 <== time request w/o payload, pevious payload reported
    RECV: 1;0;1;0;1;47.7

    procs from MySensors.cpp:

    void MySensor::request(uint8_t childSensorId, uint8_t variableType, uint8_t destination) {
    sendRoute(build(msg, nc.nodeId, destination, childSensorId, C_REQ, variableType, false));
    }

    void MySensor::requestTime(void (* _timeCallback)(unsigned long)) {
    timeCallback = _timeCallback;
    sendRoute(build(msg, nc.nodeId, GATEWAY_ADDRESS, NODE_SENSOR_ID, C_INTERNAL, I_TIME, false));
    }

    ==> add set("") to both sendRoute cmds

    1 Reply Last reply
    0
    • H Offline
      H Offline
      hek
      Admin
      wrote on last edited by
      #2

      Thanks, fixed now.

      1 Reply Last reply
      0

      Hello! It looks like you're interested in this conversation, but you don't have an account yet.

      Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

      With your input, this post could be even better 💗

      Register Login
      Reply
      • Reply as topic
      Log in to reply
      • Oldest to Newest
      • Newest to Oldest
      • Most Votes


      12

      Online

      12.0k

      Users

      11.2k

      Topics

      113.4k

      Posts


      Copyright 2025 TBD   |   Forum Guidelines   |   Privacy Policy   |   Terms of Service
      • Login

      • Don't have an account? Register

      • Login or register to search.
      • First post
        Last post
      0
      • MySensors
      • OpenHardware.io
      • Categories
      • Recent
      • Tags
      • Popular