Hi Chris, The extra_data column in diffviewer_filediff table has the info but not for all. for some rows it has "patched_sha1" and for some rows it has info like{"raw_insert_count": 0, "raw_delete_count": 1}. Unfortunately it doesnt helps. anywhere it stores the "patched_sha1" info for each files for the review id ?
Thank You ~John On Thursday, July 7, 2016 at 12:05:56 PM UTC+5:30, Christian Hammond wrote: > > We don't support database access outside of using the API or through our > Python modules in Review Board (as part of our extensions). It's doable, > but it's not something we want to encourage. > > If you absolutely need to, it's stored as serialized JSON data in the > diffviewer_filediff table. You'll need to tie that in with the > diffviewer_diffset table, which is associated with a > diffviewer_diffsethistory, which is associated with a > reviews_reviewrequest. We don't guarantee any form of stability for the > data in these. > > The API is your best choice for this. It'll be more straight-forward and > more future-proof. > > Christian > > -- > Christian Hammond > President/CEO of Beanbag <https://www.beanbaginc.com/> > Makers of Review Board <https://www.reviewboard.org/> > > On Wed, Jul 6, 2016 at 11:25 PM, john levin <smart...@gmail.com > <javascript:>> wrote: > >> Hi Chris, >> >> Thanks ! So far i'm not using the API's (i'm not Familiar with it) and >> communicating directly to the Database. Hence can you please let me know >> where these SHA"s stored in Database (In which Table). >> >> ~John >> >> On Thu, Jul 7, 2016 at 2:42 AM, Christian Hammond <chri...@beanbaginc.com >> <javascript:>> wrote: >> >>> Hi John, >>> >>> Easiest way, if it works for you, might be to just fetch the uploaded >>> diff and compare. If you look at the URL for Download Diff on a review >>> request, you'll see the kind of URL you'd need to compose. Generate a diff >>> locally and compare that to the diff going in. >>> >>> Alternatively, you can compare the SHAs of the files. To do this, you >>> need to loop over all files in the diff on the review request using the >>> API. You can see this resource for info: >>> >>> >>> https://www.reviewboard.org/docs/manual/2.5/webapi/2.0/resources/file-diff-list/ >>> >>> Note in the example payloads the "extra_data.patched_sha1" key. This is >>> the SHA1 of the patched file, the one going into the commit. If you want to >>> compare file contents and not the diff, you can get all these for the >>> latest diff revision on the review request, and compare those. Something to >>> note is that the file contents may be altered at some point in ways that >>> would affect the checksum (newlines normalized, file encodings being >>> applied on our end or the repository end to get contents to match up). So >>> depending on your setup, that may come into play. >>> >>> Hope that helps! >>> >>> Christian >>> >>> -- >>> Christian Hammond >>> President/CEO of Beanbag <https://www.beanbaginc.com/> >>> Makers of Review Board <https://www.reviewboard.org/> >>> >>> On Wed, Jul 6, 2016 at 9:44 AM, john levin <smart...@gmail.com >>> <javascript:>> wrote: >>> >>>> Hi Chris, >>>> >>>> Thanks for your reply ! >>>> >>>> We are validating the list of files in a review id and the files trying >>>> to commit in SVN (using precommit hook). But it is not a proper >>>> way.problem >>>> here is People can use the same review id and can make changes and commit >>>> to svn without updating the review. So we are trying to validate with diff >>>> or (some sort of CRC for that particular review id). >>>> >>>> -- >>>> Supercharge your Review Board with Power Pack: >>>> https://www.reviewboard.org/powerpack/ >>>> Want us to host Review Board for you? Check out RBCommons: >>>> https://rbcommons.com/ >>>> Happy user? Let us know! https://www.reviewboard.org/users/ >>>> --- >>>> 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...@googlegroups.com <javascript:>. >>>> For more options, visit https://groups.google.com/d/optout. >>>> >>> >>> -- >>> Supercharge your Review Board with Power Pack: >>> https://www.reviewboard.org/powerpack/ >>> Want us to host Review Board for you? Check out RBCommons: >>> https://rbcommons.com/ >>> Happy user? Let us know! https://www.reviewboard.org/users/ >>> --- >>> 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...@googlegroups.com <javascript:>. >>> For more options, visit https://groups.google.com/d/optout. >>> >> >> -- >> Supercharge your Review Board with Power Pack: >> https://www.reviewboard.org/powerpack/ >> Want us to host Review Board for you? Check out RBCommons: >> https://rbcommons.com/ >> Happy user? Let us know! https://www.reviewboard.org/users/ >> --- >> 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...@googlegroups.com <javascript:>. >> For more options, visit https://groups.google.com/d/optout. >> > > -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- 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/d/optout.