summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTimothy Pearson <kb9vqf@pearsoncomputing.net>2014-07-07 19:51:30 -0500
committerTimothy Pearson <kb9vqf@pearsoncomputing.net>2014-07-07 19:51:30 -0500
commit8231cc23f3e8b266a6b0e229865905233e867cbf (patch)
treec3392a7bf9a843f9f46de3bee0ec895ce4d2a068
parent93aee1b625df28b290eaf0779ee241ddce27cb69 (diff)
downloadtdepim-8231cc23f3e8b266a6b0e229865905233e867cbf.tar.gz
tdepim-8231cc23f3e8b266a6b0e229865905233e867cbf.zip
Fix korgac abort on startup due to oversized tray icon creation when icon widget has not yet been shown
Properly print caldav/carddav wallet warnings to prevent backtrace spew
-rw-r--r--knotes/knotesapp.cpp16
-rw-r--r--knotes/knotesapp.h2
-rw-r--r--korganizer/korgac/alarmdockwindow.cpp16
-rw-r--r--korganizer/korgac/alarmdockwindow.h4
-rw-r--r--tderesources/caldav/preferences.cpp16
-rw-r--r--tderesources/carddav/preferences.cpp16
6 files changed, 49 insertions, 21 deletions
diff --git a/knotes/knotesapp.cpp b/knotes/knotesapp.cpp
index 26aee48c3..14fd625b9 100644
--- a/knotes/knotesapp.cpp
+++ b/knotes/knotesapp.cpp
@@ -105,7 +105,7 @@ KNotesApp::KNotesApp()
KWin::setSystemTrayWindowFor( winId(), tqt_xrootwin() );
TQToolTip::add( this, i18n( "KNotes: Sticky notes for TDE" ) );
setBackgroundMode( X11ParentRelative );
- setPixmap( KSystemTray::loadSizedIcon( "knotes", TQWidget::width() ) );
+ setPixmap( KSystemTray::loadIcon( "knotes" ) );
// set the initial style
KNote::setStyle( KNotesGlobalConfig::style() );
@@ -210,7 +210,7 @@ KNotesApp::KNotesApp()
updateNoteActions();
}
-void KNotesApp::resizeEvent ( TQResizeEvent * )
+void KNotesApp::resizeTrayIcon ()
{
// Honor Free Desktop specifications that allow for arbitrary system tray icon sizes
TQPixmap origpixmap;
@@ -223,6 +223,18 @@ void KNotesApp::resizeEvent ( TQResizeEvent * )
setPixmap(scaledpixmap);
}
+void KNotesApp::resizeEvent ( TQResizeEvent * )
+{
+ // Honor Free Desktop specifications that allow for arbitrary system tray icon sizes
+ resizeTrayIcon();
+}
+
+void KNotesApp::showEvent ( TQShowEvent * )
+{
+ // Honor Free Desktop specifications that allow for arbitrary system tray icon sizes
+ resizeTrayIcon();
+}
+
KNotesApp::~KNotesApp()
{
saveNotes();
diff --git a/knotes/knotesapp.h b/knotes/knotesapp.h
index 2c2656f3a..0888e20bb 100644
--- a/knotes/knotesapp.h
+++ b/knotes/knotesapp.h
@@ -104,6 +104,7 @@ public slots:
protected:
void mousePressEvent( TQMouseEvent* );
void resizeEvent ( TQResizeEvent * );
+ void showEvent ( TQShowEvent * );
protected slots:
void slotShowNote();
@@ -122,6 +123,7 @@ protected slots:
private:
void showNote( KNote *note ) const;
void saveConfigs();
+ void resizeTrayIcon();
private slots:
void acceptConnection();
diff --git a/korganizer/korgac/alarmdockwindow.cpp b/korganizer/korgac/alarmdockwindow.cpp
index f8f15addf..1f5d5b33c 100644
--- a/korganizer/korgac/alarmdockwindow.cpp
+++ b/korganizer/korgac/alarmdockwindow.cpp
@@ -58,8 +58,8 @@ AlarmDockWindow::AlarmDockWindow( const char *name )
// Set up icons
TDEGlobal::iconLoader()->addAppDir( "korgac" );
- mPixmapEnabled = loadSizedIcon( "korgac", width() );
- mPixmapDisabled = loadSizedIcon( "korgac_disabled", width() );
+ mPixmapEnabled = loadIcon( "korgac" );
+ mPixmapDisabled = loadIcon( "korgac_disabled" );
setPixmap( alarmsEnabled ? mPixmapEnabled : mPixmapDisabled );
@@ -104,7 +104,7 @@ AlarmDockWindow::~AlarmDockWindow()
{
}
-void AlarmDockWindow::resizeEvent ( TQResizeEvent * )
+void AlarmDockWindow::resizeTrayIcon ()
{
// Honor Free Desktop specifications that allow for arbitrary system tray icon sizes
mPixmapEnabled = loadSizedIcon( "korgac", width() );
@@ -115,6 +115,16 @@ void AlarmDockWindow::resizeEvent ( TQResizeEvent * )
setPixmap( alarmsEnabled ? mPixmapEnabled : mPixmapDisabled );
}
+void AlarmDockWindow::resizeEvent ( TQResizeEvent * )
+{
+ resizeTrayIcon();
+}
+
+void AlarmDockWindow::showEvent ( TQShowEvent * )
+{
+ resizeTrayIcon();
+}
+
void AlarmDockWindow::slotUpdate( int reminders )
{
TQToolTip::remove( this );
diff --git a/korganizer/korgac/alarmdockwindow.h b/korganizer/korgac/alarmdockwindow.h
index 6f317fda8..6f6cd7bdb 100644
--- a/korganizer/korgac/alarmdockwindow.h
+++ b/korganizer/korgac/alarmdockwindow.h
@@ -52,6 +52,7 @@ class AlarmDockWindow : public KSystemTray
void mousePressEvent( TQMouseEvent * );
// void closeEvent( TQCloseEvent * );
void resizeEvent ( TQResizeEvent * );
+ void showEvent ( TQShowEvent * );
protected slots:
void slotQuit();
@@ -59,6 +60,9 @@ class AlarmDockWindow : public KSystemTray
void slotDismissAll();
private:
+ void resizeTrayIcon();
+
+ private:
TQPixmap mPixmapEnabled;
TQPixmap mPixmapDisabled;
TQString mName;
diff --git a/tderesources/caldav/preferences.cpp b/tderesources/caldav/preferences.cpp
index a4be7cc13..a9ba68785 100644
--- a/tderesources/caldav/preferences.cpp
+++ b/tderesources/caldav/preferences.cpp
@@ -47,12 +47,12 @@ bool CalDavPrefs::setWalletFolder(const TQString& folder) {
if (!mWallet->hasFolder(folder)) {
if (!mWallet->createFolder(folder)) {
ret = false;
- kdWarning() << "can't create the wallet folder for CalDAV passwords";
+ kdWarning() << "can't create the wallet folder for CalDAV passwords\n";
}
}
if (!mWallet->setFolder(folder)) {
ret = false;
- kdWarning() << "can't set the wallet folder for CalDAV passwords";
+ kdWarning() << "can't set the wallet folder for CalDAV passwords\n";
}
} else {
// the wallet is inaccessible or not configured
@@ -69,18 +69,18 @@ Wallet* CalDavPrefs::getWallet() {
// the wallet is not marked as inaccessible
if (NULL == mWallet) {
- kdDebug() << "creating wallet for " + mPrefix << '\n';
+ kdDebug() << "creating wallet for " << mPrefix << '\n';
mWallet = Wallet::openWallet(Wallet::NetworkWallet(), 0);
if (NULL == mWallet) {
mNoWallet = true; // can't open the wallet, mark it inaccessible
- kdWarning() << "can't create a wallet for CalDAV passwords";
+ kdWarning() << "can't create a wallet for CalDAV passwords\n";
} else {
if (setWalletFolder(WALLET_FOLDER)) {
// reserved
} else {
// can't set the wallet folder, remove the wallet and mark it inaccessible
- kdWarning() << "can't set the walet folder for CalDAV passwords";
+ kdWarning() << "can't set the walet folder for CalDAV passwords\n";
removeWallet(true);
}
}
@@ -115,7 +115,7 @@ bool CalDavPrefs::writePasswordToWallet(const TQString& password) {
if (NULL != w) {
int rc = w->writePassword(mPrefix + WALLET_PWD_SUFFIX, password);
if (0 != rc) {
- kdWarning() << "CalDAV: can't write password to the wallet";
+ kdWarning() << "CalDAV: can't write password to the wallet\n";
} else {
ret = true;
}
@@ -136,7 +136,7 @@ bool CalDavPrefs::readPasswordFromWallet(TQString& password) {
password = p;
ret = true;
} else {
- kdWarning() << "CalDAV: can't read password from the wallet";
+ kdWarning() << "CalDAV: can't read password from the wallet\n";
password = NO_PASSWORD;
}
}
@@ -154,7 +154,7 @@ bool CalDavPrefs::removePasswordFromWallet() {
if (0 == rc) {
ret = true;
} else {
- kdWarning() << "CalDAV: can't remove password from the wallet";
+ kdWarning() << "CalDAV: can't remove password from the wallet\n";
}
}
diff --git a/tderesources/carddav/preferences.cpp b/tderesources/carddav/preferences.cpp
index a37931741..3be1f21a8 100644
--- a/tderesources/carddav/preferences.cpp
+++ b/tderesources/carddav/preferences.cpp
@@ -46,12 +46,12 @@ bool CardDavPrefs::setWalletFolder(const TQString& folder) {
if (!mWallet->hasFolder(folder)) {
if (!mWallet->createFolder(folder)) {
ret = false;
- kdWarning() << "can't create the wallet folder for CardDAV passwords";
+ kdWarning() << "can't create the wallet folder for CardDAV passwords\n";
}
}
if (!mWallet->setFolder(folder)) {
ret = false;
- kdWarning() << "can't set the wallet folder for CardDAV passwords";
+ kdWarning() << "can't set the wallet folder for CardDAV passwords\n";
}
} else {
// the wallet is inaccessible or not configured
@@ -68,18 +68,18 @@ Wallet* CardDavPrefs::getWallet() {
// the wallet is not marked as inaccessible
if (NULL == mWallet) {
- kdDebug() << "creating wallet for " + mPrefix << '\n';
+ kdDebug() << "creating wallet for " << mPrefix << '\n';
mWallet = Wallet::openWallet(Wallet::NetworkWallet(), 0);
if (NULL == mWallet) {
mNoWallet = true; // can't open the wallet, mark it inaccessible
- kdWarning() << "can't create a wallet for CardDAV passwords";
+ kdWarning() << "can't create a wallet for CardDAV passwords\n";
} else {
if (setWalletFolder(WALLET_FOLDER)) {
// reserved
} else {
// can't set the wallet folder, remove the wallet and mark it inaccessible
- kdWarning() << "can't set the walet folder for CardDAV passwords";
+ kdWarning() << "can't set the walet folder for CardDAV passwords\n";
removeWallet(true);
}
}
@@ -114,7 +114,7 @@ bool CardDavPrefs::writePasswordToWallet(const TQString& password) {
if (NULL != w) {
int rc = w->writePassword(mPrefix + WALLET_PWD_SUFFIX, password);
if (0 != rc) {
- kdWarning() << "CardDAV: can't write password to the wallet";
+ kdWarning() << "CardDAV: can't write password to the wallet\n";
} else {
ret = true;
}
@@ -135,7 +135,7 @@ bool CardDavPrefs::readPasswordFromWallet(TQString& password) {
password = p;
ret = true;
} else {
- kdWarning() << "CardDAV: can't read password from the wallet";
+ kdWarning() << "CardDAV: can't read password from the wallet\n";
password = NO_PASSWORD;
}
}
@@ -153,7 +153,7 @@ bool CardDavPrefs::removePasswordFromWallet() {
if (0 == rc) {
ret = true;
} else {
- kdWarning() << "CardDAV: can't remove password from the wallet";
+ kdWarning() << "CardDAV: can't remove password from the wallet\n";
}
}