Checklist Extension template model

Review Request #6078 — Created July 8, 2014 and discarded

Information

rb-extension-pack

Reviewers

Created a new model to hold templates for the checklist extension. The model contains a name for the list, and a collection of template items. Items can be added, edited and removed from the model.
I followed the same schema as ReviewChecklist model.


 
Description From Last Updated

Col: 1 E302 expected 2 blank lines, found 1

reviewbotreviewbot

Col: 80 E501 line too long (80 > 79 characters)

reviewbotreviewbot

Col: 35 W291 trailing whitespace

reviewbotreviewbot

Col: 1 W293 blank line contains whitespace

reviewbotreviewbot

Col: 1 E302 expected 2 blank lines, found 1

reviewbotreviewbot

Col: 80 E501 line too long (80 > 79 characters)

reviewbotreviewbot

How about just calling this ChecklistTemplate. Adding another List on the end seems superfluous.

daviddavid

This isn't "making the review", it's the user who owns the template.

daviddavid

Do we really need to have unique IDs for items? We're not keeping track of any checked state for them.

daviddavid

Can you please call this items (or perhaps template_items). The name temp_items makes me think that they're temporary items.

daviddavid

Add a trailing comma.

daviddavid

I know it's in the code above, but I'm not sure why we have to convert to str for all …

daviddavid

You don't need the parens around the conditional.

daviddavid

You don't need the parens around the conditional.

daviddavid
SA
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        checklist/checklist/models.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        checklist/checklist/models.py
    
    
  2. checklist/checklist/models.py (Diff revision 2)
     
     
    Col: 1
     E302 expected 2 blank lines, found 1
    
  3. checklist/checklist/models.py (Diff revision 2)
     
     
    Col: 80
     E501 line too long (80 > 79 characters)
    
  4. 
      
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        checklist/checklist/models.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        checklist/checklist/models.py
    
    
  2. checklist/checklist/models.py (Diff revision 1)
     
     
    Col: 1
     E302 expected 2 blank lines, found 1
    
  3. checklist/checklist/models.py (Diff revision 1)
     
     
    Col: 80
     E501 line too long (80 > 79 characters)
    
  4. checklist/checklist/models.py (Diff revision 1)
     
     
    Col: 35
     W291 trailing whitespace
    
  5. checklist/checklist/models.py (Diff revision 1)
     
     
    Col: 1
     W293 blank line contains whitespace
    
  6. 
      
SA
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        checklist/checklist/models.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        checklist/checklist/models.py
    
    
  2. 
      
david
  1. 
      
  2. checklist/checklist/models.py (Diff revision 3)
     
     

    How about just calling this ChecklistTemplate. Adding another List on the end seems superfluous.

  3. checklist/checklist/models.py (Diff revision 3)
     
     

    This isn't "making the review", it's the user who owns the template.

  4. checklist/checklist/models.py (Diff revision 3)
     
     
     
     
     

    Do we really need to have unique IDs for items? We're not keeping track of any checked state for them.

    1. Isn't it neccesery for editing or deleting individual items?

  5. checklist/checklist/models.py (Diff revision 3)
     
     

    Can you please call this items (or perhaps template_items). The name temp_items makes me think that they're temporary items.

  6. checklist/checklist/models.py (Diff revision 3)
     
     

    Add a trailing comma.

  7. checklist/checklist/models.py (Diff revision 3)
     
     
     
     

    I know it's in the code above, but I'm not sure why we have to convert to str for all of this.

  8. checklist/checklist/models.py (Diff revision 3)
     
     

    You don't need the parens around the conditional.

  9. checklist/checklist/models.py (Diff revision 3)
     
     

    You don't need the parens around the conditional.

  10. 
      
SA
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        checklist/checklist/models.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        checklist/checklist/models.py
    
    
  2. 
      
SA
Review request changed

Status: Discarded

Loading...