Re: [SM-DEVEL] read_body_top hook

[Seth E. R. <se...@mi...>]

Hmm.  I think we may need to add another hook.  The hook is where is
currently is because it allows plugins to modify whether or not you want
to view html (The $show_html_default value).  But, there is also the need
for a hook later on too.

Seth.
> I installed the latest CVS snapshot of Squirrelmail & the plugins.
After
> playing around a bit, I noticed that delete_move_next (when configured
to
> show at the top) shows up a lot higher than I'm used to. (Above the
> Compose Addressese Folders.. menu bar, even).
>
> I looked in src/read_body.php and would like to propose the following
> changes:
>
> Comment out the read_Body_top hook around line 632
>
> $messagebody = '';
> /* do_hook('read_body_top'); */
> if ($show_html_default == 1) {
>
> and add the hook a little further down at line 648:
>
> displayPageHeader($color, $mailbox);
> formatMenuBar($mailbox, $passed_id, $passed_ent_id, $message,
$mbx_response);
> formatEnvheader($mailbox, $passed_id, $passed_ent_id, $message, $color,
> $FirstTimeSee);
> do_hook('read_body_top');
> echo '<table width="100%" cellpadding="0" cellspacing="0" align="center"
> border="0">';
>
> This is my first foray into SM development/plugins, so be gentle if this
> is a completely unacceptable (especially if it's obviously so!)
solution.
>
> Sincerely,
> Chris Hilts
> ch...@bi...
>
>
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by: OSDN - Tired of that same old
> cell phone?  Get a new here for FREE!
> https://www.inphonic.com/r.asp?r--
> squirrelmail-devel mailing list
> List Address: squ...@li...
> List Info:
https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel
> http://squirrelmail.org/cvs
>


-- 
Seth E. Randall
IT Support Specialist
Missoula Federal Credit Union
se...@mi...