From: Seth R. <se...@mi...> - 2003-08-29 00:32:00
|
I didn't get a good look at the changes that went in, are they optional? I don't even use quotas, so I see no reason to have the interface cluttered. I always figured that the other approach was just good error handling. Even with Chris' additions, the quota problem will still arise if someone doesn't hit the check box first. Seth. p dont think said: > Note that Chris has just put a different (and in some ways much simpler) > patch into CVS that addresses this issue. It's a checkbox on the > message list that allows the user to "Bypass Trash" when deleting. I > don't particularly like the idea of more crap on the interface > (especially crap that some people won't really understand), but the > implementation required very few changes to the rest of the SM codebase. > Chris, you should have chimed in on this thread to let people know that > you were working on the same issue (or did I miss a message somewhere?). > Are people happy with this solution? > > - Paul > >> -----Original Message----- >> From: squ...@li... >> [mailto:squ...@li...] On Behalf Of >> Alexandros Vellis >> Sent: Wednesday, August 27, 2003 9:56 AM >> To: squ...@li... >> Subject: Re: [SM-DEVEL] Re: [SM-PLUGINS] Direct Delete Plugin >> >> On Wed, 27 Aug 2003 10:57:13 +0200 (CEST) >> "Marc Groot Koerkamp" <ma...@sq...> wrote: >> >> > 2) We should store the capability response on login in the session > as >> > a key value array so we can access the earlier retrieved capability >> > response like: if ($aCaps['QUOTA' === true) then ... >> > All used capabilities are default set to false and after the $aCaps >> > arrray is filled a simple boolean check can be done to verify if the >> > capability exists. >> >> Er, squirrelmail does that already. :-D >> >> Everything is stored in the session. Mine (Cyrus) is: >> >> [sqimap_capabilities] => Array >> ( >> [IMAP4] => 1 >> [IMAP4rev1] => 1 >> [ACL] => 1 >> [QUOTA] => 1 >> [LITERAL+] => 1 >> [MAILBOX-REFERRALS] => 1 >> [NAMESPACE] => 1 >> [UIDPLUS] => 1 >> [ID] => 1 >> [NO_ATOMIC_RENAME] => 1 >> [UNSELECT] => 1 >> [CHILDREN] => 1 >> [MULTIAPPEND] => 1 >> [SORT] => 1 >> [THREAD] => REFERENCES >> [IDLE] => 1 >> [LOGINDISABLED] => 1 >> [LISTEXT] => 1 >> [LIST-SUBSCRIBED] => 1 >> [ANNOTATEMORE] => 1 >> [X-NETSCAPE] => 1 >> [AVELUSED] => 1 >> ) >> >> >> So there you go, in the patch I just sent, that ugly >> check_quota_capability() can get straight out of there. >> >> Alexandros >> >> >> > > > > > ------------------------------------------------------- > This sf.net email is sponsored by:ThinkGeek > Welcome to geek heaven. > http://thinkgeek.com/sf > -- > squirrelmail-devel mailing list > List Address: squ...@li... > List Archives: http://sourceforge.net/mailarchive/forum.php?forum_id=7139 > List Info: https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel > -- Seth Randall IT Support Specialist Missoula Federal Credit Union se...@mi... |