diff --git a/src/registrar/admin.py b/src/registrar/admin.py index d1cb287a3..5ca592ae7 100644 --- a/src/registrar/admin.py +++ b/src/registrar/admin.py @@ -13,7 +13,6 @@ from django.contrib.auth.admin import UserAdmin as BaseUserAdmin from django.contrib.auth.models import Group from django.contrib.contenttypes.models import ContentType from django.urls import reverse -from dateutil.relativedelta import relativedelta # type: ignore from epplibwrapper.errors import ErrorCode, RegistryError from registrar.models.user_domain_role import UserDomainRole from waffle.admin import FlagAdmin @@ -2196,24 +2195,8 @@ class DomainAdmin(ListHeaderAdmin, ImportExportModelAdmin): extra_context["state_help_message"] = Domain.State.get_admin_help_text(domain.state) extra_context["domain_state"] = domain.get_state_display() - # Pass in what the an extended expiration date would be for the expiration date modal - self._set_expiration_date_context(domain, extra_context) - return super().changeform_view(request, object_id, form_url, extra_context) - def _set_expiration_date_context(self, domain, extra_context): - """Given a domain, calculate the an extended expiration date - from the current registry expiration date.""" - years_to_extend_by = self._get_calculated_years_for_exp_date(domain) - try: - curr_exp_date = domain.registry_expiration_date - except KeyError: - # No expiration date was found. Return none. - extra_context["extended_expiration_date"] = None - else: - new_date = curr_exp_date + relativedelta(years=years_to_extend_by) - extra_context["extended_expiration_date"] = new_date - def response_change(self, request, obj): # Create dictionary of action functions ACTION_FUNCTIONS = { @@ -2241,11 +2224,9 @@ class DomainAdmin(ListHeaderAdmin, ImportExportModelAdmin): self.message_user(request, "Object is not of type Domain.", messages.ERROR) return None - years = self._get_calculated_years_for_exp_date(obj) - # Renew the domain. try: - obj.renew_domain(length=years) + obj.renew_domain() self.message_user( request, "Successfully extended the expiration date.", @@ -2270,37 +2251,6 @@ class DomainAdmin(ListHeaderAdmin, ImportExportModelAdmin): return HttpResponseRedirect(".") - def _get_calculated_years_for_exp_date(self, obj, extension_period: int = 1): - """Given the current date, an extension period, and a registry_expiration_date - on the domain object, calculate the number of years needed to extend the - current expiration date by the extension period. - """ - # Get the date we want to update to - desired_date = self._get_current_date() + relativedelta(years=extension_period) - - # Grab the current expiration date - try: - exp_date = obj.registry_expiration_date - except KeyError: - # if no expiration date from registry, set it to today - logger.warning("current expiration date not set; setting to today") - exp_date = self._get_current_date() - - # If the expiration date is super old (2020, for example), we need to - # "catch up" to the current year, so we add the difference. - # If both years match, then lets just proceed as normal. - calculated_exp_date = exp_date + relativedelta(years=extension_period) - - year_difference = desired_date.year - exp_date.year - - years = extension_period - if desired_date > calculated_exp_date: - # Max probably isn't needed here (no code flow), but it guards against negative and 0. - # In both of those cases, we just want to extend by the extension_period. - years = max(extension_period, year_difference) - - return years - # Workaround for unit tests, as we cannot mock date directly. # it is immutable. Rather than dealing with a convoluted workaround, # lets wrap this in a function. diff --git a/src/registrar/templates/django/admin/domain_change_form.html b/src/registrar/templates/django/admin/domain_change_form.html index 80888423d..6892822e7 100644 --- a/src/registrar/templates/django/admin/domain_change_form.html +++ b/src/registrar/templates/django/admin/domain_change_form.html @@ -69,16 +69,13 @@
- This will extend the expiration date by one year.
+ This will extend the expiration date by one year from today.
{# Acts as a
#}
Domain: {{ original.name }}
- {# Acts as a
#}
-