Skip to content

Commit 0c31bbf

Browse files
committed
Fixes #537 - Avoid migration errors with custom user models
1 parent c98ab73 commit 0c31bbf

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

two_factor/migrations/0003_auto_20150817_1733.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,18 +10,17 @@
1010
def migrate_phone_numbers(apps, schema_editor):
1111
PhoneDevice = apps.get_model("two_factor", "PhoneDevice")
1212
for device in PhoneDevice.objects.all():
13-
username = device.user.get_username()
1413
try:
1514
number = phonenumbers.parse(device.number)
1615
if not phonenumbers.is_valid_number(number):
17-
logger.info("User '%s' has an invalid phone number '%s'." % (username, device.number))
16+
logger.info("User '%s' has an invalid phone number '%s'." % (device.user, device.number))
1817
device.number = phonenumbers.format_number(number, phonenumbers.PhoneNumberFormat.E164)
1918
device.save()
2019
except phonenumbers.NumberParseException as e:
2120
# Do not modify/delete the device, as it worked before. However this might result in issues elsewhere,
2221
# so do log a warning.
2322
logger.warning("User '%s' has an invalid phone number '%s': %s. Please resolve this issue, "
24-
"as it might result in errors." % (username, device.number, e))
23+
"as it might result in errors." % (device.user, device.number, e))
2524

2625

2726
class Migration(migrations.Migration):

0 commit comments

Comments
 (0)