#83 RSS link does not work

closed-fixed
nobody
None
5
2008-10-20
2008-10-03
Mattias S
No

1.9.7-rc1
The RSS link on the "Topic List" page doesn't work.

Discussion

  • David Sitsky

    David Sitsky - 2008-10-11

    Thanks for that - should be fixed now. Here is the patch below since it is so simple.

    Index: lib/Codestriker/Http/Method/ListTopicsMethod.pm

    RCS file: /cvsroot/codestriker/codestriker/lib/Codestriker/Http/Method/ListTopicsMethod.pm,v
    retrieving revision 1.6
    diff -u -r1.6 ListTopicsMethod.pm
    --- lib/Codestriker/Http/Method/ListTopicsMethod.pm 8 Sep 2008 10:19:50 -0000 1.6
    +++ lib/Codestriker/Http/Method/ListTopicsMethod.pm 11 Oct 2008 00:23:13 -0000
    @@ -83,7 +83,13 @@
    sub execute {
    my ($self, $http_input, $http_output) = @_;

    - Codestriker::Action::ListTopics->process($http_input, $http_output);
    + my $action = $http_input->{query}->param('action');
    + if ($action eq "list_topics_rss") {
    + Codestriker::Action::ListTopicsRSS->process($http_input, $http_output);
    + } else {
    + Codestriker::Action::ListTopics->process($http_input, $http_output);
    + }
    +
    }

    1;

     
  • David Sitsky

    David Sitsky - 2008-10-11
    • status: open --> closed-fixed
     
  • Mattias S

    Mattias S - 2008-10-16

    1.9.7-rc-2
    Still does not work... I had to add this:
    File: UrlBuilder.pm
    Fuction: list_topics_url_rss
    Added line: $args{rss} = 1;

     
  • Mattias S

    Mattias S - 2008-10-16
    • status: closed-fixed --> open-fixed
     
  • David Sitsky

    David Sitsky - 2008-10-20

    Yes you are right - that is one way of fixing it, however that "rss" parameter is deprecated now.

    I've committed in the proper fix in ListTopicsMethod.pm which just switches on the action parameter instead. I've confirmed that it actually does work properly now. I'll make a rc3 build with this fix in it.

     
  • Mattias S

    Mattias S - 2008-10-20
    • status: open-fixed --> closed-fixed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks