author | drw <drw> | 2004-11-16 17:52:25 (UTC) |
---|---|---|
committer | drw <drw> | 2004-11-16 17:52:25 (UTC) |
commit | e83f5b12337c22f365323aa00806b638a971e3a2 (patch) (unidiff) | |
tree | fc9c4a34ffc7c2400ccd373f6947626cded6075f | |
parent | 3ba09273d96a32263bb22bfbc32fcc6290bde6e8 (diff) | |
download | opie-e83f5b12337c22f365323aa00806b638a971e3a2.zip opie-e83f5b12337c22f365323aa00806b638a971e3a2.tar.gz opie-e83f5b12337c22f365323aa00806b638a971e3a2.tar.bz2 |
Minor UI tweak - fix double entry in source feed configuration when adding a new feed
-rw-r--r-- | noncore/settings/packagemanager/oipkgconfigdlg.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/noncore/settings/packagemanager/oipkgconfigdlg.cpp b/noncore/settings/packagemanager/oipkgconfigdlg.cpp index ea0aef0..25052f8 100644 --- a/noncore/settings/packagemanager/oipkgconfigdlg.cpp +++ b/noncore/settings/packagemanager/oipkgconfigdlg.cpp | |||
@@ -514,65 +514,65 @@ void OIpkgConfigDlg::slotServerUpdate() | |||
514 | { | 514 | { |
515 | // Find selected server in list | 515 | // Find selected server in list |
516 | OConfItem *server = findConfItem( OConfItem::Source, m_serverCurrName ); | 516 | OConfItem *server = findConfItem( OConfItem::Source, m_serverCurrName ); |
517 | 517 | ||
518 | // Delete server | 518 | // Delete server |
519 | if ( server ) | 519 | if ( server ) |
520 | { | 520 | { |
521 | // Update url | 521 | // Update url |
522 | server->setValue( m_serverLocation->text() ); | 522 | server->setValue( m_serverLocation->text() ); |
523 | server->setActive( m_serverActive->isChecked() ); | 523 | server->setActive( m_serverActive->isChecked() ); |
524 | 524 | ||
525 | // Check if server name has changed, if it has then we need to replace the key in the map | 525 | // Check if server name has changed, if it has then we need to replace the key in the map |
526 | if ( m_serverCurrName != newName ) | 526 | if ( m_serverCurrName != newName ) |
527 | { | 527 | { |
528 | // Update server name | 528 | // Update server name |
529 | server->setName( newName ); | 529 | server->setName( newName ); |
530 | 530 | ||
531 | // Update list box | 531 | // Update list box |
532 | m_serverList->changeItem( newName, m_serverCurrent ); | 532 | m_serverList->changeItem( newName, m_serverCurrent ); |
533 | } | 533 | } |
534 | } | 534 | } |
535 | } | 535 | } |
536 | else | 536 | else |
537 | { | 537 | { |
538 | // Add new destination to configuration list | 538 | // Add new destination to configuration list |
539 | m_configs->append( new OConfItem( OConfItem::Source, newName, | 539 | m_configs->append( new OConfItem( OConfItem::Source, newName, |
540 | m_serverLocation->text(), m_serverActive->isChecked() ) ); | 540 | m_serverLocation->text(), m_serverActive->isChecked() ) ); |
541 | m_configs->sort(); | 541 | m_configs->sort(); |
542 | 542 | ||
543 | m_serverList->insertItem( newName ); | 543 | m_serverList->insertItem( newName ); |
544 | m_serverList->setCurrentItem( m_serverList->count() ); | 544 | m_serverList->setCurrentItem( m_serverList->count() ); |
545 | m_serverNew = false; | 545 | m_serverNew = false; |
546 | m_serverList->insertItem( newName ); | 546 | // m_serverList->insertItem( newName ); |
547 | } | 547 | } |
548 | } | 548 | } |
549 | 549 | ||
550 | void OIpkgConfigDlg::slotDestEdit( int index ) | 550 | void OIpkgConfigDlg::slotDestEdit( int index ) |
551 | { | 551 | { |
552 | m_destNew = false; | 552 | m_destNew = false; |
553 | m_destCurrent = index; | 553 | m_destCurrent = index; |
554 | 554 | ||
555 | // Find selected destination in list | 555 | // Find selected destination in list |
556 | OConfItem *destination = findConfItem( OConfItem::Destination, m_destList->currentText() ); | 556 | OConfItem *destination = findConfItem( OConfItem::Destination, m_destList->currentText() ); |
557 | 557 | ||
558 | // Display destination details | 558 | // Display destination details |
559 | if ( destination ) | 559 | if ( destination ) |
560 | { | 560 | { |
561 | m_destCurrName = destination->name(); | 561 | m_destCurrName = destination->name(); |
562 | m_destName->setText( destination->name() ); | 562 | m_destName->setText( destination->name() ); |
563 | m_destLocation->setText( destination->value() ); | 563 | m_destLocation->setText( destination->value() ); |
564 | m_destActive->setChecked( destination->active() ); | 564 | m_destActive->setChecked( destination->active() ); |
565 | m_destName->setFocus(); | 565 | m_destName->setFocus(); |
566 | } | 566 | } |
567 | } | 567 | } |
568 | 568 | ||
569 | void OIpkgConfigDlg::slotDestNew() | 569 | void OIpkgConfigDlg::slotDestNew() |
570 | { | 570 | { |
571 | m_destNew = true; | 571 | m_destNew = true; |
572 | 572 | ||
573 | m_destName->setText( QString::null ); | 573 | m_destName->setText( QString::null ); |
574 | m_destLocation->setText( QString::null ); | 574 | m_destLocation->setText( QString::null ); |
575 | m_destActive->setChecked( true ); | 575 | m_destActive->setChecked( true ); |
576 | m_destName->setFocus(); | 576 | m_destName->setFocus(); |
577 | } | 577 | } |
578 | 578 | ||