Fixed a bug which was logging out current user during a manual retrieval of emails.

- Theme system updates: theme() function return their data now.

- table(...) -> theme("table", ...")

- Break of when mail is processed by a mailhandler.
- Minor syntax cleanup and removal of redudant code.

updated mailhandler_node_submit() so it uses new functions like node_validate(). now does more consistent validation and access control ... also sends better error replies when a submission fails.

- Updated to work with register_globals turned off.  Patch by Michael Frankowski.

Patch to make the help text # comments work with cleanURLs, even when you're using a theme that sets baseurl.

Moved admin stuff into the modules section. I think all modules should behave like this. It makes life much more hierarchical and nice.

  'theme("function")' instead of '$theme->function()'.  Hopefully I didn't
  break anything.  If I did, let me know.

  The following modules need special attention (but should still work as is):

  1. The ldap_integration directory has its own user.module; I haven't updated
     the documentation bit that talks about themes.

  2. The hotlist module uses '$theme->hotlist_del_icon'.  Should be avoided
     and somehow wrapped in a (larger) call to theme().

  3. The image module has its own way of doing what 'theme()' does (or the
     former 'theme_invoke()').  Constructs like the one below (a) look wrong
     and (b) should be rewritten to properly use 'theme()'.  For example:

     $theme->box("foo", method_exists("bar") ? $theme->bar() : $bar);

     Search for "image_gallery_album", "image_gallery_home" and
     "image_thumb_browser".  I've already changed '$theme->box()' to
     'theme()'.

  4. The sidebar module uses '$theme->foreground' and '$theme->background'.
     Should be avoided and somehow wrapped in a (larger) call to theme().

  5. The org_informations module uses '$theme->separator()', which is new to
     me.

  6. The taxonomy_filter module uses '$theme->taxolist_filter_item()'.  It
     should use 'theme("taxolist_filter_item")' instead.  I haven't changed
     it because it would probably fail to resolve the default function if
     not overridden by a theme.

  7. The taxonomy_dhtml module's INSTALL file still talks about
     '$theme->image'.

  8. The xloadtree module's INSTALL file still talks about '$theme->image'.

  9. I haven't updated the style.module; too many $theme's, though I don't
     think it needs any updates.

  [I'll try to update the sandboxes later on.]

now uses table rendering functions

helper functions for attachments from http://www.devshed.com/Server_Side/PHP/PHPMail/PHPMail2/page6.html

these may be integrated into mailhandler at some point. meanwhile, they are available to module developers

Fixing this.
Thanks Remco for pointing that out.

Modules using the mailhandler hook need to return $node in order to allow
for further processing if desired.

i had commented out the deletion of processed emails but forgotten to uncomment before committing. fixed now.

added anonymous submission. error reply is only sent if admin wants it and if node/comment does not allow anonymous submission.

changed _mail_handler() hook api a bit. modules beware

better error handling of errors while posting comments (patch by marco)

- some api docs
- reformatting of mailhandler_help