[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kopete-devel
Subject:    [kopete-devel] [Bug 111688] IRC account properties automatically
From:       Tommi Rantala <tommi.rantala () cs ! helsinki ! fi>
Date:       2005-10-22 19:56:31
Message-ID: 20051022195631.4881.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=111688         




------- Additional Comments From tommi.rantala cs helsinki fi  2005-10-22 21:56 \
------- SVN commit 473173 by rantala:

CCBUG: 111688

Fix the second issue described in the report:
  "if I go to the Advanced Configuration tab and click Cancel, the next
  time I go into the account's Advanced Configuration tab, the selections
  for the Message Destinations area will all be off by one"

The setMessageDestinations() call is wrong (the values should all be +1,
so this caused the problem), and its actually not even needed.



 M  +0 -4      irceditaccountwidget.cpp  


--- branches/KDE/3.5/kdenetwork/kopete/protocols/irc/ui/irceditaccountwidget.cpp \
#473172:473173  @ -82,10 +82,6  @
 		informationReplies->setCurrentItem( config->readNumEntry( "InformationReplies", \
IRCAccount::ActiveWindow ) - 1 );  errorMessages->setCurrentItem( \
config->readNumEntry( "ErrorMessages", IRCAccount::ActiveWindow ) - 1 );  
-		account()->setMessageDestinations( serverNotices->currentItem(), \
                serverMessages->currentItem(),
-		informationReplies->currentItem(), errorMessages->currentItem()
-						     );
-
 		QStringList cmds = account()->connectCommands();
 		for( QStringList::Iterator i = cmds.begin(); i != cmds.end(); ++i )
 			new QListViewItem( commandList, *i );
_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic