Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deleting a feed in statistics page result in an error #1047

Closed
Alwaysin opened this issue Dec 25, 2015 · 4 comments
Closed

Deleting a feed in statistics page result in an error #1047

Alwaysin opened this issue Dec 25, 2015 · 4 comments
Assignees
Milestone

Comments

@Alwaysin
Copy link
Contributor

Steps to reproduce :

  • Go to http://<freshrss>/i/?c=stats&a=idle
  • Click on "delete" on one feed

Then you are redirected to http://<freshrss>/i/http%3A%2F%2F<freshrss>%2Fi%3Fc%3Dstats%26a%3Didle and you get this :

Not Found

The requested URL /i/http://<freshrss>/i?c=stats&a=idle was not found on this server.

But the feed is indeed deleted.

@Alwaysin
Copy link
Contributor Author

https://github.com/FreshRSS/FreshRSS/blob/dev/app/views/stats/idle.phtml#L29, I've replaced

formaction="<?php echo _url('feed', 'delete', 'id', $feed['id'], 'r', $current_url); ?>"

with

formaction="<?php echo _url('feed', 'delete', 'id', $feed['id']); ?>"

and it works, but we are redirected to http://<freshrss>/i/?c=subscription

@Alkarex
Copy link
Member

Alkarex commented Dec 29, 2015

@Alwaysin Could you please provide a pull request with your modification?

@marienfressinaud
Copy link
Member

The idea was to stay on the idle statistics page, I would prefer a PR which fix the redirection instead of redirecting to the subscription page...

@Alkarex
Copy link
Member

Alkarex commented Mar 6, 2016

Fixed in #1109

@Alkarex Alkarex closed this as completed Mar 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants