Edgewall Software
Modify

Opened 19 years ago

Closed 17 years ago

Last modified 14 years ago

#1816 closed defect (fixed)

Milestone Creation allows invalid characters

Reported by: thefigster@… Owned by: Christopher Lenz
Priority: high Milestone: 0.10.2
Component: roadmap Version: 0.10.4
Severity: normal Keywords: milestone name slash needinfo
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I created a milestone named, "Bugs, etc." I can submit tickets to this milestone, it appears in the roadmap, however when I try to access it I get an error message stating that, "Milestone Bugs, etc does not exist." I think this is because the comma is creating an invalid url, however I am not positive. This may be something you want to look into.

Attachments (0)

Change History (19)

comment:1 by Matthew Good, 19 years ago

Resolution: worksforme
Status: newclosed

These milestone naming issues seem to have been resolved for a while.

comment:2 by mark.m.mcmahon@…, 18 years ago

Resolution: worksforme
Status: closedreopened
Version: 0.8.40.10

This bug is still appearing on 0.10 (Windows based server, Apache 2.0.59, python 2.4.3).

Another invalid character is "/".

comment:3 by Christian Boos, 18 years ago

Milestone: 0.10.1

I'll take a look.

comment:4 by gass@…, 18 years ago

Spaces in milestones are not a valid caracter either because they get impossible to make custom tycket queries.

comment:5 by Christian Boos, 18 years ago

Priority: normalhigh

comment:6 by ThurnerRupert, 18 years ago

it happens with a milestone

asdf/ asd%ç+"*%

on 10.1-dev too.

comment:7 by ThurnerRupert, 18 years ago

probably the only invalid character should be "/"? or it should behave the same way as wiki page names, where "/" is allowed even if i'm not sure what it denotes, a subpage, or a page which includes "/" in its name.

how do other wiki's solve this, like moinmoin, mediawiki?

comment:8 by Christian Boos, 18 years ago

Removing the special treatment on milestone names (i.e. removing the milestone.name.replace('/', '%2F') all over the places) fixed the issue for me, at least on trunk.

cmlenz, could you remind me why this was introduced, and whether you think this is still needed?

comment:9 by Christian Boos, 18 years ago

Keywords: milestone name slash added
Resolution: fixed
Status: reopenedclosed

cmlenz told me this had to do with a weird problem in Apache…

I couldn't reproduce it, so I think it's best to cleanup the roadmap.py code and in the unlikely event that someone can reproduce the problem with Apache in the future, we'll address that issue at the appropriate level.

Fixed in r4196 (trunk) and r4198 (0.10-stable).

All of the following "weird" milestone names are now working for me:

  • Teßt
  • 0.10?
  • The / Test
  • milestone-0.9 & beyond …
  • (5) Build out front end functionality of sites
  • back\slash

comment:10 by chris.alex.thomas@…, 18 years ago

Resolution: fixed
Status: closedreopened

Hi,

I have just come across this problem, I am running trac 0.10.1 and I created a milestone called "Invoice / Financial area" and now when I click the milestone, it says exactly what is said above.

but this issue is closed, but how can that be if I am still having this problem with the latest code?

also, you cannot delete the item either, because it's name causes problems and using firefox webdeveloper to manually open the html, alter it and post that instead comes up with the same results, so now I am stuck with a milestone which is unusable.

output: "Internal Error

Milestone Invoice %2F Financial Area does not exist."

url: http://<trac-url>/milestone/Invoice%20%252F%20Financial%20Area

this bug isnt dead yet.

comment:11 by Tim Hatch <trac@…>, 18 years ago

Resolution: fixed
Status: reopenedclosed

This bug was present in 0.10.1 and fixed for 0.10.2. See TracDownload for how to download a new version if your package maintainer hasn't already. If the slashes cause you an issue once running 0.10.2, by all means please reopen.

comment:12 by anonymous, 17 years ago

Resolution: fixed
Status: closedreopened
Version: 0.100.10.4

I am still experiencing this bug in trac 0.10.4
I have a milestone named: test/test
When I create it, I immediately get an error:

Internal Error

Milestone test%2Ftest does not exist.



The milestone still appears in my Roadmap list though. When I click it, I get the same error.

in reply to:  12 ; comment:13 by Christian Boos, 17 years ago

Keywords: needinfo added

Replying to anonymous:

I am still experiencing this bug in trac 0.10.4

Sorry, it definitely works for me using milestone:0.10.4, when creating a new milestone from the roadmap page. Did you perhaps attempt to create the milestone with the WebAdmin (and an old version of it?)

Double check your installation against the pristine 0.10.4 code, I'm quite sure it's an installation issue.

comment:14 by Christopher Lenz, 17 years ago

Is this using mod_python?

in reply to:  13 comment:15 by anonymous, 17 years ago

Replying to cboos:

Replying to anonymous:

I am still experiencing this bug in trac 0.10.4

Sorry, it definitely works for me using milestone:0.10.4, when creating a new milestone from the roadmap page. Did you perhaps attempt to create the milestone with the WebAdmin (and an old version of it?)

Double check your installation against the pristine 0.10.4 code, I'm quite sure it's an installation issue.

I have created the new milestone from the roadmap page (using FireFox) and I definitely upgraded to 0.10.4. It might be an installation issue but I don't know how to check for that. I used the windows installer for 0.10.4 to upgrade.
I am using mod_python by the way.

comment:16 by anonymous, 17 years ago

Resolution: fixed
Status: reopenedclosed

Ok now it works. I forgot to run TRAC_Admin upgrade… my bad. My apologies.

comment:17 by Christian Boos, 17 years ago

Well, I don't think that trac-admin upgrade did play a role here… Probably you were still using some older Trac release, and once you were really using 0.10.4 (after a restart of apache?), things worked.

comment:18 by jenchitty@…, 14 years ago

There seems to be a related issue here with Trac 0.11.7, where a change to the list of Milestones takes a looong time to propagate through the database or get into some caches, or something.

  1. I create a new Milestone called "1.7.X".
  2. I try to re-target a ticket to that Milestone (it shows up in the drop down in the Ticket form), but I get the following warning when I try to submit my changes:
    Warning:  "1.7.X" is not a valid value for the milestone field.
    
  3. I check that the 1.7.X milestone exists in the roadmap, and it is there. I can even update the description.
  4. I refresh my ticket window and the "1.7.X" entry disappears from the drop-down!
  5. I refresh my ticket window again and the "1.7.X" entry re-appears!
  6. I try to re-target again, and it fails with the same warning again.
  7. I wait 15 minutes and click the "Submit changes" button on the ticket again, and it succeeds.

comment:19 by Christian Boos, 14 years ago

Hm, I wonder if we didn't do a config.touch() in this case?

Anyway, it's a bit too late now for 0.11.x, as this release line will only get security fixes (if any).

This scenario is already much better handled on 0.12.x anyway, so it's time to upgrade ;-)

Modify Ticket

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

Add Comment


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