jro_apps/redmine Commit - r4491:2ae2d3ef834c

Fixed: repository: mercurial: sort changesets by revision (#3449, #3567)....

Toshi MARUYAMA -

r4491:2ae2d3ef834c

parent child

Context file:

r4491:2ae2d3ef834c

Collapse all files

app/models/repository/mercurial.rb +15 0

             # redMine - project management software
             # Copyright (C) 2006-2007  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             # as published by the Free Software Foundation; either version 2
             # of the License, or (at your option) any later version.
             #
             # This program is distributed in the hope that it will be useful,
             # but WITHOUT ANY WARRANTY; without even the implied warranty of
             # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
             # GNU General Public License for more details.
             #
             # You should have received a copy of the GNU General Public License
             # along with this program; if not, write to the Free Software
             # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
             require 'redmine/scm/adapters/mercurial_adapter'
             class Repository::Mercurial < Repository
+              # sort changesets by revision number
+              has_many :changesets, :order => "#{Changeset.table_name}.id DESC", :foreign_key => 'repository_id'
               attr_protected :root_url
               validates_presence_of :url
               def scm_adapter
                 Redmine::Scm::Adapters::MercurialAdapter
               end
               def self.scm_name
                 'Mercurial'
               end
               def entries(path=nil, identifier=nil)
                 entries=scm.entries(path, identifier)
                 if entries
                   entries.each do |entry|
                     next unless entry.is_file?
                     # Set the filesize unless browsing a specific revision
                     if identifier.nil?
                       full_path = File.join(root_url, entry.path)
                       entry.size = File.stat(full_path).size if File.file?(full_path)
                     end
                     # Search the DB for the entry's last change
                     change = changes.find(:first, :conditions => ["path = ?", scm.with_leading_slash(entry.path)], :order => "#{Changeset.table_name}.committed_on DESC")
                     if change
                       entry.lastrev.identifier = change.changeset.revision
                       entry.lastrev.name = change.changeset.revision
                       entry.lastrev.author = change.changeset.committer
                       entry.lastrev.revision = change.revision
                     end
                   end
                 end
                 entries
               end
+              # Returns the latest changesets for +path+; sorted by revision number
+              def latest_changesets(path, rev, limit=10)
+                if path.blank?
+                  changesets.find(:all, :include => :user, :limit => limit)
+                else
+                  changes.find(:all, :include => {:changeset => :user},
+                                     :conditions => ["path = ?", path.with_leading_slash],
+                                     :order => "#{Changeset.table_name}.id DESC",
+                                     :limit => limit).collect(&:changeset)
+                end
+              end
               def fetch_changesets
                 scm_info = scm.info
                 if scm_info
                   # latest revision found in database
                   db_revision = latest_changeset ? latest_changeset.revision.to_i : -1
                   # latest revision in the repository
                   latest_revision = scm_info.lastrev
                   return if latest_revision.nil?
                   scm_revision = latest_revision.identifier.to_i
                   if db_revision < scm_revision
                     logger.debug "Fetching changesets for repository #{url}" if logger && logger.debug?
                     identifier_from = db_revision + 1
                     while (identifier_from <= scm_revision)
                       # loads changesets by batches of 100
                       identifier_to = [identifier_from + 99, scm_revision].min
                       revisions = scm.revisions('', identifier_from, identifier_to, :with_paths => true)
                       transaction do
                         revisions.each do |revision|
                           changeset = Changeset.create(:repository => self,
                                                        :revision => revision.identifier,
                                                        :scmid => revision.scmid,
                                                        :committer => revision.author,
                                                        :committed_on => revision.time,
                                                        :comments => revision.message)
                           revision.paths.each do |change|
                             changeset.create_change(change)
                           end
                         end
                       end unless revisions.nil?
                       identifier_from = identifier_to + 1
                     end
                   end
                 end
               end
             end

test/fixtures/repositories/mercurial_repository.tar.gz binary modified 0 0

NO CONTENT: modified file, binary diff hidden

test/unit/repository_mercurial_test.rb +11 0

             # redMine - project management software
             # Copyright (C) 2006-2007  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             # as published by the Free Software Foundation; either version 2
             # of the License, or (at your option) any later version.
             #
             # This program is distributed in the hope that it will be useful,
             # but WITHOUT ANY WARRANTY; without even the implied warranty of
             # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
             # GNU General Public License for more details.
             #
             # You should have received a copy of the GNU General Public License
             # along with this program; if not, write to the Free Software
             # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
             require File.expand_path('../../test_helper', __FILE__)
             class RepositoryMercurialTest < ActiveSupport::TestCase
               fixtures :projects
               # No '..' in the repository path
               REPOSITORY_PATH = RAILS_ROOT.gsub(%r{config\/\.\.}, '') + '/tmp/test/mercurial_repository'
               def setup
                 @project = Project.find(1)
                 assert @repository = Repository::Mercurial.create(:project => @project, :url => REPOSITORY_PATH)
               end
               if File.directory?(REPOSITORY_PATH)
                 def test_fetch_changesets_from_scratch
                   @repository.fetch_changesets
                   @repository.reload
                   assert_equal 6, @repository.changesets.count
                   assert_equal 11, @repository.changes.count
                   assert_equal "Initial import.\nThe repository contains 3 files.", @repository.changesets.find_by_revision('0').comments
                 end
                 def test_fetch_changesets_incremental
                   @repository.fetch_changesets
                   # Remove changesets with revision > 2
                   @repository.changesets.find(:all).each {|c| c.destroy if c.revision.to_i > 2}
                   @repository.reload
                   assert_equal 3, @repository.changesets.count
                   @repository.fetch_changesets
                   assert_equal 6, @repository.changesets.count
                 end
                 def test_entries
                   assert_equal 2, @repository.entries("sources", 2).size
                   assert_equal 1, @repository.entries("sources", 3).size
                 end
                 def test_locate_on_outdated_repository
                   # Change the working dir state
                   %x{hg -R #{REPOSITORY_PATH} up -r 0}
                   assert_equal 1, @repository.entries("images", 0).size
                   assert_equal 2, @repository.entries("images").size
                   assert_equal 2, @repository.entries("images", 2).size
                 end
                 def test_cat
                   assert @repository.scm.cat("sources/welcome_controller.rb", 2)
                   assert_nil @repository.scm.cat("sources/welcome_controller.rb")
                 end
                 def test_isodatesec
                   # Template keyword 'isodatesec' supported in Mercurial 1.0 and higher
                   if @repository.scm.class.client_version_above?([1, 0])
                     @repository.fetch_changesets
                     @repository.reload
                     rev0_committed_on = Time.gm(2007, 12, 14, 9, 22, 52)
                     assert_equal @repository.changesets.find_by_revision('0').committed_on, rev0_committed_on
                   end
                 end
+                def test_changeset_order_by_revision
+                  @repository.fetch_changesets
+                  @repository.reload
+                  c0 = @repository.latest_changeset
+                  c1 = @repository.changesets.find_by_revision('0')
+                  # sorted by revision (id), not by date
+                  assert c0.revision.to_i > c1.revision.to_i
+                  assert c0.committed_on  < c1.committed_on
+                end
               else
                 puts "Mercurial test repository NOT FOUND. Skipping unit tests !!!"
                 def test_fake; assert true end
               end
             end

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