Opened 4 years ago

Closed 4 years ago

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

Reported by: Owned by: dannyy@… proofek highest PeerReviewPlugin blocker rjollos 0.11

Description

Where is the users read from in this plaugin

rj

comment:2 in reply to: ↑ description Changed 4 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 4 years ago by David Horat

Same problem as me.

comment:4 Changed 4 years ago by esskim@…

Same problem to me, but I solved it as follows

 trac-admin  /path/to/repository
or


I referred to /peerreviewplugin/0.11/codereview/dbBackend.py

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 []
162
163 	        users = []
164 	        for row in rows:
165 	            users.append(row[0])
166 	        return users


comment:5 Changed 4 years ago by rjollos

• Resolution set to fixed
• Status changed from new to closed

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

Change Properties