houston-core 0.5.0.beta1 → 0.5.0

data/app/concerns/nosync.rb

@@ -1,5 +1,5 @@
 module Nosync
-  
+
   def nosync
     value = nosync?
     begin
@@ -9,13 +9,13 @@ module Nosync
       self.nosync = value
     end
   end
-  
+
   def nosync=(value)
     @nosync = value
   end
-  
+
   def nosync?
     !!@nosync
   end
-  
+
 end

data/app/concerns/project_adapter.rb

@@ -1,81 +1,81 @@
 module ProjectAdapter
-  
-  
+
+
   def adapters
     @adapters ||= {}
   end
-  
-  
+
+
   def has_adapter(*adapter_namespaces)
     adapter_namespaces.each do |adapter_namespace|
       adapter_module = Houston::Adapters[adapter_namespace]
       raise ArgumentError, "#{adapter_module} should respond to `adapters`" unless adapter_module.respond_to?(:adapters)
       raise ArgumentError, "#{adapter_module} should respond to `adapter`" unless adapter_module.respond_to?(:adapter)
-      
+
       adapter = Adapter.new(self, adapter_module)
       adapters[adapter.name] = adapter
-      
+
       adapter.define_methods!
-      
+
       validate adapter.validation_method
     end
   end
-  
-  
+
+
   class Adapter
-    
+
     def initialize(model, adapter_module)
       @model          = model
       @namespace      = adapter_module
       @name           = adapter_module.name
       @attribute_name = name.demodulize.underscore
     end
-    
+
     attr_reader :model, :namespace, :name, :attribute_name
-    
+
     def title
       name.demodulize.titleize
     end
-    
+
     def validation_method
       :"#{attribute_name}_configuration_is_valid"
     end
-    
+
     def define_methods!
       model.module_eval <<-RUBY
         def self.with_#{attribute_name}
           where arel_table[:#{attribute_name}_name].not_eq("None")
         end
-        
+
         def has_#{attribute_name}?
           #{attribute_name}_name != "None"
         end
-        
+
         def #{validation_method}
           #{attribute_name}_adapter.errors_with_parameters(self, *parameters_for_#{attribute_name}_adapter.values).each do |attribute, messages|
             errors.add(attribute, messages) if messages.any?
           end
         end
-        
+
         def #{attribute_name}
           @#{attribute_name} ||= #{attribute_name}_adapter
               .build(self, *parameters_for_#{attribute_name}_adapter.values)
               .extend(FeatureSupport)
         end
-        
+
         def parameters_for_#{attribute_name}_adapter
           #{attribute_name}_adapter.parameters.each_with_object({}) do |parameter, hash|
             hash[parameter] = extended_attributes[parameter.to_s]
           end
         end
-        
+
         def #{attribute_name}_adapter
           #{namespace}.adapter(#{attribute_name}_name)
         end
       RUBY
     end
-    
+
   end
-  
-  
+
+
 end

data/app/concerns/retirement.rb

@@ -1,28 +1,28 @@
 module Retirement
   extend ActiveSupport::Concern
-  
+
   module ClassMethods
     def unretired
       where(retired_at: nil)
     end
-    
+
     def retired
       where(arel_table[:retired_at].not_eq(nil))
     end
   end
-  
-  
+
+
   def retire!
     update_column(:retired_at, Time.now)
     freeze
   end
-  
+
   def unretire!
     update_column(:retired_at, nil)
   end
-  
+
   def retired?
     retired_at.present?
   end
-  
+
 end

data/app/concerns/ticket_synchronizer.rb

@@ -1,79 +1,79 @@
 module TicketSynchronizer
-  
-  
+
+
   def fetch_all
     return all if !ticket_tracker.supports?(:syncing_tickets)
-    
+
     Houston.benchmark "GET All Tickets" do
       synchronize ticket_tracker.all_tickets
     end
   end
-  
+
   def fetch_open
     return open if !ticket_tracker.supports?(:syncing_tickets)
-    
+
     Houston.benchmark "GET Open Tickets" do
       synchronize ticket_tracker.open_tickets
     end
   end
-  
+
   def find_by_number!(number)
     numbered(number, sync: true).first || (raise ActiveRecord::RecordNotFound)
   end
-  
+
   def numbered(*numbers, sync: false)
     numbers = numbers.flatten.map(&:to_i).uniq
     return none if numbers.empty?
-    
+
     results = super(*numbers)
     return results unless sync && ticket_tracker.supports?(:syncing_tickets)
-    
+
     results.concat fetch_numbered(numbers - results.map(&:number))
   end
-  
+
   def fetch_numbered(numbers)
     return [] if numbers.empty?
     return numbered(numbers) if !ticket_tracker.supports?(:syncing_tickets)
-    
+
     Houston.benchmark "GET Numbered Tickets" do
       synchronize ticket_tracker.find_tickets_numbered(numbers)
     end
   end
-  
+
   def fetch_with_query(*query)
     return [] unless ticket_tracker.respond_to?(:find_tickets!) # <-- an optional API
-    
+
     Rails.logger.info "[tickets.fetch_with_query] query: #{query.inspect}"
-    
+
     synchronize ticket_tracker.find_tickets!(*query)
   end
-  
-  
+
+
   def synchronize(unsynchronized_tickets)
     unsynchronized_tickets = unsynchronized_tickets.reject(&:nil?)
     return [] if unsynchronized_tickets.empty?
-    
+
     map_milestone_id = project.milestones.remote_id_map
-    
+
     Houston.benchmark("[tickets.synchronize] synchronizing #{unsynchronized_tickets.length} tickets") do
       numbers = unsynchronized_tickets.map(&:number)
       tickets = Ticket.unscoped { where(number: numbers) }
-      
+
       unsynchronized_tickets.map do |unsynchronized_ticket|
         ticket = tickets.detect { |ticket| ticket.number == unsynchronized_ticket.number }
         attributes = unsynchronized_ticket.attributes.merge(destroyed_at: nil)
-        
+
         # Convert remote milestone IDs to local milestone IDs
         attributes[:milestone_id] = map_milestone_id[attributes[:milestone_id]]
-        
+
         if ticket
-          
+
           # This is essentially a call to update_attributes,
           # but I broke it down so that we don't begin a
           # transaction if we don't have any changes to save.
           # This is pretty much just to reduce log verbosity.
           ticket.assign_attributes(attributes)
-          
+
           # hstore always thinks it has changed
           has_legitimate_changes = ticket.changed?
           if has_legitimate_changes && ticket.changed == %w{extended_attributes}
@@ -87,7 +87,7 @@ module TicketSynchronizer
         else
           ticket = Ticket.nosync { create(attributes) }
         end
-        
+
         # There's no reason why this shouldn't be set,
         # but in order to reduce a bunch of useless hits
         # to the cache and a bunch of log output...
@@ -96,8 +96,8 @@ module TicketSynchronizer
       end
     end
   end
-  
-  
+
+
   def create_from_remote(remote_ticket)
     attributes = remote_ticket.attributes
     if project.ticket_tracker.features.include?(:syncing_milestones)
@@ -105,16 +105,16 @@ module TicketSynchronizer
     end
     create(attributes)
   end
-  
-  
+
+
 private
-  
+
   def ticket_tracker
     project.ticket_tracker
   end
-  
+
   def project
     proxy_association.owner
   end
-  
+
 end

data/app/concerns/unique_add.rb

@@ -1,7 +1,7 @@
 module UniqueAdd
-  
+
   def add(relation)
     self << relation unless exists?(relation)
   end
-  
+
 end

data/app/controllers/api/v1/sprint_tasks_controller.rb

@@ -5,44 +5,44 @@ module Api
       before_filter :find_sprint
       before_filter :find_task, only: [:create, :destroy]
       skip_before_filter :verify_authenticity_token
-      
+
       attr_reader :sprint, :task
-      
+
       rescue_from ActiveRecord::RecordNotFound do
         head 404
       end
-      
+
       def index
         render json: sprint.tasks
           .includes(:ticket => :project)
           .map { |task| present_task(task) }
       end
-      
+
       def mine
         render json: sprint.tasks
           .includes(:ticket => :project)
           .checked_out_by(current_user)
           .map { |task| present_task(task) }
       end
-      
+
       def create
         authorize! :update, sprint
-        
+
         if sprint.completed?
           render text: "The Sprint is completed. You cannot add or remove tasks.", status: :unprocessable_entity
           return
         end
-        
+
         if sprint.locked? && !task.ticket_id.in?(sprint.ticket_ids)
           render text: "The Sprint is locked. You can add tasks for tickets that are already in the Sprint, but you can't add new tickets to the Sprint.", status: :unprocessable_entity
           return
         end
-        
+
         # Putting a task into a Sprint implies that you're able to estimate this ticket
         task.ticket.able_to_estimate! if task.ticket.respond_to?(:able_to_estimate!)
-        
+
         task.update_attributes(effort: params[:effort]) if params[:effort]
-        
+
         if task.completed? && task.completed_at < sprint.starts_at
           render text: "Task ##{task.shorthand} cannot be added to the Sprint because it was completed before the Sprint began", status: :unprocessable_entity
         elsif task.effort.nil? or task.effort.zero?
@@ -53,26 +53,26 @@ module Api
           render json: SprintTaskPresenter.new(sprint, task).to_json
         end
       end
-      
+
       def destroy
         authorize! :update, sprint
-        
+
         if sprint.completed?
           render text: "The Sprint is completed. You cannot add or remove tasks.", status: :unprocessable_entity
           return
         end
-        
+
         if sprint.locked?
           render text: "The Sprint is locked; tasks cannot be removed", status: :unprocessable_entity
           return
         end
-        
+
         SprintTask.where(sprint_id: sprint.id, task_id: task.id).delete_all
         head :ok
       end
-      
+
     private
-      
+
       def present_task(task)
         { projectSlug: task.project.slug,
           number: task.number,
@@ -83,15 +83,15 @@ module Api
           releasedAt: task.first_release_at,
           completedAt: task.completed_at }
       end
-      
+
       def find_sprint
         @sprint = Sprint.current || Sprint.new
       end
-      
+
       def find_task
         @task = Task.find_by_project_and_shorthand(params[:project_slug], params[:shorthand]) || (raise ActiveRecord::RecordNotFound)
       end
-      
+
     end
   end
 end

data/app/controllers/api/v1/ticket_tasks_controller.rb

@@ -5,22 +5,22 @@ module Api
       before_filter :find_project_and_ticket
       before_filter :find_task, only: [:update, :destroy]
       skip_before_filter :verify_authenticity_token
-      
+
       attr_reader :project, :ticket, :task
-      
+
       rescue_from ActiveRecord::RecordNotFound do
         head 404
       end
-      
+
       def index
         authorize! :read, Task
         render json: ticket.tasks.map { |task| present_task(task) }
       end
-      
+
       def create
         task = ticket.tasks.build params.slice(:description, :effort)
         authorize! :create, task
-        
+
         task.updated_by = current_user
         if task.save
           render json: present_task(task), status: :created
@@ -28,10 +28,10 @@ module Api
           render json: {errors: task.errors.full_messages}, status: :unprocessable_entity
         end
       end
-      
+
       def update
         authorize! :update, task
-        
+
         task.attributes = params.slice(:description, :effort)
         task.updated_by = current_user
         if task.save
@@ -40,16 +40,16 @@ module Api
           render json: {errors: task.errors.full_messages}, status: :unprocessable_entity
         end
       end
-      
+
       def destroy
         authorize! :destroy, task
-        
+
         task.destroy
         head :ok
       end
-      
+
     private
-    
+
       def present_task(task)
         { id: task.id,
           number: task.number,
@@ -60,16 +60,16 @@ module Api
           releasedAt: task.first_release_at,
           completedAt: task.completed_at }
       end
-      
+
       def find_project_and_ticket
         @project = Project.find_by_slug! params[:slug]
         @ticket = project.tickets.find_by_number! params[:number]
       end
-      
+
       def find_task
         @task = ticket.tasks.find params[:id]
       end
-      
+
     end
   end
 end

data/app/controllers/application_controller.rb

@@ -1,15 +1,15 @@
 class ApplicationController < ActionController::Base
   include UrlHelper
-  
+
   # Prevent CSRF attacks by raising an exception.
   # For APIs, you may want to use :null_session instead.
   protect_from_forgery with: :exception
-  
+
   before_filter :set_current_project
   after_filter :save_current_project
-  
-  
-  
+
+
+
   rescue_from CanCan::AccessDenied do |exception|
     if current_user
       if request.xhr?
@@ -23,19 +23,19 @@ class ApplicationController < ActionController::Base
       require_login
     end
   end
-  
+
   rescue_from UserCredentials::MissingCredentials do
     head 401, "X-Credentials" => "Missing Credentials"
   end
-  
+
   rescue_from UserCredentials::InvalidCredentials do
     head 401, "X-Credentials" => "Invalid Credentials"
   end
-  
+
   rescue_from UserCredentials::InsufficientPermissions do |exception|
     render text: exception.message, status: 401
   end
-  
+
   rescue_from Github::Unauthorized do |exception|
     session["user.return_to"] = request.referer
     if request.xhr?
@@ -44,44 +44,50 @@ class ApplicationController < ActionController::Base
       redirect_to oauth_consumer_path(id: "github")
     end
   end
-  
+
   rescue_from ActiveRecord::RecordNotFound do
     render file: "public/404", layout: false
   end
-  
+
   # Malformed request
   rescue_from ActionController::UnknownFormat do
     head 400
   end unless Rails.env.development?
-  
-  
-  
+
+
+
   def require_login
     redirect_to main_app.new_user_session_path
   end
-  
-  
-  
+
+
+
+  def unfurling?
+    request.env["HTTP_USER_AGENT"] =~ /^Slackbot-LinkExpanding/
+  end
+
+
+
   def after_sign_in_path_for(user)
     path = session["user_redirect_to"] || stored_location_for(user) || root_path
     path = root_path if path =~ /\/users\/(sign_in|password)/
     path
   end
-  
-  
-  
+
+
+
   def revision
     expire_revision!
     return_or_cache_revision!
   end
-  
+
   def expire_revision!
     if session[:revision_expiration].blank? || session[:revision_expiration] < Time.now.utc
       session[:revision] = nil
       Rails.logger.info "[revision] expiring"
     end
   end
-  
+
   def return_or_cache_revision!
     session[:revision] || read_revision.tap do |sha|
       session[:revision] = sha
@@ -89,17 +95,17 @@ class ApplicationController < ActionController::Base
       Rails.logger.info "[revision] sha: #{sha[0..8]}, expiration: #{session[:revision_expiration]}"
     end
   end
-  
+
   def read_revision
     revision_path = Rails.root.join("REVISION")
     File.exists?(revision_path) ? File.read(revision_path).chomp : ""
   end
-  
-  
-  
+
+
+
   def api_authenticate!
     return if current_user
-    
+
     allow_params_authentication!
     authenticate_or_request_with_http_basic do |username, password|
       params["user"] ||= {}
@@ -112,40 +118,40 @@ class ApplicationController < ActionController::Base
       end
     end
   end
-  
-  
-  
+
+
+
   helper_method :followed_projects
   def followed_projects
     return @followed_projects if defined?(@followed_projects)
     return @followed_projects = [] unless current_user
     @followed_projects = current_user.followed_projects.to_a
   end
-  
+
   helper_method :current_project
   def current_project
     @current_project ||= @project || (@default_project_slug ? Project[@default_project_slug] : (current_user && current_user.current_project))
   end
-  
+
   def set_current_project
     @default_project_slug = params[:project] if params[:project].is_a?(String)
   end
-  
+
   def save_current_project
     return unless current_user && current_project
-    
+
     current_user.current_project_id = current_project.id
     current_user.save if current_user.current_project_id_changed?
   end
-  
-  
-  
+
+
+
   def no_cache
     response.headers["Cache-Control"] = "no-cache, no-store, max-age=0, must-revalidate"
     response.headers["Pragma"] = "no-cache"
     response.headers["Expires"] = "Fri, 01 Jan 1990 00:00:00 GMT"
   end
-  
-  
-  
+
+
+
 end