Opened 7 years ago

Closed 7 years ago

#5758 closed defect (fixed)

When creating a new code review, you cannot assign a user to the review

Reported by: dannyy@… Owned by: Sebastian Marek
Priority: highest Component: PeerReviewPlugin
Severity: blocker Keywords:
Cc: Ryan J Ollos Trac Release: 0.11


Where is the users read from in this plaugin

Attachments (0)

Change History (5)

comment:1 Changed 7 years ago by Ryan J Ollos

Cc: Ryan J Ollos added; anonymous removed


comment:2 in reply to:  description Changed 7 years ago by mohammad.dabiri@…

I have just installed the plugin and am experiencing this issue.

I am running on trac trunk (0.12) hosted in Apache2.2 with mod_python (2.5) on a Windows Server 2003 machine. I am authenticating users using Active Directory (LDAP).

This problem currently makes the plugin unusable as I am not permitted to create a new review without an assigned user.

Any idea when this will be fixed?

comment:3 Changed 7 years ago by David Horat

Same problem as me.

comment:4 Changed 7 years ago by esskim@…

Same problem to me, but I solved it as follows

 trac-admin  /path/to/repository
 permission add target_reviewer CODE_REVIEW_DEV     
 permission add target_reviewer CODE_REVIEW_MGR

I referred to /peerreviewplugin/0.11/codereview/

156 	    def getPossibleUsers(self):
157 	        cursor = self.db.cursor()
158 	        cursor.execute("SELECT DISTINCT p1.username as username FROM permission p1 left join permission p2 on p1.action = p2.username WHERE p1.action = 'CODE_REVIEW_DEV' OR p2.action = 'CODE_REVIEW_DEV' OR p1.action = 'CODE_REVIEW_MGR' OR p2.action = 'CODE_REVIEW_MGR'")
159 	        rows = cursor.fetchall()
160 	        if not rows:
161 	            return []
163 	        users = []
164 	        for row in rows:
165 	            users.append(row[0])
166 	        return users

comment:5 Changed 7 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

I got pretty frustrated trying to install this plugin since I could find any installation steps. I think you found an important detail, so I added an installation section to the main page and included your finding, PeerReviewPlugin#Installation. I might take another crack at installing this, but the CodeReviewPlugin looks interesting as well.

Modify Ticket

as closed The owner will remain Sebastian Marek.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.