From 1a83ff8459c0117d415c3d9f69bca24349b4696f Mon Sep 17 00:00:00 2001
From: "Turkesh Patel (Open ERP)" <tpa@tinyerp.com>
Date: Wed, 26 Mar 2014 12:00:42 +0530
Subject: [PATCH] [IMP] do not store session_id in db update viewd post id in
 session and maintain it.

bzr revid: tpa@tinyerp.com-20140326063042-1palqxiw10uvjlvf
---
 addons/website_forum/controllers/main.py | 18 +++++++++---------
 addons/website_forum/models/forum.py     | 11 +++++++++--
 2 files changed, 18 insertions(+), 11 deletions(-)

diff --git a/addons/website_forum/controllers/main.py b/addons/website_forum/controllers/main.py
index 7e3622e526b..c049a63febb 100644
--- a/addons/website_forum/controllers/main.py
+++ b/addons/website_forum/controllers/main.py
@@ -148,16 +148,16 @@ class website_forum(http.Controller):
 
         #maintain total views on post.
         Statistics = request.registry['website.forum.post.statistics']
-        domain = [('session', '=', request.session_id), ('post_id', '=', question.id)]
+        post_obj = request.registry['website.forum.post']
         if request.session.uid:
-            domain = ['|', ('session', '=', request.session_id), ('user_id', '=', request.session.uid), ('post_id', '=', question.id)]
-        view_ids = Statistics.search(cr, uid, domain, context=context)
-        if not view_ids:
-            Statistics.create(cr, SUPERUSER_ID, {
-                'session': request.session_id,
-                'user_id': request.session.uid or False,
-                'post_id': question.id,
-            }, context=context)
+            view_ids = Statistics.search(cr, uid, [('user_id', '=', request.session.uid), ('post_id', '=', question.id)], context=context)
+            if not view_ids:
+                Statistics.create(cr, uid, {'user_id': request.session.uid, 'post_id': question.id }, context=context)
+        else:
+            request.session[request.session_id] = request.session.get(request.session_id, [])
+            if not (question.id in request.session[request.session_id]):
+                request.session[request.session_id].append(question.id)
+                post_obj._set_view_count(cr, SUPERUSER_ID, [question.id], 'views', 1, {}, context=context)
 
         answer_done = False
         for answer in question.child_ids:
diff --git a/addons/website_forum/models/forum.py b/addons/website_forum/models/forum.py
index 43ed160cc92..0fb83e92984 100644
--- a/addons/website_forum/models/forum.py
+++ b/addons/website_forum/models/forum.py
@@ -106,6 +106,14 @@ class Post(osv.Model):
             res[post.id] = post.views + 1
         return res
 
+    def _set_view_count(self, cr, uid, ids, name, value, arg, context=None):
+        if isinstance(ids, (int, long)):
+            ids = [ids]
+        for rec in self.browse(cr, uid, ids, context=context):
+            views = rec.views + value
+            cr.execute('UPDATE website_forum_post SET views=%s WHERE id=%s;',(views, rec.id))
+        return True
+
     def _get_views(self, cr, uid, ids, context=None):
         result = {}
         for statistic in self.pool.get('website.forum.post.statistics').browse(cr, uid, ids, context=context):
@@ -140,7 +148,7 @@ class Post(osv.Model):
         'child_ids': fields.one2many('website.forum.post', 'parent_id', 'Answers'),
         'child_count':fields.function(_get_child_count, string="Answers", type='integer',
             store={
-                'website.forum.post': (_get_child, [], 10),
+                'website.forum.post': (_get_child, ['child_ids','parent_id'], 10),
             }
         ),
 
@@ -234,7 +242,6 @@ class PostStatistics(osv.Model):
         'name': fields.char('Post Reason'),
         'post_id': fields.many2one('website.forum.post', 'Post', ondelete='cascade'),
         'user_id': fields.many2one('res.users', 'Created by'),
-        'session': fields.char('Session Id'),
     }
 
 class PostReason(osv.Model):