From 2d31061b3299bfecafe2a3a0133e86ba585c3876 Mon Sep 17 00:00:00 2001 From: farhaanbukhsh Date: Jul 26 2016 07:41:45 +0000 Subject: Fix name field not required in form --- diff --git a/pagure/hooks/jenkins_hook.py b/pagure/hooks/jenkins_hook.py index 0309caf..1918c4a 100644 --- a/pagure/hooks/jenkins_hook.py +++ b/pagure/hooks/jenkins_hook.py @@ -30,9 +30,8 @@ class PagureCI(BASE): active = sa.Column(sa.Boolean, nullable=False, default=False) display_name = sa.Column(sa.String(64), nullable=False, default='Jenkins') - name = sa.Column(sa.String(64)) pagure_name = sa.Column(sa.String(255)) - + jenkins_name = sa.Column(sa.String(255)) jenkins_url = sa.Column(sa.String(255), nullable=False, default='http://jenkins.fedorainfracloud.org/') @@ -47,10 +46,6 @@ class PagureCI(BASE): single_parent=True) ) - def __repr__(self): - return ''.format(self) - - class ConfigNotFound(Exception): pass @@ -75,9 +70,6 @@ def get_configs(project_name, service): class JenkinsForm(wtf.Form): '''Form to configure Jenkins hook''' - name = TextField('Name', - [validators.Required(), - validators.Length(max=64)]) pagure_name = TextField('Name of project in Pagure', [validators.Required(), @@ -108,7 +100,7 @@ class PagureCiHook(BaseHook): db_object = PagureCI backref = 'hook_pagure_ci' form_fields = [ - 'name', 'pagure_name', 'jenkins_name', + 'pagure_name', 'jenkins_name', 'jenkins_url', 'jenkins_token', 'active' ]