Merge lp:~mvo/software-center/fix-lp870847-5.2 into lp:software-center/5.2

[Michael Vogt]

This branch fixes the problem that its impossible to modify reviews
currently AFAICT.

This branch merges cleanly in both 5.2 and trunk, see also lp:~mvo/rnr-server/rnrclient-fix-lp870847 for diff targeted to the "upstream" branch.

[Gary Lasker (gary-lasker)]

Nice, simple fix! Thanks mvo.

I will merge to both branches.

review: Approve

[Gary Lasker (gary-lasker)]

Never mind, I just noticed that it's already in trunk and has been released!

Merging to the 5.2 branch.

=== modified file 'softwarecenter/backend/piston/rnrclient_pristine.py'
--- softwarecenter/backend/piston/rnrclient_pristine.py	2012-03-16 20:12:57 +0000
+++ softwarecenter/backend/piston/rnrclient_pristine.py	2012-08-17 09:36:20 +0000
@@ -180,7 +180,7 @@
     @validate('review_id', int)
     @validate('rating', int)
     @validate_pattern('summary', r'[^\n]+')
-    @validate_pattern('review_text', r'[^\n]+')
+    @validate('review_text', str)
     @returns(ReviewDetails)
     def modify_review(self, review_id, rating, summary, review_text):
         """Modify an existing review"""