Re: Unable to view diff for multiple files.
Some more info, when this is not working, the page source has the following: var gOpenAnIssue = ; instead of: var gOpenAnIssue = true; On Friday, April 19, 2013 9:37:07 PM UTC-6, Jonathan Wilson wrote: Hi, For some reason I am unable to view the diffs for multiple files on Reviewboard v1.7.6. I can only see the diff for the first file. The rest of the file diffs are stuck loading forever (with the little circular animated loading icon) I've tried with Internet Explorer 8, 9, Google Chrome and firefox and I get the same behavior. I even tried clearing my internet cache but that doesn't seem to help. Does anyone have any idea what is causing this? Thanks, -JW -- 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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
Re: Unable to view diff for multiple files.
Hi, We just ran across this issue on a recent installation ourselves; one single user out of 18 reported this behaviour. While tracking this down, I spotted a seemingly unrelated issue where gOpenAnIssue wasn't being set despite the user's account settings saying that it was. Getting my colleague to reset the Open An Issue setting on his account fixed his ability to view more than one diffed file. This appears to be related to the fact that his account was created manually through the admin pages rather than being auto-created through login. Will need to do some testing of that idea, though. Ken On Tuesday, April 23, 2013 7:16:04 PM UTC-6, Jonathan Wilson wrote: Hi Christian, Sorry for the late reply, I tried view a diff with the Chrome console and no error messages were displayed. I only received the following messages while clicking on the initial review request link: Uncaught SyntaxError: Unexpected token ; /r/593/:40 Uncaught ReferenceError: gReviewRequestId is not defined reviews.min.js:112 Uncaught TypeError: Object #Object has no method 'setFileAttachmentComments' rb-server.myreviewboard.org:415 I have 2 other colleagues who have the same issue, but the majority of my colleagues have no issues viewing multiple diffs. -JW On Monday, April 22, 2013 2:33:58 PM UTC-4, Christian Hammond wrote: On Mon, Apr 22, 2013 at 8:57 AM, Matthew Woehlke mwoehlk...@gmail.comwrote: On 2013-04-20 01:55, Christian Hammond wrote: Hi Jonathan, I don't think I've seen this. Can you open the Chrome console log, try again, and see what errors may appear? Is the server able to generate the diffs? I'm sure Christian knows much more about RB than I do :-), but multiple browser failures still sounds to me more like a server problem. While you're checking your browser console logs, it is probably worth also checking the server logs if there is any message that would indicate a problem generating the diffs. It very well might be the server. I stated on the browser side because generally, if there's a server-side error, we should be showing it. We load the diffs dynamically, so if there's a browser-side script error, we may fail to either load or render (or show the error). Christian -- Christian Hammond - chi...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
Re: Unable to view diff for multiple files.
On 2013-05-28 13:22, Kristin Thomas wrote: Some more info, when this is not working, the page source has the following: var gOpenAnIssue = ; instead of: var gOpenAnIssue = true; Um... this seems familiar. Have you upgraded through a number of RB versions? Is the user having trouble either notably older, newer, or had a long period of inactivity? Something to try: have the user go into their account settings and change the value of their 'always open an issue' option. (They can immediately change it back, just as long as they apply the change with the opposite value to force it to be rewritten.) -- Matthew -- 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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
Re: Unable to view diff for multiple files.
The two issues are related. I have absolutely no idea how the user's profile entry is ending up with that value set to.. null? Or something. Actually, for any user hitting this, I'd love to get an SQL dump of their accounts_profile table entry, if you can find it, before they fix it with the double-toggle workaround. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On May 28, 2013, at 7:29 AM, Kristin Thomas xtintho...@gmail.com wrote: Was there a resolution to this issue? We are experiencing a similar problem. We have 16 users with no problem and one user that cannot load any diffs (in IE, Chrome, or Firefox). Same error as mentioned previously: [08:21:52.611] SyntaxError: syntax error @ http://reviewboard.companyname.com/r/87/diff/#:40 [08:21:52.809] ReferenceError: gReviewRequestId is not defined @ http://reviewboard.companyname.com/static/rb/js/reviews.min.89914309c82b.js:112 [08:21:52.837] TypeError: RB.setFileAttachmentComments is not a function @ http://reviewboard.companyname.com/r/87/diff/#:414 I'll attach the log. Any help would be greatly appreciated. Kristin Thomas On Tuesday, April 23, 2013 7:16:04 PM UTC-6, Jonathan Wilson wrote: Hi Christian, Sorry for the late reply, I tried view a diff with the Chrome console and no error messages were displayed. I only received the following messages while clicking on the initial review request link: Uncaught SyntaxError: Unexpected token ; /r/593/:40 Uncaught ReferenceError: gReviewRequestId is not defined reviews.min.js:112 Uncaught TypeError: Object #Object has no method 'setFileAttachmentComments' rb-server.myreviewboard.org:415 I have 2 other colleagues who have the same issue, but the majority of my colleagues have no issues viewing multiple diffs. -JW On Monday, April 22, 2013 2:33:58 PM UTC-4, Christian Hammond wrote: On Mon, Apr 22, 2013 at 8:57 AM, Matthew Woehlke mwoehlk...@gmail.com wrote: On 2013-04-20 01:55, Christian Hammond wrote: Hi Jonathan, I don't think I've seen this. Can you open the Chrome console log, try again, and see what errors may appear? Is the server able to generate the diffs? I'm sure Christian knows much more about RB than I do :-), but multiple browser failures still sounds to me more like a server problem. While you're checking your browser console logs, it is probably worth also checking the server logs if there is any message that would indicate a problem generating the diffs. It very well might be the server. I stated on the browser side because generally, if there's a server-side error, we should be showing it. We load the diffs dynamically, so if there's a browser-side script error, we may fail to either load or render (or show the error). Christian -- Christian Hammond - chi...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com errors.txt -- 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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
Re: Cannot upload diff/patch file
What diff file are you uploading? -David On Sun, May 26, 2013 at 4:32 AM, Michael Vu micheal...@synergixtaskhub.comwrote: Dear All, I try to create a Review Request on demo site. But when uploading diff file, the UI keeps showing loading... forever, and never complete :(. Anyway to work around? -- 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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out. -- 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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2881 in reviewboard: Unable to post a review in Perforce using file rev range.
Updates: Status: Fixed Comment #5 on issue 2881 by ste...@smacleod.ca: Unable to post a review in Perforce using file rev range. http://code.google.com/p/reviewboard/issues/detail?id=2881 Fixed in master, release-0.5.x. Thanks! -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2881 in reviewboard: Unable to post a review in Perforce using file rev range.
Comment #6 on issue 2881 by ste...@smacleod.ca: Unable to post a review in Perforce using file rev range. http://code.google.com/p/reviewboard/issues/detail?id=2881 Issue 2986 has been merged into this issue. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2986 in reviewboard: Posting paths in Perforce crashes
Updates: Status: Duplicate Mergedinto: 2881 Comment #2 on issue 2986 by ste...@smacleod.ca: Posting paths in Perforce crashes http://code.google.com/p/reviewboard/issues/detail?id=2986 (No comment was entered for this change.) -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues?hl=en. For more options, visit https://groups.google.com/groups/opt_out.