##// END OF EJS Templates
scm: mercurial: code clean up model....
Toshi MARUYAMA -
r5811:a267babfe806
parent child
Show More
@@ -19,12 +19,15 require 'redmine/scm/adapters/mercurial_adapter'
19
19
20 class Repository::Mercurial < Repository
20 class Repository::Mercurial < Repository
21 # sort changesets by revision number
21 # sort changesets by revision number
22 has_many :changesets, :order => "#{Changeset.table_name}.id DESC", :foreign_key => 'repository_id'
22 has_many :changesets,
23 :order => "#{Changeset.table_name}.id DESC",
24 :foreign_key => 'repository_id'
23
25
24 attr_protected :root_url
26 attr_protected :root_url
25 validates_presence_of :url
27 validates_presence_of :url
26
28
27 FETCH_AT_ONCE = 100 # number of changesets to fetch at once
29 # number of changesets to fetch at once
30 FETCH_AT_ONCE = 100
28
31
29 def self.human_attribute_name(attribute_key_name)
32 def self.human_attribute_name(attribute_key_name)
30 attr_name = attribute_key_name
33 attr_name = attribute_key_name
@@ -84,9 +87,11 class Repository::Mercurial < Repository
84 # Sqlite3 and PostgreSQL pass.
87 # Sqlite3 and PostgreSQL pass.
85 # Is this MySQL bug?
88 # Is this MySQL bug?
86 def latest_changesets(path, rev, limit=10)
89 def latest_changesets(path, rev, limit=10)
87 changesets.find(:all, :include => :user,
90 changesets.find(:all,
91 :include => :user,
88 :conditions => latest_changesets_cond(path, rev, limit),
92 :conditions => latest_changesets_cond(path, rev, limit),
89 :limit => limit, :order => "#{Changeset.table_name}.id DESC")
93 :limit => limit,
94 :order => "#{Changeset.table_name}.id DESC")
90 end
95 end
91
96
92 def latest_changesets_cond(path, rev, limit)
97 def latest_changesets_cond(path, rev, limit)
@@ -108,7 +113,6 class Repository::Mercurial < Repository
108 cond << "#{Changeset.table_name}.id <= ?"
113 cond << "#{Changeset.table_name}.id <= ?"
109 args << last.id
114 args << last.id
110 end
115 end
111
112 unless path.blank?
116 unless path.blank?
113 cond << "EXISTS (SELECT * FROM #{Change.table_name}
117 cond << "EXISTS (SELECT * FROM #{Change.table_name}
114 WHERE #{Change.table_name}.changeset_id = #{Changeset.table_name}.id
118 WHERE #{Change.table_name}.changeset_id = #{Changeset.table_name}.id
@@ -117,7 +121,6 class Repository::Mercurial < Repository
117 args << path.with_leading_slash
121 args << path.with_leading_slash
118 args << "#{path.with_leading_slash.gsub(%r{[%_\\]}) { |s| "\\#{s}" }}/%" << '\\'
122 args << "#{path.with_leading_slash.gsub(%r{[%_\\]}) { |s| "\\#{s}" }}/%" << '\\'
119 end
123 end
120
121 [cond.join(' AND '), *args] unless cond.empty?
124 [cond.join(' AND '), *args] unless cond.empty?
122 end
125 end
123 private :latest_changesets_cond
126 private :latest_changesets_cond
General Comments 0
You need to be logged in to leave comments. Login now