Override validation for CommaSeparatedValuesField.

 






 

 


  Review Request #14350
  —
  Created Feb. 17, 2025 and submitted  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  david
 









 
 

  Djblets
 









 
 

  release-5.x
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  djblets
 









 
 

  
 






 








 





  


 
 

  
The new comma separated values field that replaced the old

django-multiselectfield dependency was falling back on the base field

validation, which wasn't handling choices correctly. This change adds an

override so we actually iterate through the value instead of just

checking it directly.

 








  


 
 

  
Verified that saving changes to a comma-separated values field using a

multi checkbox widget worked correctly.

 







 









 





  








 




 







   

    
    
Changes between revision 3 and 4



    

 

 

 

 
orig
1
2
3
4

 



    


    

      
Commits

      

 

 

  
   
  
  Summary
  ID
  Author
 



 
  
 
 
  
   
    

     Override validation for CommaSeparatedValuesField.
     
The new comma separated values field that replaced the old
django-multiselectfield dependency was falling back on the base field
validation, which wasn't handling choices correctly. This change adds an
override so we actually iterate through the value instead of just
checking it directly.

Testing Done:
Verified that saving changes to a comma-separated values field using a
multi checkbox widget worked correctly.

    

   
  		
  9456ff03f727837a508fab58c6fe12114a6c6cd5
  David Trowbridge
 


 
  
 
 
  
   
    

     Override validation for CommaSeparatedValuesField.
     
The new comma separated values field that replaced the old
django-multiselectfield dependency was falling back on the base field
validation, which wasn't handling choices correctly. This change adds an
override so we actually iterate through the value instead of just
checking it directly.

Testing Done:
Verified that saving changes to a comma-separated values field using a
multi checkbox widget worked correctly.

    

   
  		
  49ea0fc4c63a7ee9ce186c0bb5b1ec9d046fe818
  David Trowbridge
 




    


    
    

   

  

 









 

  

   
    

     
      
      djblets/db/fields/comma_separated_values_field.py
     
    

   
  

 










     

    

   

  

  

   
   Loading...