Differential D27395 Diff 75671 kcms/componentchooser/componentchooserterminal.cpp

Changeset View

Standalone View

View Options

kcms/componentchooser/componentchooserterminal.cpp

Show First 20 Lines • Show All 84 Lines • ▼ Show 20 Line(s)56 TerminalSettings settings;
85 addItem(QIcon::fromTheme(QStringLiteral("application-x-shellscript")), i18n("Other..."), QStringLiteral()); 85​ addItem(QIcon::fromTheme(QStringLiteral("application-x-shellscript")), i18n("Other..."), QStringLiteral());
86 86​
87 emit changed(false); 87​ emit changed(false);
88} 88​}
89 89​
90void CfgTerminalEmulator::save(KConfig *) 90​void CfgTerminalEmulator::save(KConfig *)
91{ 91​{
92 const QString terminal = currentData().toString(); 92​ const QString terminal = currentData().toString();
93 m_currentIndex = currentIndex(); 93​ if (terminal.isEmpty()) {
94​ return;
95​ }
94 96​
95 TerminalSettings settings; 97​ TerminalSettings settings;
96 settings.setTerminalApplication(terminal); 98​ settings.setTerminalApplication(terminal);
97 settings.save(); 99​ settings.save();
98 100​
99 m_currentIndex = currentIndex(); 101​ m_currentIndex = currentIndex();
100 102​
101 QDBusMessage message = QDBusMessage::createMethodCall(QStringLiteral("org.kde.klauncher5"), 103​ QDBusMessage message = QDBusMessage::createMethodCall(QStringLiteral("org.kde.klauncher5"),
102 QStringLiteral("/KLauncher"), 104​ QStringLiteral("/KLauncher"),
103 QStringLiteral("org.kde.KLauncher"), 105​ QStringLiteral("org.kde.KLauncher"),
104 QStringLiteral("reparseConfiguration")); 106​ QStringLiteral("reparseConfiguration"));
105 QDBusConnection::sessionBus().send(message); 107​ QDBusConnection::sessionBus().send(message);
106 emit changed(false); 108​ emit changed(false);
107} 109​}
108 110​
109void CfgTerminalEmulator::selectTerminalApp() 111​void CfgTerminalEmulator::selectTerminalApp()
110{ 112​{
111 QList<QUrl> urlList; 113​ QList<QUrl> urlList;
112 KOpenWithDialog dlg(urlList, i18n("Select preferred terminal application:"), QString(), this); 114​ KOpenWithDialog dlg(urlList, i18n("Select preferred terminal application:"), QString(), this);
113 // hide "Run in &terminal" here, we don't need it for a Terminal Application 115​ // hide "Run in &terminal" here, we don't need it for a Terminal Application
114 dlg.hideRunInTerminal(); 116​ dlg.hideRunInTerminal();
115 dlg.setSaveNewApplications(true); 117​ dlg.setSaveNewApplications(true);
116 if (dlg.exec() != QDialog::Accepted) { 118​ if (dlg.exec() != QDialog::Accepted) {
117 setCurrentIndex(m_currentIndex); 119​ setCurrentIndex(m_currentIndex);
ervinUnsubmitted
Done

This comes back often in this patch and feels a bit cryptic, probably worth having a properly named method for it.

ervin: This comes back often in this patch and feels a bit cryptic, probably worth having a properly…
118 return; 120​ return;
119 } 121​ }
120 const auto service = dlg.service(); 122​ const auto service = dlg.service();
121 123​
122 // if the selected service is already in the list 124​ // if the selected service is already in the list
123 const auto matching = model()->match(model()->index(0,0), Qt::DisplayRole, service->exec()); 125​ const auto matching = model()->match(model()->index(0,0), Qt::DisplayRole, service->exec());
124 if (!matching.isEmpty()) { 126​ if (!matching.isEmpty()) {
125 const int index = matching.at(0).row(); 127​ const int index = matching.at(0).row();
Show All 12 Lines