Testový import vrací HTTP500 příliš dlouhá jména oponentů/vedoucích

[thejoeejoee]

pokud se při importu testové práce vloží "externí" jméno vedoucího/oponenta, vyrábíme User instaci pro něj podle ekvilibristiky jména -- tohle parsování (nebo validace v importu) neumí zkontrolovat, že vyráběné jméno (resp. asi username) je příliš dlouhé a pětikiluje namísto validační chyby

řešení je asi vyrobit ten model user, pustit nad ním full_clean a ukládat až později (což by mělo Django dělat už teď, takže je zvlášní, že je to chyba)

project-thesaurus/django/apps/accounts/models/managers/user.py

Lines 26 to 67 in b9fc643

	@staticmethod
	def get_or_create_from_name(*, name: str, thesis_id: Optional[str]) -> typing.Tuple[Optional['User'], bool]:
	from apps.accounts.models import User
	name = name.replace('.', '. ').strip()
	if name.strip() == '-':
	return None, False
	if len(name) > 28:
	# two names in one cell
	name = name.split(',', 1)[0]
	degree_after = None
	if ',' in name:
	name, degree_after = name.rsplit(',', 1)
	[last_name, *degrees_before] = tuple(filter(None, name.strip().rsplit(' ', 2)[::-1]))
	first_name = degrees_before[0] if degrees_before else ''
	if thesis_id:
	# is student
	username = f'{last_name.strip()[:3].ljust(3, "0")}{thesis_id:05}'
	else:
	username = last_name.strip()
	username = slugify(username.strip().lower())
	if User.objects.filter(username=username).exists():
	pass # username = f'{username}.{thesis_id}'
	return User.objects.get_or_create(
	username=username,
	defaults=dict(
	last_name=last_name.strip(),
	first_name=first_name.strip(),
	degree_after=degree_after or None,
	degree_before=''.join(degrees_before[1:]).replace(' ', '').replace('.', '. ') or None,
	is_active=False, # activation by ldap sync
	),
	)

project-thesaurus/django/apps/thesis/models/managers/thesis_import.py

Lines 219 to 235 in b9fc643

	def _load_reviewer(self, reviewer: str):
	from apps.accounts.models import User
	from django.contrib.auth.models import Group
	if ' ' not in reviewer:
	# probably username, internal
	user = User.school_users.teachers().filter(username=reviewer).first()
	if not user:
	raise ValidationError(_('Unknown user {}').format(reviewer))
	return user
	# external
	user = User.objects.get_or_create_from_name(name=reviewer, thesis_id=None)[0]
	user.groups.add(Group.objects.get_or_create(name='teacher')[0])
	user.is_active = False
	return user