Error in Section 2:AttributeError: 'NoneType' object has no attribute 'youngest_rev'

[devexchage@…]

We are currently using 0.12dev version

Preconditions: I do have corresponding Review rights and Plugin is enabled.

Repro steps:

1. select Peer Review
2. Select Create Code Review
3. System is trying to load data in Section: "Step 2: Select the sections to be reviewed." and reports an error:

   Trac detected an internal error:
   
   AttributeError: 'NoneType' object has no attribute 'youngest_rev'
   

[anonymous]

CoderReview patch for 0.12

[anonymous]

Here is a patch for the problem

[anonymous]

that patch did not work for me.

[anonymous]

ok, i retract that last statement, i believe i do not know how to apply patches properly. any information about how to do this would be appreciated.

[anonymous]

Try the following patch. It is not a "nice" patch, as it pulls back some removed template code from Trac. But it can be used as a temporary work-around.

diff -ru 2.2-orig/codereview/peerReviewBrowser.py 2.2-new/codereview/peerReviewBrowser.py
--- 2.2-orig/codereview/peerReviewBrowser.py    2011-04-27 17:04:47.000000000 +0800
+++ 2.2-new/codereview/peerReviewBrowser.py     2011-04-27 17:05:12.000000000 +0800
@@ -87,7 +87,7 @@
         path = req.args.get('path', '/')
         rev = req.args.get('rev')
 
-        repos = self.env.get_repository(req.authname)
+        repos = self.env.get_repository(authname=req.authname)
 
         try:
             node = get_existing_node(self.env, repos, path, rev)
diff -ru 2.2-orig/codereview/peerReviewPerform.py 2.2-new/codereview/peerReviewPerform.py
--- 2.2-orig/codereview/peerReviewPerform.py    2011-04-27 17:04:47.000000000 +0800
+++ 2.2-new/codereview/peerReviewPerform.py     2011-04-27 17:05:13.000000000 +0800
@@ -131,7 +131,7 @@
             return 'error.cs', data, None
 
         #get the respository
-        repos = self.env.get_repository(req.authname)
+        repos = self.env.get_repository(authname=req.authname)
         #get the file attributes
         data['review_path'] = resultFile.Path
         data['review_version'] = resultFile.Version
diff -ru 2.2-orig/codereview/templates/peerReviewBrowser.html 2.2-new/codereview/templates/peerReviewBrowser.html
--- 2.2-orig/codereview/templates/peerReviewBrowser.html        2011-04-27 17:04:47.000000000 +0800
+++ 2.2-new/codereview/templates/peerReviewBrowser.html 2011-04-27 17:05:13.000000000 +0800
@@ -18,6 +18,18 @@
 </py:def>
 
 <div id="content" class="browser">
+  <py:def function="browser_path_links(path_links,rev=None)">
+    <py:for each="idx, part in enumerate(path_links)"><py:with
+        vars="first = idx == 0; last = idx == len(path_links) - 1"><a
+          class="${classes('pathentry', first=first)}"
+          title="${first and _('Go to root directory') or _('View %(folder)s', folder=part.name)}"
+          href="$part.href">$part.name</a><py:if
+        test="not last"><span class="pathentry sep">/</span></py:if></py:with></py:for>
+    <py:if test="rev"><span class="pathentry sep">@</span>
+      <a class="pathentry" href="${href.changeset(rev)}" title="View changeset $rev">$rev</a>
+    </py:if>
+    <br style="clear: both" />
+  </py:def>
        <h2>${browser_path_links(path_links, stickyrev)}</h2>
 
       <div id="jumprev">
@@ -127,4 +139,4 @@
 </div>
 
 </body>
-</html>
\ No newline at end of file
+</html>

[Kevin Fenzi]

Neither one of these seems to work here. ;(

trac 0.12.2

I always get the error. ideas or things to try welcome.

[overflow <fiolorlando@…>]

Change the repository name to empty (so it will the ​default repository).

[Kevin Fenzi]

Doesn't seem to matter here.

[bunk@…]

I can confirm that this plugin doesn't support multiple repositories and as such requires a default repository (I had to restart apache as well). The patches are useless.

[Martin Fiala]

a combination of both patches worked for me - multiple-repo.patch​

[Martin Fiala]

solves repository=None problem in trac 0.12.2

[Ryan J Ollos]

#8800 also has a patch for multirepository support.

[Ryan J Ollos]

In 13498:

Adapts get_repository calls to 0.12 and later API. Refs #7034.

[Ryan J Ollos]

In 13500:

Use path_links.html template from Trac. Refs #7034.

[Ryan J Ollos]

I believe it will work now if a (default) repository is defined. In the future we'll need to add support for multiple repositories.