#21090 closed New feature (fixed)

inspectdb shouldn't include 'models.' prefix if field_type is a dotted path

Reported by: manfre Owned by: nobody
Component: Core (Management commands) Version: master
Severity: Normal Keywords: inspectdb
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

To allow backends to better provide custom field types to inspectdb, it should not include the 'models.' prefix if the field_type contains a dot.

This would allow for:

id = sqlserver_ado.fields.BigAutoField()

instead of:

id = models.sqlserver_ado.fields.BigAutoField()

This ticket intentionally does not address the lack of import for sqlserver_ado.fields

Change History (3)

comment:1 Changed 21 months ago by manfre

  • Has patch set
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 21 months ago by timo

  • Easy pickings unset
  • Needs tests set
  • Triage Stage changed from Unreviewed to Accepted
  • Version changed from 1.5 to master

comment:3 Changed 21 months ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from new to closed

In e61cc87129727c66120b67c376feda3533544db1:

Fixed #21090 -- Allowed backends to provide dotted field path to inspectdb.

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