URL Problem in download manager

Found a bug? Tell us!!
Post Reply
frodeste
Regular
Posts: 56
Joined: Thu Aug 04, 2005 9:16 pm
Location: Tønsberg, Norway
Contact:

URL Problem in download manager

Post by frodeste »

Go here: http://www.stenstrom.no/blog/downloads.html

When you click on a category, the program goes to:

http://www.stenstrom.no/index.php?seren ... &thiscat=7

where as it should go to

http://www.stenstrom.no/[b]blog/[/b]index.php?serendipity[subpage]=downloadmanager&level=1&thiscat=7

This causes the application to fail.
-- frodeste
www.stenstrom.no
frodeste
Regular
Posts: 56
Joined: Thu Aug 04, 2005 9:16 pm
Location: Tønsberg, Norway
Contact:

Post by frodeste »

FYI: I put in a quick and dirty fix to this on my site, so it will work:

803c803
< $nodetb .= " <td><a hr
ef=\"./blog/index.php?serendipity[subpage]=".$this->get_config('pageurl')."&level=".($subcat['level']-1)."&am
p;thiscat=".$subcat['node_id']."\">".$subcat['payload']."</a></td>\n";
---
> $nodetb .= " <td><a hr
ef=\"./index.php?serendipity[subpage]=".$this->get_config('pageurl')."&level=".($subcat['level']-1)."&thi
scat=".$subcat['node_id']."\">".$subcat['payload']."</a></td>\n";
842c842


However, this is not the best way to do this. :)
-- frodeste
www.stenstrom.no
frodeste
Regular
Posts: 56
Joined: Thu Aug 04, 2005 9:16 pm
Location: Tønsberg, Norway
Contact:

Post by frodeste »

Same error applies to all images..
-- frodeste
www.stenstrom.no
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Hi!

Thanks for reporting this! Indeed the plugin contained those hardcoded values, I fixed that to use the s9y config values for that .

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/
frodeste
Regular
Posts: 56
Joined: Thu Aug 04, 2005 9:16 pm
Location: Tønsberg, Norway
Contact:

Post by frodeste »

Thanx. I guess that means I can updgrade to the new version any time soon. :)
-- frodeste
www.stenstrom.no
frodeste
Regular
Posts: 56
Joined: Thu Aug 04, 2005 9:16 pm
Location: Tønsberg, Norway
Contact:

Post by frodeste »

I can confirm that the url has been corrected. However, the images are not loaded.

Produced source now reads:

<td align="center"><a href="http://www.stenstrom.no/blog/plugin/dlfile_4"><img src="./plugins/serendipity_event_downloadmanager/img/download.gif" border=0 /></a><br /><b>Download this file</b></td>

whereas I think it should read:

<td align="center"><a href="http://www.stenstrom.no/blog/plugin/dlfile_4"><img src="/blog/plugins/serendipity_event_downloadmanager/img/download.gif" border=0 /></a><br /><b>Download this file</b></td>

Y fix?
-- frodeste
www.stenstrom.no
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Hi!

That's right, there was still a problem. The getRelPath() function did not really return a relative path. I hopefully fixed this with the commit I just made vor version 0.12 of the plugin.

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/
frodeste
Regular
Posts: 56
Joined: Thu Aug 04, 2005 9:16 pm
Location: Tønsberg, Norway
Contact:

Post by frodeste »

I can confirm that this bug is now resolved. Thank you.
-- frodeste
www.stenstrom.no
Post Reply