Path to 'next page' incorrect

Found a bug? Tell us!!
Post Reply
gwgraham
Regular
Posts: 7
Joined: Mon Apr 02, 2007 3:52 pm

Path to 'next page' incorrect

Post by gwgraham »

We did an svn upgrade of some 0.8 version blogs about a month ago.
For some reason all these blogs create an incorrect link to the 'next page'

e.g. ....php?/wrapper/archives/P2.html

When it should be simply

e.g. ....php?/archives/P2.html

The main page of the blog is embedded and the url is
E.g. http://www.shootingpeople.org/sxsw/blog/wrapper.php

In the admin the path to the archive is reading 'archives'

Where is this extra '/wrapper' coming from ?

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

Re: Path to 'next page' incorrect

Post by garvinhicking »

Hi!

What is your "indexFile" option set to? That 'wrapper' can stem because of the 'wrapper.php' file which is regarded as a path argument if not matching what was configured in "indexFile" in the s9y configuration...?

Best 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/
gwgraham
Regular
Posts: 7
Joined: Mon Apr 02, 2007 3:52 pm

Post by gwgraham »

Hi Garvin,

If you mean the 'index file' setting under 'paths' in the configuration admin, this is indeed set to 'wrapper.php'

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

Post by garvinhicking »

Hi!

Could you show me what the wrapper.php file looks like?

Maybe this is related to a bug that occured in the 1.2 alpha2 versions of some time ago, and was meanwhile fixed in recent 1.2 snapshots, so maybe you could check if a more recent snapshot exhibits the same behaviour?

Best 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/
gwgraham
Regular
Posts: 7
Joined: Mon Apr 02, 2007 3:52 pm

Post by gwgraham »

I think you are right. We are on this version. Should we be sticking to a stable release? I didn't do the upgrade but it was done by someone else via svn. Can you let me know if we should have done differently to get a stable version?

Anyway, here is the wrapper.php

<?php
// Let serendipity generate our content:
ob_start();
require 'index.php';
$blog_data = ob_get_contents();
ob_end_clean();

// Now we include our normal content building file.
// This one has to make use of your $blog_data variable to print
// the content where appropriate!
require 'content.php';
?>
gwgraham
Regular
Posts: 7
Joined: Mon Apr 02, 2007 3:52 pm

Post by gwgraham »

Hi I updated to 1.2-alpha3 but it still does the same thing.
This is the one I updated
http://www.shootingpeople.org/sundance/blog/wrapper.php
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Hi!

Thanks for reporting! I was able to make a patch to properly strip that wrapper-stuff out of the URL:

http://svn.berlios.de/viewcvs/serendipi ... 49&r2=1666

HTH,
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/
gwgraham
Regular
Posts: 7
Joined: Mon Apr 02, 2007 3:52 pm

Post by gwgraham »

Thank you very much! That works now.
Post Reply