Modify ↓

Opened 5 years ago

Last modified 5 years ago

#13583 assigned enhancement

Refactor component PeerReviewView

Reported by: Cinc-th Owned by: Cinc-th
Priority: normal Component: PeerReviewPlugin
Severity: normal Keywords:
Cc: Trac Release:

Description

The code esp. in process_request() is so convoluted, it's difficult to see what's happening there. Testing is hard or impossible, too.

While at it change the template names, see #10731.

Attachments (0)

Change History (2)

comment:1 Changed 5 years ago by Cinc-th

r17444:

PeerReviewPlugin: refactoring of PeerReviewView. Addeed some tests and smashed some bugs while at it.

comment:2 Changed 5 years ago by Cinc-th

In 17445:

PeerReviewPlugin: refactoring of PeerReviewView. Renamed template to peerreview_view.html.

Refs #13583, #10731

Last edited 5 years ago by Ryan J Ollos (previous) (diff)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as assigned The owner will remain Cinc-th.

Add Comment

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

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.