Re: [bme-develop] Final BeMSN Release Progress

[Simon T. <sim...@ga...>]

> Hello ...
> 
> > Hi guys,
> > 
> > I've tested my hotmail code and it works! it generates a html file 
> > that can 
> > sign in to hotmail! But we have some problems in the program 
> > structure! 
> > first thing is the password! we need it to log in to hotmail but 
> > it's 
> > stored 
> > nowhere!
> Good thing it works, that is great. You are right, the password is 
> not 
> being stored anywhere, it is only stored temporally in 
> 	char * MsnNSHandler::pass  (see MsnNSHandler::initMsnSession).
> but it is destroyed after the user signs in (see 
> MsnNSHandler::handleUSR).
> I like the idea of a User class derived from Contact where we can 
> hold 
> this information. Common::yo will be an object of this type (yo means 
> I, the pronoun, in spanish).

Yes sounds good to me. Do you need the actual password or can we store 
it MD5 hashed or something? Keeping passwords in memory as plain text 
should be avoided if possible I think.

> > 
> > Secondly we get a bunch of URL information from the hotmail server, 
> > see 
> > http://www.hypothetic.org/docs/msn/notification/miscellaneous.php, 
> > I 
> > need 
> > some of those....now they are hardcoded into the program....so 
> > where 
> > do we 
> > store them?
> How about using the MsnProtocol class which is not being used right 
> now.

Dunno what that is, but sounds ok! :D

[snip]
 
> > because we have a lot of protocol information and don't know where 
> > to 
> > store 
> > it I suggest making a class for it that we can query....Protocol or 
> > smth....or does anyone have a better idea! I think I will also 
> > change 
> > the 
> > profile info to a map structure....but leave it some time as it 
> > is!(it works 
> > now)

You got my previous email about using a few functions to get the data? 
We could just use them and store all the bits we need in a new protocol 
type structure. Or the map thing could work, but as we know what the 
fields are and which ones we need, the overhead of such a general 
container may not be worth it.

> > I really like to get started on changing the contact info classes 
> > as 
> > this is 
> > also needed for the hotmail integration(password)....I suggest 
> > making 
> > four 
> > new classes...
> > -two data classes: Contact and User(derives from contact,stores the 
> > password), these classes make it possible to cache contacts and 
> > users!

Yup OK.

> > -a ContactItem class, this is the list item that draws the contacts

Fine.

> > -a ListProducer class or smth that takes the Contact classes and 
> > turns them 
> > into corresponding ContactItem classes and puts them in a 
> > BOutlineListView(groups included)

I've said before it would be nice if objects could register their 
interest in the contact list with whatever ends up storing it, for any 
changes, so chat windows and the main contact list will both be updated 
when anybody changes their screen name for example.

> > what do you think about all of this?
> Just keep in mind that there are places where the program needs to 
> find 
> a particular Contact object (MsnBuddy as of now). This happens by 
> calling Common::getMsnBuddy, and this function works by sending a 
> BMessage to the list that holds all the Contact objects 
> (ContactListView as of now), which replys with a pointer to the 
> object 
> if it finds it. If we separate the Contact info from the 
> ContactListItem we need a looper to hold the Contact list (maybe 
> MsnNSHandler), so it can respond to Common::getMsnBuddy. 

It makes sense to me to have NS handler be responsible for the contact 
list.
 
> Other than that I think it is a good idea to separate the views/app 
> code from the protocol specific code. 

Me too.

Simon

> > regards,
> > 
> > Tim
> > 
> 
> Regards,
> Daniel
> 
> 
> 
> -------------------------------------------------------
> SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank 
> Media
> 100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
> Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
> http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
> _______________________________________________
> bme-develop mailing list
> bme...@li...
> https://lists.sourceforge.net/lists/listinfo/bme-develop