Commit dc844f01 authored by Valery Sizov's avatar Valery Sizov

Graph: small refactoring

parent c8968260
...@@ -181,7 +181,10 @@ class ProjectsController < ApplicationController ...@@ -181,7 +181,10 @@ class ProjectsController < ApplicationController
def add_refs(commit, ref_cache) def add_refs(commit, ref_cache)
if ref_cache.empty? if ref_cache.empty?
@repo.refs.each {|ref| ref_cache[ref.commit.id] ||= [];ref_cache[ref.commit.id] << ref} @repo.refs.each do |ref|
ref_cache[ref.commit.id] ||= []
ref_cache[ref.commit.id] << ref
end
end end
commit.refs = ref_cache[commit.id] if ref_cache.include? commit.id commit.refs = ref_cache[commit.id] if ref_cache.include? commit.id
commit.refs ||= [] commit.refs ||= []
......
...@@ -25,7 +25,7 @@ class GraphCommit ...@@ -25,7 +25,7 @@ class GraphCommit
commits.reverse.each_with_index do |c,i| commits.reverse.each_with_index do |c,i|
c.time = i c.time = i
days[i]=c.committed_date days[i] = c.committed_date
map[c.id] = c map[c.id] = c
heads += c.refs unless c.refs.nil? heads += c.refs unless c.refs.nil?
end end
...@@ -63,7 +63,7 @@ class GraphCommit ...@@ -63,7 +63,7 @@ class GraphCommit
m1 = mark - 1 m1 = mark - 1
marks = commit.parents.collect do |p| marks = commit.parents.collect do |p|
if map.include? p.id and map[p.id].space == 0 then if map.include? p.id and map[p.id].space == 0 then
mark_chain(m1+=1, map[p.id],map) mark_chain(m1 += 1, map[p.id],map)
else else
m1 + 1 m1 + 1
end end
...@@ -72,11 +72,4 @@ class GraphCommit ...@@ -72,11 +72,4 @@ class GraphCommit
marks.compact.max marks.compact.max
end end
def self.add_refs(commit, ref_cache)
if ref_cache.empty?
@repo.refs.each {|ref| ref_cache[ref.commit.id] ||= [];ref_cache[ref.commit.id] << ref}
end
commit.refs = ref_cache[commit.id] if ref_cache.include? commit.id
commit.refs ||= []
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment