odoo
/
odoo
2
0
Fork 0
Code Pull Requests Releases Activity

[FIX] website_forum: fix error 500 when editing an answer 

Was caused by missing tags.
Also fix also backwards-compatibility for textext selection method.
This commit is contained in:
Jeremy Kersten 2014-12-01 15:20:40 +01:00 committed by Olivier Dony
parent 3c0cf7c977
commit ad1ca233ca
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
addons/website_forum/controllers/main.py
View File

@ -223,6 +223,7 @@ class WebsiteForum(http.Controller):
question_tag_ids.append((4, tag_ids[0]))
else:
question_tag_ids.append((0, 0, {'name': tag, 'forum_id': forum.id}))
question_tag_ids = {forum.id: question_tag_ids}
elif tag_version == "select2":
question_tag_ids = Forum._tag_to_write_vals(cr, uid, [forum.id], post.get('question_tags', ''), context)
@ -403,6 +404,11 @@ class WebsiteForum(http.Controller):
Tag = request.registry['forum.tag']
Forum = request.registry['forum.forum']
tag_version = kwargs.get('tag_type', 'texttext')
vals = {
'name': kwargs.get('question_name'),
'content': kwargs.get('content'),
}
if tag_version == "texttext": # old version - retro v8 - #TODO Remove in master
if kwargs.get('question_tag') and kwargs.get('question_tag').strip('[]'):
tags = kwargs.get('question_tag').strip('[]').replace('"', '').split(",")
@ -413,15 +419,10 @@ class WebsiteForum(http.Controller):
else:
new_tag = Tag.create(cr, uid, {'name': tag, 'forum_id': forum.id}, context=context)
question_tags.append(new_tag)
tags_val = [(6, 0, question_tags)]
vals['tag_ids'] = [(6, 0, question_tags)]
elif tag_version == "select2": # new version
tags_val = Forum._tag_to_write_vals(cr, uid, [forum.id], kwargs.get('question_tag', ''), context)
vals['tag_ids'] = Forum._tag_to_write_vals(cr, uid, [forum.id], kwargs.get('question_tag', ''), context)[forum.id]
vals = {
'tag_ids': tags_val[forum.id],
'name': kwargs.get('question_name'),
'content': kwargs.get('content'),
}
request.registry['forum.post'].write(cr, uid, [post.id], vals, context=context)
question = post.parent_id if post.parent_id else post
return werkzeug.utils.redirect("/forum/%s/question/%s" % (slug(forum), slug(question)))