Modify ↓

Opened 11 years ago

Last modified 5 years ago

#10782 assigned task

Move data models to model.py

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

Description

To conform to common Trac code conventions, the data model should be contained in model.py: ReviewFilesStruct.py, ReviewerStruct.py, ReviewCommentStruct.py and CodeReviewStruct.py.

Attachments (0)

Change History (12)

comment:1 Changed 10 years ago by Ryan J Ollos

Owner: changed from Olemis Lang to Ryan J Ollos
Status: new → assigned

comment:2 Changed 8 years ago by Cinc-th

In 15165:

Created basic data model for reviews. Use it when viewing review page.

Refs #10782

comment:3 Changed 8 years ago by Cinc-th

In 15166:

Use review data model when displaying My Code Reviews page'. Only one of the tables is converted by now.

Refs #10782

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

comment:4 Changed 8 years ago by Cinc-th

In 15168:

New review model now used for entire My Code Reviews page. Major cleanups.

Refs #10782

comment:5 Changed 8 years ago by Cinc-th

In 15169:

Introduction of new file model and first usage. Allow author notes to have wiki formatting.

Refs #10782

comment:6 Changed 8 years ago by Cinc-th

In 15174:

Only use new file model. Completely removed old file data class.

Refs #10782

comment:7 Changed 8 years ago by Cinc-th

In 15179:

  • Removed more review handling methods from old model.
  • New Review model used everywhere except for search.
  • Some minor cleanups.

Refs #10782

comment:8 Changed 8 years ago by Cinc-th

Owner: changed from Ryan J Ollos to Cinc-th

comment:9 Changed 8 years ago by Cinc-th

In 15188:

Refactored user handling for new reviews.

  • Use jQuery for javascript code
  • Introduced new Reviewer model
  • Cleanups and refactoring

Refs #10782

comment:10 Changed 8 years ago by Cinc-th

In 15191:

New Vote model holding inforamtion about voting for a review.

Refs #10782

comment:11 Changed 8 years ago by Cinc-th

In 15193:

Remove obsolete class ReviewerStruct superseded by model Reviewer.

Refs #10782


Version 1, edited 5 years ago by Ryan J Ollos (previous) (next) (diff)

comment:12 Changed 5 years ago by Cinc-th

In 17433:

PeerReviewPlugin: remove obsolete file after last reference to code was removed with [17432].

Refs #10782

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.