summaryrefslogtreecommitdiff
path: root/noncore
Side-by-side diff
Diffstat (limited to 'noncore') (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/settings/aqpkg/settingsimpl.cpp114
1 files changed, 72 insertions, 42 deletions
diff --git a/noncore/settings/aqpkg/settingsimpl.cpp b/noncore/settings/aqpkg/settingsimpl.cpp
index 9f611da..e2afada 100644
--- a/noncore/settings/aqpkg/settingsimpl.cpp
+++ b/noncore/settings/aqpkg/settingsimpl.cpp
@@ -283,16 +283,26 @@ void SettingsImpl :: setupData()
//------------------ Servers tab ----------------------
void SettingsImpl :: editServer( int sel )
{
currentSelectedServer = sel;
Server *s = dataMgr->getServer( servers->currentText() );
- serverName = s->getServerName();
- servername->setText( s->getServerName() );
- serverurl->setText( s->getServerUrl() );
- active->setChecked( s->isServerActive() );
+ if ( s )
+ {
+ serverName = s->getServerName();
+ servername->setText( s->getServerName() );
+ serverurl->setText( s->getServerUrl() );
+ active->setChecked( s->isServerActive() );
+ }
+ else
+ {
+ serverName = "";
+ servername->setText( "" );
+ serverurl->setText( "" );
+ active->setChecked( false );
+ }
}
void SettingsImpl :: newServer()
{
newserver = true;
servername->setText( "" );
@@ -302,14 +312,17 @@ void SettingsImpl :: newServer()
}
void SettingsImpl :: removeServer()
{
changed = true;
Server *s = dataMgr->getServer( servers->currentText() );
- dataMgr->getServerList().removeRef( s );
- servers->removeItem( currentSelectedServer );
+ if ( s )
+ {
+ dataMgr->getServerList().removeRef( s );
+ servers->removeItem( currentSelectedServer );
+ }
}
void SettingsImpl :: changeServerDetails()
{
changed = true;
@@ -329,27 +342,28 @@ void SettingsImpl :: changeServerDetails()
newName = tmpStr;
delete tmpStr;
if ( !newserver )
{
Server *s = dataMgr->getServer( servers->currentText() );
-
- // Update url
- s->setServerUrl( serverurl->text() );
- s->setActive( active->isChecked() );
-
-
- // Check if server name has changed, if it has then we need to replace the key in the map
- if ( serverName != newName )
+ if ( s )
{
- // Update server name
- s->setServerName( newName );
- }
+ // Update url
+ s->setServerUrl( serverurl->text() );
+ s->setActive( active->isChecked() );
+
+ // Check if server name has changed, if it has then we need to replace the key in the map
+ if ( serverName != newName )
+ {
+ // Update server name
+ s->setServerName( newName );
+ }
- // Update list box
- servers->changeItem( newName, currentSelectedServer );
+ // Update list box
+ servers->changeItem( newName, currentSelectedServer );
+ }
}
else
{
Server s( newName, serverurl->text() );
dataMgr->getServerList().append( new Server( newName, serverurl->text() ) );
dataMgr->getServerList().last()->setActive( active->isChecked() );
@@ -360,18 +374,28 @@ void SettingsImpl :: changeServerDetails()
}
//------------------ Destinations tab ----------------------
void SettingsImpl :: editDestination( int sel )
{
- currentSelectedDestination = sel;
+ currentSelectedDestination = sel;
Destination *d = dataMgr->getDestination( destinations->currentText() );
- destinationName = d->getDestinationName();
- destinationname->setText( d->getDestinationName() );
- destinationurl->setText( d->getDestinationPath() );
- linkToRoot->setChecked( d->linkToRoot() );
+ if ( d )
+ {
+ destinationName = d->getDestinationName();
+ destinationname->setText( d->getDestinationName() );
+ destinationurl->setText( d->getDestinationPath() );
+ linkToRoot->setChecked( d->linkToRoot() );
+ }
+ else
+ {
+ destinationName = "";
+ destinationname->setText( "" );
+ destinationurl->setText( "" );
+ linkToRoot->setChecked( false );
+ }
}
void SettingsImpl :: newDestination()
{
newdestination = true;
destinationname->setText( "" );
@@ -381,14 +405,17 @@ void SettingsImpl :: newDestination()
}
void SettingsImpl :: removeDestination()
{
changed = true;
Destination *d = dataMgr->getDestination( destinations->currentText() );
- dataMgr->getDestinationList().removeRef( d );
- destinations->removeItem( currentSelectedDestination );
+ if ( d )
+ {
+ dataMgr->getDestinationList().removeRef( d );
+ destinations->removeItem( currentSelectedDestination );
+ }
}
void SettingsImpl :: changeDestinationDetails()
{
changed = true;
@@ -398,32 +425,35 @@ void SettingsImpl :: changeDestinationDetails()
#endif
QString newName = destinationname->text();
if ( !newdestination )
{
Destination *d = dataMgr->getDestination( destinations->currentText() );
+ if ( d )
+ {
+ // Update url
+ d->setDestinationPath( destinationurl->text() );
+ d->linkToRoot( linkToRoot->isChecked() );
- // Update url
- d->setDestinationPath( destinationurl->text() );
- d->linkToRoot( linkToRoot->isChecked() );
+ // Check if server name has changed, if it has then we need to replace the key in the map
+ if ( destinationName != newName )
+ {
+ // Update server name
+ d->setDestinationName( newName );
- // Check if server name has changed, if it has then we need to replace the key in the map
- if ( destinationName != newName )
- {
- // Update server name
- d->setDestinationName( newName );
+ // Update list box
+ destinations->changeItem( newName, currentSelectedDestination );
+ }
- // Update list box
- destinations->changeItem( newName, currentSelectedDestination );
- }
#ifdef QWS
- QString key = newName;
- key += "_linkToRoot";
- int val = d->linkToRoot();
- cfg.writeEntry( key, val );
-#endif
+ QString key = newName;
+ key += "_linkToRoot";
+ int val = d->linkToRoot();
+ cfg.writeEntry( key, val );
+#endif
+ }
}
else
{
dataMgr->getDestinationList().append( new Destination( newName, destinationurl->text() ) );
destinations->insertItem( newName );
destinations->setCurrentItem( destinations->count() );