Implement get_absolute_url() on BaseModel

[jathanism]

Proposed Changes

Base models should always provide get_absolute_url() so that each model doesn’t have to hard-code this as well.

e.g. This should suffice:

class BaseModel(models.Model):
    def get_absolute_url(self):
        return reverse(get_route_for_model(self), args=[self.pk])

Justification

Since we’re aligning around pk for all objects, then this is even easier in that we know the app_label and model_name for the model, so we can automatically have something like above. We could just as easily align around the natural_key_slug (see: #2900) as well (e.g. args=self.natural_key_slug should we so desire).

More importantly, this is another boiler plate reduction. The majority of model objects are using {app_label}.{model} as their detail route name, and pk as the unique identifier in detail routes, so this would eliminate most places where get_absolute_url() is required to be defined in the common case.

[glennmatthews]

Models that currently do not have a detail view at all, and hence no immediately appropriate get_absolute_url implementation:

• RelationshipAssociation
• All ComponentTemplateModel subclasses (RearPortTemplate, etc.)

[bryanculver]

Models that currently do not have a detail view at all, and hence no immediately appropriate get_absolute_url implementation:

• RelationshipAssociation
• All ComponentTemplateModel subclasses (RearPortTemplate, etc.)

Can we implement this in BaseModel and just pass/throw exceptions for these two cases until we have detail views for these (or reverse to a more sane detail view to view these?).

[bryanculver]

Closed with #3632