Unable to change bools in C++ struct

[mzimmers]

@JoeCFD said in Unable to change bools in C++ struct:

@mzimmers bool m_maintenance; // now
is undefined.

I realize that, and that's not acceptable. But currently I have the options of leaving it undefined, or initializing it and (somehow) making it QML-immutable in the process.

I can't even modify it if I initialize it in QML, like this:

    property schedule newSchedule: ({ "maintenance": false })

[mzimmers]

@JoeCFD said in Unable to change bools in C++ struct:

can you try:
bool m_maintenance{ false };

Same (bad) result. Thanks for the suggestion, though.

[JoeCFD]

@mzimmers
try
class Schedule

not struct Schedule
.

[mzimmers]

@JoeCFD no change in behavior.

[GrecKo]

@JoeCFD struct is just a class where everything is public by default.

[GrecKo]

@mzimmers what's the issue on this output? We don't know which field is which. Also why are you using m_maintenance in QML? That's not the name of your property.

[mzimmers]

@GrecKo I don't know how that got in there, but that wasn't how it was originally, and it's not how it is now. My screenshots above are correct.

[mzimmers]

OK, guys - here's a minimal repeatable example:

// schedule.h
#pragma once

#include <QObject>
#include <QMetaType>
#include <QtQml/qqmlregistration.h>
#include <QtQmlIntegration/QtQmlIntegration>

struct Schedule
{
    Q_GADGET
    QML_VALUE_TYPE(schedule)
    QML_STRUCTURED_VALUE
    Q_PROPERTY(bool maintenance MEMBER m_maintenance)
public:
    bool m_maintenance;
    Q_INVOKABLE Schedule() {}
    bool operator ==(const Schedule &rhs) const {return true;}
};

// Main.qml
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts
import QtQuick.Window

Window {
    id: mainWindow
    width: 640
    height: 480
    visible: true

    property schedule newSchedule: ({  })

    Switch {
        onClicked: {
            console.log("switch is " + checked)
            newSchedule.maintenance = checked
            console.log("newSchedule.maintenance is " + newSchedule.maintenance)
        }
    }
}

Output:

qml: switch is true
qml: newSchedule.maintenance is false
qml: switch is false
qml: newSchedule.maintenance is false
qml: switch is true
qml: newSchedule.maintenance is false
qml: switch is false
qml: newSchedule.maintenance is false

I can provide the cmake file as well if desired, but it doesn't contain anything unusual.

[GrecKo]

@mzimmers said in Unable to change bools in C++ struct:

bool operator ==(const Schedule &rhs) const {return true;}

If you remove this non-sensical operator== your code works as expected.

My guess is that the engine modifies a temp Schedule internally, checks the equality and assign it only if it's different. As it's always equal it won't modify it.

[mzimmers]

@GrecKo bingo!

I'd just stubbed out that comparison function to satisfy the compiler, but you were absolutely right. In my app, I'd neglected to include the maintenance member in my comparison function. Seems to work fine now.

Kind of has me wondering -- if I set my compiler options to no optimization, I wonder if it would have worked...

Anyway, thanks to EVERYONE for the help on this.