Bug: Lock recovery causes failed CI attempt

In Store.pm, around line 961, there is an errant:

  $tmp = $TWiki::revCiCmd;

This overwrites the already interpolated version of $tmp constructed about 20 lines earlier. This causes an attempt to checkin the file named (literally) %USERNAME% which fails.

Deleting this line removes the bug.

Test case

• Somehow stimulate a lock problem, then attempt to save some changes. Observe, in the web server log the cryptic ci command error.

Environment

-- RobertWithrow - 11 Jan 2002

Fix record

Thanks for pointing out. Is fixed andin TWikiAlphaRelease.

-- PeterThoeny - 16 Feb 2002

This was not correctly fixed.

The "bugfix" was to correct the fact that the ci command further up the code was being overwritten. But in the interim, variable names have changed, and the original meaning of the code lost.

Corrected code is in SVN rev 3284, DEVELOP branch

Note that this fix means that RelockRcsFiles is no longer necessary, and should be removed. Sorry MattWilkie

-- CrawfordCurrie - 23 Nov 2004

Awhhh. So much for my 3 months of TWikiContributor fame (which was mostly due to Crawford and Sven anyway). This sounds like a cleaner solution in any case.

-- MattWilkie - 24 Nov 2004