Warnings caused by Condensed Diff upon db migration due to utf string comparison resolved
Review Request #6949 — Created Feb. 14, 2015 and submitted
Modified condensed diff in diffviewer to not give out warnings. Warnings were originally given out in the condensed diff function because comparisons of strings were occurring with utf strings rather than the pure bytestring.
Since the original problem occurred during migrations of the db server. I tried running a syncdb, and starting the development server. The warning disappeared after using the fix was made.
-
This should list a bug number, and the summary/description/testing done should follow the guidelines at https://www.reviewboard.org/docs/codebase/dev/writing-good-descriptions/
- Change Summary:
-
Updated change Description and Testing to adhere to style guidelines
- Description:
-
~ Used comparison of bytestring rather than UTF string
~ Easy Fix Bug: 3698. Modified condensed diff in diffviewer to not give out warnings. Warnings were originally given out in the condensed diff function because comparisons of strings were happening with utf strings rather than the pure bytestring.
- Testing Done:
-
~ Manual Testing of Development Server.
~ Since the original problem occurred during migrations of the db server. I tried running a syncdb, and starting the development server. The warning disappeared after using the fix was made.
-
Almost there. Instead of listing the bug in the description, please put it in the "Bugs" field (on the right-hand side). The summary could also be more specific.
- Change Summary:
-
Updated summary, and moved bug description to appropriate field.
- Summary:
-
Changed comparison of bytestring rather than UTF stringWarnings caused by Condensed Diff upon db migration due to utf string comparison resolved
- Description:
-
~ Easy Fix Bug: 3698. Modified condensed diff in diffviewer to not give out warnings. Warnings were originally given out in the condensed diff function because comparisons of strings were happening with utf strings rather than the pure bytestring.
~ Modified condensed diff in diffviewer to not give out warnings. Warnings were originally given out in the condensed diff function because comparisons of strings were happening with utf strings rather than the pure bytestring.
- Change Summary:
-
Updated summary, and moved bug description to appropriate field.
- Description:
-
~ Modified condensed diff in diffviewer to not give out warnings. Warnings were originally given out in the condensed diff function because comparisons of strings were happening with utf strings rather than the pure bytestring.
~ Modified condensed diff in diffviewer to not give out warnings. Warnings were originally given out in the condensed diff function because comparisons of strings were occurring with utf strings rather than the pure bytestring.
- Change Summary:
-
Used single quotes instead of double quotes
- Branch:
-
release-2.0.xmaster
- Commit:
-
8214eff6ebec09b5e53547aad407a9f47f186c08047b1c152ebcd1db45576600344ae2b05fb37bc0
- Diff:
-
Revision 2 (+3 -2)
-
<p>There seems to be something missing from this patch because commit <code>33581a</code> does not exist in the repository and it looks like you only uploaded one commit.</p>
<p>Make sure you aren't doing <code>rbt post -r 6949 <commit></code>, as this will only upload a single commit for review.</p>