diff -r 1f6f713c526c MoinMoin/packages.py
--- a/MoinMoin/packages.py	Sat Sep 13 15:46:51 2008 +0200
+++ b/MoinMoin/packages.py	Sat Sep 13 17:19:06 2008 +0200
@@ -292,23 +292,24 @@
         @param comment:  comment related to this revision (optional)
         @param trivial:  boolean, if it is a trivial edit
         """
+        _ = self.request.getText
+        trivial = str2boolean(trivial)
+        uid = user.getUserId(self.request, author)
+        theuser = user.User(self.request, uid)
+        save_user = self.request.user
+        self.request.user = theuser
         if self.request.user.may.write(pagename):
-            _ = self.request.getText
-            trivial = str2boolean(trivial)
-            uid = user.getUserId(self.request, author)
-            theuser = user.User(self.request, uid)
-            save_user = self.request.user
-            self.request.user = theuser
             page = PageEditor(self.request, pagename, do_editor_backup=0, uid_override=author)
             try:
                 page.saveText(self.extract_file(filename).decode("utf-8"), 0, trivial=trivial, comment=comment)
-                self.msg += u"%(pagename)s added \n" % {"pagename": pagename}
             except PageEditor.Unchanged:
                 pass
-            self.request.user = save_user
-            page.clean_acl_cache()
+            else:
+                self.msg += u"%(pagename)s added \n" % {"pagename": pagename}
+                page.clean_acl_cache()
         else:
-            self.msg += u"action add revision: not enough rights - nothing done \n"
+            self.msg += u"action add revision: not enough rights for %(author)s - nothing done \n" % {"author": author}
+        self.request.user = save_user
 
     def do_renamepage(self, pagename, newpagename, author=u"Scripting Subsystem", comment=u"Renamed by the scripting subsystem."):
         """ Renames a page.