From 960d64930cacdc695e50fcf4d8dcb428a091641b Mon Sep 17 00:00:00 2001
From: Wojciech Zawistowski <wojciech.zawistowski@gmail.com>
Date: Tue, 13 May 2014 08:53:11 -0400
Subject: [PATCH] Wiki Post

---
 .../discourse/controllers/history.js.es6      | 15 +++++
 .../discourse/controllers/topic_controller.js |  4 ++
 .../javascripts/discourse/models/post.js      | 21 +++++-
 .../templates/modal/history.js.handlebars     |  5 ++
 .../discourse/templates/post.js.handlebars    |  3 +
 .../discourse/views/post_menu_view.js         | 51 ++++++++++++---
 .../javascripts/discourse/views/post_view.js  |  6 ++
 .../stylesheets/common/base/history.scss      |  3 +
 .../stylesheets/desktop/topic-post.scss       | 44 ++++++++++++-
 app/assets/stylesheets/mobile/discourse.scss  |  4 ++
 app/assets/stylesheets/mobile/topic-post.scss | 50 +++++++++++++-
 app/controllers/posts_controller.rb           | 11 ++++
 app/models/post.rb                            |  2 +-
 app/models/post_revision.rb                   | 13 +++-
 app/serializers/post_revision_serializer.rb   |  3 +-
 app/serializers/post_serializer.rb            |  3 +-
 config/locales/client.en.yml                  |  3 +
 config/locales/server.en.yml                  |  2 +
 config/routes.rb                              |  1 +
 config/site_settings.yml                      |  5 +-
 .../20140408061512_add_wiki_to_posts.rb       |  5 ++
 lib/guardian/post_guardian.rb                 | 22 ++++++-
 spec/components/guardian_spec.rb              | 65 +++++++++++++++++++
 spec/controllers/posts_controller_spec.rb     | 39 +++++++++++
 spec/models/post_revision_spec.rb             |  8 +++
 .../fixtures/site_settings_fixtures.js        |  2 +-
 26 files changed, 371 insertions(+), 19 deletions(-)
 create mode 100644 db/migrate/20140408061512_add_wiki_to_posts.rb

diff --git a/app/assets/javascripts/discourse/controllers/history.js.es6 b/app/assets/javascripts/discourse/controllers/history.js.es6
index 9e458cde97d..6f43cff99d7 100644
--- a/app/assets/javascripts/discourse/controllers/history.js.es6
+++ b/app/assets/javascripts/discourse/controllers/history.js.es6
@@ -60,6 +60,21 @@ export default Discourse.ObjectController.extend(Discourse.ModalFunctionality, {
 
   }.property("viewMode", "category_changes"),
 
+  wiki_diff: function() {
+    var viewMode = this.get("viewMode");
+    var changes = this.get("wiki_changes");
+    if (changes === null) { return; }
+
+    if (viewMode === "inline") {
+      var diff = changes["current_wiki"] ? '<i class="fa fa-pencil-square-o fa-2x"></i>' : '<span class="fa-stack"><i class="fa fa-pencil-square-o fa-stack-2x"></i><i class="fa fa-ban fa-stack-2x"></i></span>';
+      return "<div class='inline-diff'>" + diff + "</div>";
+    } else {
+      var prev = changes["previous_wiki"] ? '<i class="fa fa-pencil-square-o fa-2x"></i>' : "&nbsp;";
+      var curr = changes["current_wiki"] ? '<i class="fa fa-pencil-square-o fa-2x"></i>' : '<span class="fa-stack"><i class="fa fa-pencil-square-o fa-stack-2x"></i><i class="fa fa-ban fa-stack-2x"></i></span>';
+      return "<div class='span8'>" + prev + "</div><div class='span8 offset1'>" + curr + "</div>";
+    }
+  }.property("viewMode", "wiki_changes"),
+
   title_diff: function() {
     var viewMode = this.get("viewMode");
     if(viewMode === "side_by_side_markdown") {
diff --git a/app/assets/javascripts/discourse/controllers/topic_controller.js b/app/assets/javascripts/discourse/controllers/topic_controller.js
index da9a63c7345..467956224e3 100644
--- a/app/assets/javascripts/discourse/controllers/topic_controller.js
+++ b/app/assets/javascripts/discourse/controllers/topic_controller.js
@@ -243,6 +243,10 @@ Discourse.TopicController = Discourse.ObjectController.extend(Discourse.Selected
       }).finally(function() {
         self.set('loadingExpanded', false);
       });
+    },
+
+    toggleWiki: function(post) {
+      post.toggleProperty('wiki');
     }
   },
 
diff --git a/app/assets/javascripts/discourse/models/post.js b/app/assets/javascripts/discourse/models/post.js
index 7517a0e9589..40acf44a244 100644
--- a/app/assets/javascripts/discourse/models/post.js
+++ b/app/assets/javascripts/discourse/models/post.js
@@ -80,6 +80,25 @@ Discourse.Post = Discourse.Model.extend({
 
   }.observes('bookmarked'),
 
+  wikiChanged: function() {
+    var self = this;
+
+    Discourse.ajax('/posts/' + this.get('id') + '/wiki', {
+      type: 'PUT',
+      data: {
+        wiki: this.get('wiki') ? true : false
+      }
+    }).then(function() {
+      self.incrementProperty('version');
+    }, function(error) {
+      if (error && error.responseText) {
+        bootbox.alert($.parseJSON(error.responseText).errors[0]);
+      } else {
+        bootbox.alert(I18n.t('generic_error'));
+      }
+    });
+  }.observes('wiki'),
+
   internalLinks: function() {
     if (this.blank('link_counts')) return null;
     return this.get('link_counts').filterProperty('internal').filterProperty('title');
@@ -439,5 +458,3 @@ Discourse.Post.reopenClass({
   }
 
 });
-
-
diff --git a/app/assets/javascripts/discourse/templates/modal/history.js.handlebars b/app/assets/javascripts/discourse/templates/modal/history.js.handlebars
index 81d75ffdf01..65122ee8d72 100644
--- a/app/assets/javascripts/discourse/templates/modal/history.js.handlebars
+++ b/app/assets/javascripts/discourse/templates/modal/history.js.handlebars
@@ -35,6 +35,11 @@
           {{boundAvatar user_changes.previous imageSize="small"}} {{user_changes.previous.username}} → {{boundAvatar user_changes.current imageSize="small"}} {{user_changes.current.username}}
       </div>
     {{/if}}
+    {{#if wiki_changes}}
+      <div class="row">
+        {{{wiki_diff}}}
+      </div>
+    {{/if}}
     {{{body_diff}}}
   </div>
 </div>
diff --git a/app/assets/javascripts/discourse/templates/post.js.handlebars b/app/assets/javascripts/discourse/templates/post.js.handlebars
index 0d7c9693bd5..468fb5fb133 100644
--- a/app/assets/javascripts/discourse/templates/post.js.handlebars
+++ b/app/assets/javascripts/discourse/templates/post.js.handlebars
@@ -31,6 +31,9 @@
           </div>
         </div>
       {{/unless}}
+      {{#if wiki}}
+        <div class="wiki"><i class="fa fa-pencil-square-o fa-3x"></i></div>
+      {{/if}}
     </div>
 
     <div class='topic-body span14'>
diff --git a/app/assets/javascripts/discourse/views/post_menu_view.js b/app/assets/javascripts/discourse/views/post_menu_view.js
index d402327bb13..3a8dfe68b6d 100644
--- a/app/assets/javascripts/discourse/views/post_menu_view.js
+++ b/app/assets/javascripts/discourse/views/post_menu_view.js
@@ -21,18 +21,17 @@ Discourse.PostMenuView = Discourse.View.extend({
     'post.bookmarked',
     'post.shareUrl',
     'post.topic.deleted_at',
-    'post.replies.length'),
+    'post.replies.length',
+    'post.wiki'),
 
   render: function(buffer) {
     var post = this.get('post');
-    buffer.push("<nav class='post-controls'>");
-    this.renderReplies(post, buffer);
 
-    var postMenuView = this;
-    Discourse.get('postButtons').toArray().reverse().forEach(function(button) {
-      var renderer = "render" + button;
-      if(postMenuView[renderer]) postMenuView[renderer](post, buffer);
-    });
+    buffer.push("<nav class='post-controls'>");
+
+    this.renderReplies(post, buffer);
+    this.renderButtons(post, buffer);
+
     buffer.push("</nav>");
   },
 
@@ -61,6 +60,14 @@ Discourse.PostMenuView = Discourse.View.extend({
     return buffer.push("<i class='fa " + icon + "'></i></button>");
   },
 
+  renderButtons: function(post, buffer) {
+    var self = this;
+    Discourse.get('postButtons').toArray().reverse().forEach(function(button) {
+      var renderer = "render" + button;
+      if(self[renderer]) self[renderer](post, buffer);
+    });
+  },
+
   clickReplies: function() {
     if (this.get('post.replies.length') > 0) {
       this.set('post.replies', []);
@@ -214,6 +221,34 @@ Discourse.PostMenuView = Discourse.View.extend({
 
   clickBookmark: function() {
     this.get('post').toggleProperty('bookmarked');
+  },
+
+  renderAdmin: function(post, buffer) {
+    var currentUser = Discourse.User.current();
+    if (!currentUser || !currentUser.get('canManageTopic')) {
+      return;
+    }
+
+    buffer.push('<button title="' + I18n.t("post.controls.admin") + '" data-action="admin" class="admin"><i class="fa fa-wrench"></i>');
+
+    this.renderAdminPopup(post, buffer);
+
+    buffer.push('</button>');
+  },
+
+  renderAdminPopup: function(post, buffer) {
+    var wikiText = post.get('wiki') ? I18n.t('post.controls.unwiki') : I18n.t('post.controls.wiki');
+    buffer.push('<div class="post-admin-menu"><h3>' + I18n.t('admin_title') + '</h3><ul><li class="btn btn-admin" data-action="toggleWiki"><i class="fa fa-pencil-square-o"></i>' + wikiText +'</li></ul></div>');
+  },
+
+  clickAdmin: function() {
+    var $adminMenu = this.$('.post-admin-menu');
+    this.set('postView.adminMenu', $adminMenu);
+    $adminMenu.show();
+  },
+
+  clickToggleWiki: function() {
+    this.get('controller').send('toggleWiki', this.get('post'));
   }
 
 });
diff --git a/app/assets/javascripts/discourse/views/post_view.js b/app/assets/javascripts/discourse/views/post_view.js
index 8076cd08d55..81ba3fab4d4 100644
--- a/app/assets/javascripts/discourse/views/post_view.js
+++ b/app/assets/javascripts/discourse/views/post_view.js
@@ -43,6 +43,12 @@ Discourse.PostView = Discourse.GroupedView.extend(Ember.Evented, {
     if (this.get('controller.multiSelect') && (e.metaKey || e.ctrlKey)) {
       this.get('controller').toggledSelectedPost(this.get('post'));
     }
+
+    var $adminMenu = this.get('adminMenu');
+    if ($adminMenu && !$(e.target).is($adminMenu) && $adminMenu.has($(e.target)).length === 0) {
+      $adminMenu.hide();
+      this.set('adminMenu', null);
+    }
   },
 
   selected: function() {
diff --git a/app/assets/stylesheets/common/base/history.scss b/app/assets/stylesheets/common/base/history.scss
index 52099d85a7f..8ecac683d3b 100644
--- a/app/assets/stylesheets/common/base/history.scss
+++ b/app/assets/stylesheets/common/base/history.scss
@@ -129,4 +129,7 @@
   .modal-header {
     height: 42px;
   }
+  .fa-ban {
+    color: #f00;
+  }
 }
diff --git a/app/assets/stylesheets/desktop/topic-post.scss b/app/assets/stylesheets/desktop/topic-post.scss
index ccb5005a86c..3206398ba61 100644
--- a/app/assets/stylesheets/desktop/topic-post.scss
+++ b/app/assets/stylesheets/desktop/topic-post.scss
@@ -168,7 +168,10 @@ nav.post-controls {
         &.hidden {
           display: none;
         }
-        &.like, &.edit, &.flag, &.delete, &.share, &.bookmark, &.create {
+        &.admin {
+          position: relative;
+        }
+        &.like, &.edit, &.flag, &.delete, &.share, &.bookmark, &.create, &.admin {
           float: right;
         }
 
@@ -203,6 +206,40 @@ nav.post-controls {
           }
         }
       }
+
+  .post-admin-menu {
+    background-color: $secondary;
+    width: 205px;
+    padding: 10px;
+    border: 1px solid scale-color($primary, $lightness: 90%);
+    position: absolute;
+    text-align: left;
+    bottom: 0;
+    right: 0;
+    z-index: 1000;
+    display: none;
+
+    h3 {
+      margin-top: 0;
+      color: $primary;
+      font-size: 1em;
+    }
+
+    ul {
+      list-style: none;
+      margin: 10px 0 0 0;
+    }
+
+    li {
+      width: 176px;
+      margin-bottom: 5px;
+
+      i.fa {
+        width: 14px;
+        margin-right: 14px;
+      }
+    }
+  }
 }
 
 .embedded-posts {
@@ -733,6 +770,11 @@ blockquote { /* solo quotes */
 .topic-avatar {
  border-top: 1px solid scale-color($primary, $lightness: 90%);
  padding-top: 16px;
+
+  .wiki {
+    margin-top: 14px;
+    color: scale-color($primary, $lightness: 60%);
+  }
 }
 
 .bottom-round.contents.regular {
diff --git a/app/assets/stylesheets/mobile/discourse.scss b/app/assets/stylesheets/mobile/discourse.scss
index 81a86dfa1de..03c4c9f5d90 100644
--- a/app/assets/stylesheets/mobile/discourse.scss
+++ b/app/assets/stylesheets/mobile/discourse.scss
@@ -22,7 +22,11 @@ body {
     height: 100%;
     @include border-radius-all(5px);
     .contents {
+      float: left;
       padding: 10px 10px 10px 10px;
+      h3 {
+        float: left;
+      }
     }
     &.white {
       background-color: $secondary;
diff --git a/app/assets/stylesheets/mobile/topic-post.scss b/app/assets/stylesheets/mobile/topic-post.scss
index 7596c82b43e..f51a10915ff 100644
--- a/app/assets/stylesheets/mobile/topic-post.scss
+++ b/app/assets/stylesheets/mobile/topic-post.scss
@@ -54,7 +54,10 @@ button {
   &.hidden {
     display: none;
   }
-  &.like, &.edit, &.flag, &.delete, &.share, &.bookmark, &.create {
+  &.admin {
+    position: relative;
+  }
+  &.like, &.edit, &.flag, &.delete, &.share, &.bookmark, &.create, &.admin {
     float: right;
   }
   .read-icon {
@@ -77,6 +80,38 @@ button {
   }
 }
 
+.post-admin-menu {
+  background-color: $secondary;
+  width: 205px;
+  padding: 10px;
+  border: 1px solid scale-color($primary, $lightness: 90%);
+  position: absolute;
+  text-align: left;
+  bottom: 0;
+  left: 0;
+  z-index: 1000;
+  display: none;
+
+  h3 {
+    color: $primary;
+    font-size: 1em;
+  }
+
+  ul {
+    list-style: none;
+    margin: 10px 0 0 0;
+  }
+
+  li {
+    width: 176px;
+    margin-bottom: 5px;
+
+    i.fa {
+      width: 14px;
+      margin-right: 14px;
+    }
+  }
+}
 
 
 .embedded-posts {
@@ -353,6 +388,19 @@ iframe {
   word-wrap: break-word;
 }
 
+.wiki {
+  float: left;
+  padding: 10px;
+  color: #408040;
+  i {
+    float: left;
+  }
+  h3 {
+    float: left;
+    margin-left: 10px;
+  }
+}
+
 .modal-body {
   input[type=text] {
     font-size: 16px;
diff --git a/app/controllers/posts_controller.rb b/app/controllers/posts_controller.rb
index 221748535cb..077b66ec69b 100644
--- a/app/controllers/posts_controller.rb
+++ b/app/controllers/posts_controller.rb
@@ -208,6 +208,17 @@ class PostsController < ApplicationController
     render nothing: true
   end
 
+  def wiki
+    guardian.ensure_can_wiki!
+
+    post = find_post_from_params
+    post.wiki = params[:wiki]
+    post.version += 1
+    post.save
+
+    render nothing: true
+  end
+
   protected
 
   def find_post_revision_from_params
diff --git a/app/models/post.rb b/app/models/post.rb
index 6a5081354f3..50e7b3563a9 100644
--- a/app/models/post.rb
+++ b/app/models/post.rb
@@ -478,7 +478,7 @@ class Post < ActiveRecord::Base
   end
 
   def save_revision
-    modifications = changes.extract!(:raw, :cooked, :edit_reason, :user_id)
+    modifications = changes.extract!(:raw, :cooked, :edit_reason, :user_id, :wiki)
     # make sure cooked is always present (oneboxes might not change the cooked post)
     modifications["cooked"] = [self.cooked, self.cooked] unless modifications["cooked"].present?
     PostRevision.create!(
diff --git a/app/models/post_revision.rb b/app/models/post_revision.rb
index 9e4053abc43..3d20b924d2b 100644
--- a/app/models/post_revision.rb
+++ b/app/models/post_revision.rb
@@ -28,6 +28,17 @@ class PostRevision < ActiveRecord::Base
     }
   end
 
+  def wiki_changes
+    prev = lookup("wiki", 0)
+    cur = lookup("wiki", 1)
+    return if prev == cur
+
+    {
+        previous_wiki: prev,
+        current_wiki: cur,
+    }
+  end
+
   def title_changes
     prev = "<div>#{CGI::escapeHTML(previous("title"))}</div>"
     cur = "<div>#{CGI::escapeHTML(current("title"))}</div>"
@@ -79,7 +90,7 @@ class PostRevision < ActiveRecord::Base
     end
 
     unless val
-      if ["cooked","raw"].include?(field)
+      if ["cooked", "raw"].include?(field)
         val = post.send(field)
       else
         val = post.topic.send(field)
diff --git a/app/serializers/post_revision_serializer.rb b/app/serializers/post_revision_serializer.rb
index 149865502d0..55345f6aebd 100644
--- a/app/serializers/post_revision_serializer.rb
+++ b/app/serializers/post_revision_serializer.rb
@@ -10,7 +10,8 @@ class PostRevisionSerializer < ApplicationSerializer
              :body_changes,
              :title_changes,
              :category_changes,
-             :user_changes
+             :user_changes,
+             :wiki_changes
 
   def include_title_changes?
     object.has_topic_data?
diff --git a/app/serializers/post_serializer.rb b/app/serializers/post_serializer.rb
index 09aade673b2..907afc8036b 100644
--- a/app/serializers/post_serializer.rb
+++ b/app/serializers/post_serializer.rb
@@ -47,7 +47,8 @@ class PostSerializer < BasicPostSerializer
              :deleted_by,
              :user_deleted,
              :edit_reason,
-             :can_view_edit_history
+             :can_view_edit_history,
+             :wiki
 
 
   def moderator?
diff --git a/config/locales/client.en.yml b/config/locales/client.en.yml
index 77a1a8112e2..199d493554b 100644
--- a/config/locales/client.en.yml
+++ b/config/locales/client.en.yml
@@ -966,6 +966,9 @@ en:
             other: "Do you also want to delete the {{count}} direct replies to this post?"
           yes_value: "Yes, delete the replies too"
           no_value: "No, just this post"
+        admin: "post admin actions"
+        wiki: "Wiki post"
+        unwiki: "Unwiki post"
 
       actions:
         flag: 'Flag'
diff --git a/config/locales/server.en.yml b/config/locales/server.en.yml
index 2851090a3a0..1948b9c7632 100644
--- a/config/locales/server.en.yml
+++ b/config/locales/server.en.yml
@@ -780,6 +780,8 @@ en:
 
     min_trust_to_create_topic: "The minimum trust level required to create a new topic."
 
+    min_trust_to_edit_wiki_post: "The minimum trust level required to edit post marked as wiki."
+
     newuser_max_links: "How many links a new user can add to a post"
     newuser_max_images: "How many images a new user can add to a post"
     newuser_max_attachments: "How many attachments a new user can add to a post"
diff --git a/config/routes.rb b/config/routes.rb
index a66953b547e..d14e6f6f49b 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -225,6 +225,7 @@ Discourse::Application.routes.draw do
 
   resources :posts do
     put "bookmark"
+    put "wiki"
     get "replies"
     get "revisions/:revision" => "posts#revisions"
     put "recover"
diff --git a/config/site_settings.yml b/config/site_settings.yml
index 964ce74ac20..04eeee26c7c 100644
--- a/config/site_settings.yml
+++ b/config/site_settings.yml
@@ -58,7 +58,7 @@ basic:
   post_menu:
     client: true
     list: true
-    default: 'like|edit|flag|delete|share|bookmark|reply'
+    default: 'like|edit|flag|delete|share|bookmark|admin|reply'
   share_links:
     client: true
     list: true
@@ -333,6 +333,9 @@ trust:
   min_trust_to_create_topic:
     default: 0
     enum: 'MinTrustToCreateTopicSetting'
+  min_trust_to_edit_wiki_post:
+    default: 0
+    enum: 'MinTrustToCreateTopicSetting'
   basic_requires_topics_entered: 5
   basic_requires_read_posts: 50
   basic_requires_time_spent_mins: 15
diff --git a/db/migrate/20140408061512_add_wiki_to_posts.rb b/db/migrate/20140408061512_add_wiki_to_posts.rb
new file mode 100644
index 00000000000..26f38facc65
--- /dev/null
+++ b/db/migrate/20140408061512_add_wiki_to_posts.rb
@@ -0,0 +1,5 @@
+class AddWikiToPosts < ActiveRecord::Migration
+  def change
+    add_column :posts, :wiki, :boolean, default: false, null: false
+  end
+end
diff --git a/lib/guardian/post_guardian.rb b/lib/guardian/post_guardian.rb
index 2149494380f..ab5bc55ebf2 100644
--- a/lib/guardian/post_guardian.rb
+++ b/lib/guardian/post_guardian.rb
@@ -68,7 +68,23 @@ module PostGuardian
 
   # Editing Method
   def can_edit_post?(post)
-    is_staff? || @user.has_trust_level?(:elder) || (!post.topic.archived? && is_my_own?(post) && !post.user_deleted && !post.deleted_at && !post.edit_time_limit_expired?)
+    if is_staff? || @user.has_trust_level?(:elder)
+      return true
+    end
+
+    if post.topic.archived? || post.user_deleted || post.deleted_at
+      return false
+    end
+
+    if post.wiki && (@user.trust_level >= SiteSetting.min_trust_to_edit_wiki_post.to_i)
+      return true
+    end
+
+    if is_my_own?(post) && !post.edit_time_limit_expired?
+      return true
+    end
+
+    false
   end
 
   # Deleting Methods
@@ -129,4 +145,8 @@ module PostGuardian
   def can_change_post_owner?
     is_admin?
   end
+
+  def can_wiki?
+    is_staff? || @user.has_trust_level?(:elder)
+  end
 end
diff --git a/spec/components/guardian_spec.rb b/spec/components/guardian_spec.rb
index 78877277185..2866cb1311a 100644
--- a/spec/components/guardian_spec.rb
+++ b/spec/components/guardian_spec.rb
@@ -617,6 +617,11 @@ describe Guardian do
         Guardian.new.can_edit?(post).should be_false
       end
 
+      it 'returns false when not logged in also for wiki post' do
+        post.wiki = true
+        Guardian.new.can_edit?(post).should be_false
+      end
+
       it 'returns true if you want to edit your own post' do
         Guardian.new(post.user).can_edit?(post).should be_true
       end
@@ -626,15 +631,32 @@ describe Guardian do
         Guardian.new(post.user).can_edit?(post).should be_false
       end
 
+      it 'returns false if another regular user tries to edit a soft deleted wiki post' do
+        post.wiki = true
+        post.user_deleted = true
+        Guardian.new(coding_horror).can_edit?(post).should be_false
+      end
+
       it 'returns false if you are trying to edit a deleted post' do
         post.deleted_at = 1.day.ago
         Guardian.new(post.user).can_edit?(post).should be_false
       end
 
+      it 'returns false if another regular user tries to edit a deleted wiki post' do
+        post.wiki = true
+        post.deleted_at = 1.day.ago
+        Guardian.new(coding_horror).can_edit?(post).should be_false
+      end
+
       it 'returns false if another regular user tries to edit your post' do
         Guardian.new(coding_horror).can_edit?(post).should be_false
       end
 
+      it 'returns true if another regular user tries to edit wiki post' do
+        post.wiki = true
+        Guardian.new(coding_horror).can_edit?(post).should be_true
+      end
+
       it 'returns true as a moderator' do
         Guardian.new(moderator).can_edit?(post).should be_true
       end
@@ -668,6 +690,35 @@ describe Guardian do
         it 'returns false for another regular user trying to edit your post' do
           Guardian.new(coding_horror).can_edit?(old_post).should == false
         end
+
+        it 'returns true for another regular user trying to edit a wiki post' do
+          old_post.wiki = true
+          Guardian.new(coding_horror).can_edit?(old_post).should be_true
+        end
+
+        it 'returns false when another user has too low trust level to edit wiki post' do
+          SiteSetting.stubs(:min_trust_to_edit_wiki_post).returns(2)
+          post.wiki = true
+          coding_horror.trust_level = 1
+
+          Guardian.new(coding_horror).can_edit?(post).should be_false
+        end
+
+        it 'returns true when another user has adequate trust level to edit wiki post' do
+          SiteSetting.stubs(:min_trust_to_edit_wiki_post).returns(2)
+          post.wiki = true
+          coding_horror.trust_level = 2
+
+          Guardian.new(coding_horror).can_edit?(post).should be_true
+        end
+
+        it 'returns true for post author even when he has too low trust level to edit wiki post' do
+          SiteSetting.stubs(:min_trust_to_edit_wiki_post).returns(2)
+          post.wiki = true
+          post.user.trust_level = 1
+
+          Guardian.new(post.user).can_edit?(post).should be_true
+        end
       end
     end
 
@@ -1659,5 +1710,19 @@ describe Guardian do
       end
     end
   end
+
+  describe 'can_wiki?' do
+    it 'returns false for regular user' do
+      Guardian.new(coding_horror).can_wiki?.should be_false
+    end
+
+    it 'returns true for admin user' do
+      Guardian.new(admin).can_wiki?.should be_true
+    end
+
+    it 'returns true for elder user' do
+      Guardian.new(elder).can_wiki?.should be_true
+    end
+  end
 end
 
diff --git a/spec/controllers/posts_controller_spec.rb b/spec/controllers/posts_controller_spec.rb
index 18dd8609c42..4f40932b7fc 100644
--- a/spec/controllers/posts_controller_spec.rb
+++ b/spec/controllers/posts_controller_spec.rb
@@ -316,6 +316,45 @@ describe PostsController do
 
   end
 
+  describe "wiki" do
+
+    include_examples "action requires login", :put, :wiki, post_id: 2
+
+    describe "when logged in" do
+      let(:user) {log_in}
+      let(:post) {Fabricate(:post, user: user)}
+
+      it "raises an error if the user doesn't have permission to see the post" do
+        Guardian.any_instance.expects(:can_wiki?).returns(false)
+
+        xhr :put, :wiki, post_id: post.id, wiki: 'true'
+
+        response.should be_forbidden
+      end
+
+      it "can wiki a post" do
+        Guardian.any_instance.expects(:can_wiki?).returns(true)
+
+        xhr :put, :wiki, post_id: post.id, wiki: 'true'
+
+        post.reload
+        post.wiki.should be_true
+      end
+
+      it "can unwiki a post" do
+        wikied_post = Fabricate(:post, user: user, wiki: true)
+        Guardian.any_instance.expects(:can_wiki?).returns(true)
+
+        xhr :put, :wiki, post_id: wikied_post.id, wiki: 'false'
+
+        wikied_post.reload
+        wikied_post.wiki.should be_false
+      end
+
+    end
+
+  end
+
   describe 'creating a post' do
 
     include_examples 'action requires login', :post, :create
diff --git a/spec/models/post_revision_spec.rb b/spec/models/post_revision_spec.rb
index f122886049a..d76302ba18f 100644
--- a/spec/models/post_revision_spec.rb
+++ b/spec/models/post_revision_spec.rb
@@ -62,4 +62,12 @@ describe PostRevision do
 
   end
 
+  it "can find wiki changes" do
+    r = create_rev("wiki" => [false, true])
+
+    changes = r.wiki_changes
+    changes[:previous_wiki].should be_false
+    changes[:current_wiki].should be_true
+  end
+
 end
diff --git a/test/javascripts/fixtures/site_settings_fixtures.js b/test/javascripts/fixtures/site_settings_fixtures.js
index 08f0b26bea0..d872d13fce9 100644
--- a/test/javascripts/fixtures/site_settings_fixtures.js
+++ b/test/javascripts/fixtures/site_settings_fixtures.js
@@ -1,3 +1,3 @@
 /*jshint maxlen:10000000 */
-Discourse.SiteSettingsOriginal = {"title":"Discourse Meta","logo_url":"/assets/logo.png","logo_small_url":"/assets/logo-single.png","traditional_markdown_linebreaks":false,"top_menu":"latest|new|unread|read|starred|categories","post_menu":"like|edit|flag|delete|share|bookmark|reply","share_links":"twitter|facebook|google+|email","track_external_right_clicks":false,"must_approve_users":false,"ga_tracking_code":"UA-33736483-2","ga_domain_name":"","enable_long_polling":true,"polling_interval":3000,"anon_polling_interval":30000,"min_post_length":20,"max_post_length":16000,"min_topic_title_length":15,"max_topic_title_length":255,"min_private_message_title_length":2,"allow_uncategorized_topics":true,"min_search_term_length":3,"flush_timings_secs":5,"suppress_reply_directly_below":true,"email_domains_blacklist":"mailinator.com","email_domains_whitelist":null,"version_checks":true,"min_title_similar_length":10,"min_body_similar_length":15,"category_colors":"BF1E2E|F1592A|F7941D|9EB83B|3AB54A|12A89D|25AAE2|0E76BD|652D90|92278F|ED207B|8C6238|231F20|808281|B3B5B4|283890","max_upload_size_kb":1024,"category_featured_topics":6,"favicon_url":"/assets/favicon.ico","dynamic_favicon":false,"uncategorized_name":"uncategorized","uncategorized_color":"AB9364","uncategorized_text_color":"FFFFFF","invite_only":false,"login_required":false,"min_password_length":8,"enable_local_logins":true,"enable_google_logins":true,"enable_yahoo_logins":true,"enable_twitter_logins":true,"enable_facebook_logins":true,"enable_cas_logins":false,"enable_github_logins":true,"enable_persona_logins":true,"educate_until_posts":2,"topic_views_heat_low":1000,"topic_views_heat_medium":2000,"topic_views_heat_high":5000,"min_private_message_post_length":5,"faq_url":"","tos_url":"","privacy_policy_url":"","authorized_extensions":".jpg|.jpeg|.png|.gif|.txt","relative_date_duration":14,"delete_removed_posts_after":24,"delete_user_max_post_age":7, "default_code_lang": "lang-auto", "suppress_uncategorized_badge": true};
+Discourse.SiteSettingsOriginal = {"title":"Discourse Meta","logo_url":"/assets/logo.png","logo_small_url":"/assets/logo-single.png","traditional_markdown_linebreaks":false,"top_menu":"latest|new|unread|read|starred|categories","post_menu":"like|edit|flag|delete|share|bookmark|admin|reply","share_links":"twitter|facebook|google+|email","track_external_right_clicks":false,"must_approve_users":false,"ga_tracking_code":"UA-33736483-2","ga_domain_name":"","enable_long_polling":true,"polling_interval":3000,"anon_polling_interval":30000,"min_post_length":20,"max_post_length":16000,"min_topic_title_length":15,"max_topic_title_length":255,"min_private_message_title_length":2,"allow_uncategorized_topics":true,"min_search_term_length":3,"flush_timings_secs":5,"suppress_reply_directly_below":true,"email_domains_blacklist":"mailinator.com","email_domains_whitelist":null,"version_checks":true,"min_title_similar_length":10,"min_body_similar_length":15,"category_colors":"BF1E2E|F1592A|F7941D|9EB83B|3AB54A|12A89D|25AAE2|0E76BD|652D90|92278F|ED207B|8C6238|231F20|808281|B3B5B4|283890","max_upload_size_kb":1024,"category_featured_topics":6,"favicon_url":"/assets/favicon.ico","dynamic_favicon":false,"uncategorized_name":"uncategorized","uncategorized_color":"AB9364","uncategorized_text_color":"FFFFFF","invite_only":false,"login_required":false,"min_password_length":8,"enable_local_logins":true,"enable_google_logins":true,"enable_yahoo_logins":true,"enable_twitter_logins":true,"enable_facebook_logins":true,"enable_cas_logins":false,"enable_github_logins":true,"enable_persona_logins":true,"educate_until_posts":2,"topic_views_heat_low":1000,"topic_views_heat_medium":2000,"topic_views_heat_high":5000,"min_private_message_post_length":5,"faq_url":"","tos_url":"","privacy_policy_url":"","authorized_extensions":".jpg|.jpeg|.png|.gif|.txt","relative_date_duration":14,"delete_removed_posts_after":24,"delete_user_max_post_age":7, "default_code_lang": "lang-auto", "suppress_uncategorized_badge": true};
 Discourse.SiteSettings = jQuery.extend(true, {}, Discourse.SiteSettingsOriginal);