changelog: rename changelog to commits pages

rhodecode/apps/home/views.py

@@ -469,7 +469,7 b' class HomeView(BaseAppView):'
                                     _query=query_modifier())
                 }
 
-            if repo_context in ['commit', 'changelog']:
+            if repo_context in ['commit', 'commits']:
                 queries.extend([commit_qry, file_qry])
             elif repo_context in ['files', 'summary']:
                 queries.extend([file_qry, commit_qry])
@@ -509,7 +509,7 b' class HomeView(BaseAppView):'
                                     _query=query_modifier())
                 }
 
-            if repo_context in ['commit', 'changelog']:
+            if repo_context in ['commit', 'commits']:
                 queries.extend([commit_qry, file_qry])
             elif repo_context in ['files', 'summary']:
                 queries.extend([file_qry, commit_qry])

rhodecode/apps/journal/views.py

@@ -153,7 +153,7 b' class JournalView(BaseAppView):'
             desc = action_extra()
             _url = h.route_url('home')
             if entry.repository is not None:
-                _url = h.route_url('repo_changelog',
+                _url = h.route_url('repo_commits',
                                    repo_name=entry.repository.repo_name)
 
             feed.add_item(
@@ -199,7 +199,7 b' class JournalView(BaseAppView):'
             desc = action_extra()
             _url = h.route_url('home')
             if entry.repository is not None:
-                _url = h.route_url('repo_changelog',
+                _url = h.route_url('repo_commits',
                                    repo_name=entry.repository.repo_name)
 
             feed.add_item(

rhodecode/apps/repository/__init__.py

@@ -193,19 +193,27 b' def includeme(config):'
         name='repo_stats',
         pattern='/{repo_name:.*?[^/]}/repo_stats/{commit_id}', repo_route=True)
 
-    # Changelog
+    # Commits
+    config.add_route(
+        name='repo_commits',
+        pattern='/{repo_name:.*?[^/]}/commits', repo_route=True)
+    config.add_route(
+        name='repo_commits_file',
+        pattern='/{repo_name:.*?[^/]}/commits/{commit_id}/{f_path:.*}', repo_route=True)
+    config.add_route(
+        name='repo_commits_elements',
+        pattern='/{repo_name:.*?[^/]}/commits_elements', repo_route=True)
+    config.add_route(
+        name='repo_commits_elements_file',
+        pattern='/{repo_name:.*?[^/]}/commits_elements/{commit_id}/{f_path:.*}', repo_route=True)
+
+    # Changelog (old deprecated name for commits page)
     config.add_route(
         name='repo_changelog',
         pattern='/{repo_name:.*?[^/]}/changelog', repo_route=True)
     config.add_route(
         name='repo_changelog_file',
         pattern='/{repo_name:.*?[^/]}/changelog/{commit_id}/{f_path:.*}', repo_route=True)
-    config.add_route(
-        name='repo_changelog_elements',
-        pattern='/{repo_name:.*?[^/]}/changelog_elements', repo_route=True)
-    config.add_route(
-        name='repo_changelog_elements_file',
-        pattern='/{repo_name:.*?[^/]}/changelog_elements/{commit_id}/{f_path:.*}', repo_route=True)
 
     # Compare
     config.add_route(

rhodecode/apps/repository/tests/test_repo_changelog.py

@@ -33,8 +33,9 b' def route_path(name, params=None, **kwar'
 
     base_url = {
         'repo_changelog':'/{repo_name}/changelog',
-        'repo_changelog_file':'/{repo_name}/changelog/{commit_id}/{f_path}',
-        'repo_changelog_elements':'/{repo_name}/changelog_elements',
+        'repo_commits': '/{repo_name}/commits',
+        'repo_commits_file': '/{repo_name}/commits/{commit_id}/{f_path}',
+        'repo_commits_elements': '/{repo_name}/commits_elements',
     }[name].format(**kwargs)
 
     if params:
@@ -42,8 +43,23 b' def route_path(name, params=None, **kwar'
     return base_url
 
 
+def assert_commits_on_page(response, indexes):
+    found_indexes = [int(idx) for idx in MATCH_HASH.findall(response.body)]
+    assert found_indexes == indexes
+
+
 class TestChangelogController(TestController):
 
+    def test_commits_page(self, backend):
+        self.log_user()
+        response = self.app.get(
+            route_path('repo_commits', repo_name=backend.repo_name))
+
+        first_idx = -1
+        last_idx = -DEFAULT_CHANGELOG_SIZE
+        self.assert_commit_range_on_page(
+            response, first_idx, last_idx, backend)
+
     def test_changelog(self, backend):
         self.log_user()
         response = self.app.get(
@@ -62,6 +78,14 b' class TestChangelogController(TestContro'
                        params=dict(branch=backend.default_branch_name)),
             status=200)
 
+    @pytest.mark.backends("hg", "git")
+    def test_commits_filtered_by_branch(self, backend):
+        self.log_user()
+        self.app.get(
+            route_path('repo_commits', repo_name=backend.repo_name,
+                       params=dict(branch=backend.default_branch_name)),
+            status=200)
+
     @pytest.mark.backends("svn")
     def test_changelog_filtered_by_branch_svn(self, autologin_user, backend):
         repo = backend['svn-simple-layout']
@@ -70,27 +94,22 b' class TestChangelogController(TestContro'
                        params=dict(branch='trunk')),
             status=200)
 
-        self.assert_commits_on_page(
-            response, indexes=[15, 12, 7, 3, 2, 1])
+        assert_commits_on_page(response, indexes=[15, 12, 7, 3, 2, 1])
 
-    def test_changelog_filtered_by_wrong_branch(self, backend):
+    def test_commits_filtered_by_wrong_branch(self, backend):
         self.log_user()
         branch = 'wrong-branch-name'
         response = self.app.get(
-            route_path('repo_changelog', repo_name=backend.repo_name,
+            route_path('repo_commits', repo_name=backend.repo_name,
                        params=dict(branch=branch)),
             status=302)
-        expected_url = '/{repo}/changelog/{branch}'.format(
+        expected_url = '/{repo}/commits/{branch}'.format(
             repo=backend.repo_name, branch=branch)
         assert expected_url in response.location
         response = response.follow()
         expected_warning = 'Branch {} is not found.'.format(branch)
         assert expected_warning in response.body
 
-    def assert_commits_on_page(self, response, indexes):
-        found_indexes = [int(idx) for idx in MATCH_HASH.findall(response.body)]
-        assert found_indexes == indexes
-
     @pytest.mark.xfail_backends("svn", reason="Depends on branch support")
     def test_changelog_filtered_by_branch_with_merges(
             self, autologin_user, backend):
@@ -112,21 +131,20 b' class TestChangelogController(TestContro'
             status=200)
 
     @pytest.mark.backends("hg")
-    def test_changelog_closed_branches(self, autologin_user, backend):
+    def test_commits_closed_branches(self, autologin_user, backend):
         repo = backend['closed_branch']
         response = self.app.get(
-            route_path('repo_changelog', repo_name=repo.repo_name,
+            route_path('repo_commits', repo_name=repo.repo_name,
                        params=dict(branch='experimental')),
             status=200)
 
-        self.assert_commits_on_page(
-            response, indexes=[3, 1])
+        assert_commits_on_page(response, indexes=[3, 1])
 
     def test_changelog_pagination(self, backend):
         self.log_user()
         # pagination, walk up to page 6
         changelog_url = route_path(
-            'repo_changelog', repo_name=backend.repo_name)
+            'repo_commits', repo_name=backend.repo_name)
 
         for page in range(1, 7):
             response = self.app.get(changelog_url, {'page': page})
@@ -168,10 +186,10 b' class TestChangelogController(TestContro'
         '/vcs/exceptions.py',
         '//vcs/exceptions.py'
     ])
-    def test_changelog_with_filenode(self, backend, test_path):
+    def test_commits_with_filenode(self, backend, test_path):
         self.log_user()
         response = self.app.get(
-            route_path('repo_changelog_file', repo_name=backend.repo_name,
+            route_path('repo_commits_file', repo_name=backend.repo_name,
                        commit_id='tip', f_path=test_path),
             )
 
@@ -180,16 +198,16 b' class TestChangelogController(TestContro'
         response.mustcontain('Added not implemented hg backend test case')
         response.mustcontain('Added BaseChangeset class')
 
-    def test_changelog_with_filenode_that_is_dirnode(self, backend):
+    def test_commits_with_filenode_that_is_dirnode(self, backend):
         self.log_user()
         self.app.get(
-            route_path('repo_changelog_file', repo_name=backend.repo_name,
+            route_path('repo_commits_file', repo_name=backend.repo_name,
                        commit_id='tip', f_path='/tests'),
             status=302)
 
-    def test_changelog_with_filenode_not_existing(self, backend):
+    def test_commits_with_filenode_not_existing(self, backend):
         self.log_user()
         self.app.get(
-            route_path('repo_changelog_file', repo_name=backend.repo_name,
+            route_path('repo_commits_file', repo_name=backend.repo_name,
                        commit_id='tip', f_path='wrong_path'),
             status=302)

rhodecode/apps/repository/tests/test_repo_pullrequests.py

@@ -40,6 +40,8 b' def route_path(name, params=None, **kwar'
     base_url = {
         'repo_changelog': '/{repo_name}/changelog',
         'repo_changelog_file': '/{repo_name}/changelog/{commit_id}/{f_path}',
+        'repo_commits': '/{repo_name}/changelog',
+        'repo_commits_file': '/{repo_name}/changelog/{commit_id}/{f_path}',
         'pullrequest_show': '/{repo_name}/pull-request/{pull_request_id}',
         'pullrequest_show_all': '/{repo_name}/pull-request',
         'pullrequest_show_all_data': '/{repo_name}/pull-request-data',
@@ -998,11 +1000,11 b' class TestPullrequestsView(object):'
         assert len(target_children) == 1
 
         expected_origin_link = route_path(
-            'repo_changelog',
+            'repo_commits',
             repo_name=pull_request.source_repo.scm_instance().name,
             params=dict(branch='origin'))
         expected_target_link = route_path(
-            'repo_changelog',
+            'repo_commits',
             repo_name=pull_request.target_repo.scm_instance().name,
             params=dict(branch='target'))
         assert origin_children[0].attrib['href'] == expected_origin_link

rhodecode/apps/repository/views/repo_changelog.py

@@ -113,7 +113,7 b' class RepoChangelogView(RepoAppView):'
             h.flash('Branch {} is not found.'.format(h.escape(branch_name)),
                     category='warning')
             redirect_url = h.route_path(
-                'repo_changelog_file', repo_name=repo_name,
+                'repo_commits_file', repo_name=repo_name,
                 commit_id=branch_name, f_path=f_path or '')
             raise HTTPFound(redirect_url)
 
@@ -127,13 +127,13 b' class RepoChangelogView(RepoAppView):'
             if f_path:
                 # changelog for file
                 return h.route_path(
-                    'repo_changelog_file',
+                    'repo_commits_file',
                     repo_name=c.rhodecode_db_repo.repo_name,
                     commit_id=commit_id, f_path=f_path,
                     _query=query_params)
             else:
                 return h.route_path(
-                    'repo_changelog',
+                    'repo_commits',
                     repo_name=c.rhodecode_db_repo.repo_name, _query=query_params)
 
         c.total_cs = len(collection)
@@ -171,11 +171,18 b' class RepoChangelogView(RepoAppView):'
     @HasRepoPermissionAnyDecorator(
         'repository.read', 'repository.write', 'repository.admin')
     @view_config(
+        route_name='repo_commits', request_method='GET',
+        renderer='rhodecode:templates/commits/changelog.mako')
+    @view_config(
+        route_name='repo_commits_file', request_method='GET',
+        renderer='rhodecode:templates/commits/changelog.mako')
+    # old routes for backward compat
+    @view_config(
         route_name='repo_changelog', request_method='GET',
-        renderer='rhodecode:templates/changelog/changelog.mako')
+        renderer='rhodecode:templates/commits/changelog.mako')
     @view_config(
         route_name='repo_changelog_file', request_method='GET',
-        renderer='rhodecode:templates/changelog/changelog.mako')
+        renderer='rhodecode:templates/commits/changelog.mako')
     def repo_changelog(self):
         c = self.load_default_context()
 
@@ -224,7 +231,7 b' class RepoChangelogView(RepoAppView):'
                     except RepositoryError as e:
                         h.flash(safe_str(e), category='warning')
                         redirect_url = h.route_path(
-                            'repo_changelog', repo_name=self.db_repo_name)
+                            'repo_commits', repo_name=self.db_repo_name)
                         raise HTTPFound(redirect_url)
                 collection = list(reversed(collection))
             else:
@@ -246,14 +253,14 b' class RepoChangelogView(RepoAppView):'
             log.exception(safe_str(e))
             h.flash(safe_str(h.escape(e)), category='error')
             raise HTTPFound(
-                h.route_path('repo_changelog', repo_name=self.db_repo_name))
+                h.route_path('repo_commits', repo_name=self.db_repo_name))
 
         if partial_xhr or self.request.environ.get('HTTP_X_PJAX'):
             # case when loading dynamic file history in file view
             # loading from ajax, we don't want the first result, it's popped
             # in the code above
             html = render(
-                'rhodecode:templates/changelog/changelog_file_history.mako',
+                'rhodecode:templates/commits/changelog_file_history.mako',
                 self._get_template_context(c), self.request)
             return Response(html)
 
@@ -271,14 +278,14 b' class RepoChangelogView(RepoAppView):'
     @HasRepoPermissionAnyDecorator(
         'repository.read', 'repository.write', 'repository.admin')
     @view_config(
-        route_name='repo_changelog_elements', request_method=('GET', 'POST'),
-        renderer='rhodecode:templates/changelog/changelog_elements.mako',
+        route_name='repo_commits_elements', request_method=('GET', 'POST'),
+        renderer='rhodecode:templates/commits/changelog_elements.mako',
         xhr=True)
     @view_config(
-        route_name='repo_changelog_elements_file', request_method=('GET', 'POST'),
-        renderer='rhodecode:templates/changelog/changelog_elements.mako',
+        route_name='repo_commits_elements_file', request_method=('GET', 'POST'),
+        renderer='rhodecode:templates/commits/changelog_elements.mako',
         xhr=True)
-    def repo_changelog_elements(self):
+    def repo_commits_elements(self):
         c = self.load_default_context()
         commit_id = self.request.matchdict.get('commit_id')
         f_path = self._get_f_path(self.request.matchdict)
@@ -312,7 +319,7 b' class RepoChangelogView(RepoAppView):'
             except (RepositoryError, CommitDoesNotExistError, Exception) as e:
                 log.exception(safe_str(e))
                 raise HTTPFound(
-                    h.route_path('repo_changelog', repo_name=self.db_repo_name))
+                    h.route_path('repo_commits', repo_name=self.db_repo_name))
 
             collection = base_commit.get_path_history(
                 f_path, limit=hist_limit, pre_load=pre_load)

rhodecode/lib/helpers.py

@@ -2061,7 +2061,8 b' def reviewer_as_json(*args, **kwargs):'
 def get_repo_view_type(request):
     route_name = request.matched_route.name
     route_to_view_type = {
-        'repo_changelog': 'changelog',
+        'repo_changelog': 'commits',
+        'repo_commits': 'commits',
         'repo_files': 'files',
         'repo_summary': 'summary',
         'repo_commit': 'commit'

rhodecode/lib/vcs/backends/hg/repository.py

@@ -478,11 +478,11 b' class MercurialRepository(BaseRepository'
           ``end`` could not be found.
         """
         # actually we should check now if it's not an empty repo
-        branch_ancestors = False
         if self.is_empty():
             raise EmptyRepositoryError("There are no commits yet")
         self._validate_branch_name(branch_name)
 
+        branch_ancestors = False
         if start_id is not None:
             self._validate_commit_id(start_id)
             c_start = self.get_commit(commit_id=start_id)

rhodecode/lib/vcs/backends/svn/repository.py

@@ -277,7 +277,7 b' class SubversionRepository(base.BaseRepo'
             try:
                 commit_id = self.commit_ids[commit_idx]
             except IndexError:
-                raise CommitDoesNotExistError
+                raise CommitDoesNotExistError('No commit with idx: {}'.format(commit_idx))
 
         commit_id = self._sanitize_commit_id(commit_id)
         commit = SubversionCommit(repository=self, commit_id=commit_id)

rhodecode/public/js/rhodecode/base/keyboard-bindings.js

@@ -98,7 +98,7 b' function setRCMouseBindings(repoName, re'
         });
         Mousetrap.bind(['g c'], function(e) {
             window.location = pyroutes.url(
-                'repo_changelog', {'repo_name': repoName});
+                'repo_commits', {'repo_name': repoName});
         });
         Mousetrap.bind(['g F'], function(e) {
             window.location = pyroutes.url(

rhodecode/public/js/rhodecode/routes.js

@@ -196,14 +196,17 b' function registerRCRoutes() {'
     pyroutes.register('repo_files_edit_file', '/%(repo_name)s/edit_file/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
     pyroutes.register('repo_files_update_file', '/%(repo_name)s/update_file/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
     pyroutes.register('repo_files_add_file', '/%(repo_name)s/add_file/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
+    pyroutes.register('repo_files_upload_file', '/%(repo_name)s/upload_file/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
     pyroutes.register('repo_files_create_file', '/%(repo_name)s/create_file/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
     pyroutes.register('repo_refs_data', '/%(repo_name)s/refs-data', ['repo_name']);
     pyroutes.register('repo_refs_changelog_data', '/%(repo_name)s/refs-data-changelog', ['repo_name']);
     pyroutes.register('repo_stats', '/%(repo_name)s/repo_stats/%(commit_id)s', ['repo_name', 'commit_id']);
+    pyroutes.register('repo_commits', '/%(repo_name)s/commits', ['repo_name']);
+    pyroutes.register('repo_commits_file', '/%(repo_name)s/commits/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
+    pyroutes.register('repo_commits_elements', '/%(repo_name)s/commits_elements', ['repo_name']);
+    pyroutes.register('repo_commits_elements_file', '/%(repo_name)s/commits_elements/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
     pyroutes.register('repo_changelog', '/%(repo_name)s/changelog', ['repo_name']);
     pyroutes.register('repo_changelog_file', '/%(repo_name)s/changelog/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
-    pyroutes.register('repo_changelog_elements', '/%(repo_name)s/changelog_elements', ['repo_name']);
-    pyroutes.register('repo_changelog_elements_file', '/%(repo_name)s/changelog_elements/%(commit_id)s/%(f_path)s', ['repo_name', 'commit_id', 'f_path']);
     pyroutes.register('repo_compare_select', '/%(repo_name)s/compare', ['repo_name']);
     pyroutes.register('repo_compare', '/%(repo_name)s/compare/%(source_ref_type)s@%(source_ref)s...%(target_ref_type)s@%(target_ref)s', ['repo_name', 'source_ref_type', 'source_ref', 'target_ref_type', 'target_ref']);
     pyroutes.register('tags_home', '/%(repo_name)s/tags', ['repo_name']);

rhodecode/public/js/src/rhodecode/changelog.js

@@ -112,10 +112,10 b' var CommitsController = function () {'
         }
 
         if (urlData['commit_id'] && urlData['f_path']) {
-            return pyroutes.url('repo_changelog_elements_file', urlData);
+            return pyroutes.url('repo_commits_elements_file', urlData);
         }
         else {
-            return pyroutes.url('repo_changelog_elements', urlData);
+            return pyroutes.url('repo_commits_elements', urlData);
         }
 
     };

rhodecode/templates/base/base.mako

@@ -283,7 +283,7 b''
 
       <ul id="context-pages" class="navigation horizontal-list">
         <li class="${is_active('summary')}"><a class="menulink" href="${h.route_path('repo_summary', repo_name=c.repo_name)}"><div class="menulabel">${_('Summary')}</div></a></li>
-        <li class="${is_active('changelog')}"><a class="menulink" href="${h.route_path('repo_changelog', repo_name=c.repo_name)}"><div class="menulabel">${_('Changelog')}</div></a></li>
+        <li class="${is_active('commits')}"><a class="menulink" href="${h.route_path('repo_commits', repo_name=c.repo_name)}"><div class="menulabel">${_('Commits')}</div></a></li>
         <li class="${is_active('files')}"><a class="menulink" href="${h.route_path('repo_files', repo_name=c.repo_name, commit_id=c.rhodecode_db_repo.landing_rev[1], f_path='')}"><div class="menulabel">${_('Files')}</div></a></li>
         <li class="${is_active('compare')}"><a class="menulink" href="${h.route_path('repo_compare_select',repo_name=c.repo_name)}"><div class="menulabel">${_('Compare')}</div></a></li>
 

rhodecode/templates/changeset/changeset.mako

@@ -15,7 +15,7 b''
 </%def>
 
 <%def name="menu_bar_subnav()">
-    ${self.repo_menu(active='changelog')}
+    ${self.repo_menu(active='commits')}
 </%def>
 
 <%def name="main()">

rhodecode/templates/changeset/changeset_range.mako

@@ -25,7 +25,7 b''
 </%def>
 
 <%def name="menu_bar_subnav()">
-    ${self.repo_menu(active='changelog')}
+    ${self.repo_menu(active='commits')}
 </%def>
 
 <%def name="main()">

rhodecode/templates/commits/changelog.mako

@@ -23,7 +23,7 b''
 </%def>
 
 <%def name="menu_bar_subnav()">
-    ${self.repo_menu(active='changelog')}
+    ${self.repo_menu(active='commits')}
 </%def>
 
 <%def name="main()">
@@ -241,7 +241,7 b''
 
             $("#clear_filter").on("click", function() {
                 var filter = {'repo_name': '${c.repo_name}'};
-                window.location = pyroutes.url('repo_changelog', filter);
+                window.location = pyroutes.url('repo_commits', filter);
             });
 
             $("#branch_filter").select2({
@@ -295,7 +295,7 b''
                 else if (data.type == 'book'){
                     filter["bookmark"] = selected;
                 }
-                window.location = pyroutes.url('repo_changelog', filter);
+                window.location = pyroutes.url('repo_commits', filter);
             });
 
             commitsController = new CommitsController();

rhodecode/templates/commits/changelog_elements.mako

@@ -102,7 +102,7 b''
         ## branch
         %if commit.branch:
           <span class="tag branchtag" title="${h.tooltip(_('Branch %s') % commit.branch)}">
-             <a href="${h.route_path('repo_changelog',repo_name=c.repo_name,_query=dict(branch=commit.branch))}"><i class="icon-code-fork"></i>${h.shorter(commit.branch)}</a>
+             <a href="${h.route_path('repo_commits',repo_name=c.repo_name,_query=dict(branch=commit.branch))}"><i class="icon-code-fork"></i>${h.shorter(commit.branch)}</a>
           </span>
         %endif
 

rhodecode/templates/commits/changelog_file_history.mako

@@ -42,7 +42,7 b''
     %endfor
         <tr>
             <td colspan="6">
-                <a id="file_history_overview_full" href="${h.route_path('repo_changelog_file',repo_name=c.repo_name, commit_id=c.commit_id, f_path=c.f_path)}">
+                <a id="file_history_overview_full" href="${h.route_path('repo_commits_file',repo_name=c.repo_name, commit_id=c.commit_id, f_path=c.f_path)}">
                ${_('Show Full History')}
                 </a>
             </td>

rhodecode/templates/data_table/_dt_elements.mako

@@ -48,8 +48,8 b''
          </a>
       </li>
       <li>
-         <a title="${_('Changelog')}" href="${h.route_path('repo_changelog',repo_name=repo_name)}">
-         <span>${_('Changelog')}</span>
+         <a title="${_('Commits')}" href="${h.route_path('repo_commits',repo_name=repo_name)}">
+         <span>${_('Commits')}</span>
          </a>
       </li>
       <li>

rhodecode/templates/debug_style/code-block.html

@@ -641,7 +641,7 b' File Edit'
                 <span class="item">1.2 KiB</span>
                 <span class="item last">text/x-python</span>
                 <div class="buttons">
-                  <a class="btn btn-mini" href="/example/changelog/80ead1899f50a894889e19ffeb49c9cebf5bf045/rhodecode/websetup.py">
+                  <a class="btn btn-mini" href="/example/commits/80ead1899f50a894889e19ffeb49c9cebf5bf045/rhodecode/websetup.py">
                       <i class="icon-time"></i> history
                   </a>
 

rhodecode/templates/files/files.mako

@@ -182,7 +182,7 b''
                 if (path.indexOf("#") >= 0) {
                     path = path.slice(0, path.indexOf("#"));
                 }
-                var url = pyroutes.url('repo_changelog_file',
+                var url = pyroutes.url('repo_commits_file',
                         {'repo_name': templateContext.repo_name,
                          'commit_id': state.commit_id, 'f_path': path, 'limit': 6});
                 $('#file_history_container').show();

rhodecode/templates/files/files_edit.mako

@@ -48,7 +48,7 b''
                 <span class="item">${h.format_byte_size_binary(c.file.size)}</span>
                 <span class="item last">${c.file.mimetype}</span>
                 <div class="buttons">
-                  <a class="btn btn-mini" href="${h.route_path('repo_changelog_file',repo_name=c.repo_name, commit_id=c.commit.raw_id, f_path=c.f_path)}">
+                  <a class="btn btn-mini" href="${h.route_path('repo_commits_file',repo_name=c.repo_name, commit_id=c.commit.raw_id, f_path=c.f_path)}">
                       <i class="icon-time"></i> ${_('history')}
                   </a>
 

rhodecode/templates/pullrequests/pullrequest_show.mako

@@ -72,7 +72,7 b''
                     ## branch link is only valid if it is a branch
                     <span class="tag">
                       %if c.pull_request.source_ref_parts.type == 'branch':
-                        <a href="${h.route_path('repo_changelog', repo_name=c.pull_request.source_repo.repo_name, _query=dict(branch=c.pull_request.source_ref_parts.name))}">${c.pull_request.source_ref_parts.type}: ${c.pull_request.source_ref_parts.name}</a>
+                        <a href="${h.route_path('repo_commits', repo_name=c.pull_request.source_repo.repo_name, _query=dict(branch=c.pull_request.source_ref_parts.name))}">${c.pull_request.source_ref_parts.type}: ${c.pull_request.source_ref_parts.name}</a>
                       %else:
                         ${c.pull_request.source_ref_parts.type}: ${c.pull_request.source_ref_parts.name}
                       %endif
@@ -108,7 +108,7 b''
                     ## branch link is only valid if it is a branch
                     <span class="tag">
                       %if c.pull_request.target_ref_parts.type == 'branch':
-                        <a href="${h.route_path('repo_changelog', repo_name=c.pull_request.target_repo.repo_name, _query=dict(branch=c.pull_request.target_ref_parts.name))}">${c.pull_request.target_ref_parts.type}: ${c.pull_request.target_ref_parts.name}</a>
+                        <a href="${h.route_path('repo_commits', repo_name=c.pull_request.target_repo.repo_name, _query=dict(branch=c.pull_request.target_ref_parts.name))}">${c.pull_request.target_ref_parts.type}: ${c.pull_request.target_ref_parts.name}</a>
                       %else:
                         ${c.pull_request.target_ref_parts.type}: ${c.pull_request.target_ref_parts.name}
                       %endif

rhodecode/templates/search/search_content.mako

@@ -82,7 +82,7 b''
 
             <div class="pull-right">
               <div class="buttons">
-                <a id="file_history_overview_full" href="${h.route_path('repo_changelog_file',repo_name=entry.get('repository',''),commit_id=entry.get('commit_id', 'tip'),f_path=entry.get('f_path',''))}">
+                <a id="file_history_overview_full" href="${h.route_path('repo_commits_file',repo_name=entry.get('repository',''),commit_id=entry.get('commit_id', 'tip'),f_path=entry.get('f_path',''))}">
                    ${_('Show Full History')}
                 </a>
                  | ${h.link_to(_('Annotation'), h.route_path('repo_files:annotated', repo_name=entry.get('repository',''),commit_id=entry.get('commit_id', 'tip'),f_path=entry.get('f_path','')))}

rhodecode/templates/summary/components.mako

@@ -119,7 +119,7 b''
                             <span>0</span> ${_('Commits')}
                         % endif
                   % else:
-                      <a href="${h.route_path('repo_changelog', repo_name=c.repo_name)}">
+                      <a href="${h.route_path('repo_commits', repo_name=c.repo_name)}">
                         <i class="icon-tag"></i>
                         % if commit_rev == 1:
                             <span>${commit_rev}</span> ${_('Commit')}

rhodecode/templates/summary/summary_commits.mako

@@ -76,7 +76,7 b''
             ## branch
             %if cs.branch:
              <span class="branchtag tag" title="${h.tooltip(_('Branch %s') % cs.branch)}">
-              <a href="${h.route_path('repo_changelog',repo_name=c.repo_name,_query=dict(branch=cs.branch))}"><i class="icon-code-fork"></i>${h.shorter(cs.branch)}</a>
+              <a href="${h.route_path('repo_commits',repo_name=c.repo_name,_query=dict(branch=cs.branch))}"><i class="icon-code-fork"></i>${h.shorter(cs.branch)}</a>
              </span>
             %endif
           </div>

rhodecode/tests/vcs/test_commits.py

@@ -35,7 +35,7 b' from rhodecode.tests import get_new_dir'
 from rhodecode.tests.vcs.conftest import BackendTestMixin
 
 
-class TestBaseChangeset:
+class TestBaseChangeset(object):
 
     def test_is_deprecated(self):
         from rhodecode.lib.vcs.backends.base import BaseChangeset
@@ -176,6 +176,10 b' class TestCommitsInNonEmptyRepo(BackendT'
         commit_indexes = [c.idx for c in commits]
         assert commit_indexes == [1, 2, 3, 7, 12, 15]
 
+    def test_get_commit_by_index(self):
+        for idx in [1, 2, 3, 4]:
+            assert idx == self.repo.get_commit(commit_idx=idx).idx
+
     def test_get_commit_by_branch(self):
         for branch, commit_id in self.repo.branches.iteritems():
             assert commit_id == self.repo.get_commit(branch).raw_id