# HG changeset patch # User Aurelien Campeas # Date 1426603600 -3600 # Node ID f738180d499d6a29170cef8b28c35b1e22650aab # Parent c8bb743850c9aa0d2f7712f8bf66369d576f1392 [dbapi] retire repo.user_info which was used by the dbapi Related to #3933480. diff -r c8bb743850c9 -r f738180d499d req.py --- a/req.py Tue Mar 17 15:41:54 2015 +0100 +++ b/req.py Tue Mar 17 15:46:40 2015 +0100 @@ -81,7 +81,6 @@ A special method is needed to ensure the linked user is linked to the connection too. """ - # cnx validity is checked by the call to .user_info rset = self.eid_rset(orig_user.eid, 'CWUser') user_cls = self.vreg['etypes'].etype_class('CWUser') user = user_cls(self, rset, row=0, groups=orig_user.groups, diff -r c8bb743850c9 -r f738180d499d server/repository.py --- a/server/repository.py Tue Mar 17 15:41:54 2015 +0100 +++ b/server/repository.py Tue Mar 17 15:46:40 2015 +0100 @@ -743,15 +743,6 @@ session = self._get_session(sessionid) return session._cnx.call_service(regid, **kwargs) - def user_info(self, sessionid, props=None): - """this method should be used by client to: - * check session id validity - * update user information on each user's request (i.e. groups and - custom properties) - """ - user = self._get_session(sessionid, setcnxset=False).user - return user.eid, user.login, user.groups, user.properties - def undoable_transactions(self, sessionid, ueid=None, txid=None, **actionfilters): """See :class:`cubicweb.dbapi.Connection.undoable_transactions`""" diff -r c8bb743850c9 -r f738180d499d server/test/unittest_repository.py --- a/server/test/unittest_repository.py Tue Mar 17 15:41:54 2015 +0100 +++ b/server/test/unittest_repository.py Tue Mar 17 15:46:40 2015 +0100 @@ -236,13 +236,6 @@ # .properties() return a result set self.assertEqual(self.repo.properties().rql, 'Any K,V WHERE P is CWProperty,P pkey K, P value V, NOT P for_user U') - def test_session_api(self): - repo = self.repo - cnxid = repo.connect(self.admlogin, password=self.admpassword) - self.assertEqual(repo.user_info(cnxid), (6, 'admin', set([u'managers']), {})) - repo.close(cnxid) - self.assertRaises(BadConnectionId, repo.user_info, cnxid) - def test_shared_data_api(self): repo = self.repo cnxid = repo.connect(self.admlogin, password=self.admpassword)