Various small fixes for Forced Username

sso/forms.py

@@ -3,7 +3,7 @@ import re
 from django import forms
 from django.contrib.auth.models import User
 
-from eve_api.models.api_player import EVEAccount
+from eve_api.models.api_player import EVEAccount, EVEPlayerCharacter
 from sso.models import ServiceAccount, Service
 from reddit.models import RedditAccount
 
@@ -34,12 +34,12 @@ def UserServiceAccountForm(user):
     class ServiceAccountForm(forms.Form):
         """ Service Account Form """
 
-        character = forms.ChoiceField(chars)
-        service = forms.ChoiceField(services)
+        character = forms.ModelChoiceField(queryset=chars)
+        service = forms.ModelChoiceField(queryset=services)
 
         def clean(self):
             if not self.cleaned_data['character'].corporation.group in self.cleaned_data['service'].groups.all():
-                raise form.ValidationError("%s is not in a corporation allowed to access %s" % (self.cleaned_data['character'].name, self.cleaned_data['service'])
+                raise form.ValidationError("%s is not in a corporation allowed to access %s" % (self.cleaned_data['character'].name, self.cleaned_data['service']))
 
             return self.cleaned_data
 

sso/models.py

@@ -1,3 +1,4 @@
+import re
 import unicodedata
 
 from django.db import models

sso/views.py

@@ -97,7 +97,8 @@ def service_add(request):
             acc.user = request.user
 
             acc.service = form.cleaned_data['service']
-            acc.password = hashlib.sha1('%s%s' % form.cleaned_data['service'].name, request.user.username).hexdigest()
+            acc.character = form.cleaned_data['character']
+            acc.password = hashlib.sha1('%s%s' % (form.cleaned_data['service'].name, settings.SECRET_KEY)).hexdigest()
 
             try:
                 acc.save()

templates/sso/serviceaccount_created.html

@@ -1,10 +1,6 @@
 {% extends "base.html" %}
 
-{% if error %}
-{% block title %}Service Account Error{% endblock %}
-{% else %}
 {% block title %}Service Account{% endblock %}
-{% endif %}
 
 {% block content %}
 {% if error %}