From a5c8405b9ae39f23bcb8dad8aa0c3acdb1765f2e Mon Sep 17 00:00:00 2001
From: Andreas N <pixeldesu@nerdwerks.org>
Date: Sat, 27 Dec 2014 03:01:52 +0100
Subject: [PATCH 01/14] attempt to fix this shit

---
 app/assets/javascripts/inbox.coffee | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/assets/javascripts/inbox.coffee b/app/assets/javascripts/inbox.coffee
index 32abdec6..384c337b 100644
--- a/app/assets/javascripts/inbox.coffee
+++ b/app/assets/javascripts/inbox.coffee
@@ -42,7 +42,7 @@
       complete: (jqxhr, status) ->
         btn.button "reset"
         if succ
-          btn.attr "disabled", "disabled"  # this doesn't really work like I wanted it to…
+          btn.prop "disabled", true  # this doesn't really work like I wanted it to…
           btn[0].dataset.ibCount = 0
 
 

From 02cb3181420ad46bf0683112e242b4a7b5e04ff3 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sat, 27 Dec 2014 03:49:54 +0100
Subject: [PATCH 02/14] 8)

---
 Rakefile | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/Rakefile b/Rakefile
index 7f7ed899..02a1599a 100644
--- a/Rakefile
+++ b/Rakefile
@@ -97,4 +97,33 @@ namespace :justask do
 
     puts "Purged #{destroyed_count} dead notifications."
   end
+
+  desc "Prints lonely people."
+  task loners: :environment do
+    people = {}
+    Question.all.each do |q|
+      if q.author_is_anonymous and q.author_name != 'justask'
+        q.answers.each do |a|
+          if q.user == a.user
+            people[q.user.screen_name] ||= 0
+            people[q.user.screen_name] += 1
+            puts "#{q.user.screen_name} -- answer id #{a.id}"
+          end
+        end
+      end
+    end
+
+    max = 0
+    res = []
+    people.each { |k, v| max = v if v > max }
+    people.each { |k, v| res << k if v == max }
+    if res.length == 0
+      puts "No one?  I hope you're just on the development session."
+    else
+      puts res.length == 1 ? "And the winner is..." : "And the winners are..."
+      print "\033[5;31m"
+      res.each { |name| puts " - #{name}" }
+      print "\033[0m"
+    end
+  end
 end

From e789fb7e68378849c84997fb465c3900b69d8ac5 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sat, 27 Dec 2014 17:33:49 +0100
Subject: [PATCH 03/14] finally made that worker work.

---
 app/controllers/ajax/inbox_controller.rb | 10 ++--------
 app/workers/share_worker.rb              | 13 +++++++++++--
 2 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/app/controllers/ajax/inbox_controller.rb b/app/controllers/ajax/inbox_controller.rb
index 21ff83dc..52b4ca8a 100644
--- a/app/controllers/ajax/inbox_controller.rb
+++ b/app/controllers/ajax/inbox_controller.rb
@@ -46,14 +46,8 @@ class Ajax::InboxController < ApplicationController
       return
     end
 
-    # sharing
-    begin
-      share_to = JSON.parse params[:share]
-      current_user.services.each do |service|
-        service.post(answer) if share_to.include? service.provider
-      end
-    rescue
-    end
+    services = JSON.parse params[:share]
+    ShareWorker.perform_async(current_user.id, answer.id, services)
 
     @status = :okay
     @message = "Successfully answered question."
diff --git a/app/workers/share_worker.rb b/app/workers/share_worker.rb
index 079baaee..e607919b 100644
--- a/app/workers/share_worker.rb
+++ b/app/workers/share_worker.rb
@@ -3,7 +3,16 @@ class ShareWorker
 
   sidekiq_options queue: :share
 
-  def perform(answer_id)
-    # fuck this… for now
+  # @param user_id [Integer] the user id
+  # @param answer_id [Integer] the user id
+  # @param services [Array] array containing strings
+  def perform(user_id, answer_id, services)
+    User.find(user_id).services.each do |service|
+      begin
+        service.post(Answer.find(answer_id)) if services.include? service.provider
+      rescue => e
+        Rails.logger.error "failed to post answer #{answer_id} to #{service.provider} for user #{user_id}: #{e.message}"
+      end
+    end
   end
 end
\ No newline at end of file

From a487ed7f6931826ed22755227c32baef5918c586 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sat, 27 Dec 2014 17:37:54 +0100
Subject: [PATCH 04/14] update! raises an exception while update does not.

---
 app/models/services/twitter.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/models/services/twitter.rb b/app/models/services/twitter.rb
index 04ea97ce..aa6bece9 100644
--- a/app/models/services/twitter.rb
+++ b/app/models/services/twitter.rb
@@ -22,7 +22,7 @@ class Services::Twitter < Service
     end
 
     def post_tweet(answer)
-      client.update prepare_tweet(answer)
+      client.update! prepare_tweet(answer)
     end
 
     def prepare_tweet(answer)

From 3046a5e4b3204a6d98fce733abcb3f6b91199d80 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sat, 27 Dec 2014 21:03:18 +0100
Subject: [PATCH 05/14] fixed tweet not tweeting

---
 app/models/services/twitter.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/models/services/twitter.rb b/app/models/services/twitter.rb
index aa6bece9..b8b33d47 100644
--- a/app/models/services/twitter.rb
+++ b/app/models/services/twitter.rb
@@ -35,7 +35,7 @@ class Services::Twitter < Service
         host: APP_CONFIG['hostname'],
         protocol: (APP_CONFIG['https'] ? :https : :http)
       )
-      "#{question_content[0..55]}#{'…' if question_content.length > 56}" \
+      "#{question_content[0..54]}#{'…' if question_content.length > 55}" \
         " — #{answer_content[0..55]}#{'…' if answer_content.length > 56} #{answer_url}"
     end
 end
\ No newline at end of file

From f5feab59e024eaa5d959b08af9d0a170861c4818 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 00:02:01 +0100
Subject: [PATCH 06/14] as

---
 Rakefile | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/Rakefile b/Rakefile
index 02a1599a..df57da0c 100644
--- a/Rakefile
+++ b/Rakefile
@@ -98,6 +98,26 @@ namespace :justask do
     puts "Purged #{destroyed_count} dead notifications."
   end
 
+  desc "Fixes everything else"
+  task fix_db: :environment do
+    format = '%t (%c/%C) [%b>%i] %e'
+    total = Inbox.count
+    progress = ProgressBar.create title: 'Processing inboxes', format: format, starting_at: 0, total: total
+    destroyed_count = {
+        inbox: 0
+    }
+
+    Inbox.all.each do |n|
+      if n.question.nil?
+        n.destroy
+        destroyed_count[:inbox] += 1
+      end
+      progress.increment
+    end
+
+    puts "Purged #{destroyed_count[:inbox]} dead inbox entries."
+  end
+
   desc "Prints lonely people."
   task loners: :environment do
     people = {}

From d79394640e8733fe45b6122dd540bde6748f509e Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 00:34:56 +0100
Subject: [PATCH 07/14] fixed GitHub issue #25

---
 app/models/question.rb | 1 +
 app/models/user.rb     | 6 +++---
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/models/question.rb b/app/models/question.rb
index 740d59d6..aed4ec91 100644
--- a/app/models/question.rb
+++ b/app/models/question.rb
@@ -1,6 +1,7 @@
 class Question < ActiveRecord::Base
   belongs_to :user
   has_many :answers
+  has_many :inboxes, dependent: :destroy
 
   validates :content, length: { maximum: 255 }
 
diff --git a/app/models/user.rb b/app/models/user.rb
index 45b3626a..258d9c44 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -19,9 +19,9 @@ class User < ActiveRecord::Base
                                    dependent: :destroy
   has_many :friends,   through: :active_relationships, source: :target
   has_many :followers, through: :passive_relationships, source: :source
-  has_many :smiles
-  has_many :services
-  has_many :notifications, foreign_key: :recipient_id
+  has_many :smiles, dependent: :destroy
+  has_many :services, dependent: :destroy
+  has_many :notifications, foreign_key: :recipient_id, dependent: :destroy
 
   SCREEN_NAME_REGEX = /\A[a-zA-Z0-9_]{1,16}\z/
   WEBSITE_REGEX = /https?:\/\/([A-Za-z.\-]+)\/?(?:.*)/i

From 5a5338baa0aff838816f3bfd0e1c7e0cf9dc95fd Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 13:06:15 +0100
Subject: [PATCH 08/14] added some moar things to purge/attempt to fix!

---
 Rakefile | 44 +++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 41 insertions(+), 3 deletions(-)

diff --git a/Rakefile b/Rakefile
index df57da0c..0d6b8ca7 100644
--- a/Rakefile
+++ b/Rakefile
@@ -101,12 +101,16 @@ namespace :justask do
   desc "Fixes everything else"
   task fix_db: :environment do
     format = '%t (%c/%C) [%b>%i] %e'
-    total = Inbox.count
-    progress = ProgressBar.create title: 'Processing inboxes', format: format, starting_at: 0, total: total
     destroyed_count = {
-        inbox: 0
+        inbox: 0,
+        question: 0,
+        answer: 0,
+        smile: 0,
+        comment: 0
     }
 
+    total = Inbox.count
+    progress = ProgressBar.create title: 'Processing inboxes', format: format, starting_at: 0, total: total
     Inbox.all.each do |n|
       if n.question.nil?
         n.destroy
@@ -115,7 +119,41 @@ namespace :justask do
       progress.increment
     end
 
+    total = Question.count
+    progress = ProgressBar.create title: 'Processing questions', format: format, starting_at: 0, total: total
+    Question.all.each do |q|
+      if q.user.nil?
+        q.user_id = nil
+        q.author_is_anonymous = true
+        destroyed_count[:question] += 1
+      end
+      progress.increment
+    end
+
+    total = Answer.count
+    progress = ProgressBar.create title: 'Processing answers', format: format, starting_at: 0, total: total
+    Answer.all.each do |a|
+      if a.user.nil? or a.question.nil?
+        a.destroy
+        destroyed_count[:answer] += 1
+      end
+      progress.increment
+    end
+
+    total = Comment.count
+    progress = ProgressBar.create title: 'Processing comments', format: format, starting_at: 0, total: total
+    Comment.all.each do |c|
+      if c.user.nil? or c.answer.nil?
+        c.destroy
+        destroyed_count[:comment] += 1
+      end
+      progress.increment
+    end
+
     puts "Purged #{destroyed_count[:inbox]} dead inbox entries."
+    puts "Marked #{destroyed_count[:question]} questions as anonymous."
+    puts "Purged #{destroyed_count[:answer]} dead answers."
+    puts "Purged #{destroyed_count[:answer]} dead comments."
   end
 
   desc "Prints lonely people."

From 58517fbc0188d0eac3e7fb7a4660ecf11db4622d Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 17:59:26 +0100
Subject: [PATCH 09/14] added Redcarpet and Redis

---
 Gemfile      | 3 +++
 Gemfile.lock | 7 +++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/Gemfile b/Gemfile
index af1d4a74..06d36fed 100644
--- a/Gemfile
+++ b/Gemfile
@@ -40,11 +40,14 @@ gem 'sinatra', require: false
 
 gem 'questiongenerator', git: 'https://github.com/justask/questiongenerator.git'
 
+gem 'redcarpet'
+
 # OmniAuth and providers
 gem 'omniauth'
 gem 'omniauth-twitter'
 
 gem 'foreman'
+gem 'redis'
 
 group :development do
   gem 'spring'
diff --git a/Gemfile.lock b/Gemfile.lock
index c19e111c..ae7c7202 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -1,8 +1,8 @@
 GIT
   remote: https://github.com/justask/questiongenerator.git
-  revision: 95bfac7b8e9157702befb5a6f7ea8220037fd804
+  revision: 64d82a2b9de9df699fe5cf53bdfc90cb2bc3e771
   specs:
-    questiongenerator (0.0.1)
+    questiongenerator (0.0.2)
 
 GEM
   remote: https://rubygems.org/
@@ -214,6 +214,7 @@ GEM
     rake (10.4.2)
     rdoc (4.2.0)
       json (~> 1.4)
+    redcarpet (3.2.2)
     redis (3.2.0)
     redis-namespace (1.5.1)
       redis (~> 3.0, >= 3.0.4)
@@ -352,6 +353,8 @@ DEPENDENCIES
   rails (= 4.1.8)
   rails-assets-growl
   rails_admin
+  redcarpet
+  redis
   rspec-rails (~> 3.0.0)
   ruby-progressbar
   sass-rails (~> 4.0.3)

From 5b590d3455c333c04ce7ff881530822a762aa6c4 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 18:54:04 +0100
Subject: [PATCH 10/14] added sanitize gem

---
 Gemfile      | 1 +
 Gemfile.lock | 8 ++++++++
 2 files changed, 9 insertions(+)

diff --git a/Gemfile b/Gemfile
index 06d36fed..dd1e8e80 100644
--- a/Gemfile
+++ b/Gemfile
@@ -40,6 +40,7 @@ gem 'sinatra', require: false
 
 gem 'questiongenerator', git: 'https://github.com/justask/questiongenerator.git'
 
+gem 'sanitize'
 gem 'redcarpet'
 
 # OmniAuth and providers
diff --git a/Gemfile.lock b/Gemfile.lock
index ae7c7202..73345c77 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -77,6 +77,7 @@ GEM
     coffee-script-source (1.8.0)
     colorize (0.7.5)
     connection_pool (2.1.0)
+    crass (1.0.1)
     daemons (1.1.9)
     database_cleaner (1.3.0)
     devise (3.4.1)
@@ -152,6 +153,8 @@ GEM
     net-ssh (2.9.1)
     nokogiri (1.6.5)
       mini_portile (~> 0.6.0)
+    nokogumbo (1.2.0)
+      nokogiri
     nprogress-rails (0.1.6.3)
     oauth (0.4.7)
     omniauth (1.2.2)
@@ -239,6 +242,10 @@ GEM
     rspec-support (3.0.4)
     ruby-progressbar (1.7.0)
     safe_yaml (1.0.4)
+    sanitize (3.1.0)
+      crass (~> 1.0.1)
+      nokogiri (>= 1.4.4)
+      nokogumbo (= 1.2.0)
     sass (3.2.19)
     sass-rails (4.0.5)
       railties (>= 4.0.0, < 5.0)
@@ -357,6 +364,7 @@ DEPENDENCIES
   redis
   rspec-rails (~> 3.0.0)
   ruby-progressbar
+  sanitize
   sass-rails (~> 4.0.3)
   sdoc (~> 0.4.1)
   sidekiq

From 443ca0e6633cd2210ff3dc6f19bafcf7335bda6b Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 19:06:04 +0100
Subject: [PATCH 11/14] added sanitize initializer

---
 config/initializers/sanitize.rb | 9 +++++++++
 1 file changed, 9 insertions(+)
 create mode 100644 config/initializers/sanitize.rb

diff --git a/config/initializers/sanitize.rb b/config/initializers/sanitize.rb
new file mode 100644
index 00000000..69b8764d
--- /dev/null
+++ b/config/initializers/sanitize.rb
@@ -0,0 +1,9 @@
+EVIL_TAGS = {
+  elements: %w(quote a p i strong em del pre code table tr td th br ul ol li hr),
+  attributes: {
+    'a' => %w(href)
+  },
+  protocols: {
+    'a' => { 'href' => ['http', 'https', :relative] }
+  }
+}
\ No newline at end of file

From 0e4d7444787fef742899b4dc312bcbe1d911c6f3 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 19:06:49 +0100
Subject: [PATCH 12/14] added Markdown to answers

---
 app/helpers/markdown_helper.rb        | 21 +++++++++++++++++++++
 app/services/flavored_markdown.rb     | 26 ++++++++++++++++++++++++++
 app/views/shared/_answerbox.html.haml |  4 ++--
 3 files changed, 49 insertions(+), 2 deletions(-)
 create mode 100644 app/helpers/markdown_helper.rb
 create mode 100644 app/services/flavored_markdown.rb

diff --git a/app/helpers/markdown_helper.rb b/app/helpers/markdown_helper.rb
new file mode 100644
index 00000000..169900e5
--- /dev/null
+++ b/app/helpers/markdown_helper.rb
@@ -0,0 +1,21 @@
+module MarkdownHelper
+
+  def markdown(content)
+    md = Redcarpet::Markdown.new(FlavoredMarkdown,
+                                 filter_html: true,
+                                 escape_html: true,
+                                 no_images: true,
+                                 no_styles: true,
+                                 safe_links_only: true,
+                                 xhtml: false,
+                                 hard_wrap: true,
+                                 no_intra_emphasis: true,
+                                 tables: true,
+                                 fenced_code_blocks: true,
+                                 autolink: true,
+                                 disable_indented_code_blocks: true,
+                                 strikethrough: true,
+                                 superscript: true)
+    Sanitize.fragment(md.render(content), EVIL_TAGS).html_safe
+  end
+end
\ No newline at end of file
diff --git a/app/services/flavored_markdown.rb b/app/services/flavored_markdown.rb
new file mode 100644
index 00000000..e88f0453
--- /dev/null
+++ b/app/services/flavored_markdown.rb
@@ -0,0 +1,26 @@
+class FlavoredMarkdown < Redcarpet::Render::HTML
+  include Rails.application.routes.url_helpers
+
+  def preprocess(text)
+    wrap_mentions(text)
+  end
+
+  def wrap_mentions(text)
+    text.gsub! /(^|\s)(@\w+)/ do
+      "#{$1}[#{$2}](#{show_user_profile_path $2.tr('@', '')})"
+    end
+    text
+  end
+
+  def header(text, _header_level)
+    paragraph text
+  end
+
+  def paragraph(text)
+    "<p>#{text}</p>"
+  end
+
+  def raw_html(raw_html)
+    Rack::Utils.escape_html raw_html
+  end
+end
\ No newline at end of file
diff --git a/app/views/shared/_answerbox.html.haml b/app/views/shared/_answerbox.html.haml
index 8a53cdaa..0fc42e3b 100644
--- a/app/views/shared/_answerbox.html.haml
+++ b/app/views/shared/_answerbox.html.haml
@@ -25,14 +25,14 @@
   .panel-body
     %p
       - if @display_all.nil?
-        = a.content[0..255]
+        = markdown a.content[0..255]
         - if a.content.length > 255
           [...]
           %p
             %a.btn.btn-primary{href: show_user_answer_path(a.user.screen_name, a.id)}
               Read the entire answer
       - else
-        = a.content
+        = markdown a.content
     - if @user.nil?
       .row
         .col-md-6.col-sm-4.col-xs-7.text-left.text-muted

From 5829d4f1efebdc5c8d99e154cce9917e59f7761c Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 19:11:36 +0100
Subject: [PATCH 13/14] removed unnecessary p tag

---
 app/views/shared/_answerbox.html.haml | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/app/views/shared/_answerbox.html.haml b/app/views/shared/_answerbox.html.haml
index 0fc42e3b..fc24adae 100644
--- a/app/views/shared/_answerbox.html.haml
+++ b/app/views/shared/_answerbox.html.haml
@@ -23,16 +23,15 @@
           %p.answerbox--question-text
             = a.question.content
   .panel-body
-    %p
-      - if @display_all.nil?
-        = markdown a.content[0..255]
-        - if a.content.length > 255
-          [...]
-          %p
-            %a.btn.btn-primary{href: show_user_answer_path(a.user.screen_name, a.id)}
-              Read the entire answer
-      - else
-        = markdown a.content
+    - if @display_all.nil?
+      = markdown a.content[0..255]
+      - if a.content.length > 255
+        [...]
+        %p
+          %a.btn.btn-primary{href: show_user_answer_path(a.user.screen_name, a.id)}
+            Read the entire answer
+    - else
+      = markdown a.content
     - if @user.nil?
       .row
         .col-md-6.col-sm-4.col-xs-7.text-left.text-muted

From 7537331651fdc8763cae3e72abb60eed907f4d24 Mon Sep 17 00:00:00 2001
From: nilsding <nilsding@nilsding.org>
Date: Sun, 28 Dec 2014 19:12:28 +0100
Subject: [PATCH 14/14] changed this

---
 config/initializers/sanitize.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/initializers/sanitize.rb b/config/initializers/sanitize.rb
index 69b8764d..d6a4ffa9 100644
--- a/config/initializers/sanitize.rb
+++ b/config/initializers/sanitize.rb
@@ -1,5 +1,5 @@
 EVIL_TAGS = {
-  elements: %w(quote a p i strong em del pre code table tr td th br ul ol li hr),
+  elements: %w(blockquote a p i strong em del pre code table tr td th br ul ol li hr),
   attributes: {
     'a' => %w(href)
   },