Commit 5e6b6b37 authored by Claude Paroz's avatar Claude Paroz

Fixes #93 - Removed unused bugzilla account field

Thanks Tim Sabsch for the report.
parent 1e8c3924
......@@ -87,7 +87,7 @@ class DetailForm(forms.ModelForm):
class Meta:
model = Person
fields = ('username', 'first_name', 'last_name', 'email', 'image', 'avatar_service',
'webpage_url', 'irc_nick', 'bugzilla_account')
'webpage_url', 'irc_nick')
def clean_image(self):
url = self.cleaned_data['image']
......
from django.db import migrations
class Migration(migrations.Migration):
dependencies = [
('people', '0005_remove_person_use_gravatar'),
]
operations = [
migrations.RemoveField(
model_name='person',
name='bugzilla_account',
),
]
......@@ -36,8 +36,6 @@ class Person(User):
)
webpage_url = models.URLField(_("Web page"), null=True, blank=True)
irc_nick = models.SlugField(_("IRC nickname"), max_length=20, null=True, blank=True)
bugzilla_account = models.EmailField(_("Bugzilla account"), null=True, blank=True,
help_text=_("This should be an email address, useful if not equal to “E-mail address” field"))
activation_key = models.CharField(max_length=40, null=True, blank=True)
# Use UserManager to get the create_user method, etc.
......@@ -107,9 +105,6 @@ class Person(User):
def no_spam_email(self):
return obfuscate_email(self.email)
def no_spam_bugzilla_account(self):
return obfuscate_email(self.bugzilla_account)
@property
def name(self):
if self.first_name or self.last_name:
......
......@@ -55,7 +55,6 @@
},
{
"fields": {
"bugzilla_account": null,
"groups": [],
"image": null,
"activation_key": null,
......@@ -70,7 +69,6 @@
},
{
"fields": {
"bugzilla_account": null,
"groups": [],
"image": null,
"activation_key": null,
......@@ -85,7 +83,6 @@
},
{
"fields": {
"bugzilla_account": null,
"groups": [],
"image": null,
"activation_key": null,
......@@ -205,7 +202,7 @@
"ext_platform": "",
"maintainers": [],
"bugs_component": "test",
"bugs_base": "https://bugzilla.gnome.org",
"bugs_base": "https://gitlab.gnome.org/GNOME/gnome-hello/issues",
"vcs_type": "git",
"homepage": "",
"description": ""
......@@ -224,7 +221,7 @@
"ext_platform": "",
"maintainers": [],
"bugs_component": "general",
"bugs_base": "https://bugzilla.gnome.org",
"bugs_base": "https://gitlab.gnome.org/GNOME/zenity/issues",
"vcs_type": "git",
"homepage": "",
"description": ""
......
......@@ -89,13 +89,8 @@ class ModuleTestCase(TestCase):
response = self.client.get(reverse('module', args=[self.mod.name]))
self.assertContains(
response,
'https://bugzilla.gnome.org/buglist.cgi?product=test&keywords_type=anywords'
'&keywords=I18N+L10N&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED'
'&bug_status=REOPENED&bug_status=NEEDINFO'
)
self.assertContains(
response,
'https://bugzilla.gnome.org/enter_bug.cgi?product=test&keywords=I18N+L10N&component=test'
'<li><a href="https://gitlab.gnome.org/GNOME/gnome-hello/issues">Report a bug</a></li>',
html=True
)
self.mod.bugs_base = ''
self.mod.save()
......
......@@ -8,29 +8,24 @@
<div class="txt">
<a class="name" href="{{ person.get_absolute_url }}">{{ person.name }}</a>
<br />
<br>
{% if person.webpage_url %}
<strong>{% trans "Web:" %}</strong>
<a class="url" href="{{ person.webpage_url }}">{{ person.webpage_url }}</a>
<br />
<br>
{% endif %}
{% if person.no_spam_email %}
<strong>{% trans "Email address:" %}</strong>
<a class="email" href="mailto:{{ person.no_spam_email }}">{{ person.no_spam_email }}</a>
<br />
<br>
{% endif %}
{% if person.irc_nick %}
<strong>{% trans "IRC account:" %}</strong>
<em>{{ person.irc_nick }}</em>
<br />
{% endif %}
{% if person.no_spam_bugzilla_account %}
<strong>{% trans "Bugzilla account:" %}</strong> {{ person.no_spam_bugzilla_account }}
<br />
<br>
{% endif %}
</div>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment