Opened 10 years ago

Closed 8 years ago

Last modified 8 years ago

#21516 closed Cleanup/optimization (fixed)

Update the import path for the FormSet classes and factories in documentation.

Reported by: loic84 Owned by: Bryan Marty
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: yes UI/UX: no

Description

Since #21489, FormSet classes and factories are exposed directly on the django.forms package.

Examples in the docs should be updated to reflect this change at some point. The docs update is delayed for a couple of releases to help people who misuse the dev docs with an earlier version of Django so they don't get confusing import errors.

The original ticket (#21489) has a tentative patch and discussions relevant to this ticket.

Change History (10)

comment:1 by loic84, 10 years ago

Triage Stage: UnreviewedSomeday/Maybe

comment:2 by Tim Graham, 8 years ago

Easy pickings: set
Triage Stage: Someday/MaybeAccepted

This change was made in Django 1.7 and all older versions are now unsupported so it seems like a good time to make the change. See 1c7a83ee8e3da431d9d21dae42da8f1f89973f7c for the functions and classes to update.

comment:3 by Bryan Marty, 8 years ago

Owner: changed from nobody to Bryan Marty
Status: newassigned

comment:4 by Bryan Marty, 8 years ago

This is my first ticket / contribution to the Django project.

I have created a topic branch at:
https://github.com/bxm156/django/tree/ticket_21516

Commit:
https://github.com/bxm156/django/commit/9c117b9bd62ea090f1d8b4d7fb7345ebaaf9e836

Ran make html in docs locally, the build succeeded.

comment:5 by Simon Charette, 8 years ago

Has patch: set

comment:6 by Tim Graham, 8 years ago

Patch needs improvement: set

comment:7 by Bryan Marty, 8 years ago

I have updated my changes to reflect timgraham's feedback.

https://github.com/django/django/pull/5492

comment:8 by Tim Graham <timograham@…>, 8 years ago

Resolution: fixed
Status: assignedclosed

In 455034d4:

Fixed #21516 -- Updated imports paths for some formset functions/classes.

Since refs #21489, FormSet classes and factories are exposed on the
django.forms package.

comment:9 by Tim Graham <timograham@…>, 8 years ago

In 44f177b5:

[1.9.x] Fixed #21516 -- Updated imports paths for some formset functions/classes.

Since refs #21489, FormSet classes and factories are exposed on the
django.forms package.

Backport of 455034d4df048010de4ae0a9a2392b70d1463c61 from master

comment:10 by Tim Graham <timograham@…>, 8 years ago

In 42f25467:

[1.8.x] Fixed #21516 -- Updated imports paths for some formset functions/classes.

Since refs #21489, FormSet classes and factories are exposed on the
django.forms package.

Backport of 455034d4df048010de4ae0a9a2392b70d1463c61 from master

Note: See TracTickets for help on using tickets.
Back to Top