mirror of
				https://github.com/encode/django-rest-framework.git
				synced 2025-11-04 09:57:55 +03:00 
			
		
		
		
	Update serializers.py
Prevent iterating over a string that is supposed to be an iterable <==> Prevent read_only_fields = ('some_string)
			
			
This commit is contained in:
		
							parent
							
								
									5c87db96c5
								
							
						
					
					
						commit
						03f96988ba
					
				| 
						 | 
					@ -758,7 +758,7 @@ class ModelSerializer(Serializer):
 | 
				
			||||||
 | 
					
 | 
				
			||||||
                ret[accessor_name] = field
 | 
					                ret[accessor_name] = field
 | 
				
			||||||
        
 | 
					        
 | 
				
			||||||
        #Ensure that 'read_only_fields is an iterable
 | 
					        # Ensure that 'read_only_fields' is an iterable
 | 
				
			||||||
        assert isinstance(self.opts.read_only_fields, (list, tuple)), '`read_only_fields` must be a list or tuple' 
 | 
					        assert isinstance(self.opts.read_only_fields, (list, tuple)), '`read_only_fields` must be a list or tuple' 
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        # Add the `read_only` flag to any fields that have bee specified
 | 
					        # Add the `read_only` flag to any fields that have bee specified
 | 
				
			||||||
| 
						 | 
					@ -775,7 +775,7 @@ class ModelSerializer(Serializer):
 | 
				
			||||||
                (field_name, self.__class__.__name__))
 | 
					                (field_name, self.__class__.__name__))
 | 
				
			||||||
            ret[field_name].read_only = True
 | 
					            ret[field_name].read_only = True
 | 
				
			||||||
        
 | 
					        
 | 
				
			||||||
        # Ensure that 'write_only_fields' is an iterabe
 | 
					        # Ensure that 'write_only_fields' is an iterable
 | 
				
			||||||
        assert isinstance(self.opts.write_only_fields, (list, tuple)), '`read_only_fields` must be a list or tuple' 
 | 
					        assert isinstance(self.opts.write_only_fields, (list, tuple)), '`read_only_fields` must be a list or tuple' 
 | 
				
			||||||
        
 | 
					        
 | 
				
			||||||
        for field_name in self.opts.write_only_fields:
 | 
					        for field_name in self.opts.write_only_fields:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue
	
	Block a user