wcs: reduce querysets on page detail for user (#40675)
This commit is contained in:
parent
3b75d3db28
commit
c5c2a66a60
|
@ -22,10 +22,13 @@ from django.utils.translation import ugettext_lazy as _
|
||||||
|
|
||||||
|
|
||||||
def user_get_name_id(user):
|
def user_get_name_id(user):
|
||||||
saml_identifier = user.saml_identifiers.first()
|
if not hasattr(user, '_name_id'):
|
||||||
if saml_identifier:
|
user._name_id = None
|
||||||
return saml_identifier.name_id
|
saml_identifier = user.saml_identifiers.first()
|
||||||
return None
|
if saml_identifier:
|
||||||
|
user._name_id = saml_identifier.name_id
|
||||||
|
|
||||||
|
return user._name_id
|
||||||
|
|
||||||
|
|
||||||
class AppConfig(django.apps.AppConfig):
|
class AppConfig(django.apps.AppConfig):
|
||||||
|
|
|
@ -1228,7 +1228,7 @@ def test_view_page_with_wcs_cells_num_queries(app, admin_user):
|
||||||
app.get('/') # load once to populate caches
|
app.get('/') # load once to populate caches
|
||||||
with CaptureQueriesContext(connection) as ctx:
|
with CaptureQueriesContext(connection) as ctx:
|
||||||
app.get('/')
|
app.get('/')
|
||||||
assert len(ctx.captured_queries) == 160
|
assert len(ctx.captured_queries) == 61
|
||||||
|
|
||||||
|
|
||||||
@wcs_present
|
@wcs_present
|
||||||
|
|
Loading…
Reference in New Issue