Failed to edit review

Status changed to 'Confirmed' because the bug affects multiple users.

I get the same thing: it fails every time. Both for reviews I wrote a half-year ago and ones I just wrote. I've attempted to edit reviews a few times over the last week so this is not intermittent.

Error details:

"Unknown error communicating with server. Check your log and consider raising a bug report if this error persists".

What logs should I be looking for?

Here's stdout and stderr from running software-center in a terminal, annotated with # to show what I'm doing at each point:

2012-04-28 15:19:00,325 - softwarecenter.ui.gtk3.app - INFO - setting up proxy 'None'
2012-04-28 15:19:00,330 - softwarecenter.db.database - INFO - open() database: path=None use_axi=True use_agent=True
2012-04-28 15:19:19,787 - softwarecenter.db.database - INFO - reopen() database
2012-04-28 15:19:19,787 - softwarecenter.db.database - INFO - open() database: path=None use_axi=True use_agent=True
2012-04-28 15:19:27,410 - softwarecenter.ui.gtk3.gmenusearch - WARNING - could not load GMenu path: Failed to look up menu_file for "settings.menu"

# I click the Edit Review dialog
(software-center:11803): Gtk-WARNING **: Unable to show 'none': Operation not supported
# Edit Review dialog is visible
# Attempting to submit modification. Error is displayed: "Unknown error communicating with server. Check your log and consider raising a bug report if this error persists")
# Closed Edit Review dialog
2012-04-28 15:19:36,251 - softwarecenter.ui.gtk3.app - INFO - software-center-agent finished with status 0
2012-04-28 15:19:36,252 - softwarecenter.db.database - INFO - reopen() database
2012-04-28 15:19:36,252 - softwarecenter.db.database - INFO - open() database: path=None use_axi=True use_agent=True
2012-04-28 15:19:36,375 - softwarecenter.ui.gtk3.gmenusearch - WARNING - could not load GMenu path: Failed to look up menu_file for "settings.menu"

2012-04-28 15:20:57,928 - softwarecenter.backend.spawn_helper - WARNING - exit code 1 from helper for '['/usr/share/software-center/modify_review_gtk3.py', '--parent-xid', '', '--iconname', 'psensor', '--datadir', '/usr/share/software-center/', '--review-id', '43043']'
2012-04-28 15:20:57,928 - softwarecenter.backend.spawn_helper - WARNING - got error from helper: '
(modify_review_gtk3.py:11843): Gtk-WARNING **: Unknown property: GtkGrid.n-rows

(modify_review_gtk3.py:11843): Gdk-CRITICAL **: gdk_window_invalidate_rect_full: assertion `GDK_IS_WINDOW (window)' failed

(modify_review_gtk3.py:11843): Gdk-CRITICAL **: gdk_window_invalidate_rect_full: assertion `GDK_IS_WINDOW (window)' failed
modify_review
Traceback (most recent call last):
  File "/usr/share/software-center/softwarecenter/ui/gtk3/review_gui_helper.py", line 242, in _submit_modify_if_pending
    rating=rating)
  File "/usr/lib/python2.7/dist-packages/piston_mini_client/validators.py", line 65, in wrapper
    return func(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/piston_mini_client/validators.py", line 65, in wrapper
    return func(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/piston_mini_client/validators.py", line 44, in wrapper
    return func(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/piston_mini_client/validators.py", line 40, in wrapper
    pattern))
ValidationException: Argument 'review_text' must match pattern '[^\n]+$'
Argument 'review_text' must match pattern '[^\n]+$'
'

I think Bug #996280 is a duplicate of this one.
Anyone knows how to fix this yet? Didn't know it was this old.

I got the same error (Ubuntu Software Centre 5.2.4 on Ubuntu 12.04).

I got the same Traceback as Dylan in my .xsession-errors log.

My edited review had newlines in it, so I tried removing them and retrying, and it worked. Looks like an incorrect regexp in the code.

Thanks everyone. We have a fix for this planned and I will work on this soon. I appreciate everyone's help and comments, these clarified the issue nicely!

Best,
Gary

While doing a review of the ca-escalated bugs I came accross this one and realized that currently modify_review is not working at all AFAICT for anyone and any review. I bumped the priority therefore.

This bug was fixed in the package software-center - 5.3.7

---------------
software-center (5.3.7) quantal; urgency=low

  [ Michael Vogt ]
  * lp:~mvo/software-center/cleanup-in-config-code:
    - consolidate and simplify config handling for
      all configuration values
  * lp:~mvo/software-center/distro-detection-robustness:
    - tweak the distro detection to make it more robust
      against changes in the case for the plattform string
      and also makes the filenames/classnames more pythonic
  * lp:~mvo/software-center/fix-seperator-crash:
    - remove instance of self.separator_login as we do not
      use this anymore, prevents a crash
  * lp:~mvo/software-center/improve-test-for-config:
    - additional unit tests for the config handling code
  * lp:~mvo/software-center/set-ssl-stuff-in-webkitwidget:
    - move the webkit.py file into 'widgets' and consolidate
      all of the webkit settings there
  * lp:~mvo/software-center/use-lxml:
    - use python-lxml instead of xml.etree as the former is
      quite a bit faster
  * lp:~mvo/software-center/new-pep8 :
    - use pep8 comandline instead of "import pep8" to make it work again
      on quantal where the internal API of pep8 has changed
  * merged 5.2 branch that includes fix for LP: #1026494
  * lp:~mvo/software-center/cache-for-parse-applications-menu:
    - cache the result of parse_applications menu so it is run
      only once for better startup performance
  * lp:~mvo/software-center/lp913966:
    - support the new gnome 3 in the "where is it" feature
      (LP: #913966)
  * lp:~mvo/software-center/add-currency-property:
    - add a "currency" property to the appdetails and use
      it in the UI
  * lp:~mvo/software-center/no-piston-data-means-error:
    - if we don't get data at all from the webservice/cache in
      piston-generic-helper, emit a error signal
  * lp:~mvo/software-center/rec-small-cleanup:
    - small duplication cleanup
  * lp:~mvo/software-center/dep8:
    - enable automatic dep8 based testsuite running
      on the remote server
  * lp:~mvo/software-center/fix-new-pep8:
    - fix all pep8 issues in quantal branch test
  * softwarecenter/db/categories.py, tests/test_categories.py:
    - do not use cached categories in tests
  * lp:~mvo/software-center/lobby-testfixes:
    - fix tests when categories cache is used with multiple DBs
  * lp:~mvo/software-center/fix-lp870847-5.2:
    - fix incorrect validation pattern in modify_review call, this
      should fix the ablity of users to modify their reviews
      (LP: #870847)

  [ Gary Lasker ]
  * lp:~gary-lasker/software-center/fix-purchaseview-test:
   - fix the purchase view unit test to conform to the new config
     code
  * lp:~gary-lasker/software-center/display-cached-recommendations:
   - display cached recommendations in the lobby view in the case of
     no network access
  * lp:~gary-lasker/software-center/rec-tiny-cleanup:
    - additional bit of cleanup in the recommendations UI code
  * lp:~gary-lasker/software-center/fix-new-pep8-precise-fix:
    - small fix to allow trunk to continue to run on Precise
 -- Michael Vogt <email address hidden> Fri, 17 Aug 2012 11:57:20 +0200

This is in the precise-proposed queue now and will get released to precise-proposed when the SRU team had a chance to look at it.

Hello Dereck, or anyone else affected,

Accepted software-center into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/software-center/5.2.6 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

I was able to submit a comment (to cheese) and edit it too.

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

This bug was fixed in the package software-center - 5.2.6

---------------
software-center (5.2.6) precise-proposed; urgency=low

  * lp:~mvo/software-center/fix-lp870847-5.2:
    - fix incorrect validation pattern in modify_review call, this
      should fix the ablity of users to modify their reviews
      (LP: #870847)
  * test/gtk3/test_catview.py:
    - skip some automatic tests are depend on host system state
 -- Michael Vogt <email address hidden> Mon, 27 Aug 2012 12:03:48 +0200