Issue 2550695
Created on 2011-03-28 05:50 by jerrykan, last changed 2016-04-30 02:10 by rouilj.
msg4264 |
Author: [hidden] (jerrykan) |
Date: 2011-03-28 05:50 |
|
Create a query that has one of the 'no sort or group' radio buttons
selected. When you go to edit this query the settings for the 'no sort
or group' option are not retained, but are instead set to the default.
This can be annoying then wanting to make minor changes to existing
queries and forget to set the sort or group options again.
See patch for a proposed fix
|
msg4265 |
Author: [hidden] (ber) |
Date: 2011-03-30 11:04 |
|
John, thanks for feedback and the patch.
I wonder if other templates also have the issue.
|
msg4298 |
Author: [hidden] (jerrykan) |
Date: 2011-04-18 01:45 |
|
Bernhard, I'm not sure if there would be similar issues in other
templates, but my guess would be probably not.
The saved queries seem to be a part of the systems that doesn't see a
whole log of use (at least for us). We have been using roundup for about
18 months and only one user (out of ~50 frequent users) picked up on
this bug. I am assuming if there were similar problems in the other
"high use" areas, someone would have noticed by now.
|
msg4348 |
Author: [hidden] (satchit) |
Date: 2011-07-18 22:15 |
|
The patch works as expected:
- I tested this against the demo instance before and after applying the
patch.
- Ensured all the tests pass (this is just a change to the HTML code,
but still wanted to make sure).
Based on my review, this patch is good to go.
A note:
Possibly because of a change in the original file (issue.search.html),
the offset of the second part of the patch (line 167) has changed by one
(now line 168):
$ patch -p0 < issue.search.html.patch
patching file share/roundup/templates/classic/html/issue.search.html
Hunk #2 succeeded at 168 (offset 1 line).
Since patch can handle this, IMO no change is required for the patch.
|
msg4351 |
Author: [hidden] (ber) |
Date: 2011-07-19 07:55 |
|
Thanks Satchit, Thanks John!
I've committed the patch!
I wonder if we should add some update instructions
for people migrating. They would have to update their template,
right?
|
msg4352 |
Author: [hidden] (ber) |
Date: 2011-07-19 07:55 |
|
Commited with rev4642
|
msg4744 |
Author: [hidden] (rouilj) |
Date: 2013-01-11 03:35 |
|
Since this patch appears to have been commited, should the issue
be changed to status:closed resolution:fixed?
|
msg4750 |
Author: [hidden] (ber) |
Date: 2013-01-11 10:17 |
|
I guess I was wondering if we need to add some update instructions.
(Which was my last question)
|
msg4779 |
Author: [hidden] (jerrykan) |
Date: 2013-01-25 12:45 |
|
I guess figuring out the best way to document changes in templates might
be a good start. Part of the problem with roundup is that it is so
flexible, and people will generally add their own customisations to
templates.
Other than generating a diff of all of the template files for each
template just before release, is there a good way to document changes in
any concise way?
Personally I think the best way to upgrade is to do a three-way merge
with the old version, new version, and the customised version of the
template... but that isn't really documentation.
|
msg5546 |
Author: [hidden] (rouilj) |
Date: 2016-04-30 02:09 |
|
This patch was released on/in:
2012-05-15: 1.4.20
to documenting patches to the templates is still an open issue.
I claim the ticket number in the CHANGES.txt file is sufficient
to find the ticket that should have a patch attached or a reference to
the hg checkin that fixed the issue.
The user can pull the diff from there.
I don't think supplying a diff for every release really scales.
Is there a simple set of directions that will allow the end user/admin
to generate a diff on their own that they can try to apply?
|
|
Date |
User |
Action |
Args |
2016-04-30 02:10:24 | rouilj | set | status: new -> fixed type: behavior |
2016-04-30 02:09:18 | rouilj | set | messages:
+ msg5546 |
2013-01-25 12:45:26 | jerrykan | set | messages:
+ msg4779 |
2013-01-11 10:17:39 | ber | set | messages:
+ msg4750 |
2013-01-11 03:35:06 | rouilj | set | nosy:
+ rouilj messages:
+ msg4744 |
2011-07-19 07:55:42 | ber | set | messages:
+ msg4352 |
2011-07-19 07:55:22 | ber | set | messages:
+ msg4351 |
2011-07-18 22:15:28 | satchit | set | nosy:
+ satchit messages:
+ msg4348 |
2011-04-18 01:45:53 | jerrykan | set | messages:
+ msg4298 |
2011-03-30 11:04:17 | ber | set | nosy:
+ ber messages:
+ msg4265 |
2011-03-28 05:50:33 | jerrykan | create | |
|