thredded 0.13.0 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 633ce6690d9ffc55e74596d44c9c6547778700e6
-  data.tar.gz: ab7417716f223e806bf189263da331ebe9b1635f
+  metadata.gz: 8ca544efca243747b5f8b3d11c65c5376999c016
+  data.tar.gz: 6d08bb441e304681531bee3977d32d66e9854f60
 SHA512:
-  metadata.gz: a513cd6fd50b5b1e3dc4f3af5a3d81719b942df9ef66f27f32c9e874ea1784ec39bd41752c94694bdf80b59fa24b1d203dcb31322d7294f43d1aefd79781197a
-  data.tar.gz: 6ff341b70414e6fdbcb79faf28c97290dae8cedb4be683171ef33fad0453d7b3ac012447d535f460d12cb52f9f3b01a6dfcc09a02f2b875237d56baab00479e1
+  metadata.gz: d1601695337ccb89d4c2f2b058f183d22ae33eef5381c1165a3368ef93ab72915f232224a9d8c394c9c8cbeac46931b01be9be3c9af556731b8fe0fa2469381c
+  data.tar.gz: c25c917fed02ad08cd38e88dad77990c76f21165d04a7c684efe8431c5bb12b8d1ed37742550268dd16a3e8795a41c28e0d9f5af98451ac9dd6bbdbff172bfc0

data/README.md

@@ -98,7 +98,7 @@ Then, see the rest of this Readme for more information about using and customizi
 Add the gem to your Gemfile:
 
 ```ruby
-gem 'thredded', '~> 0.13.0'
+gem 'thredded', '~> 0.14.0'
 ```
 
 Add the Thredded [initializer] to your parent app by running the install generator.
@@ -229,7 +229,9 @@ Include thredded JavaScripts in your `application.js`:
 
 Thredded is fully compatible with deferred and async script loading.
 
-##### Rails UJS version
+##### Alternative JavaScript dependencies
+
+<details><summary><b>Rails UJS version</b></summary>
 
 By default, thredded loads `rails-ujs`. If you're using Rails before v5.1, you need to add `rails-ujs` to
 your Gemfile.
@@ -240,8 +242,9 @@ If you'd like it to use `jquery_ujs` instead, run this command from your app dir
 mkdir -p app/assets/javascripts/thredded/dependencies/
 printf '//= require jquery3\n//= require jquery_ujs\n' > app/assets/javascripts/thredded/dependencies/ujs.js
 ```
+</details>
 
-##### Timeago version
+<details><summary><b>Timeago version</b></summary>
 
 By default, thredded loads `timeago.js`.
 
@@ -261,6 +264,7 @@ E.g. for Brazilian Portuguese with jquery.timeago:
  //= require locales/jquery.timeago.pt-br
  //= require thredded/thredded
  ```
+</details>
 
 #### Thredded page title and ID
 
@@ -376,8 +380,8 @@ change_column_default :thredded_user_preferences, :auto_follow_topics, 1
 
 ## I18n
 
-Thredded is mostly internationalized. It is currently available in English, Brazilian Portuguese, Polish, Russian,
-and Spanish.
+Thredded is mostly internationalized. It is currently available in English, Brazilian Portuguese, Chinese (Simplified),
+German, Polish, Russian, French, and Spanish.
 We welcome PRs adding support for new languages.
 
 Here are the steps to ensure the best support for your language if it isn't English:
@@ -387,9 +391,13 @@ Here are the steps to ensure the best support for your language if it isn't Engl
 2. Require the translations for timeago.js in your JavaScript. E.g. for Brazilian Portuguese:
 
    ```js
+   //= require thredded/dependencies/timeago
    //= require timeago/locales/pt_BR
+   //= require thredded
    ```
 
+   Note that it is important that timeago and its locales are required *before* `//= require thredded`.
+
 3. To generate URL slugs for messageboards, categories, and topics with support for more language than English,
    you can use a gem like [babosa](https://github.com/norman/babosa).
    Add babosa to your Gemfile and uncomment the `Thredded.slugifier` proc for babosa in the initializer.
@@ -425,21 +433,13 @@ The methods used by Thredded for determining the permissions are described below
 
 #### Posting to messageboards
 
-1. A list of messageboards that a given user can post in.
+A list of messageboards that a given user can post in.
 
   ```ruby
   # @return [ActiveRecord::Relation<Thredded::Messageboard>] messageboards that the user can post in
   thredded_can_write_messageboards
   ```
 
-2. A list of users that can post to a given list of messageboards.
-
-  ```ruby
-  # @param messageboards [Array<Thredded::Messageboard>]
-  # @return [ActiveRecord::Relation<User>] users that can post to the given messageboards
-  self.thredded_messageboards_writers(messageboards)
-  ```
-
 #### Messaging other users (posting to private topics)
 
 A list of users a given user can message:
@@ -451,19 +451,12 @@ thredded_can_message_users
 
 #### Moderating messageboards
 
-1. A list of messageboards that a given user can moderate:
+A list of messageboards that a given user can moderate:
 
   ```ruby
   # @return [ActiveRecord::Relation<Thredded::Messageboard>] messageboards that the user can moderate
   thredded_can_moderate_messageboards
   ```
-2. A list of users that can moderate a given list of messageboards:
-
-  ```ruby
-  # @param messageboards [Array<Thredded::Messageboard>]
-  # @return [ActiveRecord::Relation<User>] users that can moderate the given messageboards
-  self.thredded_messageboards_moderators(messageboards)
-  ```
 
 #### Admin permissions
 
@@ -603,9 +596,20 @@ run tasks that maintain the test database.
 By default, SQLite is used in development and test. On Travis, the tests will run using SQLite, PostgreSQL, MySQL,
 and all the supported Rails versions.
 
-The test suite requires [PhantomJS](http://phantomjs.org) to be present in the path.
-To install it, run `sudo apt-get install phantomjs` on Ubuntu or Debian, or `brew install phantomjs` on Mac.
-For other operating systems, refer to the [PhantomJS documentation](http://phantomjs.org/download.html).
+The test suite requires Chromium v59+ and its WebDriver installed:
+
+On Ubuntu, run:
+
+```bash
+sudo apt-get install chromium-chromedriver
+```
+
+On Mac, run:
+
+```bash
+brew cask install chromium
+brew install chromedriver
+```
 
 ### Ruby
 

data/app/assets/images/thredded/lock.svg

@@ -0,0 +1 @@
+<svg viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><path d="M10 16c0-1.104.896-2 2-2s2 .896 2 2c0 .738-.404 1.376-1 1.723v2.277h-2v-2.277c-.596-.347-1-.985-1-1.723zm11-6v14h-18v-14h3v-4c0-3.313 2.687-6 6-6s6 2.687 6 6v4h3zm-13 0h8v-4c0-2.206-1.795-4-4-4s-4 1.794-4 4v4zm11 2h-14v10h14v-10z"/></svg>

data/app/assets/javascripts/thredded/components/mention_autocompletion.es6

@@ -1,7 +1,8 @@
 //= require thredded/components/user_textcomplete
 
 const ThreddedMentionAutocompletion = {
-  MATCH_RE: /(^@|\s@)"?([\w.,\- ()]+)$/,
+  MATCH_RE: /(^@|\s@)"?([\w., \-()]+[\w.,\-()])$/,
+  // the last letter has to not be a space so it doesn't match after replacement
   DROPDOWN_MAX_COUNT: 6,
 
   init(form, textarea) {

data/app/assets/javascripts/thredded/components/post_form.es6

@@ -1,4 +1,4 @@
-//= require autosize
+//= require thredded/dependencies/autosize
 //= require thredded/core/on_page_load
 //= require thredded/components/mention_autocompletion
 //= require thredded/components/preview_area

data/app/assets/javascripts/thredded/components/time_stamps.es6

@@ -8,7 +8,7 @@
       if (!threddedContainer) return;
       timeago().render(
         document.querySelectorAll(COMPONENT_SELECTOR),
-        threddedContainer.getAttribute('data-thredded-locale'));
+        threddedContainer.getAttribute('data-thredded-locale').replace('-', '_'));
     });
     document.addEventListener('turbolinks:before-cache', () => {
       timeago.cancel();

data/app/assets/javascripts/thredded/components/topic_form.es6

@@ -1,4 +1,4 @@
-//= require autosize
+//= require thredded/dependencies/autosize
 //= require thredded/core/on_page_load
 //= require thredded/components/mention_autocompletion
 //= require thredded/components/preview_area

data/app/assets/javascripts/thredded/components/users_select.es6

@@ -1,6 +1,6 @@
 //= require thredded/core/on_page_load
 //= require thredded/components/user_textcomplete
-//= require autosize
+//= require thredded/dependencies/autosize
 
 (() => {
   const Thredded = window.Thredded;

data/app/assets/javascripts/thredded/dependencies/autosize.js

@@ -0,0 +1 @@
+//= require autosize.min

data/app/assets/javascripts/thredded/dependencies.js

@@ -1,6 +1,6 @@
 //= require thredded/dependencies/timeago
 //= require thredded/dependencies/ujs
-//= require autosize
+//= require thredded/dependencies/autosize
 //= require thredded/dependencies/textcomplete
 
 // We are not currently using any features that require the Babel polyfill

data/app/assets/stylesheets/thredded/base/_variables.scss

@@ -91,6 +91,10 @@ $thredded-badge-active-background: $thredded-action-color !default;
 $thredded-badge-inactive-color: $thredded-button-color !default;
 $thredded-badge-inactive-background: rgba($thredded-text-color, 0.3) !default;
 
+// Locked topics
+$thredded-badge-locked-color: $thredded-alert-warning-background;
+$thredded-badge-locked-background: $thredded-alert-warning-color;
+
 // Onebox
 $thredded-onebox-favicons: (
   'amazon': 'favicons/amazon.png',

data/app/assets/stylesheets/thredded/components/_form-list.scss

@@ -14,9 +14,10 @@
     transition: all 0.15s ease-out 0s;
   }
 
-  label + label {
+  label ~ label {
     display: inline-block;
     font-weight: normal;
+    margin-right: 15px;
   }
 }
 

data/app/assets/stylesheets/thredded/components/_messageboard.scss

@@ -49,11 +49,20 @@
   font-size: 0.875em;
 }
 
-.thredded--messageboard--meta {
+.thredded--messageboard--meta--icon {
+  @extend %thredded--icon;
+  opacity: 0.8;
+  width: 1rem;
+  height: 1rem;
+}
+
+.thredded--messageboard--meta--counts {
   @extend %thredded--heading;
   color: $thredded-secondary-text-color;
   display: inline-block;
+  font-size: inherit;
   font-weight: normal;
+  margin-bottom: 0;
   vertical-align: baseline;
 }
 

data/app/assets/stylesheets/thredded/components/_onebox.scss

@@ -195,15 +195,6 @@ pre.onebox code {
   }
 }
 
-// Onebox - Google Maps
-.maps-onebox {
-  // The onebox gem adds a div with inline styles that breaks mobile
-  // Hide the div
-  > div:first-child {
-    display: none;
-  }
-}
-
 // Onebox - Twitter - Status
 aside.onebox.twitterstatus .onebox-body {
   h4 {

data/app/assets/stylesheets/thredded/components/_post.scss

@@ -49,13 +49,19 @@
   vertical-align: baseline;
   width: 1.75rem; // 28px
 
-  @include thredded-media-content-breakout {
+  @media (min-width: $thredded-grid-container-max-width + $thredded-content-breakout-min-width + 0.00001rem) {
     height: 2.25rem; // 36px
     left: -3rem;
     position: absolute;
     top: -0.5rem;
     width: 2.25rem; // 36px
   }
+
+  @media (min-width: $thredded-grid-container-max-width + $thredded-content-breakout-min-width + 9.5rem + 0.00001rem) {
+    height: 3.25rem;
+    left: -4.125rem;
+    width: 3.25rem;
+  }
 }
 
 .thredded--post--topic {

data/app/assets/stylesheets/thredded/components/_topics.scss

@@ -172,3 +172,15 @@
     display: none;
   }
 }
+
+.thredded--topic-locked {
+  .thredded--topics--posts-count {
+    color: $thredded-badge-locked-color;
+    background-color: $thredded-badge-locked-background;
+  }
+}
+
+.thredded--topic--locked-notice {
+  @extend %thredded--alert;
+  @extend %thredded--alert--warning;
+}

data/app/commands/thredded/create_messageboard.rb

@@ -12,14 +12,14 @@ module Thredded
 
     # @return [boolean] true if the messageboard was created and seeded with a topic successfully.
     def run
-      Messageboard.transaction do
+      Thredded::Messageboard.transaction do
         fail ActiveRecord::Rollback unless @messageboard.save
-        topic = Topic.create!(
+        topic = Thredded::Topic.create!(
           messageboard: @messageboard,
           user: @user,
           title: first_topic_title
         )
-        Post.create!(
+        Thredded::Post.create!(
           messageboard: @messageboard,
           user: @user,
           postable: topic,

data/app/commands/thredded/mark_all_read.rb

@@ -10,7 +10,7 @@ module Thredded
         last_post = topic.posts.order_oldest_first.last
         total_pages = topic.posts.page(1).total_pages
 
-        UserPrivateTopicReadState.touch!(
+        Thredded::UserPrivateTopicReadState.touch!(
           user.id,
           topic.id,
           last_post,

data/app/commands/thredded/notify_following_users.rb

@@ -9,50 +9,68 @@ module Thredded
     end
 
     def run
+      subscribed_users.select! do |user|
+        # Record idempotently that the notification happened.
+        # If a notification was already created (e.g. from another thread/process),
+        # this will return false due to the unique constraint on the table
+        # and the user will be excluded.
+        subscribed_via_any_notifier?(user) && record_as_notified_successful?(user)
+      end
       Thredded.notifiers.each do |notifier|
         notifiable_users = targeted_users(notifier)
-        notifiable_users = notifiable_users.select do |user|
-          # Create a notification for the user.
-          # If a notification was already created (from another thread/process),
-          # this will return false due to the unique constraint on the table
-          # and the user will be excluded.
-          Thredded::UserPostNotification.create_from_post_and_user(@post, user)
-        end
         next if notifiable_users.empty?
         notifier.new_post(@post, notifiable_users)
       end
     end
 
     def targeted_users(notifier)
-      users_subscribed_via(notifier).reject do |user|
-        already_notified_user_ids.include?(user.id)
+      subscribed_users.select do |user|
+        user_subscribed_via?(user, notifier)
       end
     end
 
-    def users_subscribed_via(notifier)
-      subscribed_users.select do |user|
-        NotificationsForFollowedTopics
-          .detect_or_default(user.thredded_notifications_for_followed_topics, notifier).enabled? &&
-          MessageboardNotificationsForFollowedTopics
-            .detect_or_default(messageboard_notifier_prefs_by_user_id[user.id], notifier).enabled?
-      end
+    def user_subscribed_via?(user, notifier)
+      Thredded::NotificationsForFollowedTopics
+        .detect_or_default(user.thredded_notifications_for_followed_topics, notifier).enabled? &&
+        Thredded::MessageboardNotificationsForFollowedTopics
+          .detect_or_default(messageboard_notifier_prefs_by_user_id[user.id], notifier).enabled?
     end
 
+    # @return [Array<User>]
     def subscribed_users
       @subscribed_users ||=
-        @post.postable.followers.includes(:thredded_notifications_for_followed_topics).reject do |user|
-          user == @post.user || !Thredded::PostPolicy.new(user, @post).read?
+        @post.postable.followers.includes(:thredded_notifications_for_followed_topics).select do |user|
+          !already_notified?(user) && !originator?(user) && can_read_post?(user)
         end
     end
 
-    def already_notified_user_ids
-      @notified_user_ids ||= Set.new Thredded::UserPostNotification.notified_user_ids(@post)
+    private
+
+    def originator?(user)
+      user == @post.user
     end
 
-    private
+    def can_read_post?(user)
+      Thredded::PostPolicy.new(user, @post).read?
+    end
+
+    def already_notified?(user)
+      # We memoize the set so that records created during this task do not affect the result,
+      # so that a user can receive notifications via multiple notifiers.
+      @already_notified_user_ids ||= Set.new Thredded::UserPostNotification.notified_user_ids(@post)
+      @already_notified_user_ids.include?(user.id)
+    end
+
+    def subscribed_via_any_notifier?(user)
+      Thredded.notifiers.any? { |notifier| user_subscribed_via?(user, notifier) }
+    end
+
+    def record_as_notified_successful?(user)
+      Thredded::UserPostNotification.create_from_post_and_user(@post, user)
+    end
 
     def messageboard_notifier_prefs_by_user_id
-      @messageboard_notifier_prefs_by_user_id ||= MessageboardNotificationsForFollowedTopics
+      @messageboard_notifier_prefs_by_user_id ||= Thredded::MessageboardNotificationsForFollowedTopics
         .where(user_id: subscribed_users.map(&:id))
         .for_messageboard(@post.messageboard).group_by(&:user_id)
     end

data/app/commands/thredded/notify_private_topic_users.rb

@@ -26,7 +26,7 @@ module Thredded
 
     def only_those_with_this_notifier_enabled(users, notifier)
       users.select do |user|
-        NotificationsForPrivateTopics
+        Thredded::NotificationsForPrivateTopics
           .detect_or_default(user.thredded_notifications_for_private_topics, notifier).enabled?
       end
     end

data/app/controllers/concerns/thredded/new_post_params.rb

@@ -11,7 +11,7 @@ module Thredded
         .merge(ip: request.remote_ip).tap do |p|
         quote_id = p.delete(:quote_post_id)
         if quote_id
-          post = Post.find(quote_id)
+          post = Thredded::Post.find(quote_id)
           authorize_reading post
           p[:quote_post] = post
         end

data/app/controllers/concerns/thredded/new_private_post_params.rb

@@ -11,7 +11,7 @@ module Thredded
         .merge(ip: request.remote_ip).tap do |p|
         quote_id = p.delete(:quote_private_post_id)
         if quote_id
-          post = PrivatePost.find(quote_id)
+          post = Thredded::PrivatePost.find(quote_id)
           authorize_reading post
           p[:quote_post] = post
         end

data/app/controllers/thredded/application_controller.rb

@@ -105,7 +105,7 @@ module Thredded
     # @return [Thredded::Messageboard]
     # @raise [Thredded::Errors::MessageboardNotFound] if the messageboard with the given slug does not exist.
     def messageboard
-      @messageboard ||= Messageboard.friendly_find!(params[:messageboard_id])
+      @messageboard ||= Thredded::Messageboard.friendly_find!(params[:messageboard_id])
     end
 
     def messageboard_or_nil

data/app/controllers/thredded/autocomplete_users_controller.rb

@@ -5,7 +5,7 @@ module Thredded
     MAX_RESULTS = 20
 
     def index
-      authorize_creating PrivateTopicForm.new(user: thredded_current_user).private_topic
+      authorize_creating Thredded::PrivateTopicForm.new(user: thredded_current_user).private_topic
       users = params.key?(:q) ? users_by_prefix : users_by_ids
       render json: {
         results: users.map do |user|

data/app/controllers/thredded/messageboard_groups_controller.rb

@@ -3,12 +3,12 @@
 module Thredded
   class MessageboardGroupsController < Thredded::ApplicationController
     def new
-      @messageboard_group = MessageboardGroup.new
+      @messageboard_group = Thredded::MessageboardGroup.new
       authorize @messageboard_group, :create?
     end
 
     def create
-      @messageboard_group = MessageboardGroup.new(messageboard_group_params)
+      @messageboard_group = Thredded::MessageboardGroup.new(messageboard_group_params)
       authorize @messageboard_group, :create?
 
       if @messageboard_group.save

data/app/controllers/thredded/messageboards_controller.rb

@@ -8,17 +8,17 @@ module Thredded
     after_action :verify_policy_scoped, except: %i[new create edit update]
 
     def index
-      @groups = MessageboardGroupView.grouped(policy_scope(Messageboard.all))
+      @groups = Thredded::MessageboardGroupView.grouped(policy_scope(Thredded::Messageboard.all))
     end
 
     def new
-      @messageboard = Messageboard.new
-      @messageboard_group = MessageboardGroup.all
+      @messageboard = Thredded::Messageboard.new
+      @messageboard_group = Thredded::MessageboardGroup.all
       authorize_creating @messageboard
     end
 
     def create
-      @messageboard = Messageboard.new(messageboard_params)
+      @messageboard = Thredded::Messageboard.new(messageboard_params)
       authorize_creating @messageboard
       if Thredded::CreateMessageboard.new(@messageboard, thredded_current_user).run
         redirect_to root_path
@@ -28,12 +28,12 @@ module Thredded
     end
 
     def edit
-      @messageboard = Messageboard.friendly_find!(params[:id])
+      @messageboard = Thredded::Messageboard.friendly_find!(params[:id])
       authorize @messageboard, :update?
     end
 
     def update
-      @messageboard = Messageboard.friendly_find!(params[:id])
+      @messageboard = Thredded::Messageboard.friendly_find!(params[:id])
       authorize @messageboard, :update?
       if @messageboard.update(messageboard_params)
         redirect_to messageboard_topics_path(@messageboard), notice: I18n.t('thredded.messageboard.updated_notice')
@@ -47,7 +47,7 @@ module Thredded
     def messageboard_params
       params
         .require(:messageboard)
-        .permit(:name, :description, :messageboard_group_id)
+        .permit(:name, :description, :messageboard_group_id, :locked)
     end
   end
 end

data/app/controllers/thredded/moderation_controller.rb

@@ -28,7 +28,7 @@ module Thredded
         thredded_current_user,
         moderatable_posts
           .order_newest_first
-          .preload(:user, :postable)
+          .preload(:user, :postable, :messageboard)
           .page(current_page)
       )
       maybe_set_last_moderated_record_flash

data/app/controllers/thredded/post_permalinks_controller.rb

@@ -3,7 +3,7 @@
 module Thredded
   class PostPermalinksController < Thredded::ApplicationController
     def show
-      post = Post.find(params[:id])
+      post = Thredded::Post.find(params[:id])
       authorize post, :read?
       redirect_to post_url(post, user: thredded_current_user), status: :found
     end

data/app/controllers/thredded/post_previews_controller.rb

@@ -6,14 +6,14 @@ module Thredded
 
     # Preview a new post
     def preview
-      @post = Post.new(post_params)
-      @post.postable = Topic.friendly_find!(params[:topic_id])
+      @post = Thredded::Post.new(post_params)
+      @post.postable = Thredded::Topic.friendly_find!(params[:topic_id])
       render_preview
     end
 
     # Preview an update to an existing post
     def update
-      @post = Post.find(params[:post_id])
+      @post = Thredded::Post.find(params[:post_id])
       @post.assign_attributes(post_params)
       render_preview
     end

data/app/controllers/thredded/posts_controller.rb

@@ -12,12 +12,16 @@ module Thredded
     after_action :verify_authorized
 
     def new
-      @post_form = PostForm.new(user: thredded_current_user, topic: parent_topic, post_params: new_post_params)
+      @post_form = Thredded::PostForm.new(
+        user: thredded_current_user, topic: parent_topic, post_params: new_post_params
+      )
       authorize_creating @post_form.post
     end
 
     def create
-      @post_form = PostForm.new(user: thredded_current_user, topic: parent_topic, post_params: new_post_params)
+      @post_form = Thredded::PostForm.new(
+        user: thredded_current_user, topic: parent_topic, post_params: new_post_params
+      )
       authorize_creating @post_form.post
 
       if @post_form.save
@@ -28,7 +32,7 @@ module Thredded
     end
 
     def edit
-      @post_form = PostForm.for_persisted(post)
+      @post_form = Thredded::PostForm.for_persisted(post)
       authorize @post_form.post, :update?
       return redirect_to(canonical_topic_params) unless params_match?(canonical_topic_params)
       render
@@ -36,7 +40,7 @@ module Thredded
 
     def update
       authorize post, :update?
-      post.update_attributes(new_post_params)
+      post.update(new_post_params)
 
       redirect_to post_path(post, user: thredded_current_user)
     end
@@ -76,7 +80,7 @@ module Thredded
     end
 
     def parent_topic
-      Topic
+      Thredded::Topic
         .where(messageboard: messageboard)
         .friendly
         .find(params[:topic_id])

data/app/controllers/thredded/preferences_controller.rb

@@ -19,10 +19,10 @@ module Thredded
     private
 
     def init_preferences
-      @preferences = UserPreferencesForm.new(
+      @preferences = Thredded::UserPreferencesForm.new(
         user:         thredded_current_user,
         messageboard: messageboard_or_nil,
-        messageboards: policy_scope(Messageboard.all),
+        messageboards: policy_scope(Thredded::Messageboard.all),
         params: preferences_params
       )
     end

data/app/controllers/thredded/private_post_permalinks_controller.rb

@@ -4,7 +4,7 @@ module Thredded
   class PrivatePostPermalinksController < Thredded::ApplicationController
     before_action :thredded_require_login!
     def show
-      private_post = PrivatePost.find(params[:id])
+      private_post = Thredded::PrivatePost.find(params[:id])
       authorize private_post, :read?
       redirect_to post_url(private_post, user: thredded_current_user), status: :found
     end