Add a DB converter for the model field

[francoisfreitag]

#558

[kalekseev]

This is a breaking change, would be better to release it in v8...

[francoisfreitag]

Oops, that’s right. Sorry for the trouble! I’ll update the release note text to highlight that.

[chris-erickson]

How would we get the old behavior back?

[francoisfreitag]

Depending on what you expect, I would say either:

• call str() on the PhoneNumber object (or the property for a specific format, such as .as_international, .as_e164, ...)
• subclass the phonenumber_field.modelfields.PhoneNumberField and implement from_db_value() as

    def from_db_value(self, value, expression, connection):
        return value

EDIT: Thanks @Neumsy for pointing out that I mixed up the call to remove (to_python) with the field method name (from_db_value).

[Neumsy]

• subclass the phonenumber_field.modelfields.PhoneNumberField and implement to_python() as

    def to_python(self, value):
        return value

When using this method, shouldn't the suggestion be to override the from_db_value method? The calling to_python method is not part of the class inheritance, so overriding it in the class doesn't produce the desired outcome.

    def from_db_value(self, value, expression, connection):
        return str(value)