Verified Commit 1e240368 authored by Shashwat Jolly's avatar Shashwat Jolly Committed by Daniel Vrátil
Browse files

Fix bug where time of single instance of recurring event could not be changed

Summary:
BUG: 410758

- Removed QObject::connect calls in recurrenceactions.cpp as the required "clicked" signals are already handled in DialogButtonsHelper in kmessagebox
- Updated variable type for storing return value of createKMessageBox

Reviewers: mlaurent

Reviewed By: mlaurent

Subscribers: winterz, mlaurent, kde-pim

Tags: #kde_pim

Differential Revision: https://phabricator.kde.org/D28149
parent b939eacb
......@@ -212,8 +212,6 @@ int RecurrenceActions::questionSelectedAllCancel(const QString &message, const Q
QPushButton *okButton = buttonBox->button(QDialogButtonBox::Ok);
okButton->setDefault(true);
okButton->setShortcut(Qt::CTRL | Qt::Key_Return);
QObject::connect(buttonBox, &QDialogButtonBox::accepted, dialog.data(), &QDialog::accept);
QObject::connect(buttonBox, &QDialogButtonBox::rejected, dialog.data(), &QDialog::reject);
buttonBox->button(QDialogButtonBox::Ok)->setDefault(true);
bool checkboxResult = false;
......@@ -250,12 +248,10 @@ int RecurrenceActions::questionSelectedFutureAllCancel(const QString &message, c
QPushButton *okButton = buttonBox->button(QDialogButtonBox::Ok);
okButton->setDefault(true);
okButton->setShortcut(Qt::CTRL | Qt::Key_Return);
QObject::connect(buttonBox, &QDialogButtonBox::accepted, dialog.data(), &QDialog::accept);
QObject::connect(buttonBox, &QDialogButtonBox::rejected, dialog.data(), &QDialog::reject);
buttonBox->button(QDialogButtonBox::Ok)->setDefault(true);
bool checkboxResult = false;
int result = KMessageBox::createKMessageBox(
QDialogButtonBox::StandardButton result = KMessageBox::createKMessageBox(
dialog,
buttonBox,
QMessageBox::Question,
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment