jro_apps/redmine Commit - r5811:a267babfe806

scm: mercurial: code clean up model....

Toshi MARUYAMA -

r5811:a267babfe806

parent child

Context file:

r5811:a267babfe806

Collapse all files

app/models/repository/mercurial.rb +9 -6

              class Repository::Mercurial < Repository
                # sort changesets by revision number
-               has_many :changesets, :order => "#{Changeset.table_name}.id DESC", :foreign_key => 'repository_id'
+               has_many :changesets,
+                        :order       => "#{Changeset.table_name}.id DESC",
+                        :foreign_key => 'repository_id'
                attr_protected :root_url
                validates_presence_of :url
-               FETCH_AT_ONCE = 100  # number of changesets to fetch at once
+               # number of changesets to fetch at once
+               FETCH_AT_ONCE = 100
                def self.human_attribute_name(attribute_key_name)
                  attr_name = attribute_key_name
                # Sqlite3 and PostgreSQL pass.
                # Is this MySQL bug?
                def latest_changesets(path, rev, limit=10)
-                 changesets.find(:all, :include => :user,
+                 changesets.find(:all,
+                                 :include    => :user,
                                  :conditions => latest_changesets_cond(path, rev, limit),
-                                 :limit => limit, :order => "#{Changeset.table_name}.id DESC")
+                                 :limit      => limit,
+                                 :order      => "#{Changeset.table_name}.id DESC")
                end
                def latest_changesets_cond(path, rev, limit)
                    cond << "#{Changeset.table_name}.id <= ?"
                    args << last.id
                  end
                  unless path.blank?
                    cond << "EXISTS (SELECT * FROM #{Change.table_name}
                               WHERE #{Change.table_name}.changeset_id = #{Changeset.table_name}.id
                    args << path.with_leading_slash
                    args << "#{path.with_leading_slash.gsub(%r{[%_\\]}) { |s| "\\#{s}" }}/%" << '\\'
                  end
                  [cond.join(' AND '), *args] unless cond.empty?
                end
                private :latest_changesets_cond

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages