Linked articles

app/controllers/tickets_controller.rb

@@ -24,6 +24,15 @@ class TicketsController < ApplicationController
   # this is needed for brimir integration in other sites
   before_filter :allow_cors, only: [:create, :new]
 
+  def edit_articles
+    @articles = Article.all
+    @linked_articles_ids = @ticket.articles.pluck(:id)
+
+    respond_to do |format|
+      format.html { render :layout => !request.xhr? }
+    end
+  end
+
   def ping
     @ticket = Ticket.find(params[:id])
     authorize! :read, @ticket
@@ -231,6 +240,7 @@ def ticket_params
         :deadline,
         :group_id,
         :labels_list,
+        articles_ids: [],
         attachments_attributes: [
           :file
         ])

app/models/ability.rb

@@ -39,6 +39,8 @@ def agent(user)
     can :manage, Reply, ticket: { assignee_id: user.id }
     can :manage, Reply, ticket: { group_id: user.group_ids }
 
+    can :manage, [Article, TicketArticle]
+
     can [:read, :create], Label
     can :manage, ::Template, user_id: user.id
     can :read, ::Template, public: true

app/models/article.rb

@@ -1,11 +1,16 @@
 class Article < ActiveRecord::Base
 
   validates_presence_of :title, :description, :body
+
   attr_accessor :labels_list
+
   after_save :update_labels
+  after_initialize :set_labels_list
+
   has_many :labelings, as: :labelable, dependent: :destroy
   has_many :labels, through: :labelings
-  after_initialize :set_labels_list
+  has_many :ticket_articles
+  has_many :tickets, through: :ticket_articles
 
   scope :search, ->(term) {
     if !term.nil?

app/models/ticket.rb

@@ -35,6 +35,9 @@ class Ticket < ActiveRecord::Base
 
   has_many :status_changes, dependent: :destroy
 
+  has_many :ticket_articles
+  has_many :articles, through: :ticket_articles
+
   enum status: [:open, :closed, :deleted, :waiting]
   enum priority: [:unknown, :low, :medium, :high]
 
@@ -47,10 +50,12 @@ class Ticket < ActiveRecord::Base
   after_create :create_labels
   after_create :set_start_time
   after_save :set_end_time
+  after_save :update_linked_articles
 
   attr_accessor :consumed_days, :consumed_hours, :consumed_minutes
   attr_accessor :sender
   attr_accessor :labels_list
+  attr_accessor :articles_ids
 
   def self.active_labels(status)
     label_ids = where(status: Ticket.statuses[status])
@@ -163,6 +168,12 @@ def find_matching_articles
 
   protected
 
+  def update_linked_articles
+    unless self.articles_ids.nil?
+      self.articles = Article.where(id: self.articles_ids)
+    end
+  end
+
   def set_start_time
     self.update_column(:start_time, self.created_at)
   end

app/models/ticket_article.rb

@@ -0,0 +1,4 @@
+class TicketArticle < ActiveRecord::Base
+  belongs_to :ticket
+  belongs_to :article
+end

app/views/articles/index.html.erb

@@ -26,6 +26,7 @@
   <thead>
     <tr>
       <th><%= sortable 'title', Article.human_attribute_name(:title) %></th>
+      <th><%= t(:linked_tickets_count, default: 'Linked tickets count')%></th>
       <th><%= Article.human_attribute_name(:labels) %></th>
       <th><%= sortable 'description', Article.human_attribute_name(:description) %></th>
     </tr>
@@ -34,6 +35,9 @@
     <% @articles.each do |article| %>
       <tr data-article-url="<%= article_url(article) %>">
         <td><%= link_to article.title, article %></td>
+        <td>
+          <span class="secondary ba radius label ml" title="<%= article.tickets.size %>"><%= article.tickets.size %></span>
+        </td>
         <td class="text-secondary table-nowrap">
           <% article.labels.viewable_by(current_user).each do |label| %>
             <%= render label %>

app/views/articles/show.html.erb

@@ -54,6 +54,15 @@
           </ul>
         <% end %>
         </dd>
+        <dt><%= t(:linked_tickets)%></dt>
+        <dd>
+          <% if @article.tickets.size == 0 %>
+            <%= t(:no_linked_articles)%>
+          <% end %>
+          <% @article.tickets.each do |ticket| %>
+            <div><%= link_to (ticket.subject || "<em>(#{t(:no_subject)})</em>".html_safe), ticket_path(ticket)%></div>
+          <% end %>
+        </dd>
       </div>
     </dl>
   </div>

app/views/tickets/_link_articles_form.html.erb

@@ -0,0 +1,3 @@
+<div id="link-articles" class="reveal-modal small" data-reveal>
+  <a class="close-reveal-modal">&#215;</a>
+</div>

app/views/tickets/edit_articles.html.erb

@@ -0,0 +1,16 @@
+<% if @articles.empty? %>
+  <%= t(:no_articles)%>
+<% else %>
+  <%= form_for @ticket, method: :patch do |f| %>
+    <%= hidden_field_tag "ticket[articles_ids][]", nil %>
+    <table class="full">
+      <% @articles.each do |article| %>
+        <tr>
+          <td><input type="checkbox" name="ticket[articles_ids][]" value="<%= article.id %>" <%= @linked_articles_ids.include?(article.id) ? "checked='true'" : "" %>></td>
+          <td><%= article.title%></td>
+        </tr>
+      <% end %>
+    </table>
+    <%= f.submit t(:change), class: 'button' %>
+  <% end %>
+<% end %>

app/views/tickets/show.html.erb

@@ -234,19 +234,33 @@
           <% end %>
           <% if can? :read, Article%>
             <div class="content" id="panel3">
-              <% matching_articles = @ticket.find_matching_articles %>
-              <% if matching_articles.empty? %>
-                <%= t(:no_matching_articles, default: 'No matching articles')%>
-              <% else %>
-                <dt><%= t(:matching_articles, default: 'Matching articles')%></dt>
-                <% matching_articles.each do |matching_article|%>
-                  <dd><%= link_to matching_article.title, article_path(matching_article), {target: '_BLANK'} %></dd>
+              <div>
+                <% linked_articles = @ticket.articles %>
+                <% if linked_articles.empty? %>
+                  <%= t(:no_linked_articles, default: 'No linked articles')%>
+                <% else %>
+                  <dt><%= t(:linked_articles, default: 'Linked articles')%></dt>
+                  <% linked_articles.each do |linked_article|%>
+                    <dd><%= link_to linked_article.title, article_path(linked_article), {target: '_BLANK'} %></dd>
+                  <% end %>
                 <% end %>
-              <% end %>
-
 
+                <% if can? :create, TicketArticle %>
+                  <div><a href="#" data-reveal-id="link-articles" class="tiny button radius" data-reveal-ajax="<%= edit_articles_ticket_path(@ticket)%>"><%= t(:manage)%></a></div>
+                <% end %>
 
-              <%= button_to t(:add), new_article_path, method: :get, class: 'tiny radius button mtm' %>
+              </div>
+              <div>
+                <% matching_articles = @ticket.find_matching_articles %>
+                <% if matching_articles.empty? %>
+                  <%= t(:no_matching_articles, default: 'No matching articles')%>
+                <% else %>
+                  <dt><%= t(:matching_articles, default: 'Matching articles')%></dt>
+                  <% matching_articles.each do |matching_article|%>
+                    <dd><%= link_to matching_article.title, article_path(matching_article), {target: '_BLANK'} %></dd>
+                  <% end %>
+                <% end %>
+              </div>
             </div>
           <% end %>
         </div>
@@ -261,3 +275,4 @@
 <%= render 'change_sender_form', { ticket: @ticket } %>
 <%= render 'change_group_form', { ticket: @ticket } %>
 <%= render 'change_subject_form', { ticket: @ticket } %>
+<%= render 'link_articles_form', { ticket: @ticket } %>

config/routes.rb

@@ -23,6 +23,7 @@
       post :ping
       put :closed
       get :clone
+      get :edit_articles
     end
   end
 

db/migrate/20150421192628_create_ticket_articles.rb

@@ -0,0 +1,10 @@
+class CreateTicketArticles < ActiveRecord::Migration
+  def change
+    create_table :ticket_articles do |t|
+      t.integer :ticket_id
+      t.integer :article_id
+
+      t.timestamps null: false
+    end
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150421091653) do
+ActiveRecord::Schema.define(version: 20150421192628) do
 
   create_table "articles", force: :cascade do |t|
     t.string   "title"
@@ -132,6 +132,13 @@
     t.boolean  "public",      default: false
   end
 
+  create_table "ticket_articles", force: :cascade do |t|
+    t.integer  "ticket_id"
+    t.integer  "article_id"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+  end
+
   create_table "tickets", force: :cascade do |t|
     t.string   "subject"
     t.text     "content"

test/fixtures/ticket_articles.yml

@@ -0,0 +1,9 @@
+# Read about fixtures at http://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
+
+one:
+  ticket_id: 1
+  article_id: 1
+
+two:
+  ticket_id: 1
+  article_id: 1

test/models/ticket_article_test.rb

@@ -0,0 +1,7 @@
+require 'test_helper'
+
+class TicketArticleTest < ActiveSupport::TestCase
+  # test "the truth" do
+  #   assert true
+  # end
+end