• 
  
    
 
    
 
    

    



    
    
    
    
     
    




    
 
    

  
    



    
 

 

  

  

 

    


  
    





    
 
    



  

    
 
 
    
  
    Fix issue 1539 - view watched-groups does not work
    
 
    

    



 
    
 
    

  Review Request #1699
  —
  Created June 30, 2010 and discarded

 
    


    




    




    




 
    
  
    
   Information

  
    



    
 
 
    
  bellthoven
 
    

    





    
 
 
    
  Review Board
 
    

    





    
 
 
    
  master
 
    

    





    
 
 
    
  1539
 
    

    





    
 
 
    
  
 
    

    



 
    



 
    
  
    
   Reviewers

  
    



    
 
 
    
  reviewboard
 
    

    





    
 
 
    
  
 
    

    



 
    



    


    
 
    




  

    
 
 
    
  
    As I debugged, the problem only was the lack of self.counts. As it is my first review submission, I just copied and pasted the GroupDataGrid self.counts' to the WatchedGroupDataGrid __init__ method. It's working.
    
 
    

    





  

    
 
 
    
  
    Now you can see the datagrid view, no debug page is shown as expected in issue's description.
    
 
    

    




 
    

    





    
 
    









 









 
    

    



  
    
 
    

    


    


 













    
 



 
    
  
    
   
    




 BE





   
    
   
    
  
    
 
    


 

    












    
 



 
    
  
    
   
    




 BE





   
    
   
    
  
    
 
    


 

    












    
 



 
    
  
    
   
    




 BE





   
    
   
    
  
    
 
    


 
    
  
    

   
    

   
    

    
    
Review request changed
    
    
    
    
     


      


     
    
    
   
    


  
    
  
    
  
    




    



  
    
   
    
    Change Summary:
   
    
   
    
    
    Diff file working now.
    
   
    
  
    







  
    
   
    
    Diff:
   
    
   
    
    
     Revision 3 (+11 -1)
    Show changes
     
        reviewboard/reviews/datagrids.py 
     
    
   
    
  
    





    





  
    
 
    

    












    
 





 
    
  
    
   
    



chipx86




   
    
   
    



    
  
    
 
    


 
    
  
    

   
    

   
    

    
    





  chipx86
 


    
    
    
    
     


      


     
    
    
   
    


  
    
  
    
  
    


      




    1. 
 
      
  
      
   
      Thanks. Ended up going a different route for this fix though, which is now fixed on master (c3ae1cd)
      
  
      
 
      

 
      
  


      
 
      
 
        

  

 
      


      

 
      

      2. 















    2. 
 
      
  
      
   
      
  
      
 
      

 
      
  


      
 
      
 
        

  

 
      


      

 
      

      3.