Roundup Tracker - Issues

Issue 2550723

classification
Fix propagation of @pagesize
Type: behavior Severity: normal
Components: User Interface Versions: devel
process
Status: closed fixed
:
: rouilj : ber, jerrykan, rouilj, satchit
Priority: normal : patch

Created on 2011-09-02 04:58 by jerrykan, last changed 2016-06-19 03:51 by rouilj.

Files
File name Uploaded Description Edit Remove
pagesize_prop.patch jerrykan, 2011-09-02 04:58
Messages
msg4401 Author: [hidden] (jerrykan) Date: 2011-09-02 04:58
Steps to produce the issue:
 1. start up roundup demo.py
 2. have at least two issue in the system (create two dummy ones if needed)
 3. browse to:
      http://localhost:8917/demo/issue?@startwith=1&@pagesize=0
 4. look at the "previous" link and notice it is something like:
      http://localhost:8917/demo/issue?@startwith=1&@pagesize=7
    (ie. the '@pagesize=0' is not propagated, but instead changed to
'@pagesize=7')

This is and issue if you don't want '@pagesize=0' change to
'@pagesize=7' (ie: see issue2550678)

Apply the patch and '@pagesize=0' should be propagated correctly.
msg4407 Author: [hidden] (ber) Date: 2011-09-03 20:17
Satchit,
so you already had a look at the first version of this patch.
Any spare time to check out this one as well? :)
Best,
Bernhard
msg4419 Author: [hidden] (satchit) Date: 2011-09-07 13:16
On Sep 3, 2011, at 4:17 PM, Bernhard Reiter wrote:

> 
> Bernhard Reiter <bernhard@intevation.de> added the comment:
> 
> Satchit,
> so you already had a look at the first version of this patch.
> Any spare time to check out this one as well? :)

Bernhard, I may not be able to look at it until this weekend. But I can definitely do that.

> Best,
> Bernhard
> 
> ----------
> priority:  -> normal
> 
> ________________________________________________
> Roundup tracker <issues@roundup-tracker.org>
> <http://issues.roundup-tracker.org/issue2550723>
> ________________________________________________
msg4420 Author: [hidden] (ber) Date: 2011-09-07 13:49
Satchit: Thanks that would be wonderful!
msg4517 Author: [hidden] (rouilj) Date: 2012-03-11 01:59
Satchit, did you have a chance to review this patch?
msg5611 Author: [hidden] (rouilj) Date: 2016-06-19 03:51
Verified the patch fixed the problem as described.
Manually checked that other pagesize values (5, 20) are properly
propagated before and after patch.

Checked in as: 39af8a0f3446
History
Date User Action Args
2016-06-19 03:51:41rouiljsetstatus: new -> closed
assignee: rouilj
resolution: fixed
messages: + msg5611
2012-03-11 01:59:07rouiljsetnosy: + rouilj
messages: + msg4517
2011-09-07 13:49:46bersetmessages: + msg4420
2011-09-07 13:16:55satchitsetmessages: + msg4419
2011-09-03 20:17:08bersetpriority: normal
messages: + msg4407
2011-09-03 20:14:52bersetnosy: + ber, satchit
2011-09-02 04:58:32jerrykancreate