553: If base diff path is missing for new review, path to diff disappears

dbe****@gmai***** (Google Code) (Is this you? Claim this profile.)
Dec. 28, 2008 
http://demo.review-board.org/r/new/

What steps will reproduce the problem?
1. Repo- Navi, Base path- empty, Diff- path to some diff.
2. Click create review request.

What is the expected output? What do you see instead?
Expect to see error message and still have diff path filled in.
Instead both boxes are empty (and I get the error message).

What operating system are you using? What browser?
Linux, Firefox 3.
#1 dbe****@gmai***** (Google Code) (Is this you? Claim this profile.) 
This is an RFE more than a defect. I don't see how to change the type though.
chipx86
#2 chipx86 
Sadly, Google Code doesn't let users set the type of report (yet).

I'd say it is more of a bug though. We should be able to fix it without too much trouble.

Marking this as a 1.0 fix, but we can push back to 1.5 if we need to.
  • +Milestone-Release1.0
    +Component-Reviews
    +Usability
david
#3 david
  • -Priority-Medium
    +Priority-Low
david
#4 david 
On the web server side, do we have the path on the user's disk? I didn't think so,
which makes this hard to do.
david
#5 david
  • +Confirmed
david
#6 david 
Looking through the documentation, we don't have the full path, only the filename.
This is reasonable, given the security/privacy implications here.

Unfortunately, this means we can't fix this.
  • -Confirmed
    +WontFix