Couple of strange things in 08-alpha9

Found a bug? Tell us!!
Post Reply
Martin
Regular
Posts: 90
Joined: Mon Sep 27, 2004 1:30 am
Location: Oslo
Contact:

Couple of strange things in 08-alpha9

Post by Martin »

The categories plug-in lists the categories, but not as links...

The recent entries plug-in disappears when asked to skip the frontpage entries.

-m
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Couple of strange things in 08-alpha9

Post by garvinhicking »

That was a small bug in the snapshots from the last days, has been fixed yesterday by Tom. It was about missing closing quotes in the <a> tag...

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Martin
Regular
Posts: 90
Joined: Mon Sep 27, 2004 1:30 am
Location: Oslo
Contact:

Post by Martin »

Ah, yes... Just downloaded the latest and that fixed the categories-problem. The recent entries-problem however, continues...

-m
mgroeninger
Regular
Posts: 546
Joined: Mon Dec 20, 2004 11:57 pm
Contact:

Post by mgroeninger »

I think I know what's up with the recent entries-problem..

this code is used to set the limit to the sql query:

Code: Select all

       switch($number_from_sw) {
            case 'skip':
                $sql_number = serendipity_db_limit_sql(serendipity_db_limit($serendipity['fetchLimit'], $sql_number));
                break;
        }
and it returns the word LIMIT as part of the string...

So, when the query is set with:

Code: Select all

                                       ORDER BY timestamp DESC
                                          LIMIT $sql_number";
The LIMIT appears twice... removing the LIMIT from the query seems to fix the problem...

I'll submit a patch to the dev list..
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Matthew was right, the problem came because of the double LIMIT part. I committed his fix to CVS by now!

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Post Reply