search

mogutt / TTAndroidClient

The android client of mogutt
http://tt.mogu.io/
742 stars 283 forks source link

refactoring sending picture message #21

Closed tshirtmanid closed 10 years ago

tshirtmanid commented 10 years ago

current logic is full of SHIT and error prone, thoughts:

. send picture message should be handled in the IMMessageManager, not in GUI layer . notify using broadcast, just like other features, so GUI and IMMessageManager is weak bounds . read the original logic again . read the text re-sending logic again

tshirtmanid commented 10 years ago

git log commit 3125b5762d1b310824ca74fb5d407a14cadb3c28 Author: manshan manshan@mogujie.com Date: Mon Oct 13 09:56:21 2014 +0800

finish refactoring image download feature, now codes more clean

move logics to IMMessageManager, so not in UI layer any more