Fix diff submission for p4

Review Request #7439 — Created June 19, 2015 and submitted

Information

Review Board
release-2.0.x
2f53fc3...

Reviewers

Not all SCMTools (namely Perforce) will have new revision information
for files stored in File.newInfo. In Perforce, this is actually a
timestamp so attempting to parse it as a revision will fail.

Ran unit tests.

Able to submit a P4 diff.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/diffviewer/managers.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/diffviewer/managers.py
    
    
  2. 
      
brennie
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/diffviewer/managers.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        reviewboard/diffviewer/managers.py
    
    
  2. 
      
david
  1. Typo in your summary: "attmpeting"

    What impact will this change have on other features?

    1. This shouldn't have any impact on other features. It reverts the handling for filediff dest detail to the way it was before 8c15f47ab2f29b91e3fc4ef4b62df813543bc750 .

  2. 
      
brennie
brennie
david
  1. Ship it, pending the "attmepting"

  2. 
      
brennie
Review request changed
Status:
Completed
Change Summary:
Pushed to release-2.0.x (cb9a791)