Reporting of spam screenshots disabled

Will be re-enabled later with a proper seperate page.
This commit is contained in:
Christoph Haas 2017-08-11 21:06:33 +02:00
parent 030145ca7e
commit 0fbc247aff
3 changed files with 38 additions and 22 deletions

View file

@ -1,20 +1,35 @@
// Button that reveals a dropdown/modal for admins // Button that reveals a dropdown/modal for moderation/reporting
.text-right .button-bar
button.small.dropdown.warning.button type="button" data-toggle="admin-info-#{screenshot.id}" ul.button-group
'Admin - if not screenshot.approved
.dropdown-pane data-dropdown=true id="admin-info-#{screenshot.id}" li
- if not screenshot.approved or screenshot.markedfordelete a.button.small.success[
a.button.small.success[ href=approve_screenshot_path(screenshot.id)
href=approve_screenshot_path(screenshot.id) method='post'
method='post' ] Approve screenshot
] Keep/approve screenshot - elsif screenshot.markedfordelete
a.button.small.alert[ li
href=delete_screenshot_path(screenshot.id) a.button.small.success[
onclick="return confirm('Really delete the screenshot?');" href=approve_screenshot_path(screenshot.id)
] Delete screenshot method='post'
] Keep screenshot
p Status: #{screenshot.adminstatus} - if user_signed_in?
/ p Uploader IP=#{screenshot.uploaderip} li
/ p Uploader Token=#{session[:token]} a.button.small.alert[
/ p Uploaded #{screenshot.age_days} (#{screenshot.created_at}) href=delete_screenshot_path(screenshot.id)
p Uploaded #{screenshot.age_days} onclick="return confirm('Really delete the screenshot?');"
] Delete screenshot
/ Display additional information to admins
p Status: #{screenshot.adminstatus}
p Uploader IP=#{screenshot.uploaderip}
p Uploader Token=#{session[:token]}
p Uploaded #{screenshot.age_days} (#{screenshot.created_at})
/ TODO: Move reporting screenshots to an extra page with a form and captcha
/ - else
/ / Allow anonymous users to report inappropriate screenshots
/ = form_tag(report_screenshot_path(screenshot.id))
/ = text_area_tag 'delete_reason', nil, class: 'input-group-field', maxlength: 100, rows: 3, cols: 50
/ = submit_tag 'Request removal', class: 'button alert'

View file

@ -44,8 +44,9 @@
] Delete your screenshot ] Delete your screenshot
// or is the user not related to the screenshot and the screenshot is public? // or is the user not related to the screenshot and the screenshot is public?
- elsif screenshot.approved // TODO: Move to a seperate page with a form
= render(partial: 'report_dropdown', locals: {screenshot: screenshot}) //- elsif screenshot.approved
// = render(partial: 'report_dropdown', locals: {screenshot: screenshot})
- else - else
.row.listview .row.listview

View file

@ -28,7 +28,7 @@ Rails.application.routes.draw do
# TODO: "get" is probably the wrong method to delete a screenshot # TODO: "get" is probably the wrong method to delete a screenshot
get 'delete_screenshot/:id' => 'packages#delete_screenshot', as: :delete_screenshot get 'delete_screenshot/:id' => 'packages#delete_screenshot', as: :delete_screenshot
post 'update_screenshot_description/:name/:id' => 'packages#update_screenshot_description', as: :update_screenshot_description post 'update_screenshot_description/:name/:id' => 'packages#update_screenshot_description', as: :update_screenshot_description
post 'report_screenshot/:id' => 'packages#report_screenshot', as: :report_screenshot #post 'report_screenshot/:id' => 'packages#report_screenshot', as: :report_screenshot
# TODO: "get" is probably the wrong method to delete a screenshot # TODO: "get" is probably the wrong method to delete a screenshot
get 'approve_screenshot/:id' => 'packages#approve_screenshot', as: :approve_screenshot get 'approve_screenshot/:id' => 'packages#approve_screenshot', as: :approve_screenshot
get 'about' => 'welcome#about' get 'about' => 'welcome#about'