Fixed PEP8 issues with multipoint circuits

This commit is contained in:
Nathan Gotz 2016-09-14 16:06:23 -05:00
parent 5b78bb0738
commit 510bf5ca67
6 changed files with 9 additions and 5 deletions

View File

@ -21,13 +21,14 @@ class CircuitTypeAdmin(admin.ModelAdmin):
@admin.register(Circuit)
class CircuitAdmin(admin.ModelAdmin):
list_display = ['cid', 'provider', 'type', 'tenant','install_date']
list_display = ['cid', 'provider', 'type', 'tenant', 'install_date']
list_filter = ['provider', 'type', 'tenant']
def get_queryset(self, request):
qs = super(CircuitAdmin, self).get_queryset(request)
return qs.select_related('provider', 'type', 'tenant')
@admin.register(Termination)
class TerminationAdmin(admin.ModelAdmin):
list_display = ['tid', 'circuit', 'site', 'port_speed_human',

View File

@ -150,6 +150,7 @@ class CircuitFilterForm(BootstrapMixin, CustomFieldFilterForm):
tenant = forms.MultipleChoiceField(required=False, choices=circuit_tenant_choices,
widget=forms.SelectMultiple(attrs={'size': 8}))
#
# Terminations
#
@ -197,7 +198,7 @@ class TerminationForm(BootstrapMixin, CustomFieldForm):
model = Termination
fields = [
'tid', 'site', 'rack', 'device', 'livesearch',
'interface','port_speed', 'upstream_speed', 'commit_rate',
'interface', 'port_speed', 'upstream_speed', 'commit_rate',
'xconnect_id', 'pp_info', 'comments'
]
help_texts = {

View File

@ -90,6 +90,7 @@ class Circuit(CreatedUpdatedModel):
self.install_date.isoformat() if self.install_date else '',
])
class Termination(CreatedUpdatedModel):
"""
A Termination is where a site

View File

@ -57,8 +57,7 @@ class CircuitTable(BaseTable):
provider = tables.LinkColumn('circuits:provider', args=[Accessor('provider.slug')], verbose_name='Provider')
tenant = tables.LinkColumn('tenancy:tenant', args=[Accessor('tenant.slug')], verbose_name='Tenant')
termination_count = tables.Column(accessor=Accessor('count_terminations'),
verbose_name='Terminations')
verbose_name='Terminations')
class Meta(BaseTable.Meta):
model = Circuit

View File

@ -160,6 +160,7 @@ class CircuitBulkDeleteView(PermissionRequiredMixin, BulkDeleteView):
cls = Circuit
default_redirect_url = 'circuits:circuit_list'
#
# Terminations
#
@ -189,6 +190,7 @@ def termination_add(request, pk):
'cancel_url': reverse('circuits:circuit', kwargs={'pk': circuit.pk}),
})
class TerminationEditView(PermissionRequiredMixin, ObjectEditView):
permission_required = 'circuits.change_circuit'
model = Termination
@ -197,6 +199,7 @@ class TerminationEditView(PermissionRequiredMixin, ObjectEditView):
template_name = 'circuits/termination_edit.html'
cancel_url = 'circuits:circuit_list'
@permission_required('circuits.delete_circuit')
def termination_delete(request, pk):

View File

@ -42,4 +42,3 @@ urlpatterns = [
url(r'^admin/', include(admin.site.urls)),
]