Re: [Fofredux-devel] RC 0.3 bugs & Search

[Kevin <ke...@dr...>]

Please add a test case that reporduces the failure.  I'll need to stand u=
p
a php4 environment to debug with.  I don't get this behavior on php 5.0.x=
.


> There is definitely a bug in the parse_w3cdtf () of
> magpierss/rss_utils.inc or associated. I just realized that I have not
> been reading a bunch of items from my feeds. For whatever reason, the
> publish_time is getting set to:
> 0000-00-00 00:00:00
>
> which then makes the feed items not show up in the view.
>
> You can see this in the HEAD demo:
> http://code.highearthorbit.com/fofredux/fofredux_head/view.php?feed=3D4=
2
>
> where no feed items are visible from after March 8. If you view "old
> to new" you get feed items that were at least cached up until March
> 17. But you can clearly see on the RSS feed that their have been items
> since then:
> http://www.hackaday.com/rss.xml
>
> The actual error is:
> Warning: gmmktime() expects parameter 3 to be long, string given in
> /home/.parable/highearth/code.highearthorbit.com/public/fofredux/fofred=
ux_head/magpierss/rss_utils.inc
> on line 35
>
> Which is just a warning, but then the date/time is getting set to the
> 0000 value from above and then not getting picked up in the
> fof_get_items query.
>
> Also, for anyone interested, HEAD also has an updated search
> functionality which searches within just the current view and also has
> search term highlighting.
>
> Andrew
>
> On 3/23/06, Andrew Turner <ajt...@hi...> wrote:
>> On 3/23/06, Kevin <ke...@dr...> wrote:
>> > Andrew Turner wrote:
>> > > I have the current bugs in RC 0.3:
>> > >
>> > > * Warning: gmmktime() expects parameter 3 to be long, string given
>> in
>> > > magpierss/rss_utils.inc on line 35
>> > >  - which shows up when any feed is updated.
>> > >
>> > >
>> > Which version of php are you using?   I did make changes to that fil=
e,
>> > but it runs clean on my install. (5.0.x)
>>
>> PHP 4.4.2
>>
>> >
>> > > * Warning: Unable to detect unique link for item, please report th=
e
>> > > bug to the FoFR Project in init.php on line 1598
>> > >
>> > >  - which shows up from the feed:
>> > > http://ourworld.harveyworld.com.au/jobs/default.cfm
>> > >
>> > >
>> > That feed has no URLs to uniquely identify the items.  Is it a defec=
t
>> if
>> > we've never supported horribly broken feeds such as that?   To fix
>> this,
>> > we will have to add a hash column to the items table to uniquely
>> > identify items.  Not something I'd consider slipping in during a
>> release
>> > cycle.
>> >
>>
>> Good point - I didn't necessarily think this needed to be fixed now,
>> but was worth looking at. We had discussed "fall-back" uniqueness
>> tests.
>>
>> > > * the "Previous/Next 10" (or however many) links do not abide by t=
he
>> > > "howmany" and are actually malformed (several params without actua=
l
>> > > values).
>> > >
>>
>> Can anyone else confirm this bug?
>>
>> > >
>> > > --
>> > > Andrew Turner
>> > > ajt...@hi...        42.4266N x 83.4931W
>> > > http://highearthorbit.com              Northville, Michigan, USA
>> > >
>> > >
>> > > -------------------------------------------------------
>> > > This SF.Net email is sponsored by xPML, a groundbreaking scripting
>> language
>> > > that extends applications into web and mobile media. Attend the li=
ve
>> webcast
>> > > and join the prime developer group breaking into this new coding
>> territory!
>> > > http://sel.as-us.falkag.net/sel?cmd=3Dk&kid=110944&bid$1720&dat=12=
1642
>> > > _______________________________________________
>> > > Fofredux-devel mailing list
>> > > Fof...@li...
>> > > https://lists.sourceforge.net/lists/listinfo/fofredux-devel
>> > >
>> >
>> >
>> >
>> > -------------------------------------------------------
>> > This SF.Net email is sponsored by xPML, a groundbreaking scripting
>> language
>> > that extends applications into web and mobile media. Attend the live
>> webcast
>> > and join the prime developer group breaking into this new coding
>> territory!
>> > http://sel.as-us.falkag.net/sel?cmd=3Dlnk&kid=3D110944&bid=3D241720&=
dat=3D121642
>> > _______________________________________________
>> > Fofredux-devel mailing list
>> > Fof...@li...
>> > https://lists.sourceforge.net/lists/listinfo/fofredux-devel
>> >
>>
>>
>> --
>> Andrew Turner
>> ajt...@hi...        42.4266N x 83.4931W
>> http://highearthorbit.com              Northville, Michigan, USA
>>
>
>
> --
> Andrew Turner
> ajt...@hi...        42.4266N x 83.4931W
> http://highearthorbit.com              Northville, Michigan, USA
>


--=20
Kevin