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: Closed (submitted)

Change Summary:

Pushed to release-0.9.x (1be29ab)
Loading...