Aborting undo command in an undo stack - c++

Question

I'm calling an undo command from within QML:
cppClass.undoHandler.createCommand(option)
The C++ undo-handler pushes undo command into undo stack:
void UndoHandler::createCommand(
const QString & option
)
{
m_undoStack->push(new Command(
option
));
}
The actual C++ undo command is:
class Command : public QUndoCommand
{
// ...
virtual void undo();
virtual void redo();
// ...
Logic *m_logic;
}
void Command::redo()
{
m_outputName = m_logic->run(m_option);
}
And the logic runs like this:
QString Logic::run(const QString option)
{
if ( /* some condition here */ ) {
// ** What I want to achieve:
// If this condition is met
// abort the current undo-command
// I mean pop the current undo-command which is already pushed into stack
}
}
How can I abort/pop/break the undo-command by the above condition inside the logic? I'm not sure how it should be designed, any idea?

Answer 1

You can use QUndoCommand::setObsolete() and make the command obsolete.
The boolean is used for the automatic removal of commands that are not necessary in the stack anymore. The isObsolete function is checked in the functions QUndoStack::push(), QUndoStack::undo(), QUndoStack::redo(), and QUndoStack::setIndex().

Answer 2

I used the if condition before pushing undo command into stack like this:
void UndoHandler::createCommand(
const QString & option
)
{
Command *command = new Command(option);
// Check validity before pushing undo command into undo stack
bool isValid = command->doubleCheck();
if (isValid) {
m_undoStack->push(command);
} else {
delete command;
}
}
Undo command has a new method to check the if condition:
class Command : public QUndoCommand
{
// ...
virtual void undo();
virtual void redo();
// ...
// New method to check validity
bool doubleCheck();
// ...
Logic *m_logic;
const QString m_option;
}
bool Command::doubleCheck() {
return m_logic->doubleCheck(m_option);
}
void Command::redo()
{
// Redo method has to be modified accordingly
}
The if condition is inside a separate method in logic:
bool Logic::doubleCheck(const QString option)
{
if ( /* some condition here */ )
return true;
else
return false;
}