Fix creating CC/VV diffs that have moved files.

Review Request #12893 — Created March 15, 2023 and submitted

Information

RBTools
release-4.x

Reviewers

Some code movement in the ClearCase/VersionVault client ended up breaking
the moved file case, where we tried to access a variable before defining it.
This wasn't immediately caught because my local test cases involved activities
that created a new file and then moved it, all in the same activity (which works
correctly).

Created an activity that moved an existing file. Saw the correct diff instead of
an RBTools crash.

Summary ID
Fix creating CC/VV diffs that have moved files.
Some code movement in the ClearCase/VersionVault client ended up breaking the moved file case, where we tried to access a variable before defining it. This wasn't immediately caught because my local test cases involved activities that created a new file and then moved it, all in the same activity (which works correctly). Testing Done: Created an activity that moved an existing file. Saw the correct diff instead of an RBTools crash.
c2a33d0e9eabc564088e5c2de598192f42f7f066
Description From Last Updated

line too long (84 > 79 characters) Column: 80 Error code: E501

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

david
maubin
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.x (b35001a)