[dev] Re: [cvs] commit: imp message.php imp/lib Message.php

Michael M Slusarz slusarz@bigworm.colorado.edu
Thu, 20 Jun 2002 13:00:09 -0600


Quoting Michael M Slusarz <slusarz@bigworm.colorado.edu>:

| slusarz     2002/06/20 11:56:00 PDT
| 
|   Modified files:
|     .                    message.php 
|     lib                  Message.php 
|   Log:
|   Be much smarter about when to resort the mailbox index. Move the logic
| to
|   IMP_Message:: so that it is better tied to the events that would
| trigger
|   a resort.

This responds to some concerns/critiques Chuck was raising a few days ago 
about the amount of re-sorts done while viewing messages.  The re-sort 
triggers are listed in the phpdoc above needResort() - i won't repeat them 
here.  Let me know if this looks good.

And now that I am looking at that method name, maybe i should change it to 
needReSort().  i think we could all use a resort (vacation, that is).

Now on to tackling the indexing foolishness if new messages arrive while 
moving between messages.

michael

______________________________________________
Michael Slusarz [slusarz@bigworm.colorado.edu]
The University of Colorado at Boulder