When I commit a file with review id, I found it is committed successfully even if review id is not related to this file（提交时似乎不检查 review id 对应的 review request 是不是真的与提交的文件相匹配，只检查这个 review request 的 ship 状态是不是满足 hook 的要求）

honglianglv / reviewboard-svn-hooks

Automatically exported from code.google.com/p/reviewboard-svn-hooks

0 stars 0 forks source link

What steps will reproduce the problem?
1. create a.txt and b.txt in local repository path（创建 a.txt 和 b.txt 
在本地的 svn 里）
2. svn add these two files（用 svn add 加上这个两个文件）
3. post-review b.txt, and get a review id（用 post-review 命令行工具对 
b.txt 发送一个请求，并且得到一个 review id）
4. ship this request about b.txt on review board（在 review board 
里对这个 review request 点 ship it） 
5. svn commit a.txt（回到 svn 客户端机器，用 svn 提交 a.txt 
文件，但是 review id 写上第 3 步得到的 review 
id）（在实际操作中，不排除误操作的可能，有可能不小心填
错了别的文件的已经 ship 的 review id）

What is the expected output? What do you see instead?
Fail to commit a.txt because review id is related to b.txt.（提交 a.txt 
失败，因为此时这个 review id 是与 b.txt 相关的）
But actually I do commit a.txt 
successfully.（但是实际上能够提交成功）

What version of the product are you using? On what operating system?
reviewboard-svn-hooks-0.2.1-r20.tar.gz.
post-review and svn on Windows 7.
reviewboard-svn-hooks and svn server on centos 6.2
reviewboard on ubuntu 12.04

Please provide any additional information below.
N/A

Original issue reported on code.google.com by qiuying...@gmail.com on 26 Mar 2013 at 3:08

I have tried a case. Update a diff patch about multi files to review board. This is supported by review board. So there is no way to let the review id related to single file. 我今天试了一下，了解到 reviewboard 是支持一次上传包含多个文件的 diff patch 的。这样确实是一个矛盾：我一次上传两个文件的 patch，然后 ship 了，这时，我 commit 一个文件用了 review id，commit 另一个文件就用不了该 id 了，这导致我必须同时 commit 两个文件才行，否则就会有其中一个文件 commit 不了的情况发生。但在实际使用中，要开发者时刻记着哪个 patch 跟哪些文件有关是很过分的要求。