Skip to content
Snippets Groups Projects
Commit a8b63bd6 authored by Samuel GAIST's avatar Samuel GAIST Committed by Flavio TARSETTI
Browse files

[libraries][templatetags] Pre-commit cleanup

parent dfc55c3c
No related branches found
No related tags found
2 merge requests!354Cleanup libraries,!342Django 3 migration
......@@ -27,15 +27,15 @@
from django import template
from django.conf import settings
from ...common.texts import Messages as Texts
register = template.Library()
@register.inclusion_tag('libraries/panels/table.html', takes_context=True)
@register.inclusion_tag("libraries/panels/table.html", takes_context=True)
def library_table(context, objects, owner, id):
'''Composes a library list table
"""Composes a library list table
This panel primarily exists for user's library list page.
......@@ -47,18 +47,18 @@ def library_table(context, objects, owner, id):
id: The HTML id to set on the generated table. This is handy for the
filter functionality normally available on list pages.
'''
"""
return {
'request': context['request'],
'objects': objects,
'owner': owner,
'panel_id': id,
"request": context["request"],
"objects": objects,
"owner": owner,
"panel_id": id,
}
@register.inclusion_tag('libraries/panels/actions.html', takes_context=True)
@register.inclusion_tag("libraries/panels/actions.html", takes_context=True)
def library_actions(context, obj, display_count):
'''Composes the action buttons for a particular library
"""Composes the action buttons for a particular library
This panel primarily exists for showing action buttons for a given
library taking into consideration it is being displayed for a given user.
......@@ -70,63 +70,73 @@ def library_actions(context, obj, display_count):
display_count (bool): If the set of buttons should include one with the
number of experiments using this library.
'''
request = context['request']
"""
request = context["request"]
return {
'request': request,
'object': obj,
'display_count': display_count,
'open_source': obj.open_source(request.user),
"request": request,
"object": obj,
"display_count": display_count,
"open_source": obj.open_source(request.user),
}
@register.inclusion_tag('libraries/panels/sharing.html', takes_context=True)
@register.inclusion_tag("libraries/panels/sharing.html", takes_context=True)
def library_sharing(context, obj):
'''Composes the current sharing properties and a form to change them
"""Composes the current sharing properties and a form to change them
Parameters:
obj (Library): The library object concerned for which the
buttons will be drawn.
'''
"""
return {
'request': context['request'],
'object': obj,
'owner': context['request'].user == obj.author,
'users': context['users'],
'teams': context['teams'],
"request": context["request"],
"object": obj,
"owner": context["request"].user == obj.author,
"users": context["users"],
"teams": context["teams"],
}
@register.inclusion_tag('libraries/panels/editor.html', takes_context=True)
@register.inclusion_tag("libraries/panels/editor.html", takes_context=True)
def library_editor(context, obj):
request = context['request']
request = context["request"]
return {
'owner': request.user == obj.author,
'object': obj,
'texts': Texts,
'open_source': obj.open_source(request.user),
"owner": request.user == obj.author,
"object": obj,
"texts": Texts,
"open_source": obj.open_source(request.user),
}
@register.simple_tag(takes_context=True)
def visible_referrer_libraries(context, obj):
'''Calculates the visible usage for a given library and requestor'''
"""Calculates the visible usage for a given library and requestor"""
return obj.referencing.for_user(context['request'].user, True).order_by('-creation_date').distinct()
return (
obj.referencing.for_user(context["request"].user, True)
.order_by("-creation_date")
.distinct()
)
@register.simple_tag(takes_context=True)
def visible_referrer_algorithms(context, obj):
'''Calculates the visible usage for a given library and requestor'''
"""Calculates the visible usage for a given library and requestor"""
return obj.used_by_algorithms.for_user(context['request'].user, True).order_by('-creation_date').distinct()
return (
obj.used_by_algorithms.for_user(context["request"].user, True)
.order_by("-creation_date")
.distinct()
)
@register.simple_tag(takes_context=True)
def visible_referrers_count(context, obj):
'''Calculates the visible usage for a given library and requestor'''
"""Calculates the visible usage for a given library and requestor"""
return visible_referrer_libraries(context, obj).count() + \
visible_referrer_algorithms(context, obj).count()
return (
visible_referrer_libraries(context, obj).count()
+ visible_referrer_algorithms(context, obj).count()
)
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment