my_forum 0.0.1.beta46 → 0.0.1.beta47

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1b8e08aa2bc1c01ded3147b72abc835493533068
-  data.tar.gz: 96080563f950816d16e1341416c4fc97536c036b
+  metadata.gz: 9b33c4cb82573653aa48a66e05ffa1d5b1046183
+  data.tar.gz: aed7d3f0d1e8dd3f1ef10ccb29bd85d2bd758322
 SHA512:
-  metadata.gz: 418f5c30b9a38a76a398fe8aa305cfaf43fc9ea4b709b6a09e94562ad41b04a581237a3943abde224c379bdcfc0fdbe9f4f2ea9b5f0d72b2ac9dd900853a7ace
-  data.tar.gz: cc774917b805e8bbd0eda06254f729a2196ad1c2df1034f68f723c2c5e8342dab1b5821b60bdbb2c70e389b6fac88bebfae133f43ac9ac59b4d84a357c75359b
+  metadata.gz: 7e3a752f9498e68cdc9c6df35974fde15c947ba02ad19c8efeb00d0ccb1d0e170d595b75886df92d510c52fee26e0f99b18c86ade254150bf208af2b3c4e738f
+  data.tar.gz: f5cb24e415e17ccf7eb6380be754601398f3ce6979a6397cf9cb34fd1c42fa9b02afa1ff489aba2f526859f531a2dcdb6230259c20eb1917c4a5e43333e23b32

data/app/controllers/my_forum/topics_controller.rb

@@ -62,7 +62,7 @@ module MyForum
 
     # TODO development in progress
     def get_last_readed_user_page
-      return nil unless params[:page].blank? or current_user
+      return nil unless params[:page].blank? or current_user or @topic
       user_last_page = nil
 
       latest_readed_post = LogReadMark.where(user_id: current_user.id, topic_id: @topic.id).first

data/app/models/my_forum/forum.rb

@@ -9,14 +9,17 @@ module MyForum
         SELECT
           my_forum_topics.name AS topic_name,
           my_forum_topics.forum_id,
+
+
           my_forum_topics.id,
-          my_forum_posts.created_at AS post_created_at,
-          my_forum_users.login AS user_login
-        FROM my_forum_posts
-        JOIN my_forum_topics ON my_forum_topics.id = my_forum_posts.topic_id
-        JOIN my_forum_users ON my_forum_posts.user_id = my_forum_users.id
+          my_forum_topics.latest_post_created_at AS post_created_at,
+          my_forum_topics.latest_post_login AS user_login
+
+
+        FROM my_forum_topics
+
         WHERE my_forum_topics.forum_id = #{self.id}
-        ORDER BY my_forum_posts.id DESC
+
         LIMIT 1
       ").first
     end

data/app/models/my_forum/post.rb

@@ -4,6 +4,7 @@ module MyForum
     belongs_to  :user, :counter_cache => true
 
     after_create :update_topic_latest_post
+    after_update :update_topic_latest_post, :if => :is_deleted?
 
     default_scope { where(is_deleted: false) }
 
@@ -12,7 +13,14 @@ module MyForum
     private
 
     def update_topic_latest_post
-      self.topic.update_attribute(:latest_post_id, self.id)
+      post = self.is_deleted ? Post.where(topic_id: self.topic_id).last : self
+
+      self.topic.update(
+        latest_post_id:         post.id,
+        latest_post_created_at: post.created_at,
+        latest_post_login:      post.user.login,
+        latest_post_user_id:    post.user.id
+      )
     end
   end
 end

data/db/migrate/20160122202142_add_latest_post_info_for_topic.rb

@@ -2,7 +2,23 @@ class AddLatestPostInfoForTopic < ActiveRecord::Migration
   def change
     add_column :my_forum_topics, :latest_post_created_at,   :datetime
     add_column :my_forum_topics, :latest_post_login,        :string
+    add_column :my_forum_topics, :latest_post_user_id,      :integer
 
+    count = MyForum::Topic.count
+    MyForum::Topic.find_in_batches do |group|
+      group.each do |topic|
+        puts "#{count-=1} Try to update #{topic.name}"
+        latest_post = topic.posts.last
+        latest_post_user = latest_post.user
+        next unless latest_post_user
+
+        topic.update(
+          latest_post_created_at: latest_post.created_at,
+          latest_post_login:      latest_post_user.login,
+          latest_post_user_id:    latest_post_user.id
+        )
+      end
+    end
 
   end
 end

data/lib/my_forum/version.rb

@@ -1,3 +1,3 @@
 module MyForum
-  VERSION = "0.0.1.beta46"
+  VERSION = "0.0.1.beta47"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: my_forum
 version: !ruby/object:Gem::Version
-  version: 0.0.1.beta46
+  version: 0.0.1.beta47
 platform: ruby
 authors:
 - Vitaly Omelchenko