893: Updating log directory in Admin Settings page fails to overwrite old entry?

keithho*******@gmai***** (Google Code) (Is this you? Claim this profile.)
chipx86
chipx86
Sept. 23, 2009
What's the URL of the page containing the problem?


What steps will reproduce the problem?
1. Set up new reviewboard
2. Incorrectly set log directory to non-existent directory
3. Get error re: log directory (add users etc)
4. Return to logging page to enter correct log info (old log info will be gone)
5. Get error re: log directory (add more users etc)

What is the expected output? What do you see instead?
Expect to see invalid log info and note in red etc about its validity.  It
would be nice if the directory was checked when it was saved.  Also expect
new log directory path to overwrite old info so bug is not seen again.

What operating system are you using? What browser?
Windows Vista, Firefox.  irrelevant.

Please provide any additional information below.
#1 keithho*******@gmai***** (Google Code) (Is this you? Claim this profile.)
It appears I've resolved my problem.
If you enter a non-existent folder and then attempt to change the designated log
folder, the change won't stick.
Now create the missing folder and then change the designated log folder.  It appears
this will get you out of the missing log folder trap?
chipx86
#2 chipx86
  • +Started
  • +Milestone-Release1.1
    +Component-Settings
  • +chipx86
chipx86
#3 chipx86
Fixed on master in r953a4f6.
  • -Started
    +Fixed