Summary

Fix UnboundLocalError when posting Perforce paths.

Review Request #5834 — Created May 16, 2014 and submitted May 16, 2014, 2:43 p.m.

Information

Owner

anselina

Repository

RBTools

Branch

master

Bugs

Depends On

Commit

d132fc4...

Reviewers

Groups

rbtools

People

Description

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.

Testing Done


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.

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    rbtools/clients/perforce.py
  Ignored Files:

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    rbtools/clients/perforce.py
  Ignored Files:

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-0.6.x (91666ee)