django.contrib.postgres.fields.jsonb not captured by codemod about django-codemod

[browniebroke]

If you're talking more about the undocumented deprecated transforms, I think django-codemod doesn't handle them at the moment, I don't have any objections in adding them: they are mentioned in the release notes, which I think makes them candidate for inclusion. Pull request welcome.

If you're talking about JSONField, that should be already covered by JSONModelFieldTransformer:

django-codemod/django_codemod/visitors/postgres_fields.py

Lines 23 to 29 in 1983afd

	class JSONModelFieldTransformer(BaseFuncRenameTransformer):
	"""Replace postgres' JSON field with the core JSON field."""
	deprecated_in = DJANGO_3_1
	removed_in = DJANGO_4_0
	rename_from = "django.contrib.postgres.fields.JSONField"
	rename_to = "django.db.models.JSONField"

But as you noted, the codemodder works only if imported as follow:

from django.contrib.postgres.fields import JSONField

It's true that the field may be imported from jsonb module as well, but I'm not convinced that we should support it... I already identified an edge case if the parent module is imported (#63), but solving these 2 means we would have to support at least 4 different way of importing, and I could see this opening the door to more.

It would be simpler for now to just support one way and maybe never do #63. I like to think that in a near future, what is supported by django-codemod would be enough of an incentive to import things in a predictable way: use documented imports and get the benefit of auto codemodding or do it yourself.

Probably an artifact of the way Pycharm offers imports.

Yes, I'm victim of this one myself, it sometimes suggest to import things from a module I already have in my imports, even if the thing is defined in a totally different location. A bit annoying.

[campbellmc]

Thanks for your reply.
I'm in two minds about these kind of edge cases of the undocumented kind.
But overall I think you are quite right in this case - support the typical and documented import. It does get caught by manage.py check which means the deprecation doesn't get unreported.

[browniebroke]

Hum, I think we can leave this open to cover codemodding KeyTransform and KeyTextTransform... Unless they are already handled by django-codemod (I really don't remember)?