AttributeError: 'NoneType' object has no attribute 'get_youngest_rev'

[user]

I am getting this error. I tried [13810]. But it still gives an error.

How to Reproduce

While doing a GET operation on /admin/versioncontrol/repository, Trac issued an internal error.

(please provide additional details here)

Request parameters:

{'cat_id': u'versioncontrol', 'panel_id': u'repository', 'path_info': None}

User agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2357.124 Safari/537.36

System Information

Trac	1.0.5
Babel	0.9.6
Genshi	0.6 (without speedups)
GIT	1.9.5.msysgit.1
mod_wsgi	3.5 (WSGIProcessGroup WSGIApplicationGroup %{GLOBAL})
pysqlite	2.6.3
Python	2.7.6 (default, Nov 10 2013, 19:24:18) [MSC v.1500 32 bit (Intel)]
setuptools	7.0
SQLite	3.7.6.2
Subversion	1.8.4 (r1534716)
jQuery	1.7.2
jQuery UI	1.8.21
jQuery Timepicker	1.0.1

Enabled Plugins

tracaccountmanager

0.3.2

Python Traceback

Traceback (most recent call last):
  File "build\bdist.win32\egg\trac\web\main.py", line 513, in _dispatch_request
    dispatcher.dispatch(req)
  File "build\bdist.win32\egg\trac\web\main.py", line 222, in dispatch
    resp = chosen_handler.process_request(req)
  File "build\bdist.win32\egg\trac\admin\web_ui.py", line 120, in process_request
    path_info)
  File "build\bdist.win32\egg\trac\versioncontrol\admin.py", line 334, in render_admin_panel
    for (reponame, info) in all_repos.iteritems())
  File "build\bdist.win32\egg\trac\versioncontrol\admin.py", line 334, in <genexpr>
    for (reponame, info) in all_repos.iteritems())
  File "build\bdist.win32\egg\trac\versioncontrol\admin.py", line 356, in _extend_info
    youngest_rev = repos.get_youngest_rev()
AttributeError: 'NoneType' object has no attribute 'get_youngest_rev'

[Ryan J Ollos]

Please share your [repositories] section from trac.ini and the output of trac-admin $env repository list.

[anonymous]

@rjollos.

Here is the screenshot for trac-admin $env repository list.

Name              Type  Alias      Directory
-----------------------------------------------------
(default)               (default)
project_lib       git              c:\project\lib
project_user_lib  git              c:\project\userlib

[anonymous]

I removed the empty directory (using repository remove "" which was set to be the default and it fixed this issue.

Before

Name              Type  Alias      Directory
-----------------------------------------------------
(default)               (default)
project_lib       git              c:\project\lib
project_user_lib  git              c:\project\userlib

After applying the fix

trac-admin "C:\Users\test\Bitnami Trac Stack projects\project" repository remove ""

trac-admin "C:\Users\chrisa\Bitnami Trac Stack projects\project" repository list

Name              Type  Alias  Directory
----------------------------------------------------------------------------
(default)                      C:\Users\test\Bitnami Trac Stack repository
project_lib       git          c:\project\lib
project_user_lib  git          c:\project\userlib

Thank you for your help. Just wanted to report this. Please test if the user add an empty repository from Admin page by mistake.

[Jun Omae]

Thanks for the feedback.

It is unable to add alias (default) to (default) repository via repository admin. However, the alias can be added via repository alias command.

Welcome to trac-admin 1.0.5
Interactive Trac administration console.
Copyright (C) 2003-2013 Edgewall Software

Type:  '?' or 'help' for help on commands.

Trac [/dev/shm/t12104]> repository list

Name  Type  Alias  Directory
----------------------------

Trac [/dev/shm/t12104]> repository alias '' ''
Trac [/dev/shm/t12104]> repository list

Name       Type  Alias      Directory
-------------------------------------
(default)        (default)

Trac [/dev/shm/t12104]>
$ sqlite3 /dev/shm/t12104/db/trac.db
SQLite version 3.7.9 2011-11-01 00:52:41
Enter ".help" for instructions
Enter SQL statements terminated with a ";"
sqlite> select * from repository;
1|name|
1|dir|
1|alias|

[Jun Omae]

Proposed changes in [4e0fd898c/jomae.git]. This issue occurs when an alias refer non-existent default repository.

[Ryan J Ollos]

The changes look good. Separate but related issue that we could address in another ticket:

Since the repository admin panel prevents adding an alias for a non-existent repository. We could also check that the repository name exists before allowing an alias to be added by the repository alias command.

[Jun Omae]

The changes look good.

Thanks for the reviewing.

Separate but related issue that we could address in another ticket:

Since the repository admin panel prevents adding an alias for a non-existent repository. We could also check that the repository name exists before allowing an alias to be added by the repository alias command.

Added the check for existence of target repository in repository alias in [81d0c6ae6/jomae.git].

Related issue is that referenced repository from repository aliases can be removed via admin panel and repository remove command. We should prevent removing a repository referenced from repository aliases.

[Jun Omae]

I noticed another related issue. The repository alias command allow to add an alias to an alias. The alias to an alias to a repository doesn't work. We should also prevent adding such an alias.

[Jun Omae]

Additional changes in jomae.git@t12104:

• [306897567/jomae.git] avoid creating an alias to an alias in repository alias command
• [0f7b70bde/jomae.git] avoid removing a repository used in aliases
• [f543f2eb8/jomae.git] avoid renaming a repository used in aliases

[Jun Omae]

Fixed in [14142:14143] and merged to trunk in [14144].

The issue for invalid repository aliases is filed in #12118 for next stable milestone. Fixes for the issue is needed to update message catalogs.