What version are you running?
Reviewboard 1.0.9.E1
What's the URL of the page containing the problem?
http://reviewboard.in.nds.com/reviews/r/10771/diff/3/#
What steps will reproduce the problem?
Itried 2 diffrent things, 2 different issues :
1.I tried to view diff version 2 or 3
2.I tried to commit a new review
What is the expected output? What do you see instead?
1. in a result, I wasn't able to see the new versions of files
2. after a certain time, nothing happened, so I cancelled my review dans then I see message error (see the end)
What operating system are you using? What browser?
Firefow 6.0.2 under Windows 7 SP1 64bits
Please provide any additional information below.
Warning at /reviews/api/json/reviewrequests/10771/reviews/draft/publish/
Incorrect string value: '\xEF\x81\x8A\x0AUp...' for column 'body_top' at row 1
Request Method: POST
Request URL: http://reviewboard.in.nds.com/reviews/api/json/reviewrequests/10771/reviews/draft/publish/
Django Version: 1.2.3
Exception Type: Warning
Exception Value:
Incorrect string value: '\xEF\x81\x8A\x0AUp...' for column 'body_top' at row 1
Exception Location: /usr/lib64/python2.4/warnings.py in warn_explicit, line 96
Python Executable: /usr/bin/python
Python Version: 2.4.3
Python Path: ['/srv/reviewboard/reviews/conf', '/usr/lib/python2.4/site-packages/setuptools-0.6c12dev_r85190-py2.4.egg', '/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg', '/usr/lib/python2.4/site-packages/ReviewBoard-1.0.9-py2.4.egg', '/usr/lib/python2.4/site-packages/python_memcached-1.45-py2.4.egg', '/usr/lib/python2.4/site-packages/pytz-2010l-py2.4.egg', '/usr/lib/python2.4/site-packages/flup-1.0.3.dev_20100525-py2.4.egg', '/usr/lib/python2.4/site-packages/Pygments-1.3.1-py2.4.egg', '/usr/lib/python2.4/site-packages/django_evolution-0.5-py2.4.egg', '/usr/lib/python2.4/site-packages/PIL-1.1.7-py2.4-linux-x86_64.egg', '/usr/lib/python2.4/site-packages/pydns-2.3.4-py2.4.egg', '/usr/lib/python2.4/site-packages/simplejson-2.1.1-py2.4-linux-x86_64.egg', '/usr/lib/python2.4/site-packages/MySQL_python-1.2.3-py2.4-linux-x86_64.egg', '/usr/lib/python2.4/site-packages/Djblets-0.6.6-py2.4.egg', '/usr/lib64/python24.zip', '/usr/lib64/python2.4', '/usr/lib64/python2.4/plat-linux2', '/usr/lib64/python2.4/lib-tk', '/usr/lib64/python2.4/lib-dynload', '/usr/lib64/python2.4/site-packages', '/usr/lib64/python2.4/site-packages/Numeric', '/usr/lib64/python2.4/site-packages/gtk-2.0', '/usr/lib/python2.4/site-packages']
Server time: Thu, 22 Sep 2011 12:28:23 +0530
Traceback Switch to copy-and-paste view
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/core/handlers/base.py in get_response
response = callback(request, *callback_args, **callback_kwargs) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/views/decorators/cache.py in _wrapped_view_func
response = view_func(request, *args, **kwargs) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Djblets-0.6.6-py2.4.egg/djblets/webapi/decorators.py in _checklogin
response = view_func(*args, **kwargs) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/views/decorators/http.py in inner
return func(request, *args, **kwargs) ...
▶ Local vars
/usr/lib/python2.4/site-packages/ReviewBoard-1.0.9-py2.4.egg/reviewboard/webapi/json.py in review_draft_save
review.publish() ...
▶ Local vars
/usr/lib/python2.4/site-packages/ReviewBoard-1.0.9-py2.4.egg/reviewboard/reviews/models.py in publish
self.save() ...
▶ Local vars
/usr/lib/python2.4/site-packages/ReviewBoard-1.0.9-py2.4.egg/reviewboard/reviews/models.py in save
super(Review, self).save() ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/db/models/base.py in save
self.save_base(using=using, force_insert=force_insert, force_update=force_update) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/db/models/base.py in save_base
rows = manager.using(using).filter(pk=pk_val)._update(values) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/db/models/query.py in _update
return query.get_compiler(self.db).execute_sql(None) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/db/models/sql/compiler.py in execute_sql
cursor = super(SQLUpdateCompiler, self).execute_sql(result_type) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/db/models/sql/compiler.py in execute_sql
cursor.execute(sql, params) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Djblets-0.6.6-py2.4.egg/djblets/log/middleware.py in execute
return self.cursor.execute(sql, params) ...
▶ Local vars
/usr/lib/python2.4/site-packages/Django-1.2.3-py2.4.egg/django/db/backends/mysql/base.py in execute
return self.cursor.execute(query, args) ...
▶ Local vars
/usr/lib/python2.4/site-packages/MySQL_python-1.2.3-py2.4-linux-x86_64.egg/MySQLdb/cursors.py in execute
if not self._defer_warnings: self._warning_check() ...
▶ Local vars
/usr/lib/python2.4/site-packages/MySQL_python-1.2.3-py2.4-linux-x86_64.egg/MySQLdb/cursors.py in _warning_check
warn(w[-1], self.Warning, 3) ...
▶ Local vars
/usr/lib64/python2.4/warnings.py in warn
warn_explicit(message, category, filename, lineno, module, registry) ...
▶ Local vars
/usr/lib64/python2.4/warnings.py in warn_explicit
raise message ...
▶ Local vars
Request information
GET
No GET data
POST
Variable Value
body_top
u'A reviewboard issue prevented me to view the last differences, but Bharath Kumar sent them to me by email (see below).\nwith them, it is ok for me now.\n\n\n\n\n\nHi Yvon\n\nRegarding the changes you asked in review for PeInfoEventList class .\n \n/PeInfoEventList.java-1:java:IMHP#1 (Diff revision 1) \n\n\t\t324\t public int[] searchIndexByWindow(Date start, Date end, boolean includeEndBound) throws IllegalStateException, IllegalArgumentException {\n\nDefect Type\tAlgorithm\nDefect Level\tL2\nYou have to implements this API and let searchIndexByWindow(Date start, Date end) calls searchIndexByWindow(Date start, Date end, boolean includeEndBound) includeEndBound=true.\n\nThat means, do the same as the current version of searchIndexByWindow(Date start, Date end) + add a test to remove last index or not depending of includeEndBound value.\nPlease, ensure also that any browsing using PEL are still functionnal : Menu browsing, Grid browsing, search functions.\n\n\nI have updated in the following manner . could you please tell your comments on this, any change required or it is fine.\n\n/**\n * @see mhepg.platform.service.guide.ProgramEventList#searchIndexByWindow(java.util.Date, java.util.Date)\n */\n public int[] searchIndexByWindow(Date start, Date end) throws IllegalStateException, IllegalArgumentException {\n \n return searchIndexByWindow(start, end, true);\n }\n \n /**\n * @see mhepg.platform.service.guide.ProgramEventList#searchIndexByWindow(java.util.Date, java.util.Date, boolean)\n */\n public int[] searchIndexByWindow(Date start, Date end, boolean includeEndBound) throws IllegalStateException, IllegalArgumentException {\n \n if (start == null || end == null) {\n throw new IllegalArgumentException("any parameter should be null");\n }\n if (end.before(start)) {\n throw new IllegalArgumentException("end date is before start date");\n }\n if (myIterator != null) {\n final int[] nativeResult = myIterator.searchIndexByWindow(start, end);\n if (nativeResult != null && nativeResult.length == 2 && nativeResult[0] != -1 && nativeResult[1] != -1) {\n \n if (!includeEndBound || getProgramEvent(nativeResult[1]).getStartTime().after(end)) {\n nativeResult[1]--;\n }\n return nativeResult;\n }\n }\n return NO_RESULT;\n }\n\n\nPlease let me know your comments on this.\n\n\nThanks\n\nRegards \n\nBharath Kumar R\n\nFrom: Hochart, Yvon \nSent: Monday, September 19, 2011 1:32 PM\nTo: Ramakrishnagowda, Bharathkumar\nSubject: RE: clarification needed on ReviewRequet of searchIndexByWindow(Date start, Date end, boolean includeEndBound)\n\nHi Bharathkumar,\n\nThese changes sound good. \uf04a\nUpdate the review as soon as you finished your test please, and I\u2019ll update it.\n\nRegards,\nYvon Hochart.\n\n\n\n\nDe : Ramakrishnagowda, Bharathkumar \nEnvoy\xe9 : lundi 19 septembre 2011 09:03\n\xc0 : Hochart, Yvon\nObjet : clarification needed on ReviewRequet of searchIndexByWindow(Date start, Date end, boolean includeEndBound)\n\nHi Yvon\n\n I went through the things you suggested in review request for searchIndexByWindow(Date start, Date end, boolean includeEndBound) APi of ProgramEventList. And made some tests for search through xlet and Grid search\u2019s and for Record.\n\nAnd if you want me to test any particular scenario, please suggest me on that.\n\nAnd also in the review request you have mentioned\n\n1. You have to implements this API and let searchIndexByWindow(Date start, Date end) calls searchIndexByWindow(Date start, Date end, boolean includeEndBound) includeEndBound=true.\n\n\nSo for this, I have already added this to searchIndexByWindow API as you suggested me in previous mails\n \npublic int[] searchIndexByWindow(Date start, Date end)\n {\n return searchIndexByWindow(start, end, true);\n }\n\n \n 2. That means, do the same as the current version of searchIndexByWindow(Date start, Date end) + add a test to remove last index or not depending of includeEndBound value.\n \n Defect Type:Algorithm\n \n Defect Level:L2\n\nSo for this, I have added the condition as you suggested in the previous mails.\n \n\n\npublic int[] searchIndexByWindow(Date start, Date end, boolean includeEndBound) throws IllegalStateException {\n \n|\n|\n|\n|\n|\n\n \n\nif (!includeEndBound || getProgramEventImpl(indexes[1]).getStartTime().after(end)) {\n indexes[1]--;\n \n }\n }\n \n\n\n So please suggest me if any changes needed on this.\n\nThanks\n\nRegards\n\nBHARATH KUMAR R'
body_bottom
u''
shipit
u'1'
FILES
No FILES data
COOKIES
Variable Value
collapsediffs
'True'
rbsessionid
'8850affcbdb397bec5f1e87b21d24416'
META
Variable Value
AUTH_TYPE
None
CONTENT_LENGTH
'5503'
CONTENT_TYPE
'application/x-www-form-urlencoded; charset=UTF-8'
GATEWAY_INTERFACE
'CGI/1.1'
HTTP_ACCEPT
'application/json, text/javascript, */*'
HTTP_ACCEPT_CHARSET
'ISO-8859-1,utf-8;q=0.7,*;q=0.7'
HTTP_ACCEPT_LANGUAGE
'fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3'
HTTP_CACHE_CONTROL
'no-cache'
HTTP_CONNECTION
'keep-alive'
HTTP_CONTENT_LENGTH
'5503'
HTTP_CONTENT_TYPE
'application/x-www-form-urlencoded; charset=UTF-8'
HTTP_COOKIE
'rbsessionid=8850affcbdb397bec5f1e87b21d24416; collapsediffs=True'
HTTP_HOST
'reviewboard.in.nds.com'
HTTP_PRAGMA
'no-cache'
HTTP_REFERER
'http://reviewboard.in.nds.com/reviews/r/10771/diff/3/'
HTTP_USER_AGENT
'Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0.2) Gecko/20100101 Firefox/6.0.2'
HTTP_X_REQUESTED_WITH
'XMLHttpRequest'
PATH_INFO
u'/reviews/api/json/reviewrequests/10771/reviews/draft/publish/'
PATH_TRANSLATED
None
QUERY_STRING
None
REMOTE_ADDR
'172.21.204.21'
REMOTE_HOST
None
REMOTE_IDENT
None
REMOTE_USER
None
REQUEST_METHOD
'POST'
SCRIPT_NAME
''
SERVER_NAME
'reviewboard.in.nds.com'
SERVER_PORT
80
SERVER_PROTOCOL
'HTTP/1.1'
SERVER_SOFTWARE
'mod_python'
Settings
Using settings module reviewboard.settings
Setting Value
ABSOLUTE_URL_OVERRIDES
{}
ADMINS
((u'rbadmin', u'rbadmin@nds.com'),)
ADMIN_FOR
()
ADMIN_MEDIA_PREFIX
u'/reviews/media/admin/'
AD_DOMAIN_CONTROLLER
u'indc3.in.nds.com'
AD_DOMAIN_NAME
u'IN.NDS.COM'
AD_FIND_DC_FROM_DNS
False
AD_GROUP_NAME
u''
AD_OU_NAME
u''
AD_RECURSION_DEPTH
None
AD_SEARCH_ROOT
u''
AD_USE_TLS
False
AJAX_SERIAL
1308134368
ALLOWED_INCLUDE_ROOTS
()
APPEND_SLASH
True
AUTHENTICATION_BACKENDS
('reviewboard.accounts.backends.ActiveDirectoryBackend', 'django.contrib.auth.backends.ModelBackend')
AUTH_PROFILE_MODULE
'accounts.Profile'
BANNED_IPS
()
CACHE_BACKEND
u'file:///tmp/reviewboard_cache'
CACHE_EXPIRATION_TIME
2592000
CACHE_MIDDLEWARE_KEY_PREFIX
''
CACHE_MIDDLEWARE_SECONDS
600
COMMENTS_ALLOW_PROFANITIES
False
COMMENTS_BANNED_USERS_GROUP
None
COMMENTS_FIRST_FEW
0
COMMENTS_MODERATORS_GROUP
None
COMMENTS_SKETCHY_USERS_GROUP
None
CSRF_COOKIE_DOMAIN
None
CSRF_COOKIE_NAME
'csrftoken'
CSRF_FAILURE_VIEW
'django.views.csrf.csrf_failure'
DATABASES
{'default': {'ENGINE': 'django.db.backends.mysql', 'HOST': 'localhost', 'NAME': 'reviewboard', 'OPTIONS': {}, 'PASSWORD': '********************', 'PORT': '', 'TEST_CHARSET': None, 'TEST_COLLATION': None, 'TEST_MIRROR': None, 'TEST_NAME': None, 'TIME_ZONE': 'US/Pacific', 'USER': 'rbagent'}}
DATABASE_ENGINE
'mysql'
DATABASE_HOST
'localhost'
DATABASE_NAME
'reviewboard'
DATABASE_OPTIONS
{}
DATABASE_PASSWORD
'********************'
DATABASE_PORT
''
DATABASE_ROUTERS
[]
DATABASE_USER
'rbagent'
DATETIME_FORMAT
u'N j, Y, P'
DATETIME_INPUT_FORMATS
('%Y-%m-%d %H:%M:%S', '%Y-%m-%d %H:%M', '%Y-%m-%d', '%m/%d/%Y %H:%M:%S', '%m/%d/%Y %H:%M', '%m/%d/%Y', '%m/%d/%y %H:%M:%S', '%m/%d/%y %H:%M', '%m/%d/%y')
DATE_FORMAT
u'N j, Y'
DATE_INPUT_FORMATS
('%Y-%m-%d', '%m/%d/%Y', '%m/%d/%y', '%b %d %Y', '%b %d, %Y', '%d %b %Y', '%d %b, %Y', '%B %d %Y', '%B %d, %Y', '%d %B %Y', '%d %B, %Y')
DEBUG
True
DEBUG_PROPAGATE_EXCEPTIONS
False
DECIMAL_SEPARATOR
'.'
DEFAULT_CHARSET
'utf-8'
DEFAULT_CONTENT_TYPE
'text/html'
DEFAULT_FILE_STORAGE
'django.core.files.storage.FileSystemStorage'
DEFAULT_FROM_EMAIL
u'webmaster@localhost'
DEFAULT_INDEX_TABLESPACE
''
DEFAULT_LOGGING_BACKUP_COUNT
30
DEFAULT_LOGGING_SIZE
5242880
DEFAULT_TABLESPACE
''
DISALLOWED_USER_AGENTS
()
DOMAIN_METHOD
u'http'
EMAIL_BACKEND
'django.core.mail.backends.smtp.EmailBackend'
EMAIL_HOST
u'inhc1.in.nds.com'
EMAIL_HOST_PASSWORD
'********************'
EMAIL_HOST_USER
''
EMAIL_PORT
25
EMAIL_SUBJECT_PREFIX
'[Django] '
EMAIL_USE_TLS
False
FILE_CHARSET
'utf-8'
FILE_UPLOAD_HANDLERS
('django.core.files.uploadhandler.MemoryFileUploadHandler', 'django.core.files.uploadhandler.TemporaryFileUploadHandler')
FILE_UPLOAD_MAX_MEMORY_SIZE
2621440
FILE_UPLOAD_PERMISSIONS
None
FILE_UPLOAD_TEMP_DIR
None
FIRST_DAY_OF_WEEK
0
FIXTURE_DIRS
()
FORCE_SCRIPT_NAME
''
FORMAT_MODULE_PATH
None
HTDOCS_ROOT
'/srv/reviewboard/reviews/htdocs'
IGNORABLE_404_ENDS
('mail.pl', 'mailform.pl', 'mail.cgi', 'mailform.cgi', 'favicon.ico', '.php')
IGNORABLE_404_STARTS
('/cgi-bin/', '/_vti_bin', '/_vti_inf')
INSTALLED_APPS
['django.contrib.admin', 'django.contrib.auth', 'django.contrib.contenttypes', 'django.contrib.markup', 'django.contrib.sites', 'django.contrib.sessions', 'djblets.datagrid', 'djblets.feedview', 'djblets.siteconfig', 'djblets.util', 'djblets.webapi', 'reviewboard.accounts', 'reviewboard.admin', 'reviewboard.changedescs', 'reviewboard.diffviewer', 'reviewboard.iphone', 'reviewboard.reports', 'reviewboard.reviews', 'reviewboard.scmtools', 'reviewboard.webapi', 'django_evolution']
INTERNAL_IPS
()
LANGUAGES
(('en', 'English'),)
LANGUAGES_BIDI
('he', 'ar', 'fa')
LANGUAGE_CODE
u'en-us'
LANGUAGE_COOKIE_NAME
'rblanguage'
LDAP_ANON_BIND_PASSWD
u''
LDAP_ANON_BIND_UID
u''
LDAP_BASE_DN
u''
LDAP_EMAIL_ATTRIBUTE
u''
LDAP_EMAIL_DOMAIN
u''
LDAP_TLS
False
LDAP_UID_MASK
u'uid=%s,ou=users,dc=example,dc=com'
LDAP_URI
u''
LOCALE_PATHS
()
LOCAL_ROOT
'/srv/reviewboard/reviews'
LOGGING_ALLOW_PROFILING
False
LOGGING_DIRECTORY
u'/var/log/reviewboard/'
LOGGING_ENABLED
True
LOGGING_NAME
'reviewboard'
LOGIN_REDIRECT_URL
'/accounts/profile/'
LOGIN_URL
'/reviews/account/login/'
LOGOUT_URL
'/accounts/logout/'
MANAGERS
((u'rbadmin', u'rbadmin@nds.com'),)
MEDIA_ROOT
u'/srv/reviewboard/reviews/htdocs/media'
MEDIA_SERIAL
1308134391
MEDIA_SERIAL_DIRS
['admin', 'djblets', 'rb']
MEDIA_URL
u'/reviews/media/'
MESSAGE_STORAGE
'django.contrib.messages.storage.user_messages.LegacyFallbackStorage'
MIDDLEWARE_CLASSES
('django.middleware.gzip.GZipMiddleware', 'django.middleware.common.CommonMiddleware', 'django.middleware.doc.XViewMiddleware', 'django.middleware.http.ConditionalGetMiddleware', 'django.middleware.locale.LocaleMiddleware', 'django.contrib.sessions.middleware.SessionMiddleware', 'django.contrib.auth.middleware.AuthenticationMiddleware', 'djblets.siteconfig.middleware.SettingsMiddleware', 'reviewboard.admin.middleware.LoadSettingsMiddleware', 'djblets.log.middleware.LoggingMiddleware', 'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware')
MONTH_DAY_FORMAT
u'F j'
NIS_EMAIL_DOMAIN
u''
NUMBER_GROUPING
0
PASSWORD_RESET_TIMEOUT_DAYS
'********************'
PREPEND_WWW
False
PROFANITIES_LIST
'********************'
REVIEWBOARD_ROOT
'/usr/lib/python2.4/site-packages/ReviewBoard-1.0.9-py2.4.egg/reviewboard'
ROOT_URLCONF
'djblets.util.rooturl'
SECRET_KEY
'********************'
SEND_BROKEN_LINK_EMAILS
False
SERVER_EMAIL
u'root@localhost'
SESSION_COOKIE_AGE
31536000
SESSION_COOKIE_DOMAIN
None
SESSION_COOKIE_NAME
'rbsessionid'
SESSION_COOKIE_PATH
'/reviews/'
SESSION_COOKIE_SECURE
False
SESSION_ENGINE
'django.contrib.sessions.backends.db'
SESSION_EXPIRE_AT_BROWSER_CLOSE
False
SESSION_FILE_PATH
None
SESSION_SAVE_EVERY_REQUEST
False
SETTINGS_MODULE
'reviewboard.settings'
SHORT_DATETIME_FORMAT
'm/d/Y P'
SHORT_DATE_FORMAT
'm/d/Y'
SITE_ID
1
SITE_ROOT
'/reviews/'
SITE_ROOT_URLCONF
'reviewboard.urls'
TEMPLATE_CONTEXT_PROCESSORS
('django.core.context_processors.auth', 'django.core.context_processors.debug', 'django.core.context_processors.i18n', 'django.core.context_processors.media', 'django.core.context_processors.request', 'djblets.siteconfig.context_processors.siteconfig', 'djblets.util.context_processors.settingsVars', 'djblets.util.context_processors.siteRoot', 'djblets.util.context_processors.ajaxSerial', 'djblets.util.context_processors.mediaSerial', 'reviewboard.admin.context_processors.version')
TEMPLATE_DEBUG
True
TEMPLATE_DIRS
('/usr/lib/python2.4/site-packages/ReviewBoard-1.0.9-py2.4.egg/reviewboard/templates',)
TEMPLATE_LOADERS
('django.template.loaders.filesystem.load_template_source', 'django.template.loaders.app_directories.load_template_source')
TEMPLATE_STRING_IF_INVALID
''
TEST_DATABASE_CHARSET
None
TEST_DATABASE_COLLATION
None
TEST_DATABASE_NAME
None
TEST_RUNNER
'reviewboard.test.runner'
THOUSAND_SEPARATOR
','
TIME_FORMAT
u'P'
TIME_INPUT_FORMATS
('%H:%M:%S', '%H:%M')
TIME_ZONE
'Asia/Kolkata'
TRANSACTIONS_MANAGED
False
URL_VALIDATOR_USER_AGENT
'Django/1.2.3 (http://www.djangoproject.com)'
USE_ETAGS
False
USE_I18N
False
USE_L10N
False
USE_THOUSAND_SEPARATOR
False
WEB_API_ENCODERS
('reviewboard.webapi.json.ReviewBoardAPIEncoder', 'djblets.webapi.core.BasicAPIEncoder')
YEAR_MONTH_FORMAT
u'F Y'
You're seeing this error because you have DEBUG = True in your Django settings file. Change that to False, and Django will display a standard 500 page.
