Page 1 of 1

URL Problem in download manager

Posted: Fri May 05, 2006 10:21 am
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.

Posted: Fri May 05, 2006 10:33 am
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. :)

Posted: Fri May 05, 2006 10:36 am
by frodeste
Same error applies to all images..

Posted: Fri May 05, 2006 10:51 am
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

Posted: Fri May 05, 2006 2:28 pm
by frodeste
Thanx. I guess that means I can updgrade to the new version any time soon. :)

Posted: Mon May 08, 2006 9:48 am
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?

Posted: Mon May 08, 2006 12:29 pm
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

Posted: Tue May 09, 2006 11:15 am
by frodeste
I can confirm that this bug is now resolved. Thank you.