RSS links corrupted when Drupal is installed in a sub directory

Site map are using Drupals l() and url() core functions to build all links so they should always be correct.

Have you set $base_url in settings.php? Did you by mistake end it with a slash?

Hi Fredrik,
thx for your reply.

Yes my settings.php is configured with my subdirectory.
$base_url = 'http://mydomain.com/drupal';

But with this settings "$base_url" or not my problem stays the same, it makes no difference.

Regards

I have tried to reproduce this problem without success. When I place a Drupal site in a subdirectory the link are constructed correctly.

If it was a site map bug I also believe there would have been more bug reports about it.

If you can I would recommend you to set up a fresh test site with a core theme and see if the problem persists.

My real server is under Linux.

I've just tried with new server as this :
OS : Windows Wamp 2.0
Drupal : 6.13
default language : english, no other language

1 change in settings.php :
$base_url = 'http://localhost/drupal';

1 module added :
site_map-6.x-1.x-dev (2009-Aug-12)

site_map configuration : default, nothing changed
Menus configuration : only enable "site map" in Navigation menu

Here is my Site Map HTML page :
Site map
Front page
Front page of My Drupal BETA

URL under Front Page text is : http://localhost/drupal/
URL under is RSS icon is : http://localhost/drupal//drupal/rss.xml

Did I miss something ?

Must I configure something special with Apache ?
Maybe our difference is here.

Could you please explain me your .htaccess or VirtualDocument configuration ?

Thx for your help.

Drupal core comes with a htaccess file that includes all the necessary rewrite rules needed for clean URL:s. Nothing extra is needed for the site map module.

Have you tried to set a "RewriteBase" in Drupals htaccess file? For the example site i #4 it would be

RewriteBase /drupal

Yes I've tried and it has no effect. Same problem.

I don't understand how we can have 2 different results with fresh reinstalled sites.
My test site with only sitemap module doesn't work better.

Anyone can confirm if it works or not ?

I get the same issue when updating from "6.x-1.x-dev (2009-Apr-23)" to "6.x-1.x-dev (2009-Aug-12)" on my local test machine.

6.13
default lamp installation

I've just tested a little change on my debug site i've described #4.
Same server only one change :
site_map-6.x-1.x-dev (2009-Aug-12)
site_map-6.x-1.0
=> and now it works correctly

That confirms configuration isn't concerned and it must a bug.

I just did a cvs diff -r DRUPAL-6--1-0 I had a look at the changes but can not find anything that can explain this, the links have always been build with l() or url() and that should ensure correct links always.

Can you pack you test site with a databasdump as a tar-ball and send me a link to it?

OK Fredrik I'll send you in a moment.
Can you reproduce it on your side with same versions ?

Here are my files.
My Drupal zip is to large (1.3Mo) but I made a diff before and after update.
The only difference is site_map module directory.
So I only send my settings.php file.

During my diff I think I found something that could explain.
But maybe i'm wrong.

In site_map.module line 85 :
NOK
$feedurl = variable_get('site_map_rss_front', 'rss.xml');

OK
$feedurl = 'rss.xml';

I can not reproduce this problem and to be able to look inte to it I need your complete installation.

Tar-ball it and just send me a link so I can download it, no need to upload it in public here on d.o.

Here it is
http://www.yousendit.com/download/YkxKTXRhbEpFd2RFQlE9PQ

For OK and NOK site you could find site and database backup.

Credit to you for hanging in there! I have now found the bug, sorry it took some time.

Sitemap was running RSS links through url() and then through l(). I had rebuilt the theme_site_map_feed_icon() function to use l() and missed to remove the url() part when calling it.

Have committed a fix to 6-dev now.

Thanks for helping fix this bug!

Thx a lot Fredrik !

I'm impatient to test next release.
I'm waiting for it to be generated.

Don't worry for delay. There is absolutly no reason.
Thx for your good job and for your patience.
Others could have simply reply "unreproductible" and close issue.

Thx again,
Regards

Confirmed : fixed in 6.x-1.x-dev (2009-Aug-21)

Thx again Fredrik.

Automatically closed -- issue fixed for 2 weeks with no activity.