Re: r13894: undefined function displayInternalLink

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 27, 2010 at 5:26 AM, Christian Kujau <lists@xxxxxxxxxxxxxxx> wrote:
> On Wed, 27 Jan 2010 at 05:19, Christian Kujau wrote:
>> I'm currently on r13894 and don't know when this function has been removed
>> from page_header.php[0],

revision 12549 2 and a half years ago

>>but since it's only used in
>> plugins/calendar/functions.php anyway, we might as well just define it
>> there:
>
> Or, instead of resurrecting and old function, use makeInternalLink:

That's right, except the "correct" solution is to create a template
file for the calendar header.  For now, using makeInternalLink is good
enough, but note that you forgot to *echo* that function's return
value.

CVS is fixed.  Thanks

> --- plugins/calendar/functions.php.orig 2010-01-27 14:04:41.203362743 +0100
> +++ plugins/calendar/functions.php      2010-01-27 14:23:51.063362978 +0100
> @@ -36,11 +36,11 @@ function calendar_header() {
>          html_tag( 'tr' ) .
>          html_tag( 'td', '', 'left', '', 'width="100%"' );
>
> -    displayInternalLink("plugins/calendar/calendar.php?year=$year&amp;month=$month",_("Month View"),"right");
> +    makeInternalLink("plugins/calendar/calendar.php?year=$year&amp;month=$month",_("Month View"),"right");
>     echo "&nbsp;&nbsp;\n";
> -    displayInternalLink("plugins/calendar/day.php?year=$year&amp;month=$month&amp;day=$day",_("Day View"),"right");
> +    makeInternalLink("plugins/calendar/day.php?year=$year&amp;month=$month&amp;day=$day",_("Day View"),"right");
>     echo "&nbsp;&nbsp;\n";
> -    // displayInternalLink("plugins/calendar/event_create.php?year=$year&amp;month=$month&amp;day=$day",_("Add Event"),"right");
> +    // makeInternalLink("plugins/calendar/event_create.php?year=$year&amp;month=$month&amp;day=$day",_("Add Event"),"right");
>     // echo "&nbsp;&nbsp;\n";
>     echo '</td></tr>';
>
>
>
> Dunno if this is a safe thing to do though.

-- 
Paul Lesniewski
SquirrelMail Team
Please support Open Source Software by donating to SquirrelMail!
http://squirrelmail.org/donate_paul_lesniewski.php

------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
-----
squirrelmail-users mailing list
Posting guidelines: http://squirrelmail.org/postingguidelines
List address: squirrelmail-users@xxxxxxxxxxxxxxxxxxxxx
List archives: http://news.gmane.org/gmane.mail.squirrelmail.user
List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-users


[Index of Archives]     [Video For Linux]     [Yosemite News]     [Yosemite Photos]     [gtk]     [KDE]     [Cyrus SASL]     [Gimp on Windows]     [Steve's Art]     [Webcams]

  Powered by Linux