3147: Unable to update repository configurations

scottma*******@gmai***** (Google Code) (Is this you? Claim this profile.)
david
 david
Dec. 9, 2013 
What version are you running?
  Review Board 1.7.18

What's the URL of the page containing the problem?
    https://http://internal.name.company/admin/db/scmtools/repository/6/

What steps will reproduce the problem?
1. Log in to Review Board using any admin account
2. Click on Admin -> repositories -> any repository
3. Click on Save

What is the expected output? What do you see instead?
  I expect the changes to be saved, instead, I see a red highlight
   stating "Please correct the error below." (no other indication of what the problem could be at all that I observe.)

Looking at the other config areas after the error message, I do notice an issue in the Internal State dialog area, indicating "Expecting property name: line 1 column1 (char 1)" and the extra data box contains: "{u'repository_plan': u'', u'bug_tracker_use_hosting': False}" (no quotes, just the content between them).


What operating system are you using? What browser?
centos 2.6.32-279.19.1.el6.x86_64
Chrome, Version 31.0.1650.57 m
IE version 9.0.8112.16421

Please provide any additional information below.
I do see info in the reviewboard.log getting repository information from pysvn, but not much else.
#1 raja****@gmai***** (Google Code) (Is this you? Claim this profile.) 
I posted the same on the reviewboard-dev group list:
https://groups.google.com/forum/#!topic/reviewboard-dev/n2E7skjMWX8

It seems the JSON validate is failing and it probably needs to be skipped because of the data being used in extra_data field.
#2 raja****@gmai***** (Google Code) (Is this you? Claim this profile.) 
if you wanted to update it, you could manually remove the extra_data field when saving and the update goes through fine. This is okay if you dint change anything thats in the extra_data (repository_plan or bug_tracker_user_hosting)
david
#3 david
  • +PendingReview
  • +david
david
#4 david 
Fixed in djblets release-0.7.x (ef48938). We'll get a release out soon.
  • -PendingReview
    +Fixed