IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-16 18:24:52 -06:00
Code Issues Packages Projects Releases Wiki Activity

DEV: Deprecate unused route instead of removing it. (#14030)

Browse Source 
We'll no longer use this route, so we'll return a 301 with the new URL just in case someone is still using it.
This commit is contained in:
Roman Rizzi 2021-08-12 15:54:14 -03:00 committed by GitHub
parent 65f6d46045
commit 54ad0ecc72
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
app/controllers/posts_controller.rb
View File

@ -554,6 +554,13 @@ class PostsController < ApplicationController
render body: nil
end
def flagged_posts
deprecate('posts#flagged_posts is deprecated. Please use /review instead.', since: '2.8.0.beta4', drop_from: '2.9')
review_queue_url = path("/review?status=all&type=ReviewableFlaggedPost&username=#{params[:username]}")
redirect_to review_queue_url, status: 301
end
def deleted_posts
params.permit(:offset, :limit)
guardian.ensure_can_see_deleted_posts!

1
config/routes.rb
View File

@ -564,6 +564,7 @@ Discourse::Application.routes.draw do
get "posts/:id/reply-ids" => "posts#reply_ids"
get "posts/:id/reply-ids/all" => "posts#all_reply_ids"
get "posts/:username/deleted" => "posts#deleted_posts", constraints: { username: RouteFormat.username }
get "posts/:username/flagged" => "posts#flagged_posts", constraints: { username: RouteFormat.username }
%w{groups g}.each do |root_path|
resources :groups, id: RouteFormat.username, path: root_path do