• 
      

    Exclude gridonly param when rendering paginator.

    Review Request #7230 — Created April 19, 2015 and submitted

    Information

    Djblets
    master
    6294e97...

    Reviewers

    This is a fix for issue 3794 where editing the columns in a datagrid would lead to page rendering problems when switching to a different page via the paginator.

    The issue is the inclusion of the 'gridonly=1' parameter in the paginator URLs that result after editing the columns. render_paginator() is updated to exclude the gridonly parameter when producing the new URL query string.

    (Note: I'm a little out of my comfort zone here, but this seems to be a sufficient solution.)

    Tested with internal dev server and confirmed that switching pages via the paginator no longer results in rendering issues.
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          djblets/datagrid/grids.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          djblets/datagrid/grids.py
      
      
    2. 
        
    brennie
    1. I'm not super familiar with Djblets, but this seems reasonable to me.

      I'm going to hold off on landing this until someone more familiar with Djblets reviews it.

    2. 
        
    chipx86
    1. Ship It!
    2. 
        
    gmyers
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-0.9.x (1be29ab)