• 
      

    Fix UnboundLocalError when posting Perforce paths.

    Review Request #5834 — Created May 16, 2014 and submitted

    Information

    RBTools
    master
    d132fc4...

    Reviewers

    Posting Perforce paths using rbt post would sometimes result in an
    UnboundLocalError because the local variable new_path was referenced before
    assignment. This is a trivial fix, changing the argument from new_path to
    old_path.

    • Ran unit tests.
    • Ran rbt post //p4-test2/foo@2,@3 (which previously resulted in the error),
      and saw that the review request was created successfully.
    reviewbot
    1. This is a review from Review Bot.
        Tool: PEP8 Style Checker
        Processed Files:
          rbtools/clients/perforce.py
        Ignored Files:
      
      
    2. 
        
    reviewbot
    1. This is a review from Review Bot.
        Tool: Pyflakes
        Processed Files:
          rbtools/clients/perforce.py
        Ignored Files:
      
      
    2. 
        
    david
    1. Ship It!

    2. 
        
    anselina
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-0.6.x (91666ee)