[Konversation-devel] Review Request 112733: Fix numerous topic related encryption bugs for 1.5

Travis McHenry wordsizzle at gmail.com
Sun Sep 15 06:54:52 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112733/
-----------------------------------------------------------

Review request for Konversation.


Description
-------

Fix numerous topic related encryption bugs

Added a QSortFilterProxyModel that is put behind the various
views that use the topicHistoryModel. When equipped with a
'cipher' object it will begin deciphering encrypted content

Took the decryption for topics out of server.cpp, this caused
decrypted text to get into the model, they are now decrypted
in the cipherFilterModel, and in the channel messages that go
to ircview. When the cipher is deleted, the topics in topicLine
and in TopicHistoryView return to their encrypted forms. They
remain decrypted in ircview however.

Changed some of the logic in decrypting topics so that when
nothing is decrypted we return nothing, and when something IS
decrypted, we return only the decrypted text, no prefixes. These
prefixes are now added by the cipherFilterModel. The ERROR:
prefix denoting unencrypted text has now been changed to (u) to
be more in line with the (e) for encrypted text, and make it the
same character length, so it's easier to strip later.


Diffs
-----

  src/viewer/channeloptionsdialog.cpp c1d19e2cdac7cc9fa9f7601ceed4fb657c83eca4 
  src/irc/topichistorymodel.cpp ace208558ba51f72076d145a915ed6c12e36e69c 
  src/irc/topichistorymodel.h f3c42ab8e732de8edde83f972248bb00442dbc0a 
  src/irc/server.cpp cd60d70c3bddc9d87c2783bab7c5fc79c27d6628 
  src/irc/channel.cpp 6911473e19919e24975fcca3b1a6ff14fa04c420 
  src/irc/channel.h 355bf61d4099fdd80f1b1d7056943bc88030aa8a 
  src/cipher.cpp 514e3903397ad85a75e836ca238138c77e82d4cd 

Diff: http://git.reviewboard.kde.org/r/112733/diff/


Testing
-------


Thanks,

Travis McHenry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konversation-devel/attachments/20130915/f4a269f8/attachment.html>


More information about the Konversation-devel mailing list