Re: [Phpgwtimetrack-developers] phpgw->link()

[Bob S. <bo...@pr...>]

Go ahead and commit it, as long as everything is already uptodate
with CVS. I have not yet been committing changes to CVS for
the 0.9.10 updates (table name changes, etc). Persinally,
I think it's kind of stupid the way they keep flipping back and forth
on this issue. I see no reason that $PHP_SELF should NOT
work (except in cases where php is being run as a CGI
method and not a module.
Bob

----- Original Message -----
From: "Camden Spiller" <ca...@ar...>
To: <php...@li...>
Sent: Sunday, March 18, 2001 7:11 PM
Subject: Re: [Phpgwtimetrack-developers] phpgw->link()


> I checked out the latest timetrack cvs yesterday along with updating my
copy
> of phpGroupWare-0.9.9-patches it was back to throwing errors with all
those
> PHP_SELF links.
>
> I found a message Jengo sent to the list on 3/8/01 stating the new rules
for
> link():
> 1) You _MUST_ send it the file name ... no more of this <a href="' .
> $phpgw->link()
> . '">   I made the poor choice to add the $PHP_SELF feature in the past.
> Basicly,
> it was becuase I was too lazy to type the filename ... :)
> 2) You have to pass the appname AND the first slash
> Example:  link('/calendar/edit.php')
> 3) webserver_url will be added automaticly ... don't pass it
>
> I've already got the link() changes ready to commit, but let me know if
you
> want me to hold off.
>
> Thanks,
>
> Camden
>
>
> ----- Original Message -----
> From: "Bob Schader" <bo...@pr...>
> To: <php...@li...>
> Sent: Sunday, March 18, 2001 5:19 PM
> Subject: Re: [Phpgwtimetrack-developers] phpgw->link()
>
>
> > Camden,
> > I already thought I had this fixed by just changing any
> > $PHP_SELF var's to "" instead. It was working fine
> > with 0.9.9 the last time I checked.
> > Bob
> >
> > ----- Original Message -----
> > From: "Camden Spiller" <csp...@su...>
> > To: <php...@li...>
> > Sent: Sunday, March 18, 2001 1:08 AM
> > Subject: [Phpgwtimetrack-developers] phpgw->link()
> >
> >
> > > I've converted the all the $phpgw->link()'s to
> > > $phpgw->link("/timetrack/pagename.php") as required per 0.9.9 ... any
> > reason
> > > not to commit this to CVS now?
> > >
> > > Camden
> > >
> > >
> > > _______________________________________________
> > > Phpgwtimetrack-developers mailing list
> > > Php...@li...
> > > http://lists.sourceforge.net/lists/listinfo/phpgwtimetrack-developers
> > >
> >
> >
> > _______________________________________________
> > Phpgwtimetrack-developers mailing list
> > Php...@li...
> > http://lists.sourceforge.net/lists/listinfo/phpgwtimetrack-developers
> >
>
>
> _______________________________________________
> Phpgwtimetrack-developers mailing list
> Php...@li...
> http://lists.sourceforge.net/lists/listinfo/phpgwtimetrack-developers
>