Log message for revision 119486: Report success or failure (when known) of creating a new user with the addzope2user script.
Changed: U Zope/trunk/doc/CHANGES.rst U Zope/trunk/src/Products/Five/browser/tests/pages.txt U Zope/trunk/src/Products/Five/browser/tests/pages_ftest.txt U Zope/trunk/src/Products/Five/browser/tests/resource_ftest.txt U Zope/trunk/src/Products/Five/browser/tests/test_defaultview.py U Zope/trunk/src/Products/Five/browser/tests/test_menu.py U Zope/trunk/src/Products/Five/browser/tests/test_scriptsecurity.py U Zope/trunk/src/Products/Five/component/makesite.txt U Zope/trunk/src/Products/Five/skin/tests/test_standardmacros.py U Zope/trunk/src/Products/Five/viewlet/directives.txt U Zope/trunk/src/Zope2/Startup/zopectl.py U Zope/trunk/src/Zope2/utilities/adduser.py -=- Modified: Zope/trunk/doc/CHANGES.rst =================================================================== --- Zope/trunk/doc/CHANGES.rst 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/doc/CHANGES.rst 2011-01-10 23:55:20 UTC (rev 119486) @@ -23,6 +23,9 @@ Features Added ++++++++++++++ +- Report success or failure (when known) of creating a new user with + the addzope2user script. + - Moved subset id calculation in `OFS.OrderSupport.moveObjectsByDelta` to a new helper method, patch by Tom Gross. Modified: Zope/trunk/src/Products/Five/browser/tests/pages.txt =================================================================== --- Zope/trunk/src/Products/Five/browser/tests/pages.txt 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/browser/tests/pages.txt 2011-01-10 23:55:20 UTC (rev 119486) @@ -17,7 +17,7 @@ wouldn't have all the rights to do traversal etc.: >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) >>> self.login('manager') Now for some actual testing... Modified: Zope/trunk/src/Products/Five/browser/tests/pages_ftest.txt =================================================================== --- Zope/trunk/src/Products/Five/browser/tests/pages_ftest.txt 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/browser/tests/pages_ftest.txt 2011-01-10 23:55:20 UTC (rev 119486) @@ -55,8 +55,8 @@ those; we start by adding two users: >>> uf = self.folder.acl_users - >>> uf._doAddUser('viewer', 'secret', [], []) - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('viewer', 'secret', [], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) >>> protected_view_names = [ ... 'eagle.txt', 'falcon.html', 'owl.html', 'flamingo.html', Modified: Zope/trunk/src/Products/Five/browser/tests/resource_ftest.txt =================================================================== --- Zope/trunk/src/Products/Five/browser/tests/resource_ftest.txt 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/browser/tests/resource_ftest.txt 2011-01-10 23:55:20 UTC (rev 119486) @@ -20,7 +20,7 @@ ... glob.glob('%s/*.css' % _prefix))] >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) Image resource Modified: Zope/trunk/src/Products/Five/browser/tests/test_defaultview.py =================================================================== --- Zope/trunk/src/Products/Five/browser/tests/test_defaultview.py 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/browser/tests/test_defaultview.py 2011-01-10 23:55:20 UTC (rev 119486) @@ -39,7 +39,7 @@ As a last act of preparation, we create a manager login: >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) We can specify another default view with browser:defaultView: Modified: Zope/trunk/src/Products/Five/browser/tests/test_menu.py =================================================================== --- Zope/trunk/src/Products/Five/browser/tests/test_menu.py 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/browser/tests/test_menu.py 2011-01-10 23:55:20 UTC (rev 119486) @@ -87,7 +87,7 @@ Let's create a manager user account and log in. >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) >>> self.login('manager') >>> newInteraction() Modified: Zope/trunk/src/Products/Five/browser/tests/test_scriptsecurity.py =================================================================== --- Zope/trunk/src/Products/Five/browser/tests/test_scriptsecurity.py 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/browser/tests/test_scriptsecurity.py 2011-01-10 23:55:20 UTC (rev 119486) @@ -74,7 +74,7 @@ Now let's create a manager user account and log in: >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) >>> self.login('manager') We can now view them all: @@ -122,7 +122,7 @@ wouldn't have all the rights to do traversal etc.: >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) >>> self.login('manager') >>> protected_view_names = [ Modified: Zope/trunk/src/Products/Five/component/makesite.txt =================================================================== --- Zope/trunk/src/Products/Five/component/makesite.txt 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/component/makesite.txt 2011-01-10 23:55:20 UTC (rev 119486) @@ -26,7 +26,7 @@ ------------- >>> uf = app.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) Create the test browser we'll be using: Modified: Zope/trunk/src/Products/Five/skin/tests/test_standardmacros.py =================================================================== --- Zope/trunk/src/Products/Five/skin/tests/test_standardmacros.py 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/skin/tests/test_standardmacros.py 2011-01-10 23:55:20 UTC (rev 119486) @@ -18,7 +18,7 @@ """Test standard macros >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) >>> self.login('manager') >>> from Products.Five.tests.testing import manage_addFiveTraversableFolder Modified: Zope/trunk/src/Products/Five/viewlet/directives.txt =================================================================== --- Zope/trunk/src/Products/Five/viewlet/directives.txt 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Products/Five/viewlet/directives.txt 2011-01-10 23:55:20 UTC (rev 119486) @@ -379,7 +379,7 @@ manager account: >>> uf = self.folder.acl_users - >>> uf._doAddUser('manager', 'r00t', ['Manager'], []) + >>> _ignored = uf._doAddUser('manager', 'r00t', ['Manager'], []) Finally, we need to setup a traversable folder. Otherwise, Five won't get do its view lookup magic: Modified: Zope/trunk/src/Zope2/Startup/zopectl.py =================================================================== --- Zope/trunk/src/Zope2/Startup/zopectl.py 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Zope2/Startup/zopectl.py 2011-01-10 23:55:20 UTC (rev 119486) @@ -420,9 +420,10 @@ self.options.python , 'import Zope2; ' 'app = Zope2.app(); ' - 'app.acl_users._doAddUser(\'%s\', \'%s\', [\'Manager\'], []); ' + 'result = app.acl_users._doAddUser(\'%s\', \'%s\', [\'Manager\'], []); ' 'import transaction; ' 'transaction.commit(); ' + 'print \'Created user:\', result' ) % (name, password) os.system(cmdline) Modified: Zope/trunk/src/Zope2/utilities/adduser.py =================================================================== --- Zope/trunk/src/Zope2/utilities/adduser.py 2011-01-10 23:53:23 UTC (rev 119485) +++ Zope/trunk/src/Zope2/utilities/adduser.py 2011-01-10 23:55:20 UTC (rev 119486) @@ -33,7 +33,13 @@ finder = ZopeFinder(argv) finder.filter_warnings() app = finder.get_app() - adduser(app, user, pwd) + result = adduser(app, user, pwd) + if result: + print "User %s created." % user + else: + print "Got no result back. User creation may have failed." + print "Maybe the user already exists and nothing is done then." + print "Or the implementation does not give info when it succeeds." if __name__ == '__main__': main() _______________________________________________ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins