[geeklog-devel] Geeklog.net 2.1.0 Beta 1 Upgrade

Tom websitemaster at cogeco.net
Sun Mar 23 10:36:00 EDT 2014


Thanks for finding the point where the "1" gets added. Looking at the code I
don't see anywhere that function subst gets checked if it returns true.  A
string is assumed returned and the code continues. Your fix should work fine
for at least preventing the "1" from showing.

I still need to figure out why it happens in the first place sometimes for
instance caching (why the featured story was replaced by a "1") .

Tom

-----Original Message-----
From: geeklog-devel-bounces at lists.geeklog.net
[mailto:geeklog-devel-bounces at lists.geeklog.net] On Behalf Of Wim Niemans
Sent: March-23-14 9:10 AM
To: Geeklog Development
Subject: Re: [geeklog-devel] Geeklog.net 2.1.0 Beta 1 Upgrade

Having the code examined, learns the following:

1. the cache routines make extensive use of 'file_get_contents' and handle
the return values in a buggy fashion.

2. The '1' is generated in the function subst($varname) by returning true:

           } else if (isset($this->varvals[$varname]) OR empty($varname)) {
               return $this->slow_subst($varname);
           } else {
               // $varname does not reference a file so return
               if ($this->debug & 4) {
                   echo "<p><b>subst:</b> varname $varname does not
reference a file</p>\n";
               }
               return true;
           }

3. Changing 'return true' into " return '' " removes the '1' occurrence.

4. The construct is present twice in the function. See lines 705, 738.

I am not sure if this change will do, or would generate other noise. 
Though I find it most strange for return true values in this function.

After all, this '1' is generated by parsing a unset varname ?? Really?

Wim


On 21 Mar 2014, at 11:40, Tom <websitemaster at cogeco.net> wrote:

> Thanks Wim. It gives me a place to start.
> 
> Tom
> 
> -----Original Message-----
> From: geeklog-devel-bounces at lists.geeklog.net
> [mailto:geeklog-devel-bounces at lists.geeklog.net] On Behalf Of Wim 
> Niemans
> Sent: March-21-14 5:58 AM
> To: Geeklog Development
> Subject: Re: [geeklog-devel] Geeklog.net 2.1.0 Beta 1 Upgrade
> 
> It looks like the function file_get_contents can return a false or a 
> false-alike since php5.
> 
> Wim
> 
> On 19 Mar 2014, at 22:27, Tom <websitemaster at cogeco.net> wrote:
> 
>> This problem has happened again so I have disabled template caching 
>> on Geeklog.net. If anyone else notices a "1" where it really 
>> shouldn't be let me know.
>> 
>> When I get a chance I will look into the issue.
>> 
>> Thanks
>> 
>> Tom
>> 
>> -----Original Message-----
>> From: geeklog-devel-bounces at lists.geeklog.net
>> [mailto:geeklog-devel-bounces at lists.geeklog.net] On Behalf Of Wim 
>> Niemans
>> Sent: March-13-14 7:42 AM
>> To: Geeklog Development
>> Subject: Re: [geeklog-devel] Geeklog.net 2.1.0 Beta 1 Upgrade
>> 
>> 
>> Filed a bug for this already.
>> See http://project.geeklog.net/tracking/view.php?id=1750
>> 
>> On 13 Mar 2014, at 12:30, Dirk Haun <dirk at haun-online.de> wrote:
>> 
>>> Tom <websitemaster at cogeco.net> wrote:
>>> 
>>>> I have updated Geeklog.net to the still unreleased Geeklog 2.1.0 Beta
1.
>>> 
>>> When you're not logged in, there's a "1" displaying on the front 
>>> page,
>> just below the "About Geeklog" block. I don't think it's coming from 
>> any of the static pages that we display on the front page (About 
>> Geeklog is one, or the international support one, if your browser 
>> defaults to a language other than English).
>>> 
>>> Could this be because we don't currently have a featured story?
>>> 
>>> bye, Dirk
>>> 
>>> 
>>> --
>>> http://www.themobilepresenter.com/
>>> 
>>> _______________________________________________
>>> geeklog-devel mailing list
>>> geeklog-devel at lists.geeklog.net
>>> http://eight.pairlist.net/mailman/listinfo/geeklog-devel
>>> 
>> 
>> _______________________________________________
>> geeklog-devel mailing list
>> geeklog-devel at lists.geeklog.net
>> http://eight.pairlist.net/mailman/listinfo/geeklog-devel
>> 
>> _______________________________________________
>> geeklog-devel mailing list
>> geeklog-devel at lists.geeklog.net
>> http://eight.pairlist.net/mailman/listinfo/geeklog-devel
>> 
> 
> _______________________________________________
> geeklog-devel mailing list
> geeklog-devel at lists.geeklog.net
> http://eight.pairlist.net/mailman/listinfo/geeklog-devel
> 
> _______________________________________________
> geeklog-devel mailing list
> geeklog-devel at lists.geeklog.net
> http://eight.pairlist.net/mailman/listinfo/geeklog-devel
> 

_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://eight.pairlist.net/mailman/listinfo/geeklog-devel




More information about the geeklog-devel mailing list