Edgewall Software
Modify ↓

Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#12647 closed defect (fixed)

Query page ignores max parameter when its parameter is only max parameter

Reported by: Jun Omae Owned by: Ryan J Ollos
Priority: normal Milestone: 1.0.17
Component: query system Version: 1.0.11
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Fixed max ignored when only parameter in query and some parameters not overriding the default query.

API Changes:
Internal Changes:

Description

This issue can be reproduced with query:?max=1. I think query page should respect max parameter.

Attachments (0)

Change History (3)

comment:1 by Ryan J Ollos, 7 years ago

Milestone: next-stable-1.0.x → 1.0.16
Owner: set to Ryan J Ollos
Status: new → assigned

Related issue. If the default_query includes desc=1 (status!=closed&owner=$USER&order=reporter&desc=1), the order would not be overridden by query:?desc=0.

I don't understand this area in the code well enough to be certain, but here are some changes for review: log:rjollos.git:t12647. Tests are passing for me, but probably more coverage is needed.

comment:2 by Ryan J Ollos, 7 years ago

Release Notes: modified (diff)
Resolution: → fixed
Status: assigned → closed

Committed to 1.0-stable in r16272, merged to 1.2-stable in r16273, merged to trunk in r16274.

comment:3 by Ryan J Ollos, 7 years ago

Milestone: 1.0.16 → 1.0.17

Milestone renamed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Ryan J Ollos to the specified user.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.