Yeah, though I just committed the fix. Can you make sure it works
correctly on your end?

Christian

--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com



On Thu, Jan 13, 2011 at 12:29 AM, Dan <birb...@gmail.com> wrote:
> Hi Christian,
>
> You mean post a review request to http://reviews.reviewboard.org/?
>
> On Jan 13, 1:33 pm, Christian Hammond <chip...@chipx86.com> wrote:
>> Hi Dan,
>>
>> You're correct, this is a bug, and your fix looks fine. We'll have to
>> get that fix in. Want to post a review request?
>>
>> Christian
>>
>>
>>
>> On Wednesday, January 12, 2011, Dan <birb...@gmail.com> wrote:
>> > Here is a more accurate description:
>>
>> > In ReviewBoardServer.get_repository_info, it calls
>> > ReviewBoardServer.api_get to fetch the repository info from sever,
>> > from my debug, it show the return json is something like (if the
>> > repository is svn)
>>
>> > {
>> >     "stat": "ok",
>> >     "repository": {
>> >         "path": "http://192.168.1.131/svn/Android";,
>> >         "tool": "Subversion",
>> >         "id": 2,
>> >         "links": {
>> >             "info": {
>> >                 "href": "http://192.168.1.131:81/api/repositories/2/
>> > info/",
>> >                 "method": "GET"
>> >             },
>> >             "self": {
>> >                 "href": "http://192.168.1.131:81/api/repositories/
>> > 2/",
>> >                 "method":
>> >                 "GET"
>> >             }
>> >         },
>> >     "name": "local-android"
>> >     }
>> > }
>>
>> > from the data structure of the json, we can see that use "url =
>> > rsp['links']['info']['href']" of course will get a array key
>> > exception.
>> > should use url = rsp['repository']['links']['info']['href'].
>>
>> > Is this a bug?
>>
>> > On Jan 11, 9:24 pm, Dan <birb...@gmail.com> wrote:
>> >> Hi
>>
>> >> Here I have a problem with postreview.py when publish a review
>> >> request, could anyone please have a look?
>>
>> >> My enviroment is Reviewboard 1.5.1 and SVN. I downloaded postreview.py
>> >> fromhttps://github.com/reviewboard/rbtools/blob/184334a7a5ddc24172ae2aa03...,
>> >> and configed the constants necessary.
>>
>> >> When I run postreivew, it throws a array index exception at line 677:
>> >> url = rsp['links']['info']['href'], The whole piece of code is
>>
>> >>     def get_repository_info(self, rid):
>> >>         """
>> >>         Returns detailed information about a specific repository.
>> >>         """
>> >>         if self.deprecated_api:
>> >>             url = 'api/json/repositories/%s/info/' % rid
>> >>         else:
>> >>             rsp = self.api_get(
>> >>                 '%s%s/' % (self.root_resource['links']['repositories']
>> >> ['href'],
>> >>                            rid))
>> >>             url = rsp['links']['info']['href']
>>
>> >>         rsp = self.api_get(url)
>>
>> >>         return rsp['info']
>>
>> >> I added debug info for rep, it did not contain the "links" index
>> >> indeed, but it contained a "repository" index.
>>
>> >> Then I manually went to check the xml from reviewboard server, and
>> >> found the structure of the xml is like this
>> >>     <rsp>
>> >>         <repository>
>> >>             <links>
>> >>                 <info>
>> >>                     <href>
>> >> so I changed line 677 to url = rsp["repository"]['links']['info']
>> >> ['href'], finally it worked.
>>
>> >> Though it worked, I still did not know the reason and whether my
>> >> change is right or wrong. I think it is unlike to be a bug of
>> >> reviewboard, but I am too lazy to read all the code :-) so, could
>> >> anyone kindly tell me that what the real problem is and what should I
>> >> do?
>>
>> >> Any help will be highly appreciated!
>>
>> > --
>> > Want to help the Review Board project? Donate today 
>> > athttp://www.reviewboard.org/donate/
>> > Happy user? Let us know athttp://www.reviewboard.org/users/
>> > -~----------~----~----~----~------~----~------~--~---
>> > To unsubscribe from this group, send email to 
>> > reviewboard+unsubscr...@googlegroups.com
>> > For more options, visit this group 
>> > athttp://groups.google.com/group/reviewboard?hl=en
>>
>> --
>> --
>> Christian Hammond - chip...@chipx86.com
>> Review Board -http://www.reviewboard.org
>> VMware, Inc. -http://www.vmware.com
>
> --
> Want to help the Review Board project? Donate today at 
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~----------~----~----~----~------~----~------~--~---
> To unsubscribe from this group, send email to 
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Reply via email to