only check the port for validity if it has been set

pyweb/mumble/forms.py

@@ -128,48 +128,20 @@ class MumbleAdminForm( MumbleForm ):
 		exclude = None
 	
 	def clean_port( self ):
-		""" If portno == -1 autoassign, and check if the port number is valid. """
+		""" Check if the port number is valid. """
 		
 		port = self.cleaned_data['port'];
-		if port == -1:
-			port = max( [ rec['port'] for rec in Mumble.objects.values('port') ] ) + 1;
 		
-		if port < 1 or port >= 2**16:
-			raise forms.ValidationError(
-				_("Port number %(portno)d is not within the allowed range %(minrange)d - %(maxrange)d") % {
-				'portno':   port,
-				'minrange': 1,
-				'maxrange': 2**16,
-				});
-		return port;
-	
-	def clean( self ):
-		""" Try to bind to the addr and port to verify that they are available. """
-		
-		if self.instance.id is not None or 'addr' not in self.cleaned_data or 'port' not in self.cleaned_data:
-			# Editing old instance or previous validation failed already, don't try to bind
-			return self.cleaned_data;
-		
-		addr = socket.gethostbyname( self.cleaned_data['addr'] );
-		port = self.cleaned_data['port'];
-		
-		try:
-			socktcp = socket.socket( socket.AF_INET, socket.SOCK_STREAM );
-			socktcp.bind( ( addr, port ) );
-		except socket.error, err:
-			raise forms.ValidationError( err.args[1] );
-		finally:
-			socktcp.close();
-		
-		try:
-			sockudp = socket.socket( socket.AF_INET, socket.SOCK_DGRAM  );
-			sockudp.bind( ( addr, port ) );
-		except socket.error, err:
-			raise forms.ValidationError( err.args[1] );
-		finally:
-			sockudp.close();
-		
-		return self.cleaned_data;
+		if port is not None and port != '':
+			if port < 1 or port >= 2**16:
+				raise forms.ValidationError(
+					_("Port number %(portno)d is not within the allowed range %(minrange)d - %(maxrange)d") % {
+					'portno':   port,
+					'minrange': 1,
+					'maxrange': 2**16,
+					});
+			return port;
+		return None
 
 
 class MumbleUserForm( ModelForm ):