Add missing magic_encodings. Fix blame commit select, hide blank blame. Fix git repo hooks - redo branch parse to support non-ascii branches. Refs #190, #134

Gemfile

@@ -21,7 +21,7 @@ gem 'rails-xmlrpc', '~> 0.3.6' # :git => 'git://github.com/chipiga/rails-xmlrpc.
 
 # gem 'rugged', '~> 0.16.0'
 gem 'grack', :git => 'git://github.com/rdblue/grack.git', :require => 'git_http'
-gem "grit", :git => 'git://github.com/chipiga/grit.git'
+gem "grit", :git => 'git://github.com/chipiga/grit.git' #, :path => '~/Sites/code/grit'
 gem 'charlock_holmes', '~> 0.6.8' #, :git => 'git://github.com/brianmario/charlock_holmes.git', :branch => 'bundle-icu'
 
 # Wiki

app/controllers/activity_feeds_controller.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class ActivityFeedsController < ApplicationController
   before_filter :authenticate_user!
 

app/controllers/admin/users_controller.rb

@@ -67,4 +67,4 @@ class Admin::UsersController < ApplicationController
 
     render :partial =>'users_ajax', :layout => false
   end
-end
+end

app/controllers/git/blobs_controller.rb

@@ -37,7 +37,7 @@ class Git::BlobsController < Git::BaseController
   end
 
   def blame
-    @blame = Grit::Blob.blame(@git_repository.repo, @commit.try(:id), @path)
+    @blame = Grit::Blob.blame(@git_repository.repo, @commit.id, @path)
   end
 
   def raw
@@ -55,10 +55,10 @@ class Git::BlobsController < Git::BaseController
   def set_path_blob
     @path = params[:path]
     @blob = @tree / @path
+    @commit = @git_repository.log(@treeish, @path, :max_count => 1).first
   end
 
   def find_tree
     @tree = @git_repository.tree(@treeish)
-    @commit = @git_repository.log(@treeish, @path, :max_count => 1).first # TODO WTF nil ?
   end
 end

app/helpers/activity_feeds_helper.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 module ActivityFeedsHelper
   def render_activity_feed(activity_feed)
     render :partial => activity_feed.partial, :locals => activity_feed.data.merge(:activity_feed => activity_feed)

app/helpers/build_lists_helper.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 module BuildListsHelper
   def build_list_status_color(status)
     if [BuildList::BUILD_PUBLISHED, BuildServer::SUCCESS].include? status

app/models/activity_feed.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class ActivityFeed < ActiveRecord::Base
 
   CODE = ['git_delete_branch_notification', 'git_new_push_notification', 'new_comment_commit_notification']

app/models/activity_feed_observer.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class ActivityFeedObserver < ActiveRecord::Observer
   observe :issue, :comment, :user, :build_list
 
@@ -70,7 +71,7 @@ class ActivityFeedObserver < ActiveRecord::Observer
 
     when 'GitHook'
       change_type = record.change_type
-      branch_name = record.refname.match(/\/([\w\d]+)$/)[1]
+      branch_name = record.refname.split('/').last
 
       last_commits = record.project.git_repository.repo.log(branch_name, nil).first(3)
       first_commiter = User.find_by_email(last_commits[0].author.email) unless last_commits.blank?

app/models/label.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class Label < ActiveRecord::Base
   has_many :labelings, :dependent => :destroy
   has_many :issues, :through => :labelings

app/models/labeling.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class Labeling < ActiveRecord::Base
   belongs_to :issue
   belongs_to :label

app/views/git/blobs/blame.html.haml

@@ -22,4 +22,4 @@
   - render_way = choose_render_way(@blob)
   .file
     .top= render 'top', :render_way => render_way
-    .blame_data= render 'blame_table'
+    .blame_data= render 'blame_table' if @blame.first.first.present?

db/migrate/20120124065207_create_activity_feeds.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class CreateActivityFeeds < ActiveRecord::Migration
   def self.up
     create_table :activity_feeds do |t|

db/migrate/20120219161749_add_creator_to_issue.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddCreatorToIssue < ActiveRecord::Migration
   def change
     add_column :issues, :creator_id, :integer

db/migrate/20120220175615_create_labels.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class CreateLabels < ActiveRecord::Migration
   def change
     create_table :labels do |t|

db/migrate/20120220185458_remove_repositories_owner.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class RemoveRepositoriesOwner < ActiveRecord::Migration
   def self.up
     remove_column :repositories, :owner_id

db/migrate/20120228094721_add_closed_at_to_issue.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddClosedAtToIssue < ActiveRecord::Migration
   def change
     add_column :issues, :closed_at, :datetime

db/migrate/20120228100121_add_closed_by_to_issue.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddClosedByToIssue < ActiveRecord::Migration
   def change
     add_column :issues, :closed_by, :integer

db/migrate/20120229163054_add_description_to_groups.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddDescriptionToGroups < ActiveRecord::Migration
   def change
     add_column :groups, :description, :text

db/migrate/20120229182356_add_default_values_to_is_rpm_for_projects.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddDefaultValuesToIsRpmForProjects < ActiveRecord::Migration
   def change
     Project.update_all(:is_rpm => true)

db/migrate/20120302102734_remove_name_from_groups.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class RemoveNameFromGroups < ActiveRecord::Migration
   def up
     remove_column :groups, :name

db/migrate/20120302114735_add_professional_experience_to_users.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddProfessionalExperienceToUsers < ActiveRecord::Migration
   def change
     add_column :users, :professional_experience, :text

db/migrate/20120303062601_add_site_company_and_location_to_users.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddSiteCompanyAndLocationToUsers < ActiveRecord::Migration
   def change
     add_column :users, :site, :string

db/migrate/20120303171802_add_avatar_to_users.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddAvatarToUsers < ActiveRecord::Migration
   def change
     change_table :users do |t|

db/migrate/20120306212914_add_project_to_comment.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddProjectToComment < ActiveRecord::Migration
   def up
     add_column :comments, :project_id, :integer

db/migrate/20120313130930_delete_dublicate_subscribes.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class DeleteDublicateSubscribes < ActiveRecord::Migration
   def up
     execute <<-SQL

db/migrate/20120314151558_truncate_activity_feed.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class TruncateActivityFeed < ActiveRecord::Migration
   def up
     ActivityFeed.destroy_all

db/migrate/20120314162313_add_description_to_products.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddDescriptionToProducts < ActiveRecord::Migration
   def self.up
     add_column :products, :description, :text

db/migrate/20120314223151_remove_is_template_from_products.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class RemoveIsTemplateFromProducts < ActiveRecord::Migration
   def up
     remove_column :products, :is_template

db/migrate/20120320102912_add_build_list_priority.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddBuildListPriority < ActiveRecord::Migration
   def self.up
     add_column :build_lists, :priority, :integer, :null => false, :default => 0
@@ -6,4 +7,4 @@ class AddBuildListPriority < ActiveRecord::Migration
   def self.down
     remove_column :build_lists, :priority
   end
-end
+end

db/migrate/20120321130436_add_devise_lockable.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class AddDeviseLockable < ActiveRecord::Migration
   def change
     add_column :users, :failed_attempts, :integer, :default => 0 # Only if lock strategy is :failed_attempts

db/schema.rb

@@ -1,4 +1,4 @@
-# encoding: UTF-8
+# -*- encoding : utf-8 -*-
 # This file is auto-generated from the current state of the database. Instead
 # of editing this file, please use the migrations feature of Active Record to
 # incrementally modify your database, and then regenerate this schema definition.

lib/ext/core/object.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 class Object
   def with_skip
     begin

lib/ext/gollum.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 # # -*- encoding : utf-8 -*-
 module Gollum
   class Wiki

lib/ext/grit.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 # -*- ruby encoding: utf-8 -*-
 
 module Grit

spec/controllers/activity_feeds_controller_spec.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 require 'spec_helper'
 
 describe ActivityFeedsController do

spec/factories/activity_feeds.rb

@@ -1,6 +1,7 @@
+# -*- encoding : utf-8 -*-
 # Read about factories at http://github.com/thoughtbot/factory_girl
 
 FactoryGirl.define do
   factory :activity_feed do
     end
-end
+end

spec/helpers/activity_feeds_helper_spec.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 require 'spec_helper'
 
 # Specs in this file have access to a helper object that includes

spec/helpers/build_lists_helper_spec.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 require 'spec_helper'
 
 # Specs in this file have access to a helper object that includes

spec/models/activity_feed_observer_spec.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 require 'spec_helper'
 
 describe ActivityFeedObserver do

spec/models/activity_feed_spec.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 require 'spec_helper'
 
 describe ActivityFeed do

spec/models/labels_spec.rb

@@ -1,3 +1,4 @@
+# -*- encoding : utf-8 -*-
 require 'spec_helper'
 
 describe Label do